Download & Extend
User Relationships » Issues

Undefined variable: _SESSION in user_relationship_invites_user()

Posted by bcmiller0 on February 2, 2012 at 4:12pm
Project:User Relationships
Version:6.x-1.x-dev
Component:Invites
Category:bug report
Priority:normal
Assigned:Unassigned
Status:postponed (maintainer needs more info)

Issue Summary

Getting Notice for anon user on our home page as follows:

Notice: Undefined variable: _SESSION in user_relationship_invites_user() (line 81 of www/sites/all/modules/contrib/user_relationships/user_relationship_invites/user_relationship_invites.module).

Login or register to post comments

Comments

#1

Posted by bcmiller0 on February 2, 2012 at 4:23pm

here is patch to remove hitting notice condition.

AttachmentSizeStatusTest resultOperations
user_relationships_invites-1426098-1.patch778 bytesIdleFAILED: [[SimpleTest]]: [MySQL] Unable to apply patch user_relationships_invites-1426098-1.patch. Unable to apply patch. See the log in the details link for more information.View details | Re-test

#2

Posted by bcmiller0 on February 2, 2012 at 4:29pm

oops, here is patch with proper relative paths.

AttachmentSizeStatusTest resultOperations
user_relationships_invites-1426098-2.patch882 bytesIdlePASSED: [[SimpleTest]]: [MySQL] 218 pass(es).View details | Re-test

#3

Posted by mrf on February 3, 2012 at 12:01am
Status:active» needs review

#4

Posted by mrf on April 16, 2012 at 9:09pm
Status:needs review» postponed (maintainer needs more info)

I cleaned this whole section up a bit, and included a check for the invite session variable around the whole thing.

Let me know if this removes the notice for you, I wasn't able to see that in my testing.

AttachmentSizeStatusTest resultOperations
user_relationships_invites-1426098-4.patch2.85 KBIdlePASSED: [[SimpleTest]]: [MySQL] 218 pass(es).View details | Re-test

#5

Posted by mrf on April 16, 2012 at 9:09pm
Status:postponed (maintainer needs more info)» needs review

#6

Posted by mrf on April 18, 2012 at 5:16pm
Version:6.x-1.0» 6.x-1.x-dev
Status:needs review» postponed (maintainer needs more info)

I never saw the notice so I can't really confirm if this fix works or not. If someone can test this on a site that relies on invites give me a +1 and I'll commit this.

nobody click here