Merged stuff from upstream/master
authorRoland Haeder <roland@mxchange.org>
Sat, 30 Jan 2016 12:38:25 +0000 (13:38 +0100)
committerRoland Haeder <roland@mxchange.org>
Sat, 30 Jan 2016 12:38:25 +0000 (13:38 +0100)
Signed-off-by: Roland Haeder <roland@mxchange.org>
15 files changed:
actions/apiaccountverifycredentials.php
actions/apimediaupload.php
actions/replies.php
classes/File.php
classes/File_redirection.php
classes/Notice.php
lib/router.php
plugins/Blacklist/BlacklistPlugin.php
plugins/Bookmark/classes/Bookmark.php
plugins/Event/classes/RSVP.php
plugins/ForceGroup/ForceGroupPlugin.php
plugins/GNUsocialPhotos/GNUsocialPhotosPlugin.php
plugins/OStatus/classes/Ostatus_profile.php
plugins/OStatus/lib/salmonaction.php
plugins/TwitterBridge/actions/twitterauthorization.php

index 12daf3e..f82060f 100644 (file)
@@ -57,7 +57,7 @@ class ApiAccountVerifyCredentialsAction extends ApiAuthAction
      */
     protected function handle()
     {
-        parent::handle();
+        parent::handle($args);
 
         if (!in_array($this->format, array('xml', 'json'))) {
             // TRANS: Client error displayed when coming across a non-supported API method.
index c2b4a47..86742e0 100644 (file)
@@ -52,7 +52,7 @@ class ApiMediaUploadAction extends ApiAuthAction
      */
     protected function handle()
     {
-        parent::handle();
+        parent::handle($args);
 
         // Workaround for PHP returning empty $_POST and $_FILES when POST
         // length > post_max_size in php.ini
index efe0d62..de6c0da 100644 (file)
@@ -152,7 +152,7 @@ class RepliesAction extends ShowstreamAction
         $this->elementEnd('div');
     }
 
-    public function isReadOnly(array $args = array())
+    public function isReadOnly(array $args=array())
     {
         return true;
     }
index 2390f84..7356554 100644 (file)
@@ -423,6 +423,9 @@ class File extends Managed_DataObject
             }
         }
 
+        // Debug log (convert crop to int to have TRUE being displayed as 1 and FALSE as 0)
+        /* NOISY-DEBUG: */ common_debug('[' . __METHOD__ . ':' . __LINE__ . ']: width=' . $width . ',height=' . $height . ',crop=' . intval($crop));
+
         return $image->getFileThumbnail($width, $height, $crop);
     }
 
index 4afb6d4..1cbf679 100644 (file)
@@ -397,4 +397,4 @@ class File_redirection extends Managed_DataObject
 
         return $this->file;
     }
-}
\ No newline at end of file
+}
index 53832de..2d3ed99 100644 (file)
@@ -3121,5 +3121,15 @@ class Notice extends Managed_DataObject
             print ".";
         }
         print "\n";
+
+    /**
+     * Checks whether this notice is in "private scope" (non-public notice)
+     *
+     * @return $isPrivate Whether this notice is private
+     */
+    public function isPrivateScope ()
+    {
+        return ($this->scope != Notice::SITE_SCOPE &&
+                $this->scope != Notice::PUBLIC_SCOPE);
     }
 }
index 1bbcd82..bf8cb2b 100644 (file)
@@ -182,7 +182,7 @@ class Router
 
             foreach (array('profile', 'avatar', 'password', 'im', 'oauthconnections',
                            'oauthapps', 'email', 'sms', 'url') as $s) {
-                $m->connect('settings/'.$s, array('action' => $s.'settings'));
+                $m->connect('settings/' . $s, array('action' => $s . 'settings'));
             }
 
             if (common_config('oldschool', 'enabled')) {
index 1780f42..d1ee291 100644 (file)
@@ -433,17 +433,17 @@ class BlacklistPlugin extends Plugin
         return $hostname;
     }
 
-    function onStartHandleFeedEntry($activity)
+    function onStartHandleFeedEntry(Activity $activity)
     {
         return $this->_checkActivity($activity);
     }
 
-    function onStartHandleSalmon($activity)
+    function onStartHandleSalmon(Activity $activity)
     {
         return $this->_checkActivity($activity);
     }
 
