]> git.mxchange.org Git - friendica.git/blobdiff - src/Model/Photo.php
Merge pull request #8147 from annando/fetch-post
[friendica.git] / src / Model / Photo.php
index 9e26711401cafd618c0e80c7d71c9907d8444b8c..804ae328ef4d06a5eb58889d4aea45c356266b73 100644 (file)
@@ -2,13 +2,11 @@
 
 /**
  * @file src/Model/Photo.php
- * @brief This file contains the Photo class for database interface
+ * This file contains the Photo class for database interface
  */
 namespace Friendica\Model;
 
-use Friendica\Core\Cache;
-use Friendica\Core\Config;
-use Friendica\Core\L10n;
+use Friendica\Core\Cache\Duration;
 use Friendica\Core\Logger;
 use Friendica\Core\System;
 use Friendica\Database\DBA;
@@ -30,7 +28,7 @@ require_once "include/dba.php";
 class Photo
 {
        /**
-        * @brief Select rows from the photo table and returns them as array
+        * Select rows from the photo table and returns them as array
         *
         * @param array $fields     Array of selected fields, empty for all
         * @param array $conditions Array of fields for conditions
@@ -51,7 +49,7 @@ class Photo
        }
 
        /**
-        * @brief Retrieve a single record from the photo table
+        * Retrieve a single record from the photo table
         *
         * @param array $fields     Array of selected fields, empty for all
         * @param array $conditions Array of fields for conditions
@@ -72,7 +70,7 @@ class Photo
        }
 
        /**
-        * @brief Get photos for user id
+        * Get photos for user id
         *
         * @param integer $uid        User id
         * @param string  $resourceid Rescource ID of the photo
@@ -93,7 +91,7 @@ class Photo
        }
 
        /**
-        * @brief Get a photo for user id
+        * Get a photo for user id
         *
         * @param integer $uid        User id
         * @param string  $resourceid Rescource ID of the photo
@@ -116,7 +114,7 @@ class Photo
        }
 
        /**
-        * @brief Get a single photo given resource id and scale
+        * Get a single photo given resource id and scale
         *
         * This method checks for permissions. Returns associative array
         * on success, "no sign" image info, if user has no permission,
@@ -147,7 +145,7 @@ class Photo
        }
 
        /**
-        * @brief Check if photo with given conditions exists
+        * Check if photo with given conditions exists
         *
         * @param array $conditions Array of extra conditions
         *
@@ -161,7 +159,7 @@ class Photo
 
 
        /**
-        * @brief Get Image object for given row id. null if row id does not exist
+        * Get Image object for given row id. null if row id does not exist
         *
         * @param array $photo Photo data. Needs at least 'id', 'type', 'backend-class', 'backend-ref'
         *
@@ -171,7 +169,8 @@ class Photo
         */
        public static function getImageForPhoto(array $photo)
        {
-               if (empty($photo['backend-class'])) {
+               $backendClass = DI::storageManager()->getByName($photo['backend-class'] ?? '');
+               if ($backendClass === null) {
                        // legacy data storage in "data" column
                        $i = self::selectFirst(['data'], ['id' => $photo['id']]);
                        if ($i === false) {
@@ -179,7 +178,6 @@ class Photo
                        }
                        $data = $i['data'];
                } else {
-                       $backendClass = DI::storageManager()->getByName($photo['backend-class'] ?? '');
                        $backendRef = $photo['backend-ref'] ?? '';
                        $data = $backendClass->get($backendRef);
                }
@@ -192,7 +190,7 @@ class Photo
        }
 
        /**
-        * @brief Return a list of fields that are associated with the photo table
+        * Return a list of fields that are associated with the photo table
         *
         * @return array field list
         * @throws \Exception
@@ -206,7 +204,7 @@ class Photo
        }
 
        /**
-        * @brief Construct a photo array for a system resource image
+        * Construct a photo array for a system resource image
         *
         * @param string $filename Image file name relative to code root
         * @param string $mimetype Image mime type. Defaults to "image/jpeg"
@@ -230,7 +228,7 @@ class Photo
 
 
        /**
-        * @brief store photo metadata in db and binary in default backend
+        * store photo metadata in db and binary in default backend
         *
         * @param Image   $Image     Image object with data
         * @param integer $uid       User ID
@@ -320,7 +318,7 @@ class Photo
 
 
        /**
-        * @brief Delete info from table and data from storage
+        * Delete info from table and data from storage
         *
         * @param array $conditions Field condition(s)
         * @param array $options    Options array, Optional
@@ -346,7 +344,7 @@ class Photo
        }
 
        /**
-        * @brief Update a photo
+        * Update a photo
         *
         * @param array         $fields     Contains the fields that are updated
         * @param array         $conditions Condition array with the key values
@@ -525,7 +523,7 @@ class Photo
        }
 
        /**
-        * @brief Fetch the photo albums that are available for a viewer
+        * Fetch the photo albums that are available for a viewer
         *
         * The query in this function is cost intensive, so it is cached.
         *
@@ -540,9 +538,9 @@ class Photo
                $sql_extra = Security::getPermissionsSQLByUserId($uid);
 
                $key = "photo_albums:".$uid.":".local_user().":".remote_user();
-               $albums = Cache::get($key);
+               $albums = DI::cache()->get($key);
                if (is_null($albums) || $update) {
-                       if (!Config::get("system", "no_count", false)) {
+                       if (!DI::config()->get("system", "no_count", false)) {
                                /// @todo This query needs to be renewed. It is really slow
                                // At this time we just store the data in the cache
                                $albums = q("SELECT COUNT(DISTINCT `resource-id`) AS `total`, `album`, ANY_VALUE(`created`) AS `created`
@@ -551,7 +549,7 @@ class Photo
                                        GROUP BY `album` ORDER BY `created` DESC",
                                        intval($uid),
                                        DBA::escape("Contact Photos"),
-                                       DBA::escape(L10n::t("Contact Photos"))
+                                       DBA::escape(DI::l10n()->t("Contact Photos"))
                                );
                        } else {
                                // This query doesn't do the count and is much faster
@@ -560,10 +558,10 @@ class Photo
                                        WHERE `uid` = %d  AND `album` != '%s' AND `album` != '%s' $sql_extra",
                                        intval($uid),
                                        DBA::escape("Contact Photos"),
-                                       DBA::escape(L10n::t("Contact Photos"))
+                                       DBA::escape(DI::l10n()->t("Contact Photos"))
                                );
                        }
-                       Cache::set($key, $albums, Cache::DAY);
+                       DI::cache()->set($key, $albums, Duration::DAY);
                }
                return $albums;
        }
@@ -576,7 +574,7 @@ class Photo
        public static function clearAlbumCache($uid)
        {
                $key = "photo_albums:".$uid.":".local_user().":".remote_user();
-               Cache::set($key, null, Cache::DAY);
+               DI::cache()->set($key, null, Duration::DAY);
        }
 
        /**
@@ -590,6 +588,25 @@ class Photo
                return System::createGUID(32, false);
        }
 
+       /**
+        * Extracts the rid from a local photo URI
+        *
+        * @param string $image_uri The URI of the photo
+        * @return string The rid of the photo, or an empty string if the URI is not local
+        */
+       public static function ridFromURI(string $image_uri)
+       {
+               if (!stristr($image_uri, DI::baseUrl() . '/photo/')) {
+                       return '';
+               }
+               $image_uri = substr($image_uri, strrpos($image_uri, '/') + 1);
+               $image_uri = substr($image_uri, 0, strpos($image_uri, '-'));
+               if (!strlen($image_uri)) {
+                       return '';
+               }
+               return $image_uri;
+       }
+
        /**
         * Changes photo permissions that had been embedded in a post
         *
@@ -616,12 +633,8 @@ class Photo
                }
 
                foreach ($images as $image) {
-                       if (!stristr($image, DI::baseUrl() . '/photo/')) {
-                               continue;
-                       }
-                       $image_uri = substr($image,strrpos($image,'/') + 1);
-                       $image_uri = substr($image_uri,0, strpos($image_uri,'-'));
-                       if (!strlen($image_uri)) {
+                       $image_rid = self::ridFromURI($image);
+                       if (empty($image_rid)) {
                                continue;
                        }
 
@@ -630,7 +643,7 @@ class Photo
 
                        $condition = [
                                'allow_cid' => $srch, 'allow_gid' => '', 'deny_cid' => '', 'deny_gid' => '',
-                               'resource-id' => $image_uri, 'uid' => $uid
+                               'resource-id' => $image_rid, 'uid' => $uid
                        ];
                        if (!Photo::exists($condition)) {
                                continue;
@@ -640,7 +653,7 @@ class Photo
 
                        $fields = ['allow_cid' => $str_contact_allow, 'allow_gid' => $str_group_allow,
                                        'deny_cid' => $str_contact_deny, 'deny_gid' => $str_group_deny];
-                       $condition = ['resource-id' => $image_uri, 'uid' => $uid];
+                       $condition = ['resource-id' => $image_rid, 'uid' => $uid];
                        Logger::info('Set permissions', ['condition' => $condition, 'permissions' => $fields]);
                        Photo::update($fields, $condition);
                }