X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;ds=sidebyside;f=plugins%2FOStatus%2Flib%2Fostatusqueuehandler.php;h=a4d9e527e2afed5459ba8120277f4e7b0b77298d;hb=c18020561b8a01bbb2b3fc092694a7bb0fde70f9;hp=964580cf49a208b8e2129857f8cc20a78587d468;hpb=d4e76bc25b5bdcb47d8d267fcb7b9f3b9f0b9585;p=quix0rs-gnu-social.git

diff --git a/plugins/OStatus/lib/ostatusqueuehandler.php b/plugins/OStatus/lib/ostatusqueuehandler.php
index 964580cf49..a4d9e527e2 100644
--- a/plugins/OStatus/lib/ostatusqueuehandler.php
+++ b/plugins/OStatus/lib/ostatusqueuehandler.php
@@ -66,40 +66,57 @@ class OStatusQueueHandler extends QueueHandler
             $this->pushUser();
         }
 
-        foreach ($notice->getGroups() as $group) {
-            $oprofile = Ostatus_profile::getKV('group_id', $group->id);
-            if ($oprofile) {
-                // remote group
-                if ($notice->isLocal()) {
-                    $this->pingReply($oprofile);
+        foreach ($notice->getAttentionProfiles() as $target) {
+            common_debug("OSTATUS [{$this->notice->getID()}]: Attention target profile {$target->getNickname()} ({$target->getID()})");
+            if ($target->isGroup()) {
+                common_debug("OSTATUS [{$this->notice->getID()}]: {$target->getID()} is a group");
+                $oprofile = Ostatus_profile::getKV('group_id', $target->getGroup()->getID());
+                if (!$oprofile instanceof Ostatus_profile) {
+                    // we don't save profiles like this yet, but in the future
+                    $oprofile = Ostatus_profile::getKV('profile_id', $target->getID());
                 }
-            } else {
-                // local group
-                $this->pushGroup($group->id);
-            }
-        }
-
-        if ($notice->isLocal()) {
-            // Notices generated on other sites will have already
-            // pinged their reply-targets.
+                if ($oprofile instanceof Ostatus_profile) {
+                    // remote group
+                    if ($notice->isLocal()) {
+                        common_debug("OSTATUS [{$this->notice->getID()}]: notice is local and remote group with profile ID {$target->getID()} gets a ping");
+                        $this->pingReply($oprofile);
+                    }
+                } else {
+                    common_debug("OSTATUS [{$this->notice->getID()}]: local group with profile id {$target->getID()} gets pushed out");
+                    // local group
+                    $this->pushGroup($target->getGroup());
+                }
+            } elseif ($notice->isLocal()) {
+                // Notices generated on other sites will have already
+                // pinged their reply-targets, so only do these things
+                // if the target is not a group and the notice is locally generated
 
-            foreach ($notice->getReplies() as $profile_id) {
-                $oprofile = Ostatus_profile::getKV('profile_id', $profile_id);
-                if ($oprofile) {
+                $oprofile = Ostatus_profile::getKV('profile_id', $target->getID());
+                if ($oprofile instanceof Ostatus_profile) {
+                    common_debug("OSTATUS [{$this->notice->getID()}]: Notice is local and {$target->getID()} is remote profile, getting pingReply");
                     $this->pingReply($oprofile);
                 }
             }
+        }
 
-            if (!empty($this->notice->reply_to)) {
-                $replyTo = Notice::getKV('id', $this->notice->reply_to);
-                if (!empty($replyTo)) {
-                    foreach($replyTo->getReplies() as $profile_id) {
-                        $oprofile = Ostatus_profile::getKV('profile_id', $profile_id);
-                        if ($oprofile) {
-                            $this->pingReply($oprofile);
-                        }
+        if ($notice->isLocal()) {
+            try {
+                $parent = $this->notice->getParent();
+                foreach($parent->getAttentionProfiles() as $related) {
+                    if ($related->isGroup()) {
+                        // don't ping groups in parent notices since we might not be a member of them,
+                        // though it could be useful if we study this and use it correctly
+                        continue;
+                    }
+                    common_debug("OSTATUS [{$this->notice->getID()}]: parent notice {$parent->getID()} has related profile id=={$related->getID()}");
+                    // FIXME: don't ping twice in case someone is in both notice attention spans!
+                    $oprofile = Ostatus_profile::getKV('profile_id', $related->getID());
+                    if ($oprofile instanceof Ostatus_profile) {
+                        $this->pingReply($oprofile);
                     }
                 }
+            } catch (NoParentNoticeException $e) {
+                // nothing to do then
             }
 
             foreach ($notice->getProfileTags() as $ptag) {
@@ -116,6 +133,7 @@ class OStatusQueueHandler extends QueueHandler
     function pushUser()
     {
         if ($this->user) {
+            common_debug("OSTATUS [{$this->notice->getID()}]: pushing feed for local user {$this->user->getID()}");
             // For local posts, ping the PuSH hub to update their feed.
             // http://identi.ca/api/statuses/user_timeline/1.atom
             $feed = common_local_url('ApiTimelineUser',
@@ -125,18 +143,20 @@ class OStatusQueueHandler extends QueueHandler
         }
     }
 
-    function pushGroup($group_id)
+    function pushGroup(User_group $group)
     {
+        common_debug("OSTATUS [{$this->notice->getID()}]: pushing group '{$group->getNickname()}' profile_id={$group->profile_id}");
         // For a local group, ping the PuSH hub to update its feed.
         // Updates may come from either a local or a remote user.
         $feed = common_local_url('ApiTimelineGroup',
-                                 array('id' => $group_id,
+                                 array('id' => $group->getID(),
                                        'format' => 'atom'));
-        $this->pushFeed($feed, array($this, 'groupFeedForNotice'), $group_id);
+        $this->pushFeed($feed, array($this, 'groupFeedForNotice'), $group->getID());
     }
 
     function pushPeopletag($ptag)
     {
+        common_debug("OSTATUS [{$this->notice->getID()}]: pushing peopletag '{$ptag->id}'");
         // For a local people tag, ping the PuSH hub to update its feed.
         // Updates may come from either a local or a remote user.
         $feed = common_local_url('ApiTimelineList',
@@ -146,9 +166,10 @@ class OStatusQueueHandler extends QueueHandler
         $this->pushFeed($feed, array($this, 'peopletagFeedForNotice'), $ptag);
     }
 
-    function pingReply(OStatus_profile $oprofile)
+    function pingReply(Ostatus_profile $oprofile)
     {
         if ($this->user) {
+            common_debug("OSTATUS [{$this->notice->getID()}]: pinging reply to {$oprofile->localProfile()->getNickname()} for local user '{$this->user->getID()}'");
             // For local posts, send a Salmon ping to the mentioned
             // remote user or group.
             // @fixme as an optimization we can skip this if the
@@ -236,7 +257,7 @@ class OStatusQueueHandler extends QueueHandler
                 }
             }
         }
-        if (count($batch) >= 0) {
+        if (count($batch) > 0) {
             $sub->bulkDistribute($atom, $batch);
         }
     }