]> git.mxchange.org Git - friendica.git/blobdiff - src/Model/Post/Media.php
Replace split attachment code with PostMedia objects
[friendica.git] / src / Model / Post / Media.php
index d3d26a85114adf8dfce3a88110a4d64a44006257..2d7df08096b22d498159e0deeb5cfba3dc0245ea 100644 (file)
@@ -1,6 +1,6 @@
 <?php
 /**
- * @copyright Copyright (C) 2010-2022, the Friendica project
+ * @copyright Copyright (C) 2010-2023, the Friendica project
  *
  * @license GNU AGPL version 3 or any later version
  *
@@ -21,6 +21,7 @@
 
 namespace Friendica\Model\Post;
 
+use Friendica\Content\PageInfo;
 use Friendica\Content\Text\BBCode;
 use Friendica\Core\Logger;
 use Friendica\Core\Protocol;
@@ -30,6 +31,7 @@ use Friendica\Database\DBA;
 use Friendica\DI;
 use Friendica\Model\Contact;
 use Friendica\Model\Item;
+use Friendica\Model\ItemURI;
 use Friendica\Model\Photo;
 use Friendica\Model\Post;
 use Friendica\Network\HTTPClient\Client\HttpClientAccept;
@@ -66,18 +68,19 @@ class Media
         * Insert a post-media record
         *
         * @param array $media
-        * @return void
+        * @param bool  $force
+        * @return bool
         */
-       public static function insert(array $media, bool $force = false)
+       public static function insert(array $media, bool $force = false): bool
        {
                if (empty($media['url']) || empty($media['uri-id']) || !isset($media['type'])) {
                        Logger::warning('Incomplete media data', ['media' => $media]);
-                       return;
+                       return false;
                }
 
                if (DBA::exists('post-media', ['uri-id' => $media['uri-id'], 'preview' => $media['url']])) {
                        Logger::info('Media already exists as preview', ['uri-id' => $media['uri-id'], 'url' => $media['url'], 'callstack' => System::callstack()]);
-                       return;
+                       return false;
                }
 
                // "document" has got the lowest priority. So when the same file is both attached as document
@@ -85,7 +88,12 @@ class Media
                $found = DBA::selectFirst('post-media', ['type'], ['uri-id' => $media['uri-id'], 'url' => $media['url']]);
                if (!$force && !empty($found) && (($found['type'] != self::DOCUMENT) || ($media['type'] == self::DOCUMENT))) {
                        Logger::info('Media already exists', ['uri-id' => $media['uri-id'], 'url' => $media['url'], 'callstack' => System::callstack()]);
-                       return;
+                       return false;
+               }
+
+               if (!ItemURI::exists($media['uri-id'])) {
+                       Logger::info('Media referenced URI ID not found', ['uri-id' => $media['uri-id'], 'url' => $media['url'], 'callstack' => System::callstack()]);
+                       return false;
                }
 
                $media = self::unsetEmptyFields($media);
@@ -107,6 +115,7 @@ class Media
                } else {
                        Logger::info('Nothing to update', ['media' => $media]);
                }
+               return $result;
        }
 
        /**
@@ -153,8 +162,10 @@ class Media
         */
        public static function getAttachElement(string $href, int $length, string $type, string $title = ''): string
        {
-               $media = self::fetchAdditionalData(['type' => self::DOCUMENT, 'url' => $href,
-                       'size' => $length, 'mimetype' => $type, 'description' => $title]);
+               $media = self::fetchAdditionalData([
+                       'type' => self::DOCUMENT, 'url' => $href,
+                       'size' => $length, 'mimetype' => $type, 'description' => $title
+               ]);
 
                return '[attach]href="' . $media['url'] . '" length="' . $media['size'] .
                        '" type="' . $media['mimetype'] . '" title="' . $media['description'] . '"[/attach]';
@@ -173,7 +184,7 @@ class Media
                }
 
                // Fetch the mimetype or size if missing.
