Closed (fixed)
Project:
Webform
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
15 Jun 2007 at 22:49 UTC
Updated:
1 Feb 2008 at 02:11 UTC
Jump to comment: Most recent file
Within webforms the $_SERVER and $_GET variables can be used to create things. I am requesting that $_SESSION be added to that allowed list. Towards that end I have attached a patch that I believe adds that functionality. Thank you for your consideration.
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | webform_tokens.patch | 4.29 KB | quicksketch |
| #1 | webform.module_7.patch | 2.47 KB | antgiant |
| webform.module_6.patch | 2.2 KB | antgiant |
Comments
Comment #1
antgiant commentedSince I'm requesting $_SESSION it seems to make sense to add $_COOKIE as well. So I have attached a patch that adds both to the current HEAD. However, I would much rather have just $_SESSION if forced to choose between them. Again thank you for your consideration.
Comment #2
antgiant commentedChanging title back. Sorry I'm new to this comment system.
Comment #3
antgiant commentedPatch is still good any interest in adding?
Comment #4
jugglerpm commentedI'm interested in this patch being implemented. It would help our projects.
Comment #5
designerbrent commented+2 for this. This would be very helpful!
Comment #6
quicksketchThanks, this is a great suggestion. While the patch submitted is totally fine, I made some changes to cut down on the number of copy/pasted lines of code and just put everything into a single loop. While we're at it, I also added $_REQUEST along with $_SESSION and $_COOKIE. The end result is a more efficient and flexible.
I went ahead and committed the attached patch. Thanks for everything antgiant!
Comment #7
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.