Merge branch 'master' of git@gitorious.org:statusnet/mainline

4 files changed, 52 insertions, 59 deletions

diff --git a/actions/twitapifriendships.php b/actions/twitapifriendships.php
index f2ea46910..eea8945c3 100644
--- a/actions/twitapifriendships.php
+++ b/actions/twitapifriendships.php
@@ -99,6 +99,12 @@ class TwitapifriendshipsAction extends TwitterapiAction
         $other = $this->get_profile($id);
         $user = $apidata['user']; // Alwyas the auth user
 
+	if ($user->id == $other->id) {
+	    $this->clientError(_("You cannot unfollow yourself!"),
+			       403, $apidata['content-type']);
+	    return;
+	}
+
         $sub = new Subscription();
         $sub->subscriber = $user->id;
         $sub->subscribed = $other->id;
diff --git a/actions/twittersettings.php b/actions/twittersettings.php
index 563d867a4..89169941e 100644
--- a/actions/twittersettings.php
+++ b/actions/twittersettings.php
@@ -165,7 +165,7 @@ class TwittersettingsAction extends ConnectSettingsAction
                                 ($flink->noticesync & FOREIGN_NOTICE_RECV) :
                                 false);
                 $this->elementEnd('li');
-
+            } else {
                 // preserve setting even if bidrection bridge toggled off
 
                 if ($flink && ($flink->noticesync & FOREIGN_NOTICE_RECV)) {
diff --git a/db/notice_source.sql b/db/notice_source.sql
index f590d1b97..2657763f4 100644
--- a/db/notice_source.sql
+++ b/db/notice_source.sql
@@ -21,6 +21,7 @@ VALUES
     ('identichat','identichat','http://identichat.prosody.im/', now()),
     ('IdentiFox','IdentiFox','http://www.bitbucket.org/uncryptic/identifox/', now()),
     ('identitwitch','IdentiTwitch','http://richfish.org/identitwitch/', now()),
+    ('Jiminy','Jiminy','http://code.google.com/p/jiminy/', now()),
     ('LaTwit','LaTwit','http://latwit.mac65.com/', now()),
     ('LiveTweeter', 'LiveTweeter', 'http://addons.songbirdnest.com/addon/1204', now()),
     ('livetweeter', 'livetweeter', 'http://addons.songbirdnest.com/addon/1204', now()),
diff --git a/lib/twitter.php b/lib/twitter.php
index b734d22d8..676c9b20a 100644
--- a/lib/twitter.php
+++ b/lib/twitter.php
@@ -160,6 +160,8 @@ function broadcast_twitter($notice)
             return broadcast_basicauth($notice, $flink);
         }
     }
