trac750 - Facebook app now uses XMLWriter for output (much cleaner!)

actions/facebookhome.php

@@ -28,75 +28,73 @@ class FacebookhomeAction extends FacebookAction
     {
         parent::handle($args);
 
-        $this->login();
-    }
-
-    function login()
-    {
-
-        $user = null;
-
         $facebook = get_facebook();
         $fbuid = $facebook->require_login();
 
-        # check to see whether there's already a Facebook link for this user
+        // Check to see whether there's already a Facebook link for this user
-        $flink = Foreign_link::getByForeignID($fbuid, 2); // 2 == Facebook
+        $flink = Foreign_link::getByForeignID($fbuid, FACEBOOK_SERVICE);
 
         if ($flink) {
-
+            $this->showHome($flink, null);
-            $user = $flink->getUser();
-            $this->show_home($facebook, $fbuid, $user);
-
         } else {
-
+            $this->login($fbuid);
-            # Make the user put in her Laconica creds
-            $nickname = common_canonical_nickname($this->trimmed('nickname'));
-            $password = $this->arg('password');
-
-            if ($nickname) {
-
-                if (common_check_user($nickname, $password)) {
-
-
-                    $user = User::staticGet('nickname', $nickname);
-
-                    if (!$user) {
-                        echo '<fb:error message="Coudln\'t get user!" />';
-                        $this->show_login_form();
-                    }
-
-                    $flink = DB_DataObject::factory('foreign_link');
-                    $flink->user_id = $user->id;
-                    $flink->foreign_id = $fbuid;
-                    $flink->service = 2; # Facebook
-                    $flink->created = common_sql_now();
-                    $flink->set_flags(true, false, false);
-
-                    $flink_id = $flink->insert();
-
-                    if ($flink_id) {
-                        echo '<fb:success message="You can now use Identi.ca from Facebook!" />';
-                    }
-
-                    $this->show_home($facebook, $fbuid, $user);
-
-                    return;
-                } else {
-                    echo '<fb:error message="Incorrect username or password." />';
-                }
-            }
-
-            $this->show_login_form();
         }
 
     }
 
-    function show_home($facebook, $fbuid, $user)
+    function login($fbuid)
     {
+        $nickname = common_canonical_nickname($this->trimmed('nickname'));
+        $password = $this->arg('password');
 
+        $msg = null;
+
+        if ($nickname) {
+
+            if (common_check_user($nickname, $password)) {
+
+                $user = User::staticGet('nickname', $nickname);
+
+                if (!$user) {
+                    $this->showLoginForm(_("Server error - couldn't get user!"));
+                }
+
+                $flink = DB_DataObject::factory('foreign_link');
+                $flink->user_id = $user->id;
+                $flink->foreign_id = $fbuid;
+                $flink->service = FACEBOOK_SERVICE;
+                $flink->created = common_sql_now();
+                $flink->set_flags(true, false, false);
+
+                $flink_id = $flink->insert();
+
+                // XXX: Do some error handling here
+
+                $this->showHome($flink, _('You can now use Identi.ca from Facebook!'));
+
+            } else {
+                $msg = _('Incorrect username or password.');
+            }
+        }
+
+        $this->showLoginForm($msg);
+    }
+
+    function showHome($flink, $msg)
+    {
         $this->show_header('Home');
 
+        if ($msg) {
+            common_element('fb:success', array('message' => $msg));
+        }
+
+        $facebook = get_facebook();
+        $fbuid = $facebook->require_login();
+
+        $user = $flink->getUser();
+
         echo $this->show_notices($user);
+
         $this->update_profile_box($facebook, $fbuid, $user);
 
         $this->show_footer();
@@ -112,25 +110,16 @@ class FacebookhomeAction extends FacebookAction
 
         $notice = $user->noticesWithFriends(($page-1)*NOTICES_PER_PAGE, NOTICES_PER_PAGE + 1);
 
-        echo '<ul id="notices">';
+        $cnt = $this->show_notice_list($notice);
 
-        $cnt = 0;
+        common_pagination($page > 1, $cnt > NOTICES_PER_PAGE,
-
+                          $page, 'all', array('nickname' => $user->nickname));
-        while ($notice->fetch() && $cnt <= NOTICES_PER_PAGE) {
+    }
-            $cnt++;
-
-            if ($cnt > NOTICES_PER_PAGE) {
-                break;
-            }
-
-            echo $this->render_notice($notice);
-        }
-
-        echo '<ul>';
-
-        $this->pagination($page > 1, $cnt > NOTICES_PER_PAGE,
-                          $page, 'index.php', array('nickname' => $user->nickname));
 