-               if (empty($media['mimetype']) || empty($media['size'])) {
+               if (Network::isValidHttpUrl($media['url']) && (empty($media['mimetype']) || empty($media['size']))) {
                        $timeout = DI::config()->get('system', 'xrd_timeout');
                        $curlResult = DI::httpClient()->head($media['url'], [HttpClientOptions::TIMEOUT => $timeout]);
 
@@ -229,20 +240,9 @@ class Media
                }
 
                if ($media['type'] == self::HTML) {
-                       $data = ParseUrl::getSiteinfoCached($media['url'], false);
-                       $media['preview'] = $data['images'][0]['src'] ?? null;
-                       $media['preview-height'] = $data['images'][0]['height'] ?? null;
-                       $media['preview-width'] = $data['images'][0]['width'] ?? null;
-                       $media['blurhash'] = $data['images'][0]['blurhash'] ?? null;
-                       $media['description'] = $data['text'] ?? null;
-                       $media['name'] = $data['title'] ?? null;
-                       $media['author-url'] = $data['author_url'] ?? null;
-                       $media['author-name'] = $data['author_name'] ?? null;
-                       $media['author-image'] = $data['author_img'] ?? null;
-                       $media['publisher-url'] = $data['publisher_url'] ?? null;
-                       $media['publisher-name'] = $data['publisher_name'] ?? null;
-                       $media['publisher-image'] = $data['publisher_img'] ?? null;
+                       $media = self::addPage($media);
                }
+
                return $media;
        }
 
@@ -265,8 +265,15 @@ class Media
                        return $media;
                }
 
-               if (!empty($item['plink']) && Strings::compareLink($item['plink'], $media['url']) &&
-                       parse_url($item['plink'], PHP_URL_HOST) != parse_url($item['uri'], PHP_URL_HOST)) {
+               if ($item['uri-id'] == $media['uri-id']) {
+                       Logger::info('Media-Uri-Id is identical to Uri-Id', ['uri-id' => $media['uri-id']]);
+                       return $media;
+               }
+
+               if (
+                       !empty($item['plink']) && Strings::compareLink($item['plink'], $media['url']) &&
+                       parse_url($item['plink'], PHP_URL_HOST) != parse_url($item['uri'], PHP_URL_HOST)
+               ) {
                        Logger::debug('Not a link to an activity', ['uri-id' => $media['uri-id'], 'url' => $media['url'], 'plink' => $item['plink'], 'uri' => $item['uri']]);
                        return $media;
                }
@@ -345,6 +352,31 @@ class Media
                return $media;
        }
 
+       /**
+        * Add page infos for HTML entries
+        *
+        * @param array $media
+        * @return array
+        */
+       private static function addPage(array $media): array
+       {
+               $data = ParseUrl::getSiteinfoCached($media['url'], false);
+               $media['preview'] = $data['images'][0]['src'] ?? null;
+               $media['preview-height'] = $data['images'][0]['height'] ?? null;
+               $media['preview-width'] = $data['images'][0]['width'] ?? null;
+               $media['blurhash'] = $data['images'][0]['blurhash'] ?? null;
+               $media['description'] = $data['text'] ?? null;
+               $media['name'] = $data['title'] ?? null;
+               $media['author-url'] = $data['author_url'] ?? null;
+               $media['author-name'] = $data['author_name'] ?? null;
+               $media['author-image'] = $data['author_img'] ?? null;
+               $media['publisher-url'] = $data['publisher_url'] ?? null;
+               $media['publisher-name'] = $data['publisher_name'] ?? null;
+               $media['publisher-image'] = $data['publisher_img'] ?? null;
+
+               return $media;
+       }
+
        /**
         * Fetch media data from local resources
         * @param array $media
@@ -428,25 +460,66 @@ class Media
        }
 
        /**
-        * Tests for path patterns that are usef for picture links in Friendica
+        * Tests for path patterns that are used for picture links in Friendica
         *
         * @param string $page    Link to the image page
         * @param string $preview Preview picture
         * @return boolean
         */
