- Show Facebook profile pic and favicon as dictated by Facebook Connnect policy

- Logout fixed, again (cleans up Facebook cookies now)

1 files changed, 80 insertions, 54 deletions

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('<li id="nav_logout"><a href="%s" title="%s" ' .
-                 'onclick="FB.Connect.logout(function() { goto_logout() })">%s</a></li>',
+                $html = sprintf('<li id="nav_logout"><a href="%s" title="%s" ' .
+                    'onclick="FB.Connect.logout(function() { goto_logout() })">%s</a></li>',
                     $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());
+        }
+    }
+
+}