]> git.mxchange.org Git - friendica.git/blobdiff - src/Model/Item.php
Insert a `user-contact` for every contact
[friendica.git] / src / Model / Item.php
index 86b9270fe9e899c55f7e6bf4cd2c129f1d515619..5cc72b05800bc15ebc60a92effb08baf3c553b8e 100644 (file)
@@ -363,7 +363,7 @@ class Item
                return true;
        }
 
-       private static function guid($item, $notify)
+       public static function guid($item, $notify)
        {
                if (!empty($item['guid'])) {
                        return Strings::escapeTags(trim($item['guid']));
@@ -468,18 +468,14 @@ class Item
                // Checking if there is already an item with the same guid
                $condition = ['guid' => $item['guid'], 'network' => $item['network'], 'uid' => $item['uid']];
                if (Post::exists($condition)) {
-                       Logger::notice('Found already existing item', [
-                               'guid' => $item['guid'],
-                               'uid' => $item['uid'],
-                               'network' => $item['network']
-                       ]);
+                       Logger::notice('Found already existing item', $condition);
                        return true;
                }
 
                $condition = ['uri-id' => $item['uri-id'], 'uid' => $item['uid'],
                        'network' => [$item['network'], Protocol::DFRN]];
                if (Post::exists($condition)) {
-                       Logger::notice('duplicated item with the same uri found.', $item);
+                       Logger::notice('duplicated item with the same uri found.', $condition);
                        return true;
                }
 
@@ -487,7 +483,7 @@ class Item
                if (in_array($item['network'], [Protocol::DFRN, Protocol::DIASPORA])) {
                        $condition = ['guid' => $item['guid'], 'uid' => $item['uid']];
                        if (Post::exists($condition)) {
-                               Logger::notice('duplicated item with the same guid found.', $item);
+                               Logger::notice('duplicated item with the same guid found.', $condition);
                                return true;
                        }
                } elseif ($item['network'] == Protocol::OSTATUS) {
@@ -710,7 +706,7 @@ class Item
                return GRAVITY_UNKNOWN;   // Should not happen
        }
 
-       public static function insert($item, $notify = false, $dontcache = false)
+       public static function insert(array $item, bool $notify = false, bool $post_local = true)
        {
                $orig_item = $item;
 
@@ -935,7 +931,7 @@ class Item
                        $item["private"] = self::PRIVATE;
                }
 
-               if ($notify) {
+               if ($notify && $post_local) {
                        $item['edit'] = false;
                        $item['parent'] = $parent_id;
 
@@ -957,7 +953,7 @@ class Item
                                unset($_SESSION['authenticated']);
                                unset($_SESSION['uid']);
                        }
-               } else {
+               } elseif (!$notify) {
                        Hook::callAll('post_remote', $item);
                }
 
@@ -1033,17 +1029,19 @@ class Item
                                $ev['guid']      = $item['guid'];
                                $ev['plink']     = $item['plink'];
                                $ev['network']   = $item['network'];
-                               $ev['protocol']  = $item['protocol'];
-                               $ev['direction'] = $item['direction'];
-                               $ev['source']    = $item['source'];
+                               $ev['protocol']  = $item['protocol'] ?? Conversation::PARCEL_UNKNOWN;
+                               $ev['direction'] = $item['direction'] ?? Conversation::UNKNOWN;
+                               $ev['source']    = $item['source'] ?? '';
 
                                $event = DBA::selectFirst('event', ['id'], ['uri' => $item['uri'], 'uid' => $item['uid']]);
                                if (DBA::isResult($event)) {
                                        $ev['id'] = $event['id'];
                                }
 
-                               $item['event-id'] = Event::store($ev);
-                               Logger::info('Event was stored', ['id' => $item['event-id']]);
+                               $event_id = Event::store($ev);
+                               $item = Event::getItemArrayForImportedId($event_id, $item);
+
+                               Logger::info('Event was stored', ['id' => $event_id]);
                        }
                }
 
@@ -1063,7 +1061,7 @@ class Item
 
                // Create Diaspora signature
                if ($item['origin'] && empty($item['diaspora_signed_text']) && ($item['gravity'] != GRAVITY_PARENT)) {
-                       $signed = Diaspora::createCommentSignature($uid, $item);
+                       $signed = Diaspora::createCommentSignature($item);
                        if (!empty($signed)) {
                                $item['diaspora_signed_text'] = json_encode($signed);
                        }
@@ -1143,12 +1141,13 @@ class Item
                        return 0;
                }
 