+
+    return true;
 }
 
 function broadcast_oauth($notice, $flink) {
@@ -173,34 +175,7 @@ function broadcast_oauth($notice, $flink) {
     try {
         $status = $client->statusesUpdate($statustxt);
     } catch (OAuthClientCurlException $e) {
-
-        if ($e->getMessage() == 'The requested URL returned error: 401') {
-
-            $errmsg = sprintf('User %1$s (user id: %2$s) has an invalid ' .
-                              'Twitter OAuth access token.',
-                              $user->nickname, $user->id);
-            common_log(LOG_WARNING, $errmsg);
-
-            // Bad auth token! We need to delete the foreign_link
-            // to Twitter and inform the user.
-
-            remove_twitter_link($flink);
-            return true;
-
-        } else {
-
-            // Some other error happened, so we should probably
-            // try to send again later.
-
-            $errmsg = sprintf('cURL error trying to send notice to Twitter ' .
-                              'for user %1$s (user id: %2$s) - ' .
-                              'code: %3$s message: $4$s.',
-                              $user->nickname, $user->id,
-                              $e->getCode(), $e->getMessage());
-            common_log(LOG_WARNING, $errmsg);
-
-            return false;
-        }
+        return process_error($e, $flink);
     }
 
     if (empty($status)) {
@@ -208,9 +183,9 @@ function broadcast_oauth($notice, $flink) {
         // This could represent a failure posting,
         // or the Twitter API might just be behaving flakey.
 
-        $errmsg = sprintf('No data returned by Twitter API when ' .
-                         'trying to send update for %1$s (user id %2$s).',
-                         $user->nickname, $user->id);
+        $errmsg = sprintf('Twitter bridge - No data returned by Twitter API when ' .
+                          'trying to send update for %1$s (user id %2$s).',
+                          $user->nickname, $user->id);
         common_log(LOG_WARNING, $errmsg);
 
         return false;
@@ -218,7 +193,7 @@ function broadcast_oauth($notice, $flink) {
 
     // Notice crossed the great divide
 
-    $msg = sprintf('Twitter bridge posted notice %s to Twitter using OAuth.',
+    $msg = sprintf('Twitter bridge - posted notice %s to Twitter using OAuth.',
                    $notice->id);
     common_log(LOG_INFO, $msg);
 
@@ -237,46 +212,57 @@ function broadcast_basicauth($notice, $flink)
     try {
         $status = $client->statusesUpdate($statustxt);
     } catch (BasicAuthCurlException $e) {
-
-        if ($e->getMessage() == 'The requested URL returned error: 401') {
-
-            $errmsg = sprintf('User %1$s (user id: %2$s) has an invalid ' .
-                              'Twitter screen_name/password combo.',
-                              $user->nickname, $user->id);
-            common_log(LOG_WARNING, $errmsg);
-
-            remove_twitter_link($flink);
-            return true;
-
-        } else {
-
-            $errmsg = sprintf('cURL error trying to send notice to Twitter ' .
-                              'for user %1$s (user id: %2$s) - ' .
-                              'code: %3$s message: $4$s.',
-                              $user->nickname, $user->id,
-                              $e->getCode(), $e->getMessage());
-            common_log(LOG_WARNING, $errmsg);
-
-            return false;
-        }
+        return process_error($e, $flink);
     }
 
     if (empty($status)) {
 
-        $errmsg = sprintf('No data returned by Twitter API when ' .
-                         'trying to send update for %1$s (user id %2$s).',
-                         $user->nickname, $user->id);
+        $errmsg = sprintf('Twitter bridge - No data returned by Twitter API when ' .
+                          'trying to send update for %1$s (user id %2$s).',
+                          $user->nickname, $user->id);
         common_log(LOG_WARNING, $errmsg);
 
         return false;
     }
 
-    $msg = sprintf('Twitter bridge posted notice %s to Twitter using basic auth.',
+    $msg = sprintf('Twitter bridge - posted notice %s to Twitter using basic auth.',
                    $notice->id);
     common_log(LOG_INFO, $msg);
 
     return true;
+}
+
+function process_error($e, $flink)
+{
+    $user        = $flink->getUser();
+    $errmsg      = $e->getMessage();
+    $delivered   = false;
+
+    switch($errmsg) {
+     case 'The requested URL returned error: 401':
+        $logmsg = sprintf('Twiter bridge - User %1$s (user id: %2$s) has an invalid ' .
+                          'Twitter screen_name/password combo or an invalid acesss token.',
+                          $user->nickname, $user->id);
+        $delivered = true;
+        remove_twitter_link($flink);
+        break;
+     case 'The requested URL returned error: 403':
+        $logmsg = sprintf('Twitter bridge - User %1$s (user id: %2$s) has exceeded ' .
+                          'his/her Twitter request limit.',
+                          $user->nickname, $user->id);
+        break;
+     default:
+        $logmsg = sprintf('Twitter bridge - cURL error trying to send notice to Twitter ' .
+                          'for user %1$s (user id: %2$s) - ' .
+                          'code: %3$s message: %4$s.',
+                          $user->nickname, $user->id,
+                          $e->getCode(), $e->getMessage());
+        break;
+    }
+
+    common_log(LOG_WARNING, $logmsg);
 
+    return $delivered;
 }
 
 function format_status($notice)