Tickets :: [#5287] Don't create global variables with the same name as session variables

6.0.0-beta1

8/1/25

Summary	Don't create global variables with the same name as session variables
Queue	Horde Base
Queue Version	HEAD
Type	Bug
State	Resolved
Priority	2. Medium
Owners	Horde Developers (at)
Requester	chuck (at) horde (dot) org
Created	04/23/2007 (6675 days ago)
Due
Updated	06/15/2007 (6622 days ago)
Assigned	04/23/2007 (6675 days ago)
Resolved	06/15/2007 (6622 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch	No

06/15/2007 03:33:48 PM	Chuck Hagenbuch	Comment #6 State ⇒ Resolved	Reply to this comment
Done.

06/15/2007 04:15:14 AM	Chuck Hagenbuch	Comment #5	Reply to this comment
These are what's left here: incubator/rakim ($rakim) kronolith ($attendees, $kronolith) whups ($whups, $whups_query)

04/23/2007 08:58:56 PM	Chuck Hagenbuch	Comment #4	Reply to this comment
Based on that data and some grepping, the following applications are (potentially) affected: incubator/beatnik ($beatnik) incubator/Horde_Payment ($payment) incubator/Horde_Policy ($policy) incubator/minerva ($minerva) incubator/rakim ($rakim) kronolith ($attendees, $kronolith) merk ($merk) vilma ($vilma) whups ($whups, $whups_query) (we actually did fix Wicked already)

04/23/2007 05:50:16 PM	Chuck Hagenbuch	Comment #3 New Attachment: unique_session_vars.txt	Reply to this comment
Here's a list of unique session variables used in HEAD. Most aren't used as global variable names, but any overlap here is what we need to fix.

04/23/2007 07:52:03 AM	Jan Schneider	Assigned to Horde Developers Priority ⇒ 2. Medium

04/23/2007 07:51:42 AM	Jan Schneider	Comment #2 Priority ⇒ 1. Low State ⇒ Assigned Type ⇒ Bug	Reply to this comment
I consider this even a bug.

04/23/2007 05:07:56 AM	Chuck Hagenbuch	Comment #1 Priority ⇒ 1. Low Type ⇒ Enhancement Summary ⇒ Don't create global variables with the same name as session variables Queue ⇒ Horde Base State ⇒ Accepted	Reply to this comment
(or vice versa). One place we do this currently is in Wicked; it means that Wicked doesn't work with register_globals on.