-               if (!$dontcache) {
-                       if ($notify) {
-                               Hook::callAll('post_local_end', $posted_item);
-                       } else {
-                               Hook::callAll('post_remote_end', $posted_item);
+               if ($notify) {
+                       if (!\Friendica\Content\Feature::isEnabled($posted_item['uid'], 'explicit_mentions') && ($posted_item['gravity'] == GRAVITY_COMMENT)) {
+                               Tag::createImplicitMentions($posted_item['uri-id'], $posted_item['thr-parent-id']);
                        }
+                       Hook::callAll('post_local_end', $posted_item);
+               } else {
+                       Hook::callAll('post_remote_end', $posted_item);
                }
 
                if ($posted_item['gravity'] === GRAVITY_PARENT) {
@@ -1161,8 +1160,6 @@ class Item
 
                Post\UserNotification::setNotification($posted_item['uri-id'], $posted_item['uid']);
 
-               check_user_notification($posted_item['uri-id'], $posted_item['uid']);
-
                // Distribute items to users who subscribed to their tags
                self::distributeByTags($posted_item);
 
@@ -1464,7 +1461,7 @@ class Item
                        }
                }
 
-               $distributed = self::insert($item, $notify, true);
+               $distributed = self::insert($item, $notify);
 
                if (!$distributed) {
                        Logger::info("Distributed item wasn't stored", ['uri-id' => $item['uri-id'], 'user' => $uid]);
@@ -1533,7 +1530,7 @@ class Item
                                $item['contact-id'] = $item['author-id'];
                        }
 
-                       $public_shadow = self::insert($item, false, true);
+                       $public_shadow = self::insert($item);
 
                        Logger::info('Stored public shadow', ['thread' => $itemid, 'id' => $public_shadow]);
                }
@@ -1592,7 +1589,7 @@ class Item
                unset($item['post-reason']);
                $item['contact-id'] = Contact::getIdForURL($item['author-link']);
 
-               $public_shadow = self::insert($item, false, true);
+               $public_shadow = self::insert($item);
 
                Logger::info('Stored public shadow', ['uri-id' => $item['uri-id'], 'id' => $public_shadow]);
 
@@ -1640,7 +1637,13 @@ class Item
                        return '';
                }
 
-               $ld = new Language(DI::l10n()->getAvailableLanguages());
+               $availableLanguages = DI::l10n()->getAvailableLanguages();
+               // See https://github.com/friendica/friendica/issues/10511
+               // Persian is manually added to language detection until a persian translation is provided for the interface, at
+               // which point it will be automatically available through `getAvailableLanguages()` and this should be removed.
+               $availableLanguages['fa'] = 'fa';
+
+               $ld = new Language($availableLanguages);
                $languages = $ld->detect($naked_body)->limit(0, 3)->close();
                if (is_array($languages)) {
                        return json_encode($languages);
@@ -2681,7 +2684,7 @@ class Item
                                        continue;
                                }
 
