]> git.mxchange.org Git - friendica.git/blobdiff - src/Model/Item.php
Use getByNickname as suggested in code review.
[friendica.git] / src / Model / Item.php
index df6f83aec208f29ae79623c0d756cd8552484597..e8044ecd7a93b7b469952118194167b62f675851 100644 (file)
@@ -21,7 +21,6 @@
 
 namespace Friendica\Model;
 
-use Friendica\Content\PageInfo;
 use Friendica\Content\Text\BBCode;
 use Friendica\Content\Text\HTML;
 use Friendica\Core\Hook;
@@ -35,13 +34,13 @@ use Friendica\Core\Worker;
 use Friendica\Database\DBA;
 use Friendica\DI;
 use Friendica\Model\Post;
-use Friendica\Model\Post\Media;
 use Friendica\Protocol\Activity;
 use Friendica\Protocol\ActivityPub;
 use Friendica\Protocol\Diaspora;
 use Friendica\Util\DateTimeFormat;
 use Friendica\Util\Map;
 use Friendica\Util\Network;
+use Friendica\Util\Proxy;
 use Friendica\Util\Strings;
 use Friendica\Worker\Delivery;
 use LanguageDetection\Language;
@@ -173,7 +172,7 @@ class Item
 
                Logger::info('Updating per single row method', ['fields' => $fields, 'condition' => $condition]);
 
-               $items = Post::select(['id', 'origin', 'uri-id', 'uid'], $condition);
+               $items = Post::select(['id', 'origin', 'uri-id', 'uid', 'author-network'], $condition);
 
                $notify_items = [];
 
@@ -181,9 +180,14 @@ class Item
                        if (!empty($fields['body'])) {
                                Post\Media::insertFromAttachmentData($item['uri-id'], $fields['body']);
 
+                               if ($item['author-network'] != Protocol::DFRN) {
+                                       Post\Media::insertFromRelevantUrl($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
+                               // @todo On shared postings (Diaspora style and commented reshare) don't fetch content from the shared part
                                $content_fields['raw-body'] = Post\Media::insertFromBody($item['uri-id'], $content_fields['raw-body']);
                                $content_fields['raw-body'] = self::setHashtags($content_fields['raw-body']);
                        }
@@ -928,7 +932,25 @@ class Item
                if ($notify) {
                        $item['edit'] = false;
                        $item['parent'] = $parent_id;
+
+                       // Trigger automatic reactions for addons
+                       $item['api_source'] = true;
+
+                       // We have to tell the hooks who we are - this really should be improved
+                       if (!local_user()) {
+                               $_SESSION['authenticated'] = true;
+                               $_SESSION['uid'] = $uid;
+                               $dummy_session = true;
+                       } else {
+                               $dummy_session = false;
+                       }
+
                        Hook::callAll('post_local', $item);
+
+                       if ($dummy_session) {
+                               unset($_SESSION['authenticated']);
+                               unset($_SESSION['uid']);
+                       }
                } else {
                        Hook::callAll('post_remote', $item);
                }
@@ -959,8 +981,20 @@ class Item
                        self::setOwnerforResharedItem($item);
                }
 
+               if (isset($item['attachments'])) {
+                       foreach ($item['attachments'] as $attachment) {
+                               $attachment['uri-id'] = $item['uri-id'];
+                               Post\Media::insert($attachment);
+                       }
+                       unset($item['attachments']);
+               }
+
                Post\Media::insertFromAttachmentData($item['uri-id'], $item['body']);
 
+               if (!DBA::exists('contact', ['id' => $item['author-id'], 'network' => Protocol::DFRN])) {
+                       Post\Media::insertFromRelevantUrl($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']);
@@ -1706,7 +1740,7 @@ class Item
                                        return ("[bookmark=" . str_replace("#", "#", $match[1]) . "]" . str_replace("#", "#", $match[2]) . "[/bookmark]");
                                }, $body);
 
-                       $body = preg_replace_callback("/\[attachment (.*)\](.*?)\[\/attachment\]/ism",
+                       $body = preg_replace_callback("/\[attachment (.*?)\](.*?)\[\/attachment\]/ism",
                                function ($match) {
                                        return ("[attachment " . str_replace("#", "#", $match[1]) . "]" . $match[2] . "[/attachment]");
                                }, $body);
@@ -1955,13 +1989,6 @@ class Item
                        $result = true;
                }
 
-               // Trigger automatic reactions for addons
-               $datarray['api_source'] = true;
-
-               // We have to tell the hooks who we are - this really should be improved
-               $_SESSION['authenticated'] = true;
-               $_SESSION['uid'] = $contact['uid'];
-
                return (bool)$result;
        }
 
