]> git.mxchange.org Git - friendica.git/blobdiff - src/Model/Item.php
Improved profile summary for notifications
[friendica.git] / src / Model / Item.php
index 8b1002fb2664a1b4aa3bcaca6e87bbeacf3cc3f5..60e88757e051bb9bfe2ba88639feeff3984880bc 100644 (file)
@@ -31,6 +31,7 @@ use Friendica\Core\System;
 use Friendica\Core\Worker;
 use Friendica\Database\DBA;
 use Friendica\DI;
+use Friendica\Network\HTTPException\InternalServerErrorException;
 use Friendica\Protocol\Activity;
 use Friendica\Protocol\ActivityPub;
 use Friendica\Protocol\Delivery;
@@ -78,6 +79,7 @@ class Item
        const PR_DISTRIBUTE = 79;
        const PR_PUSHED = 80;
        const PR_LOCAL = 81;
+       const PR_AUDIENCE = 82;
 
        // system.accept_only_sharer setting values
        const COMPLETION_NONE    = 1;
@@ -210,8 +212,6 @@ class Item
                                        }
                                }
 
-                               Post\Media::insertFromAttachmentData($item['uri-id'], $fields['body']);
-
                                $content_fields = ['raw-body' => trim($fields['raw-body'] ?? $fields['body'])];
 
                                // Remove all media attachments from the body and store them in the post-media table
@@ -220,6 +220,10 @@ class Item
                                $content_fields['raw-body'] = self::setHashtags($content_fields['raw-body']);
 
                                Post\Media::insertFromRelevantUrl($item['uri-id'], $content_fields['raw-body'], $fields['body'], $item['author-network']);
+
+                               Post\Media::insertFromAttachmentData($item['uri-id'], $fields['body']);
+                               $content_fields['raw-body'] = BBCode::removeAttachment($content_fields['raw-body']);
+
                                Post\Content::update($item['uri-id'], $content_fields);
                        }
 
@@ -231,7 +235,7 @@ class Item
                                Post\Media::insertFromAttachment($item['uri-id'], $fields['attach']);
                        }
 