+    function show_notice_list($notice)
+    {
+        $nl = new NoticeList($notice);
+        return $nl->show();
     }
 
 }

actions/facebookinvite.php

@@ -40,17 +40,20 @@ class FacebookinviteAction extends FacebookAction
 
         $this->show_header('Invite');
 
-        $fbml = '<fb:request-form '
+        $content = 'You have been invited to Identi.ca! ' .
-        .'action="invite.php" '
+            htmlentities('<fb:req-choice url="http://apps.facebook.com/identica_app/" label="Add"/>');
-        .'method="POST" '
-        .'invite="true" '
-        .'type="Identi.ca" '
-        .'content="You have been invited to Identi.ca! '
-        . htmlentities('<fb:req-choice url="http://apps.facebook.com/identica_app/" label="Add"/>') . '">'
-        .'<fb:multi-friend-selector showborder="false" actiontext="Invite your friends to use Identi.ca.">'
-        .'</fb:request-form>';
 
-        echo $fbml;
+        common_element_start('fb:request-form', array('action' => 'invite.php',
+                                                      'method' => 'POST',
+                                                      'invite' => 'true',
+                                                      'type' => 'Identi.ca',
+                                                      'content' => $content));
+
+        $actiontext = 'Invite your friends to use Identi.ca.';
+        common_element_start('fb:multi-friend-selector', array('showborder' => 'false',
+                                                               'actiontext' => $actiontext));
+
+        common_element_end('fb:request-form');
 
         $this->show_footer();
 

actions/facebooksettings.php

@@ -35,7 +35,6 @@ class FacebooksettingsAction extends FacebookAction
         }
     }
 
-
     function save_settings() {
 
         $noticesync = $this->arg('noticesync');
@@ -44,69 +43,60 @@ class FacebooksettingsAction extends FacebookAction
         $facebook = get_facebook();
         $fbuid = $facebook->require_login();
 
-        $flink = Foreign_link::getByForeignID($fbuid, 2); // 2 == Facebook
+        $flink = Foreign_link::getByForeignID($fbuid, FACEBOOK_SERVICE);
 
         $original = clone($flink);
         $flink->set_flags($noticesync, $replysync, false);
         $result = $flink->update($original);
 
         if ($result) {
-            echo '<fb:success message="Sync preferences saved." />';
+            $this->show_form('Sync preferences saved.', true);
+        } else {
+            $this->show_form('There was a problem saving your sync preferences!');
         }
-
-        $this->show_form();
-
     }
 