-       private static function isPictureLink(string $page, string $preview): bool
+       private static function isLinkToPhoto(string $page, string $preview): bool
        {
-               return preg_match('#/photos/.*/image/#ism', $page) && preg_match('#/photo/.*-1\.#ism', $preview);
+               return preg_match('#/photo/.*-0\.#ism', $page) && preg_match('#/photo/.*-[012]\.#ism', $preview);
+       }
+
+       /**
+        * Tests for path patterns that are used for picture links in Friendica
+        *
+        * @param string $page    Link to the image page
+        * @param string $preview Preview picture
+        * @return boolean
+        */
+       private static function isLinkToImagePage(string $page, string $preview): bool
+       {
+               return preg_match('#/photos/.*/image/#ism', $page) && preg_match('#/photo/.*-[012]\.#ism', $preview);
+       }
+
+       /**
+        * Replace the image link in Friendica image posts with a link to the image
+        *
+        * @param string $body
+        * @return string
+        */
+       public static function replaceImage(string $body): string
+       {
+               if (preg_match_all("#\[url=([^\]]+?)\]\s*\[img=([^\[\]]*)\]([^\[\]]*)\[\/img\]\s*\[/url\]#ism", $body, $pictures, PREG_SET_ORDER)) {
+                       foreach ($pictures as $picture) {
+                               if (self::isLinkToImagePage($picture[1], $picture[2])) {
+                                       $body = str_replace($picture[0], Images::getBBCodeByUrl(str_replace(['-1.', '-2.'], '-0.', $picture[2]), $picture[2], $picture[3]), $body);
+                               }
+                       }
+               }
+
+               if (preg_match_all("#\[url=([^\]]+?)\]\s*\[img\]([^\[]+?)\[/img\]\s*\[/url\]#ism", $body, $pictures, PREG_SET_ORDER)) {
+                       foreach ($pictures as $picture) {
+                               if (self::isLinkToImagePage($picture[1], $picture[2])) {
+                                       $body = str_replace($picture[0], Images::getBBCodeByUrl(str_replace(['-1.', '-2.'], '-0.', $picture[2]), $picture[2]), $body);
+                               }
+                       }
+               }
+
+               return $body;
        }
 
        /**
         * Add media links and remove them from the body
         *
         * @param integer $uriid
-        * @param string $body
+        * @param string  $body
+        * @param bool    $endmatch
+        * @param bool    $removepicturelinks
         * @return string Body without media links
         */
