This handles the case where the 'reset_form' button triggers an error for users who are not logged in.

When not logged in, the _SESSION variable is undefined.
This produces an error about _SESSION not being defined.
This patch adds a check to ensure that the variable exists before acting on it.

CommentFileSizeAuthor
#2 views-7.x-3.0-beta3-_SESSION_invalid_usage-2.patch926 bytesthekevinday
views-7.x-3.0-beta3-_SESSION_invalid_usage-1.patch941 bytesthekevinday

Comments

dawehner’s picture

dawehner
Primary language German
commented
Status: Active » Needs work

Update status.

Some kind of comment for this would be cool.
Additional take care about drupal codestyle.

thekevinday’s picture

thekevinday commented
Status: Needs work » Needs review
StatusFileSize
new views-7.x-3.0-beta3-_SESSION_invalid_usage-2.patch926 bytes

Then try this out out for size.

Is there some sort of drupal-specific way to access $_SESSION?
I would rather use that instead of directly checking and calling $_SESSION to avoid problems like this.

dawehner’s picture

dawehner
Primary language German
commented
Status: Needs review » Fixed

Cool, updated status.

dawehner’s picture

dawehner
Primary language German
commented

and commited to 7.x-3.x

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.