-    function show_form() {
+    function show_form($msg = null, $success = false) {
 
         $facebook = get_facebook();
         $fbuid = $facebook->require_login();
 
-        $flink = Foreign_link::getByForeignID($fbuid, 2); // 2 == Facebook
+        $flink = Foreign_link::getByForeignID($fbuid, FACEBOOK_SERVICE);
 
-        $this->show_header('Settings');
+        $this->show_header('Settings', $msg, $success);
 
-        $fbml = '<fb:if-section-not-added section="profile">'
+        common_element_start('fb:if-section-not-added', array('section' => 'profile'));
-            .'<h2>Add an Identi.ca box to my profile</h2>'
+        common_element('h2', null, _('Add an Identi.ca box to my profile'));
-            .'<p><fb:add-section-button section="profile"/></p>'
+        common_element_start('p');
-            .'</fb:if-section-not-added>';
+        common_element('fb:add-section-button', array('section' => 'profile'));
+        common_element_end('p');
+        common_element_end('fb:if-section-not-added');
 
-        $fbml .= '<p><fb:prompt-permission perms="status_update"><h2>Allow Identi.ca to update my Facebook status</h2></fb:prompt-permission></p>';
+        common_element_start('p');
+        common_element_start('fb:prompt-permission', array('perms' => 'status_update'));
+        common_element('h2', null, _('Allow Identi.ca to update my Facebook status'));
+        common_element_end('fb:prompt-permission');
+        common_element_end('p');
 
         if ($facebook->api_client->users_hasAppPermission('status_update')) {
 
-        $fbml .= '<form method="post" id="facebook_settings">'
+            common_element_start('form', array('method' => 'post',
-        .'<h2>Sync preferences</h2>'
+                                               'id' => 'facebook_settings'));
-        .'<p>';
+
+            common_element('h2', null, _('Sync preferences'));
+
+            common_checkbox('noticesync', _('Automatically update my Facebook status with my notices.'),
+                                ($flink) ? ($flink->noticesync & FOREIGN_NOTICE_SEND) : true);
+
+            common_checkbox('replysync', _('Send local "@" replies to Facebook.'),
+                             ($flink) ? ($flink->noticesync & FOREIGN_NOTICE_SEND_REPLY) : true);
+
+            common_submit('save', _('Save'));
+
+            common_element_end('form');
 
-        if ($flink->noticesync & FOREIGN_NOTICE_SEND) {
-            $fbml .= '<input name="noticesync" type="checkbox" class="checkbox" id="noticesync" checked="checked"/>';
-        } else {
-            $fbml .= '<input name="noticesync" type="checkbox" class="checkbox" id="noticesync">';
         }
 
-        $fbml .= '<label class="checkbox_label" for="noticesync">Automatically update my Facebook status with my notices.</label>'
-        .'</p>'
-        .'<p>';
-
-        if ($flink->noticesync & FOREIGN_NOTICE_SEND_REPLY) {
-            $fbml .= '<input name="replysync" type="checkbox" class="checkbox" id="replysync" checked="checked"/>';
-        } else {
-            $fbml .= '<input name="replysync" type="checkbox" class="checkbox" id="replysync"/>';
-        }
-
-        $fbml .= '<label class="checkbox_label" for="replysync">Send &quot;@&quot; replies to Facebook.</label>'
-        .'</p>'
-        .'<p>'
-        .'<input type="submit" id="save" name="save" class="submit" value="Save"/>'
-        .'</p>'
-        .'</form>';
-
-    }
-
-        echo $fbml;
-
         $this->show_footer();
     }
 

lib/facebookaction.php

@@ -21,7 +21,6 @@ if (!defined('LACONICA')) { exit(1); }
 
 require_once(INSTALLDIR.'/lib/facebookutil.php');
 
-
 class FacebookAction extends Action
 {
 
@@ -85,205 +84,96 @@ class FacebookAction extends Action
 
     # Display methods
 
-    function show_header($selected ='Home')
+    function show_header($selected = 'Home', $msg = null, $success = false)
     {
 
+        start_fbml();
+
         # Add a timestamp to the CSS file so Facebook cache wont ignore our changes
         $ts = filemtime(theme_file('facebookapp.css'));
         $cssurl = theme_path('facebookapp.css') . "?ts=$ts";
 
-         $header = '<link rel="stylesheet" type="text/css" href="'. $cssurl . '" />';
+        common_element('link', array('rel' => 'stylesheet',
-         # $header .='<script src="" ></script>';
+                                     'type' => 'text/css',
-          $header .= '<fb:dashboard/>';
+                                     'href' => $cssurl));
 
-          $header .=
+        common_element('fb:dashboard');
-            '<fb:tabs>'
-            .'<fb:tab-item title="Home" href="index.php" selected="' . ($selected == 'Home') .'" />'
-            .'<fb:tab-item title="Invite Friends"  href="invite.php" selected="' . ($selected == 'Invite') . '" />'
-            .'<fb:tab-item title="Settings"     href="settings.php" selected="' . ($selected == 'Settings') . '" />'
-            .'</fb:tabs>';
-          $header .= '<div id="main_body">';
 
-      echo $header;
+        common_element_start('fb:tabs');
+        common_element('fb:tab-item', array('title' => 'Home',
+                                            'href' => 'index.php',
+                                            'selected' => ($selected == 'Home')));
+        common_element('fb:tab-item', array('title' => 'Invite Friends',
+                                            'href' => 'invite.php',
+                                            'selected' => ($selected == 'Invite')));
+        common_element('fb:tab-item', array('title' => 'Settings',
+                                            'href' => 'settings.php',
+                                            'selected' => ($selected == 'Settings')));
+        common_element_end('fb:tabs');
+
+
+        if ($msg) {
+            if ($success) {
+                common_element('fb:success', array('message' => $msg));
+            } else {
+                // XXX do an error message here
+            }
+        }
+
+        common_element_start('div', 'main_body');
 
     }
 
     function show_footer()
     {
-      $footer = '</div>';
+        common_element_end('div');
-      echo $footer;
+        common_end_xml();
     }
 
-    function show_login_form()
+    function showLoginForm($msg = null)
     {
+        start_fbml();
 
-        $loginform =
+        common_element_start('a', array('href' => 'http://identi.ca'));
-            ' <h2>To add the Identi.ca application, you need to log into your Identi.ca account.</h2>'
+        common_element('img', array('src' => 'http://theme.identi.ca/identica/logo.png',
-            .'<a href="http://identi.ca/">'
+                                    'alt' => 'Identi.ca',
-            .'    <img src="http://theme.identi.ca/identica/logo.png" alt="Identi.ca" id="logo"/>'
+                                    'id' => 'logo'));
-            .'</a>'
+        common_element_end('a');
-            .'<h1 class="pagetitle">Login</h1>'
+
-            .'<div class="instructions">'
+        if ($msg) {
-            .'    <p>Login with your username and password. Don\'t have a username yet?'
+             common_element('fb:error', array('message' => $msg));
-            .'      <a href="http://identi.ca/main/register">Register</a> a new account.'
+        }
-            .'    </p>'
+
-            .'</div>'
+        common_element("h2", null,
-            .'<div id="content">'
+            _('To add the Identi.ca application, you need to log into your Identi.ca account.'));
-            .'    <form method="post" id="login">'
+
-            .'      <p>'
+
-            .'        <label for="nickname">Nickname</label>'
+        common_element_start('div', array('class' => 'instructions'));
-            .'        <input name="nickname" type="text" class="input_text" id="nickname"/>'
+        common_element_start('p');
-            .'      </p>'
+        common_raw('Login with your username and password. Don\'t have a username yet?'
-            .'      <p>'
+        .' <a href="http://identi.ca/main/register">Register</a> a new account.');
-            .'          <label for="password">Password</label>'
+        common_element_end('p');
-            .'        <input name="password" type="password" class="password" id="password"/>'
+        common_element_end('div');
-            .'      </p>'
+
-            .'      <p>'
+        common_element_start('div', array('id' => 'content'));
-            .'        <input type="submit" id="submit" name="submit" class="submit" value="Login"/>'
+        common_element_start('form', array('method' => 'post',
-            .'      </p>'
+                                               'id' => 'login',
-            .'    </form>'
+                                               'action' => 'index.php'));
-            .'    <p>'
+        common_input('nickname', _('Nickname'));
-            .'      <a href="http://identi.ca/main/recoverpassword">Lost or forgotten password?</a>'
+        common_password('password', _('Password'));
-            .'    </p>'
+
-            .'</div';
+        common_submit('submit', _('Login'));
+        common_element_end('form');
+
+        common_element_start('p');
+        common_element('a', array('href' => common_local_url('recoverpassword')),
+                       _('Lost or forgotten password?'));
+        common_element_end('p');
+        common_element_end('div');
+
+        common_end_xml();
 
-            echo $loginform;
     }
 
-    function render_notice($notice)
-    {
-
-        global $config;
-
-        $profile = $notice->getProfile();
-        $avatar = $profile->getAvatar(AVATAR_STREAM_SIZE);
-
-        $noticeurl = common_local_url('shownotice', array('notice' => $notice->id));
-
-        # XXX: we need to figure this out better. Is this right?
-        if (strcmp($notice->uri, $noticeurl) != 0 && preg_match('/^http/', $notice->uri)) {
-            $noticeurl = $notice->uri;
-        }
-
-        $html =
-        '<li class="notice_single" id="' . $notice->id . '">'
-        .'<a href="' . $profile->profileurl . '">'
-        .'<img src="';
-
-        if ($avatar) {
-            $html .= common_avatar_display_url($avatar);
-        } else {
-            $html .= common_default_avatar(AVATAR_STREAM_SIZE);
-        }
-
-        $html .=
-        '" class="avatar stream" width="'
-        . AVATAR_STREAM_SIZE . '" height="' . AVATAR_STREAM_SIZE .'"'
-        .' alt="';
-
-        if ($profile->fullname) {
-            $html .= $profile->fullname;
-        } else {
-            $html .= $profile->nickname;
-        }
-
-        $html .=
-        '"></a>'
-        .'<a href="' .    $profile->profileurl . '" class="nickname">' . $profile->nickname . '</a>'
-        .'<p class="content">' . $notice->rendered . '</p>'
-        .'<p class="time">'
-        .'<a class="permalink" href="' . $noticeurl . '" title="' . common_exact_date($notice->created) . '">' . common_date_string($notice->created) . '</a>';
-
-        if ($notice->source) {
-            $html .= _(' from ');
-            $html .= $this->source_link($notice->source);
-        }
-
-        if ($notice->reply_to) {
-            $replyurl = common_local_url('shownotice', array('notice' => $notice->reply_to));
-            $html .=
-            ' (<a class="inreplyto" href="' . $replyurl . '">' . _('in reply to...') . ')';
-        }
-
-        $html .= '</p></li>';
-
-        return $html;
-    }
-
-    function source_link($source)
-    {
-        $source_name = _($source);
-
-        $html = '<span class="noticesource">';
-
-        switch ($source) {
-         case 'web':
-         case 'xmpp':
-         case 'mail':
-         case 'omb':
-         case 'api':
-            $html .= $source_name;
-            break;
-         default:
-            $ns = Notice_source::staticGet($source);
-            if ($ns) {
-                $html .= '<a href="' . $ns->url . '">' . $ns->name . '</a>';
-            } else {
-                $html .= $source_name;
-            }
-            break;
-        }
-
-        $html .= '</span>';
-
-        return $html;
-    }
-
-    function pagination($have_before, $have_after, $page, $fbaction, $args=null)
-    {
-
-        $html = '';
-
-        if ($have_before || $have_after) {
-            $html = '<div id="pagination">';
-            $html .'<ul id="nav_pagination">';
-        }
-
-        if ($have_before) {
-            $pargs = array('page' => $page-1);
-            $newargs = ($args) ? array_merge($args,$pargs) : $pargs;
-            $html .= '<li class="before">';
-            $html .'<a href="' . $this->pagination_url($fbaction, $newargs) . '">' . _('« After') . '</a>';
-            $html .'</li>';
-        }
-
-        if ($have_after) {
-            $pargs = array('page' => $page+1);
-            $newargs = ($args) ? array_merge($args,$pargs) : $pargs;
-            $html .= '<li class="after">';
-            $html .'<a href="' . $this->pagination_url($fbaction, $newargs) . '">' . _('Before »') . '</a>';
-            $html .'</li>';
-        }
-
-        if ($have_before || $have_after) {
-            $html .= '<ul>';
-            $html .'<div>';
-        }
-    }
-
-    function pagination_url($fbaction, $args=null)
-    {
-        global $config;
-
-        $extra = '';
-
-        if ($args) {
-            foreach ($args as $key => $value) {
-                $extra .= "&${key}=${value}";
-            }
-        }
-
-        return "$fbaction?${extra}";
-    }
 
 }

lib/facebookutil.php

@@ -18,6 +18,9 @@
  */
 
 require_once(INSTALLDIR.'/extlib/facebook/facebook.php');
+require_once(INSTALLDIR.'/lib/noticelist.php');
+
+define("FACEBOOK_SERVICE", 2); // Facebook is foreign_service ID 2
 
 // Gets all the notices from users with a Facebook link since a given ID
 function get_facebook_notices($since)
@@ -38,3 +41,11 @@ function get_facebook()
     $secret = common_config('facebook', 'secret');
     return new Facebook($apikey, $secret);
 }
+
+function start_fbml($indent = true)
+{
+    global $xw;
+    $xw = new XMLWriter();
+    $xw->openURI('php://output');
+    $xw->setIndent($indent);
+}