]> git.mxchange.org Git - friendica.git/blobdiff - src/Worker/Notifier.php
Merge pull request #11402 from annando/featured-endpoint
[friendica.git] / src / Worker / Notifier.php
index a53eefa7d17d354440350dea75297a1671dea220..80628d7dba08d08fe5e55a3dabc84886e2a27038 100644 (file)
@@ -1,6 +1,6 @@
 <?php
 /**
- * @copyright Copyright (C) 2010-2021, the Friendica project
+ * @copyright Copyright (C) 2010-2022, the Friendica project
  *
  * @license GNU AGPL version 3 or any later version
  *
@@ -90,7 +90,7 @@ class Notifier
                                        'APDelivery', $cmd, $target_id, $inbox, $uid, $receivers, $post_uriid);
                        }
                } elseif ($cmd == Delivery::SUGGESTION) {
-                       $suggest = DI::fsuggest()->getById($target_id);
+                       $suggest = DI::fsuggest()->selectOneById($target_id);
                        $uid = $suggest->uid;
                        $recipients[] = $suggest->cid;
                } elseif ($cmd == Delivery::REMOVAL) {
@@ -153,7 +153,7 @@ class Notifier
                }
 
                // Should the post be transmitted to Diaspora?
-               $diaspora_delivery = true;
+               $diaspora_delivery = ($owner['account-type'] != User::ACCOUNT_TYPE_COMMUNITY);
 
                // If this is a public conversation, notify the feed hub
                $public_message = true;
@@ -223,10 +223,6 @@ class Notifier
                                $relay_to_owner = true;
                        }
 
-                       if (($cmd === Delivery::UPLINK) && (intval($parent['forum_mode']) == 1) && !$top_level) {
-                               $relay_to_owner = true;
-                       }
-
                        // until the 'origin' flag has been in use for several months
                        // we will just use it as a fallback test
                        // later we will be able to use it as the primary test of whether or not to relay.
@@ -239,13 +235,13 @@ class Notifier
                        }
 
                        // Special treatment for forum posts
-                       if (Item::isForumPost($target_item, $owner)) {
+                       if (Item::isForumPost($target_item['uri-id'])) {
                                $relay_to_owner = true;
                                $direct_forum_delivery = true;
                        }
 
                        // Avoid that comments in a forum thread are sent to OStatus
-                       if (Item::isForumPost($parent, $owner)) {
+                       if (Item::isForumPost($parent['uri-id'])) {
                                $direct_forum_delivery = true;
                        }
 
@@ -296,7 +292,7 @@ class Notifier
                                        $push_notify = false;
                                }
 
-                               Logger::log('Notify ' . $target_item["guid"] .' via PuSH: ' . ($push_notify ? "Yes":"No"), Logger::DEBUG);
+                               Logger::info('Notify ' . $target_item["guid"] .' via PuSH: ' . ($push_notify ? "Yes":"No"));
                        } elseif ($exclusive_delivery) {
                                $followup = true;
 
@@ -315,7 +311,7 @@ class Notifier
                                // don't send deletions onward for other people's stuff
 
                                if ($target_item['deleted'] && !intval($target_item['wall'])) {
-                                       Logger::log('Ignoring delete notification for non-wall item');
+                                       Logger::notice('Ignoring delete notification for non-wall item');
                                        return;
                                }
 
@@ -333,15 +329,6 @@ class Notifier
                                $deny_people  = $aclFormatter->expand($parent['deny_cid']);
                                $deny_groups  = Group::expand($uid, $aclFormatter->expand($parent['deny_gid']));
 
-                               // if our parent is a public forum (forum_mode == 1), uplink to the origional author causing
-                               // a delivery fork. private groups (forum_mode == 2) do not uplink
-                               /// @todo Possibly we should not uplink when the author is the forum itself?
-
-                               if ((intval($parent['forum_mode']) == 1) && !$top_level && ($cmd !== Delivery::UPLINK)
-                                       && ($target_item['verb'] != Activity::ANNOUNCE)) {
-                                       Worker::add($a->getQueueValue('priority'), 'Notifier', Delivery::UPLINK, $post_uriid, $sender_uid);
-                               }
-
                                foreach ($items as $item) {
                                        $recipients[] = $item['contact-id'];
                                        // pull out additional tagged people to notify (if public message)
@@ -385,7 +372,7 @@ class Notifier
                        if (($thr_parent && ($thr_parent['network'] == Protocol::OSTATUS)) || ($parent['network'] == Protocol::OSTATUS)) {
                                $diaspora_delivery = false;
 
-                               Logger::log('Some parent is OStatus for '.$target_item["guid"]." - Author: ".$thr_parent['author-id']." - Owner: ".$thr_parent['owner-id'], Logger::DEBUG);
+                               Logger::info('Some parent is OStatus for '.$target_item["guid"]." - Author: ".$thr_parent['author-id']." - Owner: ".$thr_parent['owner-id']);
 
                                // Send a salmon to the parent author
                                $probed_contact = DBA::selectFirst('contact', ['url', 'notify'], ['id' => $thr_parent['author-id']]);
@@ -446,26 +433,9 @@ class Notifier
                        if ($diaspora_delivery && !$unlisted) {
                                $batch_delivery = true;
 
-                               $participants_stmt = DBA::p(
-                                       "SELECT
-                                               `batch`, `network`, `protocol`,
-                                               ANY_VALUE(`id`) AS `id`,
-                                               ANY_VALUE(`url`) AS `url`,
-                                               ANY_VALUE(`name`) AS `name`
-                                       FROM `contact`
-                                       WHERE `network` = ?
-                                       AND `batch` != ''
-                                       AND `uid` = ?
-                                       AND `rel` != ?
-                                       AND NOT `blocked`
-                                       AND NOT `pending`
-                                       AND NOT `archive`
-                                       GROUP BY `batch`, `network`, `protocol`",
-                                       Protocol::DIASPORA,
-                                       $owner['uid'],
-                                       Contact::SHARING
-                               );
-                               $participants = DBA::toArray($participants_stmt);
+                               $participants = DBA::selectToArray('contact', ['batch', 'network', 'protocol', 'id', 'url', 'name'],
+                                       ["`network` = ? AND `batch` != '' AND `uid` = ? AND `rel` != ? AND NOT `blocked` AND NOT `pending` AND NOT `archive`", Protocol::DIASPORA, $owner['uid'], Contact::SHARING],
+                                       ['group_by' => ['batch', 'network', 'protocol']]);
 
                                // Fetch the participation list
                                // The function will ensure that there are no duplicates
@@ -475,7 +445,7 @@ class Notifier
                        $condition = ['network' => Protocol::DFRN, 'uid' => $owner['uid'], 'blocked' => false,
                                'pending' => false, 'archive' => false, 'rel' => [Contact::FOLLOWER, Contact::FRIEND]];
 
-                       $contacts = DBA::toArray(DBA::select('contact', ['id', 'url', 'addr', 'name', 'network', 'protocol'], $condition));
+                       $contacts = DBA::selectToArray('contact', ['id', 'url', 'addr', 'name', 'network', 'protocol'], $condition);
 
                        $conversants = array_merge($contacts, $participants);
 
@@ -490,7 +460,7 @@ class Notifier
                $delivery_queue_count += self::deliverOStatus($target_id, $target_item, $owner, $url_recipients, $public_message, $push_notify);
 
                if (!empty($target_item)) {
-                       Logger::log('Calling hooks for ' . $cmd . ' ' . $target_id, Logger::DEBUG);
+                       Logger::info('Calling hooks for ' . $cmd . ' ' . $target_id);
 
                        Hook::fork($a->getQueueValue('priority'), 'notifier_normal', $target_item);
 
@@ -693,7 +663,7 @@ class Notifier
        private static function notifySelfRemoval($self_user_id, $priority, $created)
        {
                $owner = User::getOwnerDataById($self_user_id);
-               if (!$owner) {
+               if (empty($self_user_id) || empty($owner)) {
                        return false;
                }
 
@@ -703,7 +673,7 @@ class Notifier
                }
 
                while($contact = DBA::fetch($contacts_stmt)) {
-                       Contact::terminateFriendship($owner, $contact, true);
+                       Contact::terminateFriendship($contact);
                }
                DBA::close($contacts_stmt);
 
@@ -744,7 +714,7 @@ class Notifier
 
                // Also don't deliver when the direct thread parent was delivered via Diaspora
                if ($thr_parent['network'] == Protocol::DIASPORA) {
-                       Logger::info('Threat parent network is Diaspora, so no AP delivery');
+                       Logger::info('Thread parent network is Diaspora, so no AP delivery');
                        return ['count' => 0, 'contacts' => []];
                }
 
@@ -759,6 +729,14 @@ class Notifier
 
                $uid = $target_item['contact-uid'] ?: $target_item['uid'];
 
+               // Update the locally stored follower list when we deliver to a forum
+               foreach (Tag::getByURIId($target_item['uri-id'], [Tag::MENTION, Tag::EXCLUSIVE_MENTION]) as $tag) {
+                       $target_contact = Contact::getByURL(Strings::normaliseLink($tag['url']), null, [], $uid);
+                       if ($target_contact && $target_contact['contact-type'] == Contact::TYPE_COMMUNITY && $target_contact['manually-approve']) {
+                               Group::updateMembersForForum($target_contact['id']);
+                       }
+               }
+
                if ($target_item['origin']) {
                        $inboxes = ActivityPub\Transmitter::fetchTargetInboxes($target_item, $uid);
 
@@ -767,12 +745,9 @@ class Notifier
                                $relay_inboxes = ActivityPub\Transmitter::addRelayServerInboxes();
                        }
 
-                       Logger::log('Origin item ' . $target_item['id'] . ' with URL ' . $target_item['uri'] . ' will be distributed.', Logger::DEBUG);
-               } elseif (Item::isForumPost($target_item, $owner)) {
-                       $inboxes = ActivityPub\Transmitter::fetchTargetInboxes($target_item, $uid, false, 0, true);
-                       Logger::log('Forum item ' . $target_item['id'] . ' with URL ' . $target_item['uri'] . ' will be distributed.', Logger::DEBUG);
+                       Logger::info('Origin item ' . $target_item['id'] . ' with URL ' . $target_item['uri'] . ' will be distributed.');
                } elseif (!DBA::exists('conversation', ['item-uri' => $target_item['uri'], 'protocol' => Conversation::PARCEL_ACTIVITYPUB])) {
-                       Logger::log('Remote item ' . $target_item['id'] . ' with URL ' . $target_item['uri'] . ' is no AP post. It will not be distributed.', Logger::DEBUG);
+                       Logger::info('Remote item ' . $target_item['id'] . ' with URL ' . $target_item['uri'] . ' is no AP post. It will not be distributed.');
                        return ['count' => 0, 'contacts' => []];
                } elseif ($parent['origin']) {
                        // Remote items are transmitted via the personal inboxes.
@@ -784,11 +759,11 @@ class Notifier
                                $relay_inboxes = ActivityPub\Transmitter::addRelayServerInboxes([]);
                        }
 
-                       Logger::log('Remote item ' . $target_item['id'] . ' with URL ' . $target_item['uri'] . ' will be distributed.', Logger::DEBUG);
+                       Logger::info('Remote item ' . $target_item['id'] . ' with URL ' . $target_item['uri'] . ' will be distributed.');
                }
 
                if (empty($inboxes) && empty($relay_inboxes)) {
-                       Logger::log('No inboxes found for item ' . $target_item['id'] . ' with URL ' . $target_item['uri'] . '. It will not be distributed.', Logger::DEBUG);
+                       Logger::info('No inboxes found for item ' . $target_item['id'] . ' with URL ' . $target_item['uri'] . '. It will not be distributed.');
                        return ['count' => 0, 'contacts' => []];
                }
 
@@ -830,15 +805,4 @@ class Notifier
 
                return ['count' => $delivery_queue_count, 'contacts' => $contacts];
        }
-
-       /**
-        * Check if the delivered item is a forum post
-        *
-        * @param array $item
-        * @return boolean
-        */
-       public static function isForumPost(array $item)
-       {
-               return ($item['gravity'] == GRAVITY_PARENT) && !empty($item['forum_mode']);
-       }
 }