@@ -2639,10 +2666,10 @@ class Item
                        unset($hook_data);
                }
 
-               $orig_body = $item['body'];
-               $item['body'] = preg_replace("/\s*\[attachment .*\].*?\[\/attachment\]\s*/ism", '', $item['body']);
+               $body = $item['body'] ?? '';
+               $item['body'] = preg_replace("/\s*\[attachment .*?\].*?\[\/attachment\]\s*/ism", "\n", $item['body']);
                self::putInCache($item);
-               $item['body'] = $orig_body;
+               $item['body'] = $body;
                $s = $item["rendered-html"];
 
                $hook_data = [
@@ -2666,19 +2693,23 @@ class Item
                if (!empty($shared['guid'])) {
                        $shared_item = Post::selectFirst(['uri-id', 'plink'], ['guid' => $shared['guid']]);
                        $shared_uri_id = $shared_item['uri-id'] ?? 0;
-                       $shared_plink = $shared_item['plink'] ?? '';
-                       $attachments = Post\Media::splitAttachments($shared_uri_id);
+                       $shared_links = [strtolower($shared_item['plink'] ?? '')];
+                       $attachments = Post\Media::splitAttachments($shared_uri_id, $shared['guid']);
                        $s = self::addVisualAttachments($attachments, $item, $s, true);
-                       $s = self::addLinkAttachment($attachments, $item, $s, true, '');
+                       $s = self::addLinkAttachment($attachments, $body, $s, true, []);
                        $s = self::addNonVisualAttachments($attachments, $item, $s, true);
+                       $shared_links = array_merge($shared_links, array_column($attachments['visual'], 'url'));
+                       $shared_links = array_merge($shared_links, array_column($attachments['link'], 'url'));
+                       $shared_links = array_merge($shared_links, array_column($attachments['additional'], 'url'));
+                       $body = preg_replace("/\s*\[share .*?\].*?\[\/share\]\s*/ism", '', $body);
                } else {
                        $shared_uri_id = 0;
-                       $shared_plink = '';
+                       $shared_links = [];
                }
 
-               $attachments = Post\Media::splitAttachments($item['uri-id']);
+               $attachments = Post\Media::splitAttachments($item['uri-id'], $item['guid'] ?? '', $shared_links);
                $s = self::addVisualAttachments($attachments, $item, $s, false);
-               $s = self::addLinkAttachment($attachments, $item, $s, false, $shared_plink);
+               $s = self::addLinkAttachment($attachments, $body, $s, false, $shared_links);
                $s = self::addNonVisualAttachments($attachments, $item, $s, false);
 
                // Map.
@@ -2712,6 +2743,12 @@ class Item
         */
        public static function containsLink(string $body, string $url)
        {
+               // Make sure that for example site parameters aren't used when testing if the link is contained in the body
+               $urlparts = parse_url($url);
+               unset($urlparts['query']);
+               unset($urlparts['fragment']);
+               $url = Network::unparseURL($urlparts);
+
                if (strpos($body, $url)) {
                        return true;
                }
@@ -2734,9 +2771,11 @@ class Item
         */
        private static function addVisualAttachments(array $attachments, array $item, string $content, bool $shared)
        {
+               $stamp1 = microtime(true);
                $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'])) {
                                continue;
@@ -2746,13 +2785,21 @@ class Item
                                'network' => $item['author-network'], 'url' => $item['author-link']];
                        $the_url = Contact::magicLinkByContact($author, $attachment['url']);
 
+                       if (!empty($attachment['preview'])) {
+                               $preview_url = Proxy::proxifyUrl(Contact::magicLinkByContact($author, $attachment['preview']));
+                       } else {
+                               $preview_url = '';
+                       }
+
                        if (($attachment['filetype'] == 'video')) {
                                /// @todo Move the template to /content as well
                                $media = Renderer::replaceMacros(Renderer::getMarkupTemplate('video_top.tpl'), [
                                        '$video' => [
-                                               'id'     => $item['author-id'],
-                                               'src'    => $the_url,
-                                               'mime'   => $attachment['mimetype'],
+                                               'id'      => $attachment['id'],
+                                               'src'     => $the_url,
+                                               'name'    => $attachment['name'] ?: $attachment['url'],
+                                               'preview' => $preview_url,
+                                               'mime'    => $attachment['mimetype'],
                                        ],
                                ]);
                                if ($item['post-type'] == Item::PT_VIDEO) {
@@ -2763,8 +2810,9 @@ class Item
                        } elseif ($attachment['filetype'] == 'audio') {
                                $media = Renderer::replaceMacros(Renderer::getMarkupTemplate('content/audio.tpl'), [
                                        '$audio' => [
-                                               'id'     => $item['author-id'],
+                                               'id'     => $attachment['id'],
                                                'src'    => $the_url,
+                                               'name'    => $attachment['name'] ?: $attachment['url'],
                                                'mime'   => $attachment['mimetype'],
                                        ],
                                ]);
@@ -2774,19 +2822,28 @@ class Item
                                        $trailing .= $media;
                                }
                        } elseif ($attachment['filetype'] == 'image') {
+                               if (empty($preview_url) && (max($attachment['width'], $attachment['height']) > 600)) {
+                                       $preview_url = Proxy::proxifyUrl($the_url, false, ($attachment['width'] > $attachment['height']) ? Proxy::SIZE_MEDIUM : Proxy::SIZE_LARGE);
+                               }
                                $media = Renderer::replaceMacros(Renderer::getMarkupTemplate('content/image.tpl'), [
                                        '$image' => [
-                                               'src'    => $the_url,
+                                               'src'    => Proxy::proxifyUrl($the_url),
+                                               'preview' => $preview_url,
                                                'attachment'   => $attachment,
                                        ],
                                ]);
-                               $trailing .= $media;
+                               // On Diaspora posts the attached pictures are leading
+                               if ($item['network'] == Protocol::DIASPORA) {
+                                       $leading .= $media;
+                               } else {
+                                       $trailing .= $media;
+                               }
                        }
                }
 
                if ($shared) {
-                       $content = str_replace(BBCode::ANCHOR, '<div class="body-attach">' . $leading . '<div class="clear"></div></div>' . BBCode::ANCHOR, $content);
-                       $content = str_replace(BBCode::ANCHOR, BBCode::ANCHOR . '<div class="body-attach">' . $trailing . '<div class="clear"></div></div>', $content);
+                       $content = str_replace(BBCode::TOP_ANCHOR, '<div class="body-attach">' . $leading . '<div class="clear"></div></div>' . BBCode::TOP_ANCHOR, $content);
+                       $content = str_replace(BBCode::BOTTOM_ANCHOR, '<div class="body-attach">' . $trailing . '<div class="clear"></div></div>' . BBCode::BOTTOM_ANCHOR, $content);
                } else {
                        if ($leading != '') {
                                $content = '<div class="body-attach">' . $leading . '<div class="clear"></div></div>' . $content;
@@ -2797,6 +2854,7 @@ class Item
                        }
                }
 
+               DI::profiler()->saveTimestamp($stamp1, 'rendering');
                return $content;
        }
 