-                       // We only need to notfiy others when it is an original entry from us.
+                       // We only need to notify others when it is an original entry from us.
                        // Only call the notifier when the item had been edited and records had been changed.
                        if ($item['origin'] && !empty($fields['edited']) && ($previous['edited'] != $fields['edited'])) {
                                $notify_items[] = $item['id'];
@@ -241,7 +245,15 @@ class Item
                DBA::close($items);
 
                foreach ($notify_items as $notify_item) {
-                       $post = Post::selectFirst(['uri-id', 'uid'], ['id' => $notify_item]);
+                       $post = Post::selectFirst([], ['id' => $notify_item]);
+
+                       if ($post['gravity'] != self::GRAVITY_PARENT) {
+                               $signed = Diaspora::createCommentSignature($post);
+                               if (!empty($signed)) {
+                                       DBA::replace('diaspora-interaction', ['uri-id' => $post['uri-id'], 'interaction' => json_encode($signed)]);
+                               }
+                       }
+
                        Worker::add(Worker::PRIORITY_HIGH, 'Notifier', Delivery::POST, (int)$post['uri-id'], (int)$post['uid']);
                }
 
@@ -408,7 +420,7 @@ class Item
                if ($notify) {
                        // We have to avoid duplicates. So we create the GUID in form of a hash of the plink or uri.
                        // We add the hash of our own host because our host is the original creator of the post.
-                       $prefix_host = DI::baseUrl()->getHostname();
+                       $prefix_host = DI::baseUrl()->getHost();
                } else {
                        $prefix_host = '';
 
@@ -585,7 +597,7 @@ class Item
        public static function isValid(array $item): bool
        {
                // When there is no content then we don't post it
-               if (($item['body'] . $item['title'] == '') && empty($item['quote-uri-id']) && (empty($item['uri-id']) || !Post\Media::existsByURIId($item['uri-id']))) {
+               if (($item['body'] . $item['title'] == '') && empty($item['quote-uri-id']) && empty($item['attachments']) && (empty($item['uri-id']) || !Post\Media::existsByURIId($item['uri-id']))) {
                        Logger::notice('No body, no title.');
                        return false;
                }
@@ -623,10 +635,6 @@ class Item
                        return false;
                }
 
-               if (!empty($item['uid']) && !self::isAllowedByUser($item, $item['uid'])) {
-                       return false;
-               }
-
                if ($item['verb'] == Activity::FOLLOW) {
                        if (!$item['origin'] && ($item['author-id'] == Contact::getPublicIdByUserId($item['uid']))) {
                                // Our own follow request can be relayed to us. We don't store it to avoid notification chaos.
@@ -820,12 +828,6 @@ class Item
        private static function prepareOriginPost(array $item): array
        {
                $item = DI::contentItem()->initializePost($item);
-
-               if (Photo::setPermissionFromBody($item['body'], $item['uid'], $item['contact-id'], $item['allow_cid'], $item['allow_gid'], $item['deny_cid'], $item['deny_gid'])) {
-                       $item['object-type'] = Activity\ObjectType::IMAGE;
-               }
-
-               $item = DI::contentItem()->moveAttachmentsFromBodyToAttach($item);
                $item = DI::contentItem()->finalizePost($item);
 
                return $item;
@@ -883,7 +885,7 @@ class Item
                /*
                 * Do we already have this item?
                 * We have to check several networks since Friendica posts could be repeated
-                * via OStatus (maybe Diasporsa as well)
+                * via OStatus (maybe Diaspora as well)
                 */
                $duplicate = self::getDuplicateID($item);
                if ($duplicate) {
@@ -900,6 +902,8 @@ class Item
                        $item['post-type'] = empty($item['title']) ? self::PT_NOTE : self::PT_ARTICLE;
                }
 
+               $defined_permissions = isset($item['allow_cid']) && isset($item['allow_gid']) && isset($item['deny_cid']) && isset($item['deny_gid']) && isset($item['private']);
+
                $item['wall']          = intval($item['wall'] ?? 0);
                $item['extid']         = trim($item['extid'] ?? '');
                $item['author-name']   = trim($item['author-name'] ?? '');
@@ -939,7 +943,7 @@ class Item
                $item['inform']        = trim($item['inform'] ?? '');
                $item['file']          = trim($item['file'] ?? '');
 
-               // Communities aren't working with the Diaspora protoccol
+               // Communities aren't working with the Diaspora protocol
                if (($uid != 0) && ($item['network'] == Protocol::DIASPORA)) {
                        $user = User::getById($uid, ['account-type']);
                        if ($user['account-type'] == Contact::TYPE_COMMUNITY) {
@@ -994,13 +998,6 @@ class Item
                                return 0;
                        }
 
-                       // If the thread originated from this node, we check the permission against the thread starter
-                       $condition = ['uri-id' => $toplevel_parent['uri-id'], 'wall' => true];
-                       $localTopLevelParent = Post::selectFirst(['uid'], $condition);
-                       if (!empty($localTopLevelParent['uid']) && !self::isAllowedByUser($item, $localTopLevelParent['uid'])) {
-                               return 0;
-                       }
-
                        $parent_id             = $toplevel_parent['id'];
                        $item['parent-uri']    = $toplevel_parent['uri'];
                        $item['parent-uri-id'] = $toplevel_parent['uri-id'];
@@ -1008,7 +1005,7 @@ class Item
                        $item['wall']          = $toplevel_parent['wall'];
 
                        // Reshares have to keep their permissions to allow forums to work
-                       if (!$item['origin'] || ($item['verb'] != Activity::ANNOUNCE)) {
+                       if (!$defined_permissions && (!$item['origin'] || ($item['verb'] != Activity::ANNOUNCE))) {
                                $item['allow_cid']     = $toplevel_parent['allow_cid'];
                                $item['allow_gid']     = $toplevel_parent['allow_gid'];
                                $item['deny_cid']      = $toplevel_parent['deny_cid'];
@@ -1031,7 +1028,7 @@ class Item
                         * This differs from the above settings as it subtly allows comments from
                         * email correspondents to be private even if the overall thread is not.
                         */
-                       if ($toplevel_parent['private']) {
+                       if (!$defined_permissions && $toplevel_parent['private']) {
                                $item['private'] = $toplevel_parent['private'];
                        }
 
@@ -1052,6 +1049,14 @@ class Item
                        }
                }
 
+               if ($item['origin']) {
+                       if (Photo::setPermissionFromBody($item['body'], $item['uid'], $item['contact-id'], $item['allow_cid'], $item['allow_gid'], $item['deny_cid'], $item['deny_gid'])) {
+                               $item['object-type'] = Activity\ObjectType::IMAGE;
+                       }
+
+                       $item = DI::contentItem()->moveAttachmentsFromBodyToAttach($item);
+               }
+
                $item['parent-uri-id'] = ItemURI::getIdByURI($item['parent-uri']);
                $item['thr-parent-id'] = ItemURI::getIdByURI($item['thr-parent']);
 
@@ -1070,7 +1075,7 @@ class Item
                }
 
                // ACL settings
-               if (!empty($item['allow_cid'] . $item['allow_gid'] . $item['deny_cid'] . $item['deny_gid'])) {
+               if (!$defined_permissions && !empty($item['allow_cid'] . $item['allow_gid'] . $item['deny_cid'] . $item['deny_gid'])) {
                        $item['private'] = self::PRIVATE;
                }
 
@@ -1152,8 +1157,6 @@ class Item
                        $item['body']     = BBCode::removeSharedData($item['body']);
                }
 
-               Post\Media::insertFromAttachmentData($item['uri-id'], $item['body']);
-
                // Remove all media attachments from the body and store them in the post-media table
                $item['raw-body'] = Post\Media::insertFromBody($item['uri-id'], $item['raw-body']);
                $item['raw-body'] = self::setHashtags($item['raw-body']);
@@ -1161,6 +1164,10 @@ class Item
                $author = Contact::getById($item['author-id'], ['network']);
                Post\Media::insertFromRelevantUrl($item['uri-id'], $item['raw-body'], $item['body'], $author['network'] ?? '');
 
+               Post\Media::insertFromAttachmentData($item['uri-id'], $item['body']);
+               $item['body']     = BBCode::removeAttachment($item['body']);
+               $item['raw-body'] = BBCode::removeAttachment($item['raw-body']);
+
                // Check for hashtags in the body and repair or add hashtag links
                $item['body'] = self::setHashtags($item['body']);
 
@@ -1220,10 +1227,13 @@ class Item
                        Post\Thread::insert($item['uri-id'], $item);
                }
 
-               if (!in_array($item['verb'], self::ACTIVITIES)) {
+               // The content of activities normally doesn't matter - except for likes from Misskey
+               if (!in_array($item['verb'], self::ACTIVITIES) || in_array($item['verb'], [Activity::LIKE, Activity::DISLIKE]) && !empty($item['body']) && (mb_strlen($item['body']) == 1)) {
                        Post\Content::insert($item['uri-id'], $item);
                }
 
+               $item['parent'] = $parent_id;
+
                // Create Diaspora signature
                if ($item['origin'] && empty($item['diaspora_signed_text']) && ($item['gravity'] != self::GRAVITY_PARENT)) {
                        $signed = Diaspora::createCommentSignature($item);
@@ -1331,6 +1341,19 @@ class Item
                $transmit = $notify || ($posted_item['visible'] && ($parent_origin || $posted_item['origin']));
 
                if ($transmit) {
+                       if ($posted_item['uid'] && Contact\User::isBlocked($posted_item['author-id'], $posted_item['uid'])) {
+                               Logger::info('Message from blocked author will not be relayed', ['item' => $posted_item['id'], 'uri' => $posted_item['uri'], 'cid' => $posted_item['author-id']]);
+                               $transmit = false;
+                       }
+                       if ($transmit && $posted_item['uid'] && Contact\User::isBlocked($posted_item['owner-id'], $posted_item['uid'])) {
+                               Logger::info('Message from blocked owner will not be relayed', ['item' => $posted_item['id'], 'uri' => $posted_item['uri'], 'cid' => $posted_item['owner-id']]);
+                               $transmit = false;
+                       }
+                       if ($transmit && !empty($posted_item['causer-id']) && $posted_item['uid'] && Contact\User::isBlocked($posted_item['causer-id'], $posted_item['uid'])) {
+                               Logger::info('Message from blocked causer will not be relayed', ['item' => $posted_item['id'], 'uri' => $posted_item['uri'], 'cid' => $posted_item['causer-id']]);
+                               $transmit = false;
+                       }
+
                        // Don't relay participation messages
                        if (($posted_item['verb'] == Activity::FOLLOW) &&
                                (!$posted_item['origin'] || ($posted_item['author-id'] != Contact::getPublicIdByUserId($uid)))) {
@@ -1381,10 +1404,16 @@ class Item
         *
         * @param integer $uri_id
         * @return void
+        * @throws InternalServerErrorException
+        * @throws \ImagickException
         */
        public static function updateDisplayCache(int $uri_id)
        {
                $item = Post::selectFirst(self::DISPLAY_FIELDLIST, ['uri-id' => $uri_id]);
+               if (!$item) {
+                       return;
+               }
+
                self::prepareBody($item, false, false, true);
        }
 
@@ -1486,7 +1515,7 @@ class Item
 
                $users = [];
 
-               /// @todo add a field "pcid" in the contact table that referrs to the public contact id.
+               /// @todo add a field "pcid" in the contact table that refers to the public contact id.
                $owner = DBA::selectFirst('contact', ['url', 'nurl', 'alias'], ['id' => $parent['owner-id']]);
                if (!DBA::isResult($owner)) {
                        return;
@@ -1596,7 +1625,7 @@ class Item
 
                if (($uid != 0) && (($item['gravity'] == self::GRAVITY_PARENT) || $is_reshare) &&
                        DI::pConfig()->get($uid, 'system', 'accept_only_sharer') == self::COMPLETION_NONE &&
-                       !in_array($item['post-reason'], [self::PR_FOLLOWER, self::PR_TAG, self::PR_TO, self::PR_CC, self::PR_ACTIVITY])) {
+                       !in_array($item['post-reason'], [self::PR_FOLLOWER, self::PR_TAG, self::PR_TO, self::PR_CC, self::PR_ACTIVITY, self::PR_AUDIENCE])) {
                        Logger::info('Contact is not a follower, thread will not be stored', ['author' => $item['author-link'], 'uid' => $uid, 'uri-id' => $uri_id, 'post-reason' => $item['post-reason']]);
                        return 0;
                }
@@ -1708,7 +1737,7 @@ class Item
                        if (!empty($item['event-id'])) {
                                $event_post = Post::selectFirst(['event-id'], ['uri-id' => $item['uri-id'], 'uid' => $uid]);
                                if (!empty($event_post['event-id'])) {
-                                       $event = DBA::selectFirst('event', ['edited', 'start', 'finish', 'summary', 'desc', 'location', 'nofinish', 'adjust'], ['id' => $item['event-id']]);
+                                       $event = DBA::selectFirst('event', ['edited', 'start', 'finish', 'summary', 'desc', 'location', 'nofinish'], ['id' => $item['event-id']]);
                                        if (!empty($event)) {
                                                // We aren't using "Event::store" here, since we don't want to trigger any further action
                                                $ret = DBA::update('event', $event, ['id' => $event_post['event-id']]);
@@ -1741,16 +1770,16 @@ class Item
                $item['origin'] = 0;
                $item['wall'] = 0;
 
-               $item['contact-id'] = self::contactId($item);
-
                $notify = false;
                if ($item['gravity'] == self::GRAVITY_PARENT) {
                        $contact = DBA::selectFirst('contact', [], ['id' => $item['contact-id'], 'self' => false]);
                        if (DBA::isResult($contact)) {
                                $notify = self::isRemoteSelf($contact, $item);
+                               $item['wall'] = (bool)$notify;
                        }
                }
 
+               $item['contact-id'] = self::contactId($item);
                $distributed = self::insert($item, $notify);
 
                if (!$distributed) {
@@ -2021,7 +2050,11 @@ class Item
                }
 
                // Glue it together to be able to make a hash from it
-               $host_id = implode('/', $parsed);
+               if (!empty($parsed)) {
+                       $host_id = implode('/', $parsed);
+               } else {
+                       $host_id = $uri;
+               }
 
                // Use a mixture of several hashes to provide some GUID like experience
                return hash('crc32', $hostPart) . '-' . hash('joaat', $host_id) . '-' . hash('fnv164', $host_id);
@@ -2041,7 +2074,7 @@ class Item
                        $guid = System::createUUID();
                }
 
-               return DI::baseUrl()->get() . '/objects/' . $guid;
+               return DI::baseUrl() . '/objects/' . $guid;
        }
 
        /**
@@ -2221,7 +2254,7 @@ class Item
 
                if ($owner['page-flags'] == User::PAGE_FLAGS_PRVGROUP) {
                        $allow_cid = '';
-                       $allow_gid = '<' . Group::FOLLOWERS . '>';
+                       $allow_gid = '<' . Circle::FOLLOWERS . '>';
                        $deny_cid  = '';
                        $deny_gid  = '';
                        self::performActivity($item['id'], 'announce', $uid, $allow_cid, $allow_gid, $deny_cid, $deny_gid);
@@ -2281,7 +2314,7 @@ class Item
                }
 
                // Prevent to forward already forwarded posts
-               if ($datarray['app'] == DI::baseUrl()->getHostname()) {
+               if ($datarray['app'] == DI::baseUrl()->getHost()) {
                        Logger::info('Already forwarded (second test)');
                        return false;
                }
@@ -2481,22 +2514,27 @@ class Item
        /**
         * Returns an array of contact-ids that are allowed to see this object
         *
-        * @param array $obj        Item array with at least uid, allow_cid, allow_gid, deny_cid and deny_gid
-        * @param bool  $check_dead Prunes unavailable contacts from the result
+        * @param array $obj              Item array with at least uid, allow_cid, allow_gid, deny_cid and deny_gid
+        * @param bool  $check_dead       Prunes unavailable contacts from the result
+        * @param bool  $expand_followers Expand the list of followers
         * @return array
         * @throws \Exception
         */
-       public static function enumeratePermissions(array $obj, bool $check_dead = false): array
+       public static function enumeratePermissions(array $obj, bool $check_dead = false, bool $expand_followers = true): array
        {
-               $aclFormater = DI::aclFormatter();
+               $aclFormatter = DI::aclFormatter();
+
+               if (!$expand_followers && (!empty($obj['deny_cid']) || !empty($obj['deny_gid']))) {
+                       $expand_followers = true;
+               }
 
-               $allow_people = $aclFormater->expand($obj['allow_cid']);
-               $allow_groups = Group::expand($obj['uid'], $aclFormater->expand($obj['allow_gid']), $check_dead);
-               $deny_people  = $aclFormater->expand($obj['deny_cid']);
-               $deny_groups  = Group::expand($obj['uid'], $aclFormater->expand($obj['deny_gid']), $check_dead);
-               $recipients   = array_unique(array_merge($allow_people, $allow_groups));
-               $deny         = array_unique(array_merge($deny_people, $deny_groups));
-               $recipients   = array_diff($recipients, $deny);
+               $allow_people  = $aclFormatter->expand($obj['allow_cid']);
+               $allow_circles = Circle::expand($obj['uid'], $aclFormatter->expand($obj['allow_gid']), $check_dead, $expand_followers);
+               $deny_people   = $aclFormatter->expand($obj['deny_cid']);
+               $deny_circles  = Circle::expand($obj['uid'], $aclFormatter->expand($obj['deny_gid']), $check_dead);
+               $recipients    = array_unique(array_merge($allow_people, $allow_circles));
+               $deny          = array_unique(array_merge($deny_people, $deny_circles));
+               $recipients    = array_diff($recipients, $deny);
                return $recipients;
        }
 
@@ -2602,7 +2640,7 @@ class Item
         *            Activity verb. One of
         *            like, unlike, dislike, undislike, attendyes, unattendyes,
         *            attendno, unattendno, attendmaybe, unattendmaybe,
-        *            announce, unannouce
+        *            announce, unannounce
         * @param int    $uid
         * @param string $allow_cid
         * @param string $allow_gid
@@ -2862,9 +2900,9 @@ class Item
                        /*
                         * Authenticated visitor. Unless pre-verified,
                         * check that the contact belongs to this $owner_id
-                        * and load the groups the visitor belongs to.
+                        * and load the circles the visitor belongs to.
                         * If pre-verified, the caller is expected to have already
-                        * done this and passed the groups into this function.
+                        * done this and passed the circles into this function.
                         */
                        $permissionSets = DI::permissionSet()->selectByContactId($remote_user, $owner_id);
 
@@ -2987,6 +3025,7 @@ class Item
                if (!$is_preview) {
                        $item['body'] = preg_replace("#\s*\[attachment .*?].*?\[/attachment]\s*#ism", "\n", $item['body']);
                        $item['body'] = Post\Media::removeFromEndOfBody($item['body'] ?? '');
+                       $item['body'] = Post\Media::replaceImage($item['body']);
                }
 
                $body = $item['body'];
@@ -2996,32 +3035,50 @@ class Item
 
                $fields = ['uri-id', 'uri', 'body', 'title', 'author-name', 'author-link', 'author-avatar', 'guid', 'created', 'plink', 'network', 'has-media', 'quote-uri-id', 'post-type'];
 
-               $shared_uri_id = 0;
-               $shared_links  = [];
+               $shared_uri_id      = 0;
+               $shared_links       = [];
+               $quote_shared_links = [];
 
                $shared = DI::contentItem()->getSharedPost($item, $fields);
                if (!empty($shared['post'])) {
                        $shared_item  = $shared['post'];
                        $shared_item['body'] = Post\Media::removeFromEndOfBody($shared_item['body']);
+                       $shared_item['body'] = Post\Media::replaceImage($shared_item['body']);
                        $quote_uri_id = $shared['post']['uri-id'];
                        $shared_links[] = strtolower($shared['post']['uri']);
                        $item['body'] = BBCode::removeSharedData($item['body']);
                } elseif (empty($shared_item['uri-id']) && empty($item['quote-uri-id']) && ($item['network'] != Protocol::DIASPORA)) {
                        $media = Post\Media::getByURIId($item['uri-id'], [Post\Media::ACTIVITY]);
                        if (!empty($media)) {
-                               $shared_item = Post::selectFirst($fields, ['plink' => $media[0]['url'], 'uid' => [$item['uid'], 0]]);
+                               $shared_item = Post::selectFirst($fields, ['uri-id' => $media[0]['media-uri-id'], 'uid' => [$item['uid'], 0]]);
+                               if (empty($shared_item['uri-id'])) {
+                                       $shared_item = Post::selectFirst($fields, ['plink' => $media[0]['url'], 'uid' => [$item['uid'], 0]]);
+                               } elseif (!in_array(strtolower($media[0]['url']), $shared_links)) {
+                                       $shared_links[] = strtolower($media[0]['url']);
+                               }
 
                                if (empty($shared_item['uri-id'])) {
                                        $shared_item = Post::selectFirst($fields, ['uri' => $media[0]['url'], 'uid' => [$item['uid'], 0]]);
                                        $shared_links[] = strtolower($media[0]['url']);
                                }
 
-                               $quote_uri_id = $shared_item['uri-id'] ?? 0;
+                               if (!empty($shared_item['uri-id'])) {
+                                       $data = BBCode::getAttachmentData($shared_item['body']);
+                                       if (!empty($data['url'])) {
+                                               $quote_shared_links[] = $data['url'];
+                                       }
+
+                                       $quote_uri_id = $shared_item['uri-id'];
+                               }
                        }
                }
 
                if (!empty($quote_uri_id)) {
-                       $item['body'] .= "\n" . DI::contentItem()->createSharedBlockByArray($shared_item);
+                       if (isset($shared_item['plink'])) {
+                               $item['body'] .= "\n" . DI::contentItem()->createSharedBlockByArray($shared_item);
+                       } else {
+                               DI::logger()->warning('Missing plink in shared item', ['item' => $item, 'shared' => $shared, 'quote_uri_id' => $quote_uri_id, 'shared_item' => $shared_item]);
+                       }
                }
 
                if (!empty($shared_item['uri-id'])) {
@@ -3048,9 +3105,14 @@ class Item
                // Compile eventual content filter reasons
                $filter_reasons = [];
                if (!$is_preview && DI::userSession()->getPublicContactId() != $item['author-id']) {
-                       if (Contact\User::isCollapsed($item['author-id'], $item['uid'])) {
+                       if (!empty($item['user-blocked-author']) || !empty($item['user-blocked-owner'])) {
+                               $filter_reasons[] = DI::l10n()->t('%s is blocked', $item['author-name']);
+                       } elseif (!empty($item['user-ignored-author']) || !empty($item['user-ignored-owner'])) {
+                               $filter_reasons[] = DI::l10n()->t('%s is ignored', $item['author-name']);
+                       } elseif (!empty($item['user-collapsed-author']) || !empty($item['user-collapsed-owner'])) {
                                $filter_reasons[] = DI::l10n()->t('Content from %s is collapsed', $item['author-name']);
                        }
+
                        if (!empty($item['content-warning']) && (!DI::userSession()->getLocalUserId() || !DI::pConfig()->get(DI::userSession()->getLocalUserId(), 'system', 'disable_cw', false))) {
                                $filter_reasons[] = DI::l10n()->t('Content warning: %s', $item['content-warning']);
                        }
@@ -3085,12 +3147,14 @@ class Item
                }
 
                if (!empty($shared_attachments)) {
+                       $s = self::addGallery($s, $shared_attachments, $item['uri-id']);
                        $s = self::addVisualAttachments($shared_attachments, $shared_item, $s, true);
-                       $s = self::addLinkAttachment($shared_uri_id ?: $item['uri-id'], $shared_attachments, $body, $s, true, []);
+                       $s = self::addLinkAttachment($shared_uri_id ?: $item['uri-id'], $shared_attachments, $body, $s, true, $quote_shared_links);
                        $s = self::addNonVisualAttachments($shared_attachments, $item, $s, true);
                        $body = BBCode::removeSharedData($body);
                }
 
+               $s = self::addGallery($s, $attachments, $item['uri-id']);
                $s = self::addVisualAttachments($attachments, $item, $s, false);
                $s = self::addLinkAttachment($item['uri-id'], $attachments, $body, $s, false, $shared_links);
                $s = self::addNonVisualAttachments($attachments, $item, $s, false);
@@ -3140,6 +3204,24 @@ class Item
                ]);
        }
 
+       /**
+        * Modify links to pictures to links for the "Fancybox" gallery
+        *
+        * @param string $s
+        * @param array $attachments
+        * @param integer $uri_id
+        * @return string
+        */
+       private static function addGallery(string $s, array $attachments, int $uri_id): string
+       {
+               foreach ($attachments['visual'] as $attachment) {
+                       if (empty($attachment['preview']) || ($attachment['type'] != Post\Media::IMAGE)) {
+                               continue;
+                       }
+                       $s = str_replace('<a href="' . $attachment['url'] . '"', '<a data-fancybox="' . $uri_id . '" href="' . $attachment['url'] . '"', $s);
+               }
+               return $s;
+       }
 
        /**
         * Check if the body contains a link
@@ -3203,12 +3285,18 @@ class Item
 
                foreach ($attachments['visual'] as $attachment) {
                        if (!empty($attachment['preview'])) {
+                               if (Network::isLocalLink($attachment['preview'])) {
+                                       continue;
+                               }
                                $proxy   = Post\Media::getPreviewUrlForId($attachment['id'], Proxy::SIZE_LARGE);
                                $search  = ['[img=' . $attachment['preview'] . ']', ']' . $attachment['preview'] . '[/img]'];
                                $replace = ['[img=' . $proxy . ']', ']' . $proxy . '[/img]'];
 
                                $body = str_replace($search, $replace, $body);
                        } elseif ($attachment['filetype'] == 'image') {
+                               if (Network::isLocalLink($attachment['url'])) {
+                                       continue;
+                               }
                                $proxy   = Post\Media::getUrlForId($attachment['id']);
                                $search  = ['[img=' . $attachment['url'] . ']', ']' . $attachment['url'] . '[/img]'];
                                $replace = ['[img=' . $proxy . ']', ']' . $proxy . '[/img]'];
@@ -3288,7 +3376,7 @@ class Item
                                if (self::containsLink($item['body'], $src_url)) {
                                        continue;
                                }
-                               $images[] = ['src' => $src_url, 'preview' => $preview_url, 'attachment' => $attachment];
+                               $images[] = ['src' => $src_url, 'preview' => $preview_url, 'attachment' => $attachment, 'uri_id' => $item['uri-id']];
                        }
                }
 
@@ -3406,7 +3494,7 @@ class Item
                DI::profiler()->stopRecording();
 
                if (isset($data['url']) && !in_array(strtolower($data['url']), $ignore_links)) {
-                       if (!empty($data['description']) || !empty($data['image']) || !empty($data['preview'])) {
+                       if (!empty($data['description']) || !empty($data['image']) || !empty($data['preview']) || (!empty($data['title']) && !Strings::compareLink($data['title'], $data['url']))) {
                                $parts = parse_url($data['url']);
                                if (!empty($parts['scheme']) && !empty($parts['host'])) {
                                        if (empty($data['provider_name'])) {
@@ -3671,7 +3759,7 @@ class Item
                        return is_numeric($hookData['item_id']) ? $hookData['item_id'] : 0;
                }
 
-               $fetched_uri = ActivityPub\Processor::fetchMissingActivity($uri);
+               $fetched_uri = ActivityPub\Processor::fetchMissingActivity($uri, [], '', ActivityPub\Receiver::COMPLETION_MANUAL, $uid);
 
                if ($fetched_uri) {
                        $item_id = self::searchByLink($fetched_uri, $uid);
@@ -3688,43 +3776,6 @@ class Item
                return 0;
        }
 
-       /**
-        * Check a prospective item array against user-level permissions
-        *
-        * @param array $item Expected keys: uri, gravity, and
-        *                    author-link if is author-id is set,
-        *                    owner-link if is owner-id is set,
-        *                    causer-link if is causer-id is set.
-        * @param int   $user_id Local user ID
-        * @return bool
-        * @throws \Exception
-        */
-       protected static function isAllowedByUser(array $item, int $user_id): bool
-       {
-               if (!empty($item['author-id']) && Contact\User::isBlocked($item['author-id'], $user_id)) {
-                       Logger::notice('Author is blocked by user', ['author-link' => $item['author-link'], 'uid' => $user_id, 'item-uri' => $item['uri']]);
-                       return false;
-               }
-
-               if (!empty($item['owner-id']) && Contact\User::isBlocked($item['owner-id'], $user_id)) {
-                       Logger::notice('Owner is blocked by user', ['owner-link' => $item['owner-link'], 'uid' => $user_id, 'item-uri' => $item['uri']]);
-                       return false;
-               }
-
-               // The causer is set during a thread completion, for example because of a reshare. It countains the responsible actor.
-               if (!empty($item['causer-id']) && Contact\User::isBlocked($item['causer-id'], $user_id)) {
-                       Logger::notice('Causer is blocked by user', ['causer-link' => $item['causer-link'] ?? $item['causer-id'], 'uid' => $user_id, 'item-uri' => $item['uri']]);
-                       return false;
-               }
-
-               if (!empty($item['causer-id']) && ($item['gravity'] === self::GRAVITY_PARENT) && Contact\User::isIgnored($item['causer-id'], $user_id)) {
-                       Logger::notice('Causer is ignored by user', ['causer-link' => $item['causer-link'] ?? $item['causer-id'], 'uid' => $user_id, 'item-uri' => $item['uri']]);
-                       return false;
-               }
-
-               return true;
-       }
-
        /**
         * Fetch the uri-id of a quote
         *