]> git.mxchange.org Git - friendica.git/blobdiff - src/Model/Post/UserNotification.php
Merge pull request #12298 from annando/api-suggestions
[friendica.git] / src / Model / Post / UserNotification.php
index d14eb7f3be556a9c7b1cd94ecee028367bbc4337..b26fb292bc6b84d664a87689e2df1b2fc3ba4442 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
  *
@@ -30,10 +30,11 @@ use Friendica\Database\DBA;
 use Friendica\Database\DBStructure;
 use Friendica\DI;
 use Friendica\Model\Contact;
+use Friendica\Model\Item;
 use Friendica\Model\Post;
 use Friendica\Model\Subscription;
 use Friendica\Model\Tag;
-use Friendica\Navigation\Notifications;
+use Friendica\Model\User;
 use Friendica\Network\HTTPException;
 use Friendica\Protocol\Activity;
 use Friendica\Util\Strings;
@@ -50,6 +51,7 @@ class UserNotification
        const TYPE_ACTIVITY_PARTICIPATION = 32;
        const TYPE_DIRECT_THREAD_COMMENT  = 64;
        const TYPE_SHARED                 = 128;
+       const TYPE_FOLLOW                 = 256;
 
        /**
         * Insert a new user notification entry
@@ -66,7 +68,7 @@ class UserNotification
                        throw new BadMethodCallException('Empty URI_id');
                }
 
-               $fields = DBStructure::getFieldsForTable('post-user-notification', $data);
+               $fields = DI::dbaDefinition()->truncateFieldsForTable('post-user-notification', $data);
 
                $fields['uri-id'] = $uri_id;
                $fields['uid']    = $uid;
@@ -90,7 +92,7 @@ class UserNotification
                        throw new BadMethodCallException('Empty URI_id');
                }
 
-               $fields = DBStructure::getFieldsForTable('post-user-notification', $data);
+               $fields = DI::dbaDefinition()->truncateFieldsForTable('post-user-notification', $data);
 
                // Remove the key fields
                unset($fields['uri-id']);
@@ -176,12 +178,28 @@ class UserNotification
                        return;
                }
 
+               $user = User::getById($uid, ['account-type', 'account_removed', 'account_expired']);
+               if (in_array($user['account-type'], [User::ACCOUNT_TYPE_COMMUNITY, User::ACCOUNT_TYPE_RELAY])) {
+                       return;
+               }
+
+               if ($user['account_removed'] || $user['account_expired']) {
+                       return;
+               }
+
+               $author = Contact::getById($item['author-id'], ['contact-type']);
+               if (empty($author)) {
+                       return;
+               }
+
                $notification_type = self::TYPE_NONE;
 
                if (self::checkShared($item, $uid)) {
                        $notification_type = $notification_type | self::TYPE_SHARED;
                        self::insertNotificationByItem(self::TYPE_SHARED, $uid, $item);
                        $notified = true;
+               } elseif ($author['contact-type'] == Contact::TYPE_COMMUNITY) {
+                       return;
                } else {
                        $notified = false;
                }
@@ -189,11 +207,16 @@ class UserNotification
                $profiles = self::getProfileForUser($uid);
 
                // Fetch all contacts for the given profiles
-               $contacts = [];
+               $contacts    = [];
+               $iscommunity = false;
 
-               $ret = DBA::select('contact', ['id'], ['uid' => 0, 'nurl' => $profiles]);
+               $ret = DBA::select('contact', ['id', 'contact-type'], ['uid' => 0, 'nurl' => $profiles]);
                while ($contact = DBA::fetch($ret)) {
                        $contacts[] = $contact['id'];
+
+                       if ($contact['contact-type'] == Contact::TYPE_COMMUNITY) {
+                               $iscommunity = true;
+                       }
                }
                DBA::close($ret);
 
@@ -202,7 +225,7 @@ class UserNotification
                        return;
                }
 
-               if (self::checkExplicitMention($item, $profiles)) {
+               if (($item['verb'] != Activity::ANNOUNCE) && self::checkExplicitMention($item, $profiles)) {
                        $notification_type = $notification_type | self::TYPE_EXPLICIT_TAGGED;
                        if (!$notified) {
                                self::insertNotificationByItem(self::TYPE_EXPLICIT_TAGGED, $uid, $item);
@@ -210,7 +233,7 @@ class UserNotification
                        }
                }
 
-               if (self::checkImplicitMention($item, $profiles)) {
+               if (($item['verb'] != Activity::ANNOUNCE) && self::checkImplicitMention($item, $profiles)) {
                        $notification_type = $notification_type | self::TYPE_IMPLICIT_TAGGED;
                        if (!$notified) {
                                self::insertNotificationByItem(self::TYPE_IMPLICIT_TAGGED, $uid, $item);
@@ -226,7 +249,7 @@ class UserNotification
                        }
                }
 
-               if (self::checkDirectCommentedThread($item, $contacts)) {
+               if (!$iscommunity && self::checkDirectCommentedThread($item, $contacts)) {
                        $notification_type = $notification_type | self::TYPE_DIRECT_THREAD_COMMENT;
                        if (!$notified) {
                                self::insertNotificationByItem(self::TYPE_DIRECT_THREAD_COMMENT, $uid, $item);
@@ -234,7 +257,7 @@ class UserNotification
                        }
                }
 
-               if (self::checkCommentedThread($item, $contacts)) {
+               if (($item['verb'] != Activity::ANNOUNCE) && self::checkCommentedThread($item, $contacts)) {
                        $notification_type = $notification_type | self::TYPE_THREAD_COMMENT;
                        if (!$notified) {
                                self::insertNotificationByItem(self::TYPE_THREAD_COMMENT, $uid, $item);
@@ -242,7 +265,7 @@ class UserNotification
                        }
                }
 
-               if (self::checkCommentedParticipation($item, $contacts)) {
+               if (($item['verb'] != Activity::ANNOUNCE) && self::checkCommentedParticipation($item, $contacts)) {
                        $notification_type = $notification_type | self::TYPE_COMMENT_PARTICIPATION;
                        if (!$notified) {
                                self::insertNotificationByItem(self::TYPE_COMMENT_PARTICIPATION, $uid, $item);
@@ -250,15 +273,27 @@ class UserNotification
                        }
                }
 
-               if (self::checkActivityParticipation($item, $contacts)) {
+               if (($item['verb'] != Activity::ANNOUNCE) && self::checkFollowParticipation($item, $contacts)) {
+                       $notification_type = $notification_type | self::TYPE_FOLLOW;
+                       if (!$notified) {
+                               self::insertNotificationByItem(self::TYPE_FOLLOW, $uid, $item);
+                               $notified = true;
+                       }
+               }
+
+               if (($item['verb'] != Activity::ANNOUNCE) && self::checkActivityParticipation($item, $contacts)) {
                        $notification_type = $notification_type | self::TYPE_ACTIVITY_PARTICIPATION;
                        if (!$notified) {
                                self::insertNotificationByItem(self::TYPE_ACTIVITY_PARTICIPATION, $uid, $item);
                        }
                }
 
+               if (empty($notification_type)) {
+                       return;
+               }
+
                // Only create notifications for posts and comments, not for activities
-               if (empty($notification_type) || !in_array($item['gravity'], [GRAVITY_PARENT, GRAVITY_COMMENT])) {
+               if (($item['gravity'] == Item::GRAVITY_ACTIVITY) && ($item['verb'] != Activity::ANNOUNCE)) {
                        return;
                }
 
@@ -280,18 +315,18 @@ class UserNotification
         */
        private static function insertNotificationByItem(int $type, int $uid, array $item): void
        {
-               if (($item['gravity'] == GRAVITY_ACTIVITY) &&
+               if (($item['verb'] != Activity::ANNOUNCE) && ($item['gravity'] == Item::GRAVITY_ACTIVITY) &&
                        !in_array($type, [self::TYPE_DIRECT_COMMENT, self::TYPE_DIRECT_THREAD_COMMENT])) {
                        // Activities are only stored when performed on the user's post or comment
                        return;
                }
 
-               $notification = (new Notifications\Factory\Notification(DI::logger()))->createForUser(
+               $notification = DI::notificationFactory()->createForUser(
                        $uid,
                        $item['vid'],
                        $type,
                        $item['author-id'],
-                       $item['gravity'] == GRAVITY_ACTIVITY ? $item['thr-parent-id'] : $item['uri-id'],
+                       $item['gravity'] == Item::GRAVITY_ACTIVITY ? $item['thr-parent-id'] : $item['uri-id'],
                        $item['parent-uri-id']
                );
 
@@ -306,7 +341,7 @@ class UserNotification
        /**
         * Add a notification entry
         *
-        * @param int    $actor Contact ID of the actor
+        * @param int    $actor Public contact ID of the actor
         * @param string $verb  One of the Activity verb constant values
         * @param int    $uid   User ID
         * @return boolean
@@ -314,7 +349,7 @@ class UserNotification
         */
        public static function insertNotification(int $actor, string $verb, int $uid): bool
        {
-               $notification = (new Notifications\Factory\Notification(DI::logger()))->createForRelationship(
+               $notification = DI::notificationFactory()->createForRelationship(
                        $uid,
                        $actor,
                        $verb
@@ -393,30 +428,23 @@ class UserNotification
        private static function checkShared(array $item, int $uid): bool
        {
                // Only check on original posts and reshare ("announce") activities, otherwise return
-               if (($item['gravity'] != GRAVITY_PARENT) && ($item['verb'] != Activity::ANNOUNCE)) {
+               if (($item['gravity'] != Item::GRAVITY_PARENT) && ($item['verb'] != Activity::ANNOUNCE)) {
                        return false;
                }
 
+               // Don't notify about reshares by communities of our own posts or each time someone comments
+               if (($item['verb'] == Activity::ANNOUNCE) && DBA::exists('contact', ['id' => $item['contact-id'], 'contact-type' => Contact::TYPE_COMMUNITY])) {
+                       $post = Post::selectFirst(['origin', 'gravity'], ['uri-id' => $item['thr-parent-id'], 'uid' => $uid]);
+                       if (!$post || $post['origin'] || ($post['gravity'] != Item::GRAVITY_PARENT)) {
+                               return false;
+                       }
+               }
+
                // Check if the contact posted or shared something directly
                if (DBA::exists('contact', ['id' => $item['contact-id'], 'notify_new_posts' => true])) {
                        return true;
                }
 
-               // The following check doesn't make sense on activities, so quit here
-               if ($item['verb'] == Activity::ANNOUNCE) {
-                       return false;
-               }
-
-               // Check if the contact is a mentioned forum
-               $tags = DBA::select('tag-view', ['url'], ['uri-id' => $item['uri-id'], 'type' => [Tag::MENTION, Tag::EXCLUSIVE_MENTION]]);
-               while ($tag = DBA::fetch($tags)) {
-                       $condition = ['nurl' => Strings::normaliseLink($tag['url']), 'uid' => $uid, 'notify_new_posts' => true, 'contact-type' => Contact::TYPE_COMMUNITY];
-                       if (DBA::exists('contact', $condition)) {
-                               return true;
-                       }
-               }
-               DBA::close($tags);
-
                return false;
        }
 
@@ -474,7 +502,7 @@ class UserNotification
         */
        private static function checkCommentedThread(array $item, array $contacts): bool
        {
-               $condition = ['parent' => $item['parent'], 'author-id' => $contacts, 'deleted' => false, 'gravity' => GRAVITY_PARENT];
+               $condition = ['parent' => $item['parent'], 'author-id' => $contacts, 'deleted' => false, 'gravity' => Item::GRAVITY_PARENT];
                return Post::exists($condition);
        }
 
@@ -488,7 +516,7 @@ class UserNotification
         */
        private static function checkDirectComment(array $item, array $contacts): bool
        {
-               $condition = ['uri' => $item['thr-parent'], 'uid' => $item['uid'], 'author-id' => $contacts, 'deleted' => false, 'gravity' => GRAVITY_COMMENT];
+               $condition = ['uri' => $item['thr-parent'], 'uid' => $item['uid'], 'author-id' => $contacts, 'deleted' => false, 'gravity' => Item::GRAVITY_COMMENT];
                return Post::exists($condition);
        }
 
@@ -502,7 +530,7 @@ class UserNotification
         */
        private static function checkDirectCommentedThread(array $item, array $contacts): bool
        {
-               $condition = ['uri' => $item['thr-parent'], 'uid' => $item['uid'], 'author-id' => $contacts, 'deleted' => false, 'gravity' => GRAVITY_PARENT];
+               $condition = ['uri' => $item['thr-parent'], 'uid' => $item['uid'], 'author-id' => $contacts, 'deleted' => false, 'gravity' => Item::GRAVITY_PARENT];
                return Post::exists($condition);
        }
 
@@ -516,7 +544,21 @@ class UserNotification
         */
        private static function checkCommentedParticipation(array $item, array $contacts): bool
        {
-               $condition = ['parent' => $item['parent'], 'author-id' => $contacts, 'deleted' => false, 'gravity' => GRAVITY_COMMENT];
+               $condition = ['parent' => $item['parent'], 'author-id' => $contacts, 'deleted' => false, 'gravity' => Item::GRAVITY_COMMENT];
+               return Post::exists($condition);
+       }
+
+       /**
+        * Check if the user follows this thread
+        *
+        * @param array $item
+        * @param array $contacts Array of contact IDs
+        * @return bool The user follows the thread
+        * @throws Exception
+        */
+       private static function checkFollowParticipation(array $item, array $contacts): bool
+       {
+               $condition = ['parent' => $item['parent'], 'author-id' => $contacts, 'deleted' => false, 'gravity' => Item::GRAVITY_ACTIVITY, 'verb' => Activity::FOLLOW];
                return Post::exists($condition);
        }
 
@@ -530,7 +572,7 @@ class UserNotification
         */
        private static function checkActivityParticipation(array $item, array $contacts): bool
        {
-               $condition = ['parent' => $item['parent'], 'author-id' => $contacts, 'deleted' => false, 'gravity' => GRAVITY_ACTIVITY];
+               $condition = ['parent' => $item['parent'], 'author-id' => $contacts, 'deleted' => false, 'gravity' => Item::GRAVITY_ACTIVITY];
                return Post::exists($condition);
        }
 }