line 44 of masquerade.module

CommentFileSizeAuthor
#4 masquerade_987610_d7.patch497 byteshadsie
#1 masquerade-987610.patch755 bytesmikeytown2

Comments

mikeytown2’s picture

mikeytown2 commented
Status: Active » Needs review
StatusFileSize
new masquerade-987610.patch755 bytes

pressflow flavor of drupal

afreeman’s picture

afreeman commented
Status: Needs review » Reviewed & tested by the community

Reviewed, patch applies cleanly, doesn't appear to break anything in testing.

hadsie’s picture

hadsie commented

Also works on D7 (though with some minor offset)

hadsie’s picture

hadsie commented
StatusFileSize
new masquerade_987610_d7.patch497 bytes
afreeman’s picture

afreeman commented

Patched in masquerade-7.x-1.x-dev

robcarr’s picture

robcarr
Primary language English
Location Perthshire, Scotland
commented

Works for me too. Thanks @mikeytown2

andypost’s picture

andypost
he/him
Primary language Russian
commented
Status: Reviewed & tested by the community » Fixed

http://drupalcode.org/project/masquerade.git/commit/ae3f56f

Status: Fixed » Closed (fixed)

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

asb’s picture

asb commented

I just ran into the same issue:

WD php: Notice: Undefined variable: _SESSION in masquerade_init() (line 43 of /var/www/drupal/sites/all/modules/masquerade/masquerade.module).

The patch from #1 applies cleanly against Pressflow 6.22.x and fixes the issue:

# patch < masquerade-987610.patch 
patching file masquerade.module
Hunk #1 succeeded at 40 (offset -1 lines).

Thanks! (& waiting for a new release ;)

asb’s picture

asb commented
Status: Closed (fixed) » Active

Reopening since I'm still getting massive amounts (read: thousands) of these errors with the latest "recommended" and "supported" release of 'Masquerade'. Please roll a new stable release, it's more than ten months after this issue has been patched. Thank you!

andypost’s picture

andypost
he/him
Primary language Russian
commented
Status: Active » Fixed

I've rolled-out 6.x-1.7 http://drupal.org/node/1321888

asb’s picture

asb commented

Thanks a lot!

Status: Fixed » Closed (fixed)

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

  • Commit 4431720 on master, 8.x-2.x, 8.x-2.x-admin-menu, 8.x-1.x-1836516 by afreeman: 
    #987610 by hadsie: Fixes Undefined variable: _SESSION in masquerade_init...

  • Commit 4431720 on master, 8.x-2.x, 8.x-2.x-admin-menu, 8.x-1.x-1836516 by afreeman: 
    #987610 by hadsie: Fixes Undefined variable: _SESSION in masquerade_init...