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.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dawehner’s picture

Status: Active » Needs work

Update status.

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

thekevinday’s picture

Status: Needs work » Needs review
FileSize
926 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

Status: Needs review » Fixed

Cool, updated status.

dawehner’s picture

and commited to 7.x-3.x

Status: Fixed » Closed (fixed)

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