I was struggling as I couldn't log out from both Drupal and Facebook,
I was continuously having this error reported:

Notice: Undefined variable: _SESSION in Facebook->clearPersistentData() (line 130 of /home/domains/xxxx.net/public_html/sites/all/libraries/facebook-php-sdk/src/facebook.php).

so that I posted something in this issue #1809964: Notice: Undefined variable: _SESSION in Facebook->clearPersistentData() (line 130

As I was continuously tried to she to an unknown page, then I realized that it might be due to the language identifier in the url … as I am using a multi language site (with entity translation).

I then discovered that it is caused by the missing of a little "trailing slash" in the facebook_connect.js file, in charge of selecting the proper jquery a.link selector …

Thus I solved this issue of mine, and I hope probably also the previous issue … #1809964: Notice: Undefined variable: _SESSION in Facebook->clearPersistentData() (line 130.

Here attached is the diff file.

Comments

itamair’s picture

Of course this bug fix should be committed as soon as ..

Dave Cohen’s picture

Assigned: itamair » Unassigned
Status: Fixed » Needs review

I would have seen this sooner if it's status were not marked fixed.

Dave Cohen’s picture

StatusFileSize
new773 bytes

I don't think the solution belongs in fb_connect.js, but rather fb_connect.module as in the attached patch. Please let me know if this solves the problem.

Dave Cohen’s picture

Status: Needs review » Fixed

I went ahead and pushed this. I'm pretty confident the change is safe.

Status: Fixed » Closed (fixed)

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