Closed (fixed)
Project:
Drupal for Facebook
Version:
6.x-3.x-dev
Component:
Facebook Connect
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
29 Sep 2009 at 19:41 UTC
Updated:
12 Nov 2010 at 20:30 UTC
Jump to comment: Most recent file
Comments
Comment #1
Dave Cohen commentedInteresting one, thanks for reporting.
I consider the attached patch a hack. But with facebook hacks are a way of life.
Comment #2
gstout commentedSo... I can't log out period. In some ways I find this funny as it might be great for user retention.
Going to /logout doesn't log me out of facebook or Drupal. This happened in the last dev and in the beta I put on today. Hope that helps
Happy to test ideas
Comment #3
Dave Cohen commentedClicking a logout link should work, although typing the /logout url into the browser doesn't, because it wont log you out of facebook. So you'll be logged in again, thanks to their single sign on.
The patch above is way out of date. The code above might work though. It should be placed in fb_user_user() for $op=='logout'.
Comment #4
gstout commentedWe don't have a log-out link because we have done away with our user account system in favor of a third party membership system.
Now only editors log-in by typing /user and logout with /logout.
Hmm. If one needed to logout, could then first got to facebook.com and logout and then come back to the site and type /logout? Is there a FB logout button (I've never seen one) that i could implement with fbml?
I guess this is not a big deal but I'm sure some one will complain to me at some point.
Dave thanks for your great work and for you on time beta delivery. It was like Christmas!
Comment #5
Dave Cohen commentedIf the user logs in using facebook connect, then goes to facebook (in say another tab) and logs out there, there is no need to return to your site and logout. They are already logged out.
You could use drupal's hook_menu_alter to change the permissions for /logout, so that only your admins/editors can use it. Then you can simply show it on the page, and clicking it would log users out properly.
Comment #6
Dave Cohen commentedHere's a re-roll of the patch.
Comment #7
Dave Cohen commentedno reviews, but checked this in anyway.
Comment #8
almaudoh commentedI had this same issue and I applied your diff. It's working fine now. Thanks.