From eaf2a98daf2ad135f6daa1bcab391fbee7df3183 Mon Sep 17 00:00:00 2001 From: Zach Copley Date: Fri, 22 May 2009 00:54:22 +0000 Subject: - Show Facebook profile pic and favicon as dictated by Facebook Connnect policy - Logout fixed, again (cleans up Facebook cookies now) --- plugins/FBConnect/FBConnectPlugin.php | 134 ++++++++++++++++++++-------------- 1 file changed, 80 insertions(+), 54 deletions(-) (limited to 'plugins/FBConnect/FBConnectPlugin.php') diff --git a/plugins/FBConnect/FBConnectPlugin.php b/plugins/FBConnect/FBConnectPlugin.php index 36dee0448..4fb9a7ca4 100644 --- a/plugins/FBConnect/FBConnectPlugin.php +++ b/plugins/FBConnect/FBConnectPlugin.php @@ -40,7 +40,6 @@ require_once INSTALLDIR . '/plugins/FBConnect/FBConnectSettings.php'; require_once INSTALLDIR . '/plugins/FBConnect/FBCLoginGroupNav.php'; require_once INSTALLDIR . '/plugins/FBConnect/FBCSettingsNav.php'; - /** * Plugin to enable Facebook Connect * @@ -53,7 +52,6 @@ require_once INSTALLDIR . '/plugins/FBConnect/FBCSettingsNav.php'; class FBConnectPlugin extends Plugin { - function __construct() { parent::__construct(); @@ -84,29 +82,27 @@ class FBConnectPlugin extends Plugin 'media type you accept'), 406); } - header('Content-Type: '.$type); $action->extraHeaders(); $action->startXML('html', - '-//W3C//DTD XHTML 1.0 Strict//EN', - 'http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd'); + '-//W3C//DTD XHTML 1.0 Strict//EN', + 'http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd'); $language = $action->getLanguage(); - $action->elementStart('html', array('xmlns' => 'http://www.w3.org/1999/xhtml', - 'xmlns:fb' => 'http://www.facebook.com/2008/fbml', - 'xml:lang' => $language, - 'lang' => $language)); + $action->elementStart('html', + array('xmlns' => 'http://www.w3.org/1999/xhtml', + 'xmlns:fb' => 'http://www.facebook.com/2008/fbml', + 'xml:lang' => $language, + 'lang' => $language)); return false; - } function onEndShowLaconicaScripts($action) { - $action->element('script', array('type' => 'text/javascript', 'src' => 'http://static.ak.connect.facebook.com/js/api_lib/v0.4/FeatureLoader.js.php'), @@ -140,77 +136,92 @@ class FBConnectPlugin extends Plugin if ($user) { + $action->menuItem(common_local_url('all', array('nickname' => $user->nickname)), + _('Home'), _('Personal profile and friends timeline'), false, 'nav_home'); + $action->menuItem(common_local_url('profilesettings'), + _('Account'), _('Change your email, avatar, password, profile'), false, 'nav_account'); + if (common_config('xmpp', 'enabled')) { + $action->menuItem(common_local_url('imsettings'), + _('Connect'), _('Connect to IM, SMS, Twitter'), false, 'nav_connect'); + } else { + $action->menuItem(common_local_url('smssettings'), + _('Connect'), _('Connect to SMS, Twitter'), false, 'nav_connect'); + } + $action->menuItem(common_local_url('invite'), + _('Invite'), + sprintf(_('Invite friends and colleagues to join you on %s'), + common_config('site', 'name')), + false, 'nav_invitecontact'); + $flink = Foreign_link::getByUserId($user->id, FACEBOOK_CONNECT_SERVICE); + $fbuid = 0; if ($flink) { - $facebook = getFacebook(); + try { - if ($facebook->api_client->users_isAppUser($flink->foreign_id) || - $facebook->api_client->added) { + $facebook = getFacebook(); + $fbuid = getFacebook()->get_loggedin_user(); + + } catch (Exception $e) { + common_log(LOG_WARNING, + 'Problem getting Facebook client: ' . + $e->getMessage()); + } - // XXX: We need to replace this with a proper mini-icon and only after - // checing the FB Connect JavaScript lib method to see what the Connect - // status is. Checking Connect status looks to be impossible with the - // PHP client. + // Display Facebook Logged in indicator w/Facebook favicon - $action->elementStart('li'); + if ($fbuid > 0) { + + $action->elementStart('li', array('id' => 'nav_fb')); $action->elementStart('fb:profile-pic', array('uid' => $flink->foreign_id, - 'facebook-logo' => 'true', 'linked' => 'false', - 'width' => 32, - 'height' => 32)); + 'width' => 16, + 'height' => 16)); $action->elementEnd('fb:profile-pic'); + + $iconurl = common_path('/plugins/FBConnect/fbfavicon.ico'); + $action->element('img', array('src' => $iconurl)); + $action->elementEnd('li'); - } + } } - - $action->menuItem(common_local_url('all', array('nickname' => $user->nickname)), - _('Home'), _('Personal profile and friends timeline'), false, 'nav_home'); - $action->menuItem(common_local_url('profilesettings'), - _('Account'), _('Change your email, avatar, password, profile'), false, 'nav_account'); - if (common_config('xmpp', 'enabled')) { - $action->menuItem(common_local_url('imsettings'), - _('Connect'), _('Connect to IM, SMS, Twitter'), false, 'nav_connect'); - } else { - $action->menuItem(common_local_url('smssettings'), - _('Connect'), _('Connect to SMS, Twitter'), false, 'nav_connect'); - } - $action->menuItem(common_local_url('invite'), - _('Invite'), - sprintf(_('Invite friends and colleagues to join you on %s'), - common_config('site', 'name')), - false, 'nav_invitecontact'); - // Need to override the Logout link to make it do FB stuff + // Need to override the Logout link to make it do FB stuff + + if ($fbuid > 0) { - $logout_url = common_local_url('logout'); - $title = _('Logout from the site'); - $text = _('Logout'); + $logout_url = common_local_url('logout'); + $title = _('Logout from the site'); + $text = _('Logout'); - $html = sprintf('', + $html = sprintf('', $logout_url, $title, $text); - $action->raw($html); + $action->raw($html); + } else { + $action->menuItem(common_local_url('logout'), + _('Logout'), _('Logout from the site'), false, 'nav_logout'); + } } else { if (!common_config('site', 'closed')) { $action->menuItem(common_local_url('register'), - _('Register'), _('Create an account'), false, 'nav_register'); + _('Register'), _('Create an account'), false, 'nav_register'); } $action->menuItem(common_local_url('openidlogin'), - _('OpenID'), _('Login with OpenID'), false, 'nav_openid'); + _('OpenID'), _('Login with OpenID'), false, 'nav_openid'); $action->menuItem(common_local_url('login'), - _('Login'), _('Login to the site'), false, 'nav_login'); + _('Login'), _('Login to the site'), false, 'nav_login'); } $action->menuItem(common_local_url('doc', array('title' => 'help')), - _('Help'), _('Help me!'), false, 'nav_help'); + _('Help'), _('Help me!'), false, 'nav_help'); $action->menuItem(common_local_url('peoplesearch'), - _('Search'), _('Search for people or text'), false, 'nav_search'); + _('Search'), _('Search for people or text'), false, 'nav_search'); return false; } @@ -220,7 +231,7 @@ class FBConnectPlugin extends Plugin $action_name = get_class($action); $login_actions = array('LoginAction', 'RegisterAction', - 'OpenidloginAction', 'FacebookStart'); + 'OpenidloginAction', 'FBConnectLoginAction'); if (in_array($action_name, $login_actions)) { $nav = new FBCLoginGroupNav($action); @@ -240,6 +251,21 @@ class FBConnectPlugin extends Plugin return true; } -} + function onEndLogout($action) + { + try { + $facebook = getFacebook(); + $fbuid = $facebook->get_loggedin_user(); + + if ($fbuid > 0) { + $facebook->logout(common_local_url('public')); + } + } catch (Exception $e) { + common_log(LOG_WARNING, 'Could\'t logout of Facebook: ' . + $e->getMessage()); + } + } + +} -- cgit v1.2.3-54-g00ecf