]> 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 cc5b16d1e7d73f3aeb1e656cf744e0367d17f9be..2d7df08096b22d498159e0deeb5cfba3dc0245ea 100644 (file)
@@ -265,6 +265,11 @@ class Media
                        return $media;
                }
 
+               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)
@@ -489,7 +494,7 @@ class Media
                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], '[url=' . str_replace(['-1.', '-2.'], '-0.', $picture[2]) . '][img=' . $picture[2] . ']' . $picture[3] . '[/img][/url]', $body);
+                                       $body = str_replace($picture[0], Images::getBBCodeByUrl(str_replace(['-1.', '-2.'], '-0.', $picture[2]), $picture[2], $picture[3]), $body);
                                }
                        }
                }
@@ -497,7 +502,7 @@ class Media
                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], '[url=' . str_replace(['-1.', '-2.'], '-0.', $picture[2]) . '][img]' . $picture[2] . '[/img][/url]', $body);
+                                       $body = str_replace($picture[0], Images::getBBCodeByUrl(str_replace(['-1.', '-2.'], '-0.', $picture[2]), $picture[2]), $body);
                                }
                        }
                }
@@ -869,113 +874,6 @@ class Media
                return DBA::delete('post-media', ['id' => $id]);
        }
 
-       /**
-        * Split the attachment media in the three segments "visual", "link" and "additional"
-        *
-        * @param int    $uri_id URI id
-        * @param array  $links list of links that shouldn't be added
-        * @param bool   $has_media
-        * @return array attachments
-        */
-       public static function splitAttachments(int $uri_id, array $links = [], bool $has_media = true): array
-       {
-               $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;
-                       }
-
-                       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($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 $attachments;
-       }
-
        /**
         * Add media attachments to the body
         *
@@ -1004,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) {