@@ -2804,19 +2862,31 @@ class Item
         * Add link attachment to the content
         *
         * @param array  $attachments
-        * @param array  $item
+        * @param string $body
         * @param string $content
         * @param bool   $shared
+        * @param array  $ignore_links A list of URLs to ignore
         * @return string modified content
         */
-       private static function addLinkAttachment(array $attachments, array $item, string $content, bool $shared, string $ignore_link)
+       private static function addLinkAttachment(array $attachments, string $body, string $content, bool $shared, array $ignore_links)
        {
+               $stamp1 = microtime(true);
                // @ToDo Check only for audio and video
                $preview = empty($attachments['visual']);
 
                if (!empty($attachments['link'])) {
                        foreach ($attachments['link'] as $link) {
-                               if (!Strings::compareLink($link['url'], $ignore_link)) {
+                               $found = false;
+                               foreach ($ignore_links as $ignore_link) {
+                                       if (Strings::compareLink($link['url'], $ignore_link)) {
+                                               $found = true;
+                                       }
+                               }
+                               // @todo Judge between the links to use the one with most information
+                               if (!$found && (empty($attachment) || !empty($link['author-name']) ||
+                                       (empty($attachment['name']) && !empty($link['name'])) ||
+                                       (empty($attachment['description']) && !empty($link['description'])) ||
+                                       (empty($attachment['preview']) && !empty($link['preview'])))) {
                                        $attachment = $link;
                                }
                        }
@@ -2824,32 +2894,78 @@ class Item
 
                if (!empty($attachment)) {
                        $data = [
-                               'author_img'     => $attachment['author-image'] ?? '',
-                               'author_name'    => $attachment['author-name'] ?? '',
-                               'author_url'     => $attachment['author-url'] ?? '',
-                               'publisher_img'  => $attachment['publisher-image'] ?? '',
-                               'publisher_name' => $attachment['publisher-name'] ?? '',
-                               'publisher_url'  => $attachment['publisher-url'] ?? '',
-                               'text'           => $attachment['description'] ?? '',
-                               'title'          => $attachment['name'] ?? '',
-                               'type'           => 'link',
-                               'url'            => $attachment['url'] ?? '',
-                       ];
+                               'after' => '',
+                               'author_name' => $attachment['author-name'] ?? '',
+                               'author_url' =>  $attachment['author-url'] ?? '',
+                               'description' => $attachment['description'] ?? '',
+                               'image' => '',
+                               'preview' => '',
+                               'provider_name' => $attachment['publisher-name'] ?? '',
+                               'provider_url' => $attachment['publisher-url'] ?? '',
+                               'text' => '',
+                               'title' => $attachment['name'] ?? '',
+                               'type' => 'link',
+                               'url' => $attachment['url']];
+
+                       if ($preview) {
+                               if ($attachment['preview-width'] >= 500) {
+                                       $data['image'] = $attachment['preview'] ?? '';
+                               } else {
+                                       $data['preview'] = $attachment['preview'] ?? '';
+                               }
+                       }
 
-                       if ($preview && !empty($attachment['preview']) && !empty($attachment['preview-height']) && !empty($attachment['preview-width'])) {
-                               $data['images'][] = ['src' => $attachment['preview'],
-                                       'width' => $attachment['preview-width'], 'height' => $attachment['preview-height']];
+                       if (!empty($data['description']) && !empty($content)) {
+                               similar_text($data['description'], $content, $percent);
+                       } else {
+                               $percent = 0;
+                       }
+
+                       if (!empty($data['description']) && (($data['title'] == $data['description']) || ($percent > 95) || (strpos($content, $data['description']) !== false))) {
+                               $data['description'] = '';
+                       }
+
+                       if (($data['author_name'] ?? '') == ($data['provider_name'] ?? '')) {
+                               $data['author_name'] = '';
                        }
-                       $footer = PageInfo::getFooterFromData($data);
-               } elseif (preg_match("/.*(\[attachment.*?\].*?\[\/attachment\]).*/ism", $item['body'], $match)) {
-                       $footer = $match[1];
+
+                       if (($data['author_url'] ?? '') == ($data['provider_url'] ?? '')) {
+                               $data['author_url'] = '';
+                       }
+               } elseif (preg_match("/.*(\[attachment.*?\].*?\[\/attachment\]).*/ism", $body, $match)) {
+                       $data = BBCode::getAttachmentData($match[1]);
                }
+               DI::profiler()->saveTimestamp($stamp1, 'rendering');
+
+               if (isset($data['url']) && !in_array($data['url'], $ignore_links)) {
+                       if (!empty($data['description']) || !empty($data['image']) || !empty($data['preview'])) {
+                               $parts = parse_url($data['url']);
+                               if (!empty($parts['scheme']) && !empty($parts['host'])) {
+                                       if (empty($data['provider_name'])) {
+                                               $data['provider_name'] = $parts['host'];
+                                       }
+                                       if (empty($data['provider_url']) || empty(parse_url($data['provider_url'], PHP_URL_SCHEME))) {
+                                               $data['provider_url'] = $parts['scheme'] . '://' . $parts['host'];
+
+                                               if (!empty($parts['port'])) {
+                                                       $data['provider_url'] .= ':' . $parts['port'];
+                                               }
+                                       }
+                               }
+
+                               // @todo Use a template
+                               $rendered = BBCode::convertAttachment('', BBCode::INTERNAL, false, $data);
+                       } elseif (!self::containsLink($content, $data['url'])) {
+                               $rendered = Renderer::replaceMacros(Renderer::getMarkupTemplate('content/link.tpl'), [
+                                       '$url'  => $data['url'],
+                                       '$title' => $data['title'],
+                               ]);
+                       } else {
+                               return $content;
+                       }
 
-               if (!empty($footer)) {
-                       // @todo Use a template
-                       $rendered = BBCode::convert($footer);
                        if ($shared) {
-                               return str_replace(BBCode::ANCHOR, BBCode::ANCHOR . $rendered, $content);
+                               return str_replace(BBCode::BOTTOM_ANCHOR, BBCode::BOTTOM_ANCHOR . $rendered, $content);
                        } else {
                                return $content . $rendered;
                        }
@@ -2867,6 +2983,7 @@ class Item
         */
        private static function addNonVisualAttachments(array $attachments, array $item, string $content)
        {
+               $stamp1 = microtime(true);
                $trailing = '';
                foreach ($attachments['additional'] as $attachment) {
                        if (strpos($item['body'], $attachment['url'])) {
@@ -2892,6 +3009,7 @@ class Item
                        $content .= '<div class="body-attach">' . $trailing . '<div class="clear"></div></div>';
                }
 
+               DI::profiler()->saveTimestamp($stamp1, 'rendering');
                return $content;
        }
 
@@ -2909,18 +3027,19 @@ class Item
                                'href' => "display/" . $item['guid'],
                                'orig' => "display/" . $item['guid'],
                                'title' => DI::l10n()->t('View on separate page'),
-                               'orig_title' => DI::l10n()->t('view on separate page'),
+                               'orig_title' => DI::l10n()->t('View on separate page'),
                        ];
 
                        if (!empty($item['plink'])) {
-                               $ret["href"] = DI::baseUrl()->remove($item['plink']);
-                               $ret["title"] = DI::l10n()->t('link to source');
+                               $ret['href'] = DI::baseUrl()->remove($item['plink']);
+                               $ret['title'] = DI::l10n()->t('Link to source');
                        }
                } elseif (!empty($item['plink']) && ($item['private'] != self::PRIVATE)) {
                        $ret = [
                                'href' => $item['plink'],
                                'orig' => $item['plink'],
-                               'title' => DI::l10n()->t('link to source'),
+                               'title' => DI::l10n()->t('Link to source'),
+                               'orig_title' => DI::l10n()->t('Link to source'),
                        ];
                } else {
                        $ret = [];
@@ -3158,4 +3277,41 @@ class Item
 
                return true;
        }
+
+       /**
+        * Improve the data in shared posts
+        *
+        * @param array $item
+        * @return string body
+        */
+       public static function improveSharedDataInBody(array $item)
+       {
+               $shared = BBCode::fetchShareAttributes($item['body']);
+               if (empty($shared['link'])) {
+                       return $item['body'];   
+               }
+               
+               $id = self::fetchByLink($shared['link']);
+               Logger::info('Fetched shared post', ['uri-id' => $item['uri-id'], 'id' => $id, 'author' => $shared['profile'], 'url' => $shared['link'], 'guid' => $shared['guid'], 'callstack' => System::callstack()]);
+               if (!$id) {
+                       return $item['body'];   
+               }
+
+               $shared_item = Post::selectFirst(['author-name', 'author-link', 'author-avatar', 'plink', 'created', 'guid', 'title', 'body'], ['id' => $id]);
+               if (!DBA::isResult($shared_item)) {
+                       return $item['body'];   
+               }
+
+               $shared_content = BBCode::getShareOpeningTag($shared_item['author-name'], $shared_item['author-link'], $shared_item['author-avatar'], $shared_item['plink'], $shared_item['created'], $shared_item['guid']);
+
+               if (!empty($shared_item['title'])) {
+                       $shared_content .= '[h3]'.$shared_item['title'].'[/h3]'."\n";
+               }
+
+               $shared_content .= $shared_item['body'];
+
+               $item['body'] = preg_replace("/\[share.*?\](.*)\[\/share\]/ism", $shared_content . '[/share]', $item['body']);
+               Logger::info('New shared data', ['uri-id' => $item['uri-id'], 'id' => $id, 'shared_item' => $shared_item]);
+               return $item['body'];
+       }
 }