-                               if ((local_user() == $item['uid']) && ($item['private'] == self::PRIVATE) && ($item['contact-id'] != $app->contact['id']) && ($item['network'] == Protocol::DFRN)) {
+                               if ((local_user() == $item['uid']) && ($item['private'] == self::PRIVATE) && ($item['contact-id'] != $app->getContactId()) && ($item['network'] == Protocol::DFRN)) {
                                        $img_url = 'redir/' . $item['contact-id'] . '?url=' . urlencode($mtch[1]);
                                        $item['body'] = str_replace($mtch[0], '[img]' . $img_url . '[/img]', $item['body']);
                                }
@@ -2722,22 +2725,6 @@ class Item
                $item['hashtags'] = $tags['hashtags'];
                $item['mentions'] = $tags['mentions'];
 
-               // Compile eventual content filter reasons
-               $filter_reasons = [];
-               if (!$is_preview && public_contact() != $item['author-id']) {
-                       if (!empty($item['content-warning']) && (!local_user() || !DI::pConfig()->get(local_user(), 'system', 'disable_cw', false))) {
-                               $filter_reasons[] = DI::l10n()->t('Content warning: %s', $item['content-warning']);
-                       }
-
-                       $hook_data = [
-                               'item' => $item,
-                               'filter_reasons' => $filter_reasons
-                       ];
-                       Hook::callAll('prepare_body_content_filter', $hook_data);
-                       $filter_reasons = $hook_data['filter_reasons'];
-                       unset($hook_data);
-               }
-
                $body = $item['body'] ?? '';
                $shared = BBCode::fetchShareAttributes($body);
                if (!empty($shared['guid'])) {
@@ -2761,6 +2748,24 @@ class Item
                $item['body'] = $body;
                $s = $item["rendered-html"];
 
+               // Compile eventual content filter reasons
+               $filter_reasons = [];
+               if (!$is_preview && public_contact() != $item['author-id']) {
+                       if (!empty($item['content-warning']) && (!local_user() || !DI::pConfig()->get(local_user(), 'system', 'disable_cw', false))) {
+                               $filter_reasons[] = DI::l10n()->t('Content warning: %s', $item['content-warning']);
+                       }
+
+                       $item['attachments'] = $attachments;
+
+                       $hook_data = [
+                               'item' => $item,
+                               'filter_reasons' => $filter_reasons
+                       ];
+                       Hook::callAll('prepare_body_content_filter', $hook_data);
+                       $filter_reasons = $hook_data['filter_reasons'];
+                       unset($hook_data);
+               }
+
                $hook_data = [
                        'item' => $item,
                        'html' => $s,
@@ -2798,8 +2803,8 @@ class Item
                }
 
                // Replace friendica image url size with theme preference.
-               if (!empty($a->theme_info['item_image_size'])) {
-                       $ps = $a->theme_info['item_image_size'];
+               if (!empty($a->getThemeInfoValue('item_image_size'))) {
+                       $ps = $a->getThemeInfoValue('item_image_size');
                        $s = preg_replace('|(<img[^>]+src="[^"]+/photo/[0-9a-f]+)-[0-9]|', "$1-" . $ps, $s);
                }
 
@@ -2854,7 +2859,7 @@ class Item
         */
        private static function replaceVisualAttachments(array $attachments, string $body)
        {
-               $stamp1 = microtime(true);
+               DI::profiler()->startRecording('rendering');
 
                foreach ($attachments['visual'] as $attachment) {
                        if (!empty($attachment['preview'])) {
@@ -2863,7 +2868,7 @@ class Item
                                $body = str_replace($attachment['url'], Post\Media::getUrlForId($attachment['id']), $body);
                        }
                }
-               DI::profiler()->saveTimestamp($stamp1, 'rendering');
+               DI::profiler()->stopRecording();
                return $body;
        }
 
@@ -2877,13 +2882,13 @@ class Item
         */
        private static function addVisualAttachments(array $attachments, array $item, string $content, bool $shared)
        {
-               $stamp1 = microtime(true);
+               DI::profiler()->startRecording('rendering');
                $leading = '';
                $trailing = '';
 
                // @todo In the future we should make a single for the template engine with all media in it. This allows more flexibilty.
                foreach ($attachments['visual'] as $attachment) {
-                       if (self::containsLink($item['body'], $attachment['url'], $attachment['type'])) {
+                       if (self::containsLink($item['body'], $attachment['preview'] ?? $attachment['url'], $attachment['type'])) {
                                continue;
                        }
 
@@ -2953,7 +2958,7 @@ class Item
                        }
                }
 
-               DI::profiler()->saveTimestamp($stamp1, 'rendering');
+               DI::profiler()->stopRecording();
                return $content;
        }
 
@@ -2969,7 +2974,7 @@ class Item
         */
        private static function addLinkAttachment(int $uriid, array $attachments, string $body, string $content, bool $shared, array $ignore_links)
        {
-               $stamp1 = microtime(true);
+               DI::profiler()->startRecording('rendering');
                // @ToDo Check only for audio and video
                $preview = empty($attachments['visual']);
 
@@ -3034,7 +3039,7 @@ class Item
                } elseif (preg_match("/.*(\[attachment.*?\].*?\[\/attachment\]).*/ism", $body, $match)) {
                        $data = BBCode::getAttachmentData($match[1]);
                }
-               DI::profiler()->saveTimestamp($stamp1, 'rendering');
+               DI::profiler()->stopRecording();
 
                if (isset($data['url']) && !in_array($data['url'], $ignore_links)) {
                        if (!empty($data['description']) || !empty($data['image']) || !empty($data['preview'])) {
@@ -3082,7 +3087,7 @@ class Item
         */
        private static function addNonVisualAttachments(array $attachments, array $item, string $content)
        {
-               $stamp1 = microtime(true);
+               DI::profiler()->startRecording('rendering');
                $trailing = '';
                foreach ($attachments['additional'] as $attachment) {
                        if (strpos($item['body'], $attachment['url'])) {
@@ -3108,7 +3113,7 @@ class Item
                        $content .= '<div class="body-attach">' . $trailing . '<div class="clear"></div></div>';
                }
 
-               DI::profiler()->saveTimestamp($stamp1, 'rendering');
+               DI::profiler()->stopRecording();
                return $content;
        }