Cast field data when ATTR_EMULATE_PREPARES is enabled

[friendica.git] / src / Model / UserItem.php

diff --git a/src/Model/UserItem.php b/src/Model/UserItem.php
index 4e4ca6c94c87aa44535777731e06c11017e4de5e..38e9adc6ea26d451375347d144525b46ebfeebd1 100644 (file)
--- a/src/Model/UserItem.php
+++ b/src/Model/UserItem.php
@@ -51,8 +51,8 @@ class UserItem
         */
        public static function setNotification(int $iid)
        {
-               $fields = ['id', 'uri-id', 'uid', 'body', 'parent', 'gravity', 'tag',
-                       'contact-id', 'thr-parent', 'parent-uri', 'author-id', 'verb'];
+               $fields = ['id', 'uri-id', 'parent-uri-id', 'uid', 'body', 'parent', 'gravity', 'tag',
+                       'private', 'contact-id', 'thr-parent', 'parent-uri', 'author-id', 'verb'];
                $item = Item::selectFirst($fields, ['id' => $iid, 'origin' => false]);
                if (!DBA::isResult($item)) {
                        return;
@@ -63,14 +63,26 @@ class UserItem
                        return;
                }
 
-               // fetch all users in the thread
+               if ($item['uid'] == 0) {
+                       $uids = [];
+               } else {
+                       // Always include the item user
+                       $uids = [$item['uid']];
+               }
+
+               // Add every user who participated so far in this thread
+               // This can only happen with participations on global items. (means: uid = 0) 
                $users = DBA::p("SELECT DISTINCT(`contact`.`uid`) FROM `item`
                        INNER JOIN `contact` ON `contact`.`id` = `item`.`contact-id` AND `contact`.`uid` != 0
                        WHERE `parent` IN (SELECT `parent` FROM `item` WHERE `id`=?)", $iid);
                while ($user = DBA::fetch($users)) {
-                       self::setNotificationForUser($item, $user['uid']);
+                       $uids[] = $user['uid'];
                }
                DBA::close($users);
+
+               foreach (array_unique($uids) as $uid) {
+                       self::setNotificationForUser($item, $uid);
+               }
        }
 
        /**
@@ -144,7 +156,9 @@ class UserItem
 
                Logger::info('Set notification', ['iid' => $item['id'], 'uid' => $uid, 'notification-type' => $notification_type]);
 
-               DBA::update('user-item', ['notification-type' => $notification_type], ['iid' => $item['id'], 'uid' => $uid], true);
+               $fields = ['notification-type' => $notification_type];
+               Post\User::update($item['uri-id'], $uid, $fields);
+               DBA::update('user-item', $fields, ['iid' => $item['id'], 'uid' => $uid], true);
        }
 
        /**
@@ -217,7 +231,7 @@ class UserItem
                        return true;
                }
 
-               // Don't continue when the item had been an announce activity
+               // The following check doesn't make sense on activities, so quit here
                if ($item['verb'] == Activity::ANNOUNCE) {
                        return false;
                }