-    function _checkActivity($activity)
+    function _checkActivity(Activity $activity)
     {
         $actor = $activity->actor;
 
index b593bc1..7fd3f53 100644 (file)
@@ -92,12 +92,16 @@ class Bookmark extends Managed_DataObject
      * @return  Bookmark            The found bookmark object.
      * @throws  NoResultException   When you don't find it after all.
      */
+<<<<<<< .merge_file_ZPs3Af
     static public function fromStored(Notice $stored)
     {
         return self::getByPK(array('uri' => $stored->getUri()));
     }
 
     public function getStored()
+=======
+    static function getByNotice(Notice $notice)
+>>>>>>> .merge_file_eD7Wwf
     {
         return Notice::getByKeys(array('uri' => $this->getUri()));
     }
@@ -157,7 +161,12 @@ class Bookmark extends Managed_DataObject
      *
      * @return Bookmark the Bookmark object
      */
+<<<<<<< .merge_file_ZPs3Af
     static function saveActivityObject(ActivityObject $actobj, Notice $stored)
+=======
+    static function saveNew(Profile $profile, $title, $url, $rawtags, $description,
+                            array $options=array())
+>>>>>>> .merge_file_eD7Wwf
     {
         $url = null;
         // each extra element is array('tagname', array('attr'=>'val', ...), 'content')
@@ -177,11 +186,20 @@ class Bookmark extends Managed_DataObject
             throw new ClientException(sprintf(_m('Expected exactly 1 link rel=related in a Bookmark, got %1$d.'), count($relLinkEls)));
         }
 
+<<<<<<< .merge_file_ZPs3Af
         if (!strlen($actobj->title)) {
             throw new ClientException(_m('You must provide a non-empty title.'));
         }
         if (!common_valid_http_url($url)) {
             throw new ClientException(_m('Only web bookmarks can be posted (HTTP or HTTPS).'));
+=======
+        if (array_key_exists('uri', $options)) {
+            $other = Bookmark::getKV('uri', $options['uri']);
+            if (!empty($other)) {
+                // TRANS: Client exception thrown when trying to save a new bookmark that already exists.
+                throw new ClientException(_m('Bookmark already exists.'));
+            }
+>>>>>>> .merge_file_eD7Wwf
         }
 
         try {
index f6b8c0a..7fa76d7 100644 (file)
@@ -91,7 +91,11 @@ class RSVP extends Managed_DataObject
         );
     }
 
+<<<<<<< .merge_file_oq2F7C
     static public function beforeSchemaUpdate()
+=======
+    function saveNew(Profile $profile, $event, $verb, array $options = array())
+>>>>>>> .merge_file_L3oA6C
     {
         $table = strtolower(get_called_class());
         $schema = Schema::get();
index 56e3335..a27436e 100644 (file)
@@ -49,7 +49,7 @@ class ForceGroupPlugin extends Plugin
      * @param Notice $notice
      * @return boolean event hook return
      */
-    function onStartNoticeDistribute($notice)
+    function onStartNoticeDistribute(Notice $notice)
     {
         $profile = $notice->getProfile();
 
index 906509f..5e0f3f7 100644 (file)
@@ -54,7 +54,7 @@ class GNUsocialPhotosPlugin extends Plugin
         return true;
     }
 
-    function onStartNoticeDistribute($notice)
+    function onStartNoticeDistribute(Notice $notice)
     {
         common_log(LOG_INFO, "event: StartNoticeDistribute");
         if (GNUsocialPhotoTemp::$tmp) {
@@ -84,7 +84,7 @@ class GNUsocialPhotosPlugin extends Plugin
     }
 
 
-    function onStartHandleFeedEntry($activity)
+    function onStartHandleFeedEntry(Activity $activity)
     {
         common_log(LOG_INFO, 'photo plugin: onEndAtomPubNewActivity');
         $oprofile = Ostatus_profile::ensureActorProfile($activity);
@@ -121,7 +121,7 @@ class GNUsocialPhotosPlugin extends Plugin
         return true;
     }
 
-    function onStartShowNoticeItem($action)
+    function onStartShowNoticeItem(Action $action)
     {
         $photo = GNUsocialPhoto::getKV('notice_id', $action->notice->id);
         if($photo) { 
index 07432d7..b4df92e 100644 (file)
@@ -894,10 +894,12 @@ class Ostatus_profile extends Managed_DataObject
         $temp_filename = tempnam(common_get_temp_dir(), 'listener_avatar');
         try {
             $imgData = HTTPClient::quickGet($url);
+
             // Make sure it's at least an image file. ImageFile can do the rest.
             if (false === getimagesizefromstring($imgData)) {
                 throw new UnsupportedMediaException(_('Downloaded group avatar was not an image.'));
             }
+
             file_put_contents($temp_filename, $imgData);
             unset($imgData);    // No need to carry this in memory.
 
index 2f48042..f328f09 100644 (file)
@@ -79,8 +79,17 @@ class SalmonAction extends Action
             $this->clientError($e->getMessage());
         }
 
+<<<<<<< .merge_file_l1W8if
         // Cryptographic verification test, throws exception on failure
         $magic_env->verify($this->actor);
+=======
+        // Cryptographic verification test
+        if (!$magic_env->verify($this->actor)) {
+            common_debug("Salmon signature verification failed.");
+            // TRANS: Client error.
+            $this->clientError(_m('Salmon signature verification failed.'));
+        }
+>>>>>>> .merge_file_ISiK5f
 
         return true;
     }
index c9b892b..d3df46b 100644 (file)
@@ -56,17 +56,50 @@ class TwitterauthorizationAction extends FormAction
     var $access_token = null;
     var $verifier     = null;
 
+<<<<<<< .merge_file_Bq71r4
     protected $needLogin = false;   // authorization page can also be used to create a new user
+=======
+    /**
+     * Initialize class members. Looks for 'oauth_token' parameter.
+     *
+     * @param array $args misc. arguments
+     *
+     * @return boolean true
+     */
+    function prepare(array $args=array())
+    {
+        parent::prepare($args);
+>>>>>>> .merge_file_Ad5nH4
 
     protected function doPreparation()
     {
         $this->oauth_token = $this->arg('oauth_token');
         $this->verifier    = $this->arg('oauth_verifier');
 
+<<<<<<< .merge_file_Bq71r4
         if ($this->scoped instanceof Profile) {
             try {
                 $flink = Foreign_link::getByUserID($this->scoped->getID(), TWITTER_SERVICE);
                 $fuser = $flink->getForeignUser();
+=======
+        return true;
+    }
+
+    /**
+     * Handler method
+     *
+     * @param array $args is ignored since it's now passed in in prepare()
+     *
+     * @return nothing
+     */
+    function handle(array $args=array())
+    {
+        parent::handle($args);
+
+        if (common_logged_in()) {
+            $user  = common_current_user();
+            $flink = Foreign_link::getByUserID($user->id, TWITTER_SERVICE);
+>>>>>>> .merge_file_Ad5nH4
 
                 // If there's already a foreign link record and a foreign user
                 // (no exceptions were thrown when fetching either of them...)