-       public static function insertFromBody(int $uriid, string $body, bool $endmatch = false): string
+       public static function insertFromBody(int $uriid, string $body, bool $endmatch = false, bool $removepicturelinks = false): string
        {
                $endmatchpattern = $endmatch ? '\z' : '';
                // Simplify image codes
@@ -455,13 +528,26 @@ class Media
                $attachments = [];
                if (preg_match_all("#\[url=([^\]]+?)\]\s*\[img=([^\[\]]*)\]([^\[\]]*)\[\/img\]\s*\[/url\]$endmatchpattern#ism", $body, $pictures, PREG_SET_ORDER)) {
                        foreach ($pictures as $picture) {
-                               if (!self::isPictureLink($picture[1], $picture[2])) {
-                                       continue;
+                               if (self::isLinkToImagePage($picture[1], $picture[2])) {
+                                       $body = str_replace($picture[0], '', $body);
+                                       $image = str_replace(['-1.', '-2.'], '-0.', $picture[2]);
+                                       $attachments[$image] = [
+                                               'uri-id' => $uriid, 'type' => self::IMAGE, 'url' => $image,
+                                               'preview' => $picture[2], 'description' => $picture[3]
+                                       ];
+                               } elseif (self::isLinkToPhoto($picture[1], $picture[2])) {
+                                       $body = str_replace($picture[0], '', $body);
+                                       $attachments[$picture[1]] = [
+                                               'uri-id' => $uriid, 'type' => self::IMAGE, 'url' => $picture[1],
+                                               'preview' => $picture[2], 'description' => $picture[3]
+                                       ];
+                               } elseif ($removepicturelinks) {
+                                       $body = str_replace($picture[0], '', $body);
+                                       $attachments[$picture[1]] = [
+                                               'uri-id' => $uriid, 'type' => self::UNKNOWN, 'url' => $picture[1],
+                                               'preview' => $picture[2], 'description' => $picture[3]
+                                       ];
                                }
-                               $body = str_replace($picture[0], '', $body);
-                               $image = str_replace('-1.', '-0.', $picture[2]);
-                               $attachments[$image] = ['uri-id' => $uriid, 'type' => self::IMAGE, 'url' => $image,
-                                       'preview' => $picture[2], 'description' => $picture[3]];
                        }
                }
 
@@ -474,13 +560,26 @@ class Media
 
                if (preg_match_all("#\[url=([^\]]+?)\]\s*\[img\]([^\[]+?)\[/img\]\s*\[/url\]$endmatchpattern#ism", $body, $pictures, PREG_SET_ORDER)) {
                        foreach ($pictures as $picture) {
-                               if (!self::isPictureLink($picture[1], $picture[2])) {
-                                       continue;
+                               if (self::isLinkToImagePage($picture[1], $picture[2])) {
+                                       $body = str_replace($picture[0], '', $body);
+                                       $image = str_replace(['-1.', '-2.'], '-0.', $picture[2]);
+                                       $attachments[$image] = [
+                                               'uri-id' => $uriid, 'type' => self::IMAGE, 'url' => $image,
+                                               'preview' => $picture[2], 'description' => null
+                                       ];
+                               } elseif (self::isLinkToPhoto($picture[1], $picture[2])) {
+                                       $body = str_replace($picture[0], '', $body);
+                                       $attachments[$picture[1]] = [
+                                               'uri-id' => $uriid, 'type' => self::IMAGE, 'url' => $picture[1],
+                                               'preview' => $picture[2], 'description' => null
+                                       ];
+                               } elseif ($removepicturelinks) {
+                                       $body = str_replace($picture[0], '', $body);
+                                       $attachments[$picture[1]] = [
+                                               'uri-id' => $uriid, 'type' => self::UNKNOWN, 'url' => $picture[1],
+                                               'preview' => $picture[2], 'description' => null
+                                       ];
                                }
-                               $body = str_replace($picture[0], '', $body);
-                               $image = str_replace('-1.', '-0.', $picture[2]);
-                               $attachments[$image] = ['uri-id' => $uriid, 'type' => self::IMAGE, 'url' => $image,
-                                       'preview' => $picture[2], 'description' => null];
                        }
                }
 
@@ -536,6 +635,21 @@ class Media
                return $body;
        }
 
+       /**
+        * Remove media from the body
+        *
+        * @param string $body
+        * @return string
+        */
+       public static function removeFromBody(string $body): string
+       {
+               do {
+                       $prebody = $body;
+                       $body = self::insertFromBody(0, $body, false, true);
+               } while ($prebody != $body);
+               return $body;
+       }
+
        /**
         * Add media links from a relevant url in the body
         *
@@ -543,7 +657,7 @@ class Media
         * @param string $body
         * @return void
         */
-       public static function insertFromRelevantUrl(int $uriid, string $body)
+       public static function insertFromRelevantUrl(int $uriid, string $body, string $fullbody, string $network)
        {
                // Remove all hashtags and mentions
                $body = preg_replace("/([#@!])\[url\=(.*?)\](.*?)\[\/url\]/ism", '', $body);
@@ -552,7 +666,15 @@ class Media
                if (preg_match_all("/\[url\](https?:.*?)\[\/url\]/ism", $body, $matches)) {
                        foreach ($matches[1] as $url) {
                                Logger::info('Got page url (link without description)', ['uri-id' => $uriid, 'url' => $url]);
-                               self::insert(['uri-id' => $uriid, 'type' => self::UNKNOWN, 'url' => $url]);
+                               $result = self::insert(['uri-id' => $uriid, 'type' => self::UNKNOWN, 'url' => $url], false, $network);
+                               if ($result && !in_array($network, [Protocol::ACTIVITYPUB, Protocol::OSTATUS, Protocol::DIASPORA])) {
+                                       self::revertHTMLType($uriid, $url, $fullbody);
+                                       Logger::debug('Revert HTML type', ['uri-id' => $uriid, 'url' => $url]);
+                               } elseif ($result) {
+                                       Logger::debug('Media had been added', ['uri-id' => $uriid, 'url' => $url]);
+                               } else {
+                                       Logger::debug('Media had not been added', ['uri-id' => $uriid, 'url' => $url]);
+                               }
                        }
                }
 
@@ -560,11 +682,36 @@ class Media
                if (preg_match_all("/\[url\=(https?:.*?)\].*?\[\/url\]/ism", $body, $matches)) {
                        foreach ($matches[1] as $url) {
                                Logger::info('Got page url (link with description)', ['uri-id' => $uriid, 'url' => $url]);
-                               self::insert(['uri-id' => $uriid, 'type' => self::UNKNOWN, 'url' => $url]);
+                               $result = self::insert(['uri-id' => $uriid, 'type' => self::UNKNOWN, 'url' => $url], false, $network);
+                               if ($result && !in_array($network, [Protocol::ACTIVITYPUB, Protocol::OSTATUS, Protocol::DIASPORA])) {
+                                       self::revertHTMLType($uriid, $url, $fullbody);
+                                       Logger::debug('Revert HTML type', ['uri-id' => $uriid, 'url' => $url]);
+                               } elseif ($result) {
+                                       Logger::debug('Media has been added', ['uri-id' => $uriid, 'url' => $url]);
+                               } else {
+                                       Logger::debug('Media has not been added', ['uri-id' => $uriid, 'url' => $url]);
+                               }
                        }
                }
        }
 
+       /**
+        * Revert the media type of links to UNKNOWN for DFRN posts when they aren't attached
+        *
+        * @param integer $uriid
+        * @param string $url
+        * @param string $body
+        * @return void
+        */
+       private static function revertHTMLType(int $uriid, string $url, string $body)
+       {
+               $attachment = BBCode::getAttachmentData($body);
+               if (!empty($attachment['url']) && Network::getUrlMatch($attachment['url'], $url)) {
+                       return;
+               }
+               DBA::update('post-media', ['type' => self::UNKNOWN], ['uri-id' => $uriid, 'type' => self::HTML, 'url' => $url]);
+       }
+
        /**
         * Add media links from the attachment field
         *
@@ -575,7 +722,7 @@ class Media
        public static function insertFromAttachmentData(int $uriid, string $body)
        {
                $data = BBCode::getAttachmentData($body);
-               if (empty($data))  {
+               if (empty($data)) {
                        return;
                }
 
@@ -633,7 +780,7 @@ class Media
         */
        public static function getByURIId(int $uri_id, array $types = [])
        {
-               $condition = ['uri-id' => $uri_id];
+               $condition = ["`uri-id` = ? AND `type` != ?", $uri_id, self::UNKNOWN];
 
                if (!empty($types)) {
                        $condition = DBA::mergeConditions($condition, ['type' => $types]);
@@ -642,6 +789,41 @@ class Media
                return DBA::selectToArray('post-media', [], $condition, ['order' => ['id']]);
        }
 
+       public static function getByURL(int $uri_id, string $url, array $types = [])
+       {
+               $condition = ["`uri-id` = ? AND `url` = ? AND `type` != ?", $uri_id, $url, self::UNKNOWN];
+
+               if (!empty($types)) {
+                       $condition = DBA::mergeConditions($condition, ['type' => $types]);
+               }
+
+               return DBA::selectFirst('post-media', [], $condition);
+       }
+
+       /**
+        * Retrieves the media attachment with the provided media id.
+        *
+        * @param int $id  id
+        * @return array|bool Array on success, false on error
+        * @throws \Exception
+        */
+       public static function getById(int $id)
+       {
+               return DBA::selectFirst('post-media', [], ['id' => $id]);
+       }
+
+       /**
+        * Update post-media entries
+        *
+        * @param array $fields
+        * @param int $id
+        * @return bool
+        */
+       public static function updateById(array $fields, int $id): bool
+       {
+               return DBA::update('post-media', $fields, ['id' => $id]);
+       }
+
        /**
         * Checks if media attachments are associated with the provided item ID.
         *
@@ -652,7 +834,7 @@ class Media
         */
        public static function existsByURIId(int $uri_id, array $types = []): bool
        {
-               $condition = ['uri-id' => $uri_id];
+               $condition = ["`uri-id` = ? AND `type` != ?", $uri_id, self::UNKNOWN];
 
                if (!empty($types)) {
                        $condition = DBA::mergeConditions($condition, ['type' => $types]);
@@ -662,108 +844,34 @@ class Media
        }
 
        /**
-        * Split the attachment media in the three segments "visual", "link" and "additional"
+        * Delete media by uri-id and media type
         *
-        * @param int    $uri_id URI id
-        * @param array  $links list of links that shouldn't be added
-        * @param bool   $has_media
-        * @return array attachments
+        * @param int $uri_id URI id
+        * @param array $types Media types
+        * @return bool result of deletion
+        * @throws \Exception
         */
-       public static function splitAttachments(int $uri_id, array $links = [], bool $has_media = true): array
+       public static function deleteByURIId(int $uri_id, array $types = []): bool
        {
-               $attachments = ['visual' => [], 'link' => [], 'additional' => []];
-
-               if (!$has_media) {
-                       return $attachments;
-               }
-
-               $media = self::getByURIId($uri_id);
-               if (empty($media)) {
-                       return $attachments;
-               }
-
-               $heights = [];
-               $selected = '';
-               $previews = [];
-
-               foreach ($media as $medium) {
-                       foreach ($links as $link) {
-                               if (Strings::compareLink($link, $medium['url'])) {
-                                       continue 2;
-                               }
-                       }
-
-                       // Avoid adding separate media entries for previews
-                       foreach ($previews as $preview) {
-                               if (Strings::compareLink($preview, $medium['url'])) {
-                                       continue 2;
-                               }
-                       }
-
-                       // Currently these two types are ignored here.
-                       // Posts are added differently and contacts are not displayed as attachments.
-                       if (in_array($medium['type'], [self::ACCOUNT, self::ACTIVITY])) {
-                               continue;
-                       }
-
-                       if (!empty($medium['preview'])) {
-                               $previews[] = $medium['preview'];
-                       }
-
-                       $type = explode('/', explode(';', $medium['mimetype'] ?? '')[0]);
-                       if (count($type) < 2) {
-                               Logger::info('Unknown MimeType', ['type' => $type, 'media' => $medium]);
-                               $filetype = 'unkn';
-                               $subtype = 'unkn';
-                       } else {
-                               $filetype = strtolower($type[0]);
-                               $subtype = strtolower($type[1]);
-                       }
-
-                       $medium['filetype'] = $filetype;
-                       $medium['subtype'] = $subtype;
-
-                       if ($medium['type'] == self::HTML || (($filetype == 'text') && ($subtype == 'html'))) {
-                               $attachments['link'][] = $medium;
-                               continue;
-                       }
+               $condition = ['uri-id' => $uri_id];
 
-                       if (in_array($medium['type'], [self::AUDIO, self::IMAGE]) ||
-                               in_array($filetype, ['audio', 'image'])) {
-                               $attachments['visual'][] = $medium;
-                       } elseif (($medium['type'] == self::VIDEO) || ($filetype == 'video')) {
-                               if (!empty($medium['height'])) {
-                                       // Peertube videos are delivered in many different resolutions. We pick a moderate one.
-                                       // Since only Peertube provides a "height" parameter, this wouldn't be executed
-                                       // when someone for example on Mastodon was sharing multiple videos in a single post.
-                                       $heights[$medium['height']] = $medium['url'];
-                                       $video[$medium['url']] = $medium;
-                               } else {
-                                       $attachments['visual'][] = $medium;
-                               }
-                       } else {
-                               $attachments['additional'][] = $medium;
-                       }
+               if (!empty($types)) {
+                       $condition = DBA::mergeConditions($condition, ['type' => $types]);
                }
 
-               if (!empty($heights)) {
-                       ksort($heights);
-                       foreach ($heights as $height => $url) {
-                               if (empty($selected) || $height <= 480) {
-                                       $selected = $url;
-                               }
-                       }
-
-                       if (!empty($selected)) {
-                               $attachments['visual'][] = $video[$selected];
-                               unset($video[$selected]);
-                               foreach ($video as $element) {
-                                       $attachments['additional'][] = $element;
-                               }
-                       }
-               }
+               return DBA::delete('post-media', $condition);
+       }
 
-               return $attachments;
+       /**
+        * Delete media by id
+        *
+        * @param int $id media id
+        * @return bool result of deletion
+        * @throws \Exception
+        */
+       public static function deleteById(int $id): bool
+       {
+               return DBA::delete('post-media', ['id' => $id]);
        }
 
        /**
@@ -786,7 +894,7 @@ class Media
                }
                $original_body = $body;
 
-               $body = preg_replace("/\s*\[attachment .*?\].*?\[\/attachment\]\s*/ism", '', $body);
+               $body = BBCode::removeAttachment($body);
 
                foreach (self::getByURIId($uriid, $types) as $media) {
                        if (Item::containsLink($body, $media['preview'] ?? $media['url'], $media['type'])) {
@@ -794,19 +902,7 @@ class Media
                        }
 
                        if ($media['type'] == self::IMAGE) {
-                               if (!empty($media['preview'])) {
-                                       if (!empty($media['description'])) {
-                                               $body .= "\n[url=" . $media['url'] . "][img=" . $media['preview'] . ']' . $media['description'] .'[/img][/url]';
-                                       } else {
-                                               $body .= "\n[url=" . $media['url'] . "][img]" . $media['preview'] .'[/img][/url]';
-                                       }
-                               } else {
-                                       if (!empty($media['description'])) {
-                                               $body .= "\n[img=" . $media['url'] . ']' . $media['description'] .'[/img]';
-                                       } else {
-                                               $body .= "\n[img]" . $media['url'] .'[/img]';
-                                       }
-                               }
+                               $body .= "\n" . Images::getBBCodeByUrl($media['url'], $media['preview'], $media['description'] ?? '');
                        } elseif ($media['type'] == self::AUDIO) {
                                $body .= "\n[audio]" . $media['url'] . "[/audio]\n";
                        } elseif ($media['type'] == self::VIDEO) {
@@ -821,6 +917,92 @@ class Media
                return $body;
        }
 
+       /**
+        * Add an [attachment] element to the body for a given uri-id with a HTML media element
+        *
+        * @param integer $uriid
+        * @param string $body
+        * @return string
+        */
+       public static function addHTMLAttachmentToBody(int $uriid, string $body): string
+       {
+               if (preg_match("/.*(\[attachment.*?\].*?\[\/attachment\]).*/ism", $body, $match)) {
+                       return $body;
+               }
+
+               $links = self::getByURIId($uriid, [self::HTML]);
+               if (empty($links)) {
+                       return $body;
+               }
+
+               $data = [
+                       'type' => 'link',
+                       'url'  => $links[0]['url'],
+                       'title' => $links[0]['name'],
+                       'text' => $links[0]['description'],
+                       'publisher_name' => $links[0]['publisher-name'],
+                       'publisher_url' => $links[0]['publisher-url'],
+                       'publisher_img' => $links[0]['publisher-image'],
+                       'author_name' => $links[0]['author-name'],
+                       'author_url' => $links[0]['author-url'],
+                       'author_img' => $links[0]['author-image'],
+                       'images' => [[
+                               'src' => $links[0]['preview'],
+                               'height' => $links[0]['preview-height'],
+                               'width' => $links[0]['preview-width'],
+                       ]]
+               ];
+               $body .= "\n" . PageInfo::getFooterFromData($data);
+
+               return $body;
+       }
+
+       /**
+        * Add a link to the body for a given uri-id with a HTML media element
+        *
+        * @param integer $uriid
+        * @param string $body
+        * @return string
+        */
+       public static function addHTMLLinkToBody(int $uriid, string $body): string
+       {
+               $links = self::getByURIId($uriid, [self::HTML]);
+               if (empty($links)) {
+                       return $body;
+               }
+
+               if (strpos($body, $links[0]['url'])) {
+                       return $body;
+               }
+
+               if (!empty($links[0]['name']) && ($links[0]['name'] != $links[0]['url'])) {
+                       return $body . "\n[url=" . $links[0]['url'] . ']' . $links[0]['name'] . "[/url]";
+               } else {
+                       return $body . "\n[url]" . $links[0]['url'] . "[/url]";
+               }
+       }
+
+       /**
+        * Add an [attachment] element to the body and a link to raw-body for a given uri-id with a HTML media element
+        *
+        * @param array $item
+        * @return array
+        */
+       public static function addHTMLAttachmentToItem(array $item): array
+       {
+               if (($item['gravity'] == Item::GRAVITY_ACTIVITY) || empty($item['uri-id'])) {
+                       return $item;
+               }
+
+               $item['body'] = self::addHTMLAttachmentToBody($item['uri-id'], $item['body']);
+
+               if (!empty($item['raw-body'])) {
+                       $item['raw-body'] = self::addHTMLLinkToBody($item['uri-id'], $item['raw-body']);
+               }
+
+               return $item;
+       }
+
        /**
         * Get preview link for given media id
         *