Further to #308748: Store location information in session rather than global variable, the use of a configuration setting to trigger the use of a global variable or session variable makes it difficult for other modules to use this functionality, since it could be turned off at any time by a site admin.

Instead, the variable setting should be in a function that other modules and this one can call as needed.

Comments

amitgoyal’s picture

Assigned: Unassigned » amitgoyal
Status: Active » Closed (fixed)

Thanks for the suggestion, Nedjo.

We have removed the use of a configuration setting to trigger the use of a global variable or session variable and implemented hook_init to set the data in session variable.

We have incorporated the changes in D6 and D7 versions. D5 version is no longer supported.