]> 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 012ec6b62162a4e84d5853690ba879c0e52133b6..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)
@@ -463,7 +468,7 @@ class Media
         */
        private static function isLinkToPhoto(string $page, string $preview): bool
        {
-               return preg_match('#/photo/.*-0\.#ism', $page) && preg_match('#/photo/.*-[01]\.#ism', $preview);
+               return preg_match('#/photo/.*-0\.#ism', $page) && preg_match('#/photo/.*-[012]\.#ism', $preview);
        }
 
        /**
@@ -475,7 +480,34 @@ class Media
         */
        private static function isLinkToImagePage(string $page, string $preview): bool
        {
-               return preg_match('#/photos/.*/image/#ism', $page) && preg_match('#/photo/.*-[01]\.#ism', $preview);
+               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;
        }
 
        /**
@@ -498,7 +530,7 @@ class Media
                        foreach ($pictures as $picture) {
                                if (self::isLinkToImagePage($picture[1], $picture[2])) {
                                        $body = str_replace($picture[0], '', $body);
-                                       $image = str_replace('-1.', '-0.', $picture[2]);
+                                       $image = str_replace(['-1.', '-2.'], '-0.', $picture[2]);
                                        $attachments[$image] = [
                                                'uri-id' => $uriid, 'type' => self::IMAGE, 'url' => $image,
                                                'preview' => $picture[2], 'description' => $picture[3]
@@ -530,7 +562,7 @@ class Media
                        foreach ($pictures as $picture) {
                                if (self::isLinkToImagePage($picture[1], $picture[2])) {
                                        $body = str_replace($picture[0], '', $body);
-                                       $image = str_replace('-1.', '-0.', $picture[2]);
+                                       $image = str_replace(['-1.', '-2.'], '-0.', $picture[2]);
                                        $attachments[$image] = [
                                                'uri-id' => $uriid, 'type' => self::IMAGE, 'url' => $image,
                                                'preview' => $picture[2], 'description' => null
@@ -635,7 +667,7 @@ class Media
                        foreach ($matches[1] as $url) {
                                Logger::info('Got page url (link without description)', ['uri-id' => $uriid, 'url' => $url]);
                                $result = self::insert(['uri-id' => $uriid, 'type' => self::UNKNOWN, 'url' => $url], false, $network);
-                               if ($result && ($network == Protocol::DFRN)) {
+                               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) {
@@ -651,7 +683,7 @@ class Media
                        foreach ($matches[1] as $url) {
                                Logger::info('Got page url (link with description)', ['uri-id' => $uriid, 'url' => $url]);
                                $result = self::insert(['uri-id' => $uriid, 'type' => self::UNKNOWN, 'url' => $url], false, $network);
-                               if ($result && ($network == Protocol::DFRN)) {
+                               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) {
@@ -757,6 +789,17 @@ 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.
         *
@@ -831,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
         *
@@ -966,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) {