Merge branch '0.9.x' into 1.0.x

diff --git a/EVENTS.txt b/EVENTS.txt
index 57f2bf6bccb1922298adae1bf89954e113184f04..9d77c4f38dc49b46d9bc3887a53bd72db7f909e1 100644 (file)
--- a/EVENTS.txt
+++ b/EVENTS.txt
@@ -1017,3 +1017,27 @@ StartAdminPanelNav: Before displaying the first item in the list of admin panels
 
 EndAdminPanelNav: After displaying the last item in the list of admin panels
 - $nav The AdminPanelNav widget
+
+StartActivityObjectFromNotice: When converting a notice to an activity:object
+- $notice: The notice being converted
+- &$object: The resulting object. Fill this and return false to override defaults.
+
+EndActivityObjectFromNotice: After converting a notice to an activity:object
+- $notice: The notice being converted
+- &$object: The resulting object. Can be edited
+
+StartActivityObjectFromProfile: When converting a profile to an activity:object
+- $profile: The profile being converted
+- &$object: The (empty) object. Fill it up and return false to override defaults.
+
+EndActivityObjectFromProfile: After converting a profile to an activity:object
+- $profile: The profile being converted
+- &$object: The finished object. Can be tweaked
+
+StartActivityObjectFromGroup: When converting a group to an activity:object
+- $group: The group being converted
+- &$object: The (empty) object. Fill and return false to override.
+
+EndActivityObjectFromGroup:  After converting a group to an activity:object
+- $group:  The group being converted
+- &$object: The finished object. Tweak as needed.

diff --git a/lib/activity.php b/lib/activity.php
index c3a984a7b994f289995dccf521ee0e4eb8b83fda..3458c76b9a6d3ef61f33659abde74011d226a61d 100644 (file)
--- a/lib/activity.php
+++ b/lib/activity.php
@@ -349,32 +349,7 @@ class Activity
         if ($this->verb == ActivityVerb::POST && count($this->objects) == 1) {
 
             $obj = $this->objects[0];
-
-            $xs->element('id', null, $obj->id);
-            $xs->element('activity:object-type', null, $obj->type);
-            
-            if (!empty($obj->title)) {
-                $xs->element('title', null, $obj->title);
-            } else {
-                // XXX need a better default title
-                $xs->element('title', null, _('Post'));
-            }
-
-            if (!empty($obj->content)) {
-                $xs->element('content', array('type' => 'html'), $obj->content);
-            }
-            
-            if (!empty($obj->summary)) {
-                $xs->element('summary', null, $obj->summary);
-            }
-            
-            if (!empty($obj->link)) {
-                $xs->element('link', array('rel' => 'alternate',
-                                           'type' => 'text/html'),
-                             $obj->link);
-            }
-
-            // XXX: some object types might have other values here.
+                       $obj->outputTo($xs, null);
 
         } else {
             $xs->element('id', null, $this->id);
@@ -408,12 +383,12 @@ class Activity
                 $xs->element('name', array(), $this->actor->title);
             }
             $xs->elementEnd('author');
-            $xs->raw($this->actor->asString('activity:actor'));
+            $this->actor->outputTo($xs, 'activity:actor');
         }
 
         if ($this->verb != ActivityVerb::POST || count($this->objects) != 1) {
             foreach($this->objects as $object) {
-                $xs->raw($object->asString());
+                $object->outputTo($xs, 'activity:object');
             }
         }
 
@@ -467,7 +442,7 @@ class Activity
         }
 
         if ($this->target) {
-            $xs->raw($this->target->asString('activity:target'));
+            $this->target->outputTo($xs, 'activity:target');
         }
 
         foreach ($this->categories as $cat) {

diff --git a/lib/activityobject.php b/lib/activityobject.php
index e89c8db4e9afee4e611a0e54d41ac150b9d14fc5..61614935f1fa0ecb91f865a91f3a56e1b6b31638 100644 (file)
--- a/lib/activityobject.php
+++ b/lib/activityobject.php
@@ -381,13 +381,18 @@ class ActivityObject
     static function fromNotice(Notice $notice)
     {
         $object = new ActivityObject();
+               
+               if (Event::handle('StartActivityObjectFromNotice', array($notice, &$object))) {
 
-        $object->type    = ActivityObject::NOTE;
+                       $object->type    = ActivityObject::NOTE;
 
-        $object->id      = $notice->uri;
-        $object->title   = $notice->content;
-        $object->content = $notice->rendered;
-        $object->link    = $notice->bestUrl();
+                       $object->id      = $notice->uri;
+                       $object->title   = $notice->content;
+                       $object->content = $notice->rendered;
+                       $object->link    = $notice->bestUrl();
+
+                       Event::handle('EndActivityObjectFromNotice', array($notice, &$object));
+               }
 
         return $object;
     }
@@ -396,57 +401,62 @@ class ActivityObject
     {
         $object = new ActivityObject();
 
-        $object->type   = ActivityObject::PERSON;
-        $object->id     = $profile->getUri();
-        $object->title  = $profile->getBestName();
-        $object->link   = $profile->profileurl;
-
-        $orig = $profile->getOriginalAvatar();
-
-        if (!empty($orig)) {
-            $object->avatarLinks[] = AvatarLink::fromAvatar($orig);
-        }
+               if (Event::handle('StartActivityObjectFromProfile', array($profile, &$object))) {
 
-        $sizes = array(
-            AVATAR_PROFILE_SIZE,
-            AVATAR_STREAM_SIZE,
-            AVATAR_MINI_SIZE
-        );
+                       $object->type   = ActivityObject::PERSON;
+                       $object->id     = $profile->getUri();
+                       $object->title  = $profile->getBestName();
+                       $object->link   = $profile->profileurl;
 
-        foreach ($sizes as $size) {
-            $alink  = null;
-            $avatar = $profile->getAvatar($size);
+                       $orig = $profile->getOriginalAvatar();
 
-            if (!empty($avatar)) {
-                $alink = AvatarLink::fromAvatar($avatar);
-            } else {
-                $alink = new AvatarLink();
-                $alink->type   = 'image/png';
-                $alink->height = $size;
-                $alink->width  = $size;
-                $alink->url    = Avatar::defaultImage($size);
-
-                if ($size == AVATAR_PROFILE_SIZE) {
-                    // Hack for Twitter import: we don't have a 96x96 image,
-                    // but we do have a 73x73 image. For now, fake it with that.
-                    $avatar = $profile->getAvatar(73);
-                    if ($avatar) {
-                        $alink = AvatarLink::fromAvatar($avatar);
-                        $alink->height= $size;
-                        $alink->width = $size;
-                    }
-                }
-            }
+                       if (!empty($orig)) {
+                               $object->avatarLinks[] = AvatarLink::fromAvatar($orig);
+                       }
 
-            $object->avatarLinks[] = $alink;
-        }
-
-        if (isset($profile->lat) && isset($profile->lon)) {
-            $object->geopoint = (float)$profile->lat
-                . ' ' . (float)$profile->lon;
-        }
+                       $sizes = array(
+                AVATAR_PROFILE_SIZE,
+                AVATAR_STREAM_SIZE,
+                AVATAR_MINI_SIZE
+            );
 
-        $object->poco = PoCo::fromProfile($profile);
+                       foreach ($sizes as $size) {
+                               $alink  = null;
+                               $avatar = $profile->getAvatar($size);
+
+                               if (!empty($avatar)) {
+                                       $alink = AvatarLink::fromAvatar($avatar);
+                               } else {
+                                       $alink = new AvatarLink();
+                                       $alink->type   = 'image/png';
+                                       $alink->height = $size;
+                                       $alink->width  = $size;
+                                       $alink->url    = Avatar::defaultImage($size);
+
+                                       if ($size == AVATAR_PROFILE_SIZE) {
+                                               // Hack for Twitter import: we don't have a 96x96 image,
+                                               // but we do have a 73x73 image. For now, fake it with that.
+                                               $avatar = $profile->getAvatar(73);
+                                               if ($avatar) {
+                                                       $alink = AvatarLink::fromAvatar($avatar);
+                                                       $alink->height= $size;
+                                                       $alink->width = $size;
+                                               }
+                                       }
+                               }
+
+                               $object->avatarLinks[] = $alink;
+                       }
+
+                       if (isset($profile->lat) && isset($profile->lon)) {
+                               $object->geopoint = (float)$profile->lat
+                                       . ' ' . (float)$profile->lon;
+                       }
+
+                       $object->poco = PoCo::fromProfile($profile);
+
+                       Event::handle('EndActivityObjectFromProfile', array($profile, &$object));
+               }
 
         return $object;
     }
@@ -455,43 +465,42 @@ class ActivityObject
     {
         $object = new ActivityObject();
 
-        $object->type   = ActivityObject::GROUP;
-        $object->id     = $group->getUri();
-        $object->title  = $group->getBestName();
-        $object->link   = $group->getUri();
+               if (Event::handle('StartActivityObjectFromGroup', array($group, &$object))) {
 
-        $object->avatarLinks[] = AvatarLink::fromFilename(
-            $group->homepage_logo,
-            AVATAR_PROFILE_SIZE
-        );
+                       $object->type   = ActivityObject::GROUP;
+                       $object->id     = $group->getUri();
+                       $object->title  = $group->getBestName();
+                       $object->link   = $group->getUri();
 
-        $object->avatarLinks[] = AvatarLink::fromFilename(
-            $group->stream_logo,
-            AVATAR_STREAM_SIZE
-        );
+                       $object->avatarLinks[] = AvatarLink::fromFilename($group->homepage_logo,
+                                                                                                                         AVATAR_PROFILE_SIZE);
 
-        $object->avatarLinks[] = AvatarLink::fromFilename(
-            $group->mini_logo,
-            AVATAR_MINI_SIZE
-        );
+                       $object->avatarLinks[] = AvatarLink::fromFilename($group->stream_logo,
+                                                                                                                         AVATAR_STREAM_SIZE);
 
-        $object->poco = PoCo::fromGroup($group);
+                       $object->avatarLinks[] = AvatarLink::fromFilename($group->mini_logo,
+                                                                                                                         AVATAR_MINI_SIZE);
 
-        return $object;
-    }
+                       $object->poco = PoCo::fromGroup($group);
 
-    function asString($tag='activity:object')
-    {
-        $xs = new XMLStringer(true);
+                       Event::handle('EndActivityObjectFromGroup', array($group, &$object));
+               }
 
-        $xs->elementStart($tag);
+        return $object;
+    }
+       
+       function outputTo($xo, $tag='activity:object')
+       {
+               if (!empty($tag)) {
+                       $xo->elementStart($tag);
+               }
 
-        $xs->element('activity:object-type', null, $this->type);
+        $xo->element('activity:object-type', null, $this->type);
 
-        $xs->element(self::ID, null, $this->id);
+        $xo->element(self::ID, null, $this->id);
 
         if (!empty($this->title)) {
-            $xs->element(
+            $xo->element(
                 self::TITLE,
                 null,
                 common_xml_safe_str($this->title)
@@ -499,7 +508,7 @@ class ActivityObject
         }
 
         if (!empty($this->summary)) {
-            $xs->element(
+            $xo->element(
                 self::SUMMARY,
                 null,
                 common_xml_safe_str($this->summary)
@@ -508,7 +517,7 @@ class ActivityObject
 
         if (!empty($this->content)) {
             // XXX: assuming HTML content here
-            $xs->element(
+            $xo->element(
                 ActivityUtils::CONTENT,
                 array('type' => 'html'),
                 common_xml_safe_str($this->content)
@@ -516,7 +525,7 @@ class ActivityObject
         }
 
         if (!empty($this->link)) {
-            $xs->element(
+            $xo->element(
                 'link',
                 array(
                     'rel' => 'alternate',
@@ -531,7 +540,7 @@ class ActivityObject
             || $this->type == ActivityObject::GROUP) {
 
             foreach ($this->avatarLinks as $avatar) {
-                $xs->element(
+                $xo->element(
                     'link', array(
                         'rel'  => 'avatar',
                         'type'         => $avatar->type,
@@ -545,7 +554,7 @@ class ActivityObject
         }
 
         if (!empty($this->geopoint)) {
-            $xs->element(
+            $xo->element(
                 'georss:point',
                 null,
                 $this->geopoint
@@ -553,10 +562,26 @@ class ActivityObject
         }
 
         if (!empty($this->poco)) {
-            $xs->raw($this->poco->asString());
+            $xo->raw($this->poco->asString());
         }
 
-        $xs->elementEnd($tag);
+        foreach ($this->extra as $el) {
+            list($extraTag, $attrs, $content) = $el;
+            $xo->element($extraTag, $attrs, $content);
+        }
+
+               if (!empty($tag)) {
+                       $xo->elementEnd($tag);
+               }
+
+        return;
+       }
+
+    function asString($tag='activity:object')
+    {
+        $xs = new XMLStringer(true);
+
+               $this->outputTo($xs, $tag);
 
         return $xs->getString();
     }

diff --git a/plugins/OStatus/classes/FeedSub.php b/plugins/OStatus/classes/FeedSub.php
index 140f323846498c91817d6dc59a20d083dbebd04d..7756f6a234325a5a50e751ce65c4c168bc7cdd7f 100644 (file)
--- a/plugins/OStatus/classes/FeedSub.php
+++ b/plugins/OStatus/classes/FeedSub.php
@@ -487,7 +487,15 @@ class FeedSub extends Memcached_DataObject
                 if ($their_hmac === $our_hmac) {
                     return true;
                 }
-                common_log(LOG_ERR, __METHOD__ . ": ignoring PuSH with bad SHA-1 HMAC: got $their_hmac, expected $our_hmac");
+                if (common_config('feedsub', 'debug')) {
+                    $tempfile = tempnam(sys_get_temp_dir(), 'feedsub-receive');
+                    if ($tempfile) {
+                        file_put_contents($tempfile, $post);
+                    }
+                    common_log(LOG_ERR, __METHOD__ . ": ignoring PuSH with bad SHA-1 HMAC: got $their_hmac, expected $our_hmac for feed $this->uri on $this->huburi; saved to $tempfile");
+                } else {
+                    common_log(LOG_ERR, __METHOD__ . ": ignoring PuSH with bad SHA-1 HMAC: got $their_hmac, expected $our_hmac for feed $this->uri on $this->huburi");
+                }
             } else {
                 common_log(LOG_ERR, __METHOD__ . ": ignoring PuSH with bogus HMAC '$hmac'");
             }