]> git.mxchange.org Git - friendica.git/blobdiff - src/Model/Tag.php
Issue 13221: Diaspora posts are now stored correctly
[friendica.git] / src / Model / Tag.php
index 729ef0dd2bbeff6133d973c554e230ca688c2275..1645dc1255b540a9f66742b3c987f6a7bd93e78b 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
  *
@@ -31,6 +31,7 @@ use Friendica\Database\DBA;
 use Friendica\DI;
 use Friendica\Protocol\ActivityPub;
 use Friendica\Util\DateTimeFormat;
+use Friendica\Util\Network;
 use Friendica\Util\Strings;
 
 /**
@@ -49,14 +50,16 @@ class Tag
         */
        const IMPLICIT_MENTION  = 8;
        /**
-        * An exclusive mention transmits the post only to the target account without transmitting it to the followers, usually a forum.
+        * An exclusive mention transmits the post only to the target account without transmitting it to the followers, usually a group.
         */
        const EXCLUSIVE_MENTION = 9;
 
-       const TO  = 10;
-       const CC  = 11;
-       const BTO = 12;
-       const BCC = 13;
+       const TO         = 10;
+       const CC         = 11;
+       const BTO        = 12;
+       const BCC        = 13;
+       const AUDIENCE   = 14;
+       const ATTRIBUTED = 15;
 
        const ACCOUNT             = 1;
        const GENERAL_COLLECTION  = 2;
@@ -73,13 +76,14 @@ class Tag
        /**
         * Store tag/mention elements
         *
-        * @param integer $uriid
-        * @param integer $type
-        * @param string  $name
-        * @param string  $url
-        * @param integer $target
+        * @param integer $uriId
+        * @param integer $type Tag type
+        * @param string  $name Tag name
+        * @param string  $url Contact URL (optional)
+        * @param integer $target Target (default: null)
+        * @return void
         */
-       public static function store(int $uriid, int $type, string $name, string $url = '', int $target = null)
+       public static function store(int $uriId, int $type, string $name, string $url = '', int $target = null)
        {
                if ($type == self::HASHTAG) {
                        // Trim Unicode non-word characters
@@ -88,7 +92,7 @@ class Tag
                        $tags = explode(self::TAG_CHARACTER[self::HASHTAG], $name);
                        if (count($tags) > 1) {
                                foreach ($tags as $tag) {
-                                       self::store($uriid, $type, $tag, $url);
+                                       self::store($uriId, $type, $tag, $url);
                                }
                                return;
                        }
@@ -101,7 +105,7 @@ class Tag
                $cid = 0;
                $tagid = 0;
 
-               if (in_array($type, [self::MENTION, self::EXCLUSIVE_MENTION, self::IMPLICIT_MENTION, self::TO, self::CC, self::BTO, self::BCC])) {
+               if (in_array($type, [self::MENTION, self::EXCLUSIVE_MENTION, self::IMPLICIT_MENTION, self::TO, self::CC, self::BTO, self::BCC, self::AUDIENCE, self::ATTRIBUTED])) {
                        if (empty($url)) {
                                // No mention without a contact url
                                return;
@@ -118,7 +122,7 @@ class Tag
                                $tag = DBA::selectFirst('tag', ['name', 'type'], ['url' => $url]);
                                if (!empty($tag)) {
                                        if ($tag['name'] != substr($name, 0, 96)) {
-                                               DBA::update('tag', ['name' => substr($name, 0, 96)], ['url' => $url]);
+                                               DBA::update('tag', ['name' => substr($name, 0, 96)], ['url' => $url, 'type' => $tag['type']]);
                                        }
                                        if (!empty($target) && ($tag['type'] != $target)) {
                                                DBA::update('tag', ['type' => $target], ['url' => $url]);
@@ -128,7 +132,7 @@ class Tag
                }
 
                if (empty($cid)) {
-                       if (!in_array($type, [self::TO, self::CC, self::BTO, self::BCC])) {
+                       if (!in_array($type, [self::TO, self::CC, self::BTO, self::BCC, self::AUDIENCE, self::ATTRIBUTED])) {
                                if (($type != self::HASHTAG) && !empty($url) && ($url != $name)) {
                                        $url = strtolower($url);
                                } else {
@@ -142,7 +146,7 @@ class Tag
                        }
                }
 
-               $fields = ['uri-id' => $uriid, 'type' => $type, 'tid' => $tagid, 'cid' => $cid];
+               $fields = ['uri-id' => $uriId, 'type' => $type, 'tid' => $tagid, 'cid' => $cid];
 
                if (in_array($type, [self::MENTION, self::EXCLUSIVE_MENTION, self::IMPLICIT_MENTION])) {
                        $condition = $fields;
@@ -155,7 +159,7 @@ class Tag
 
                DBA::insert('post-tag', $fields, Database::INSERT_IGNORE);
 
-               Logger::info('Stored tag/mention', ['uri-id' => $uriid, 'tag-id' => $tagid, 'contact-id' => $cid, 'name' => $name, 'type' => $type, 'callstack' => System::callstack(8)]);
+               Logger::debug('Stored tag/mention', ['uri-id' => $uriId, 'tag-id' => $tagid, 'contact-id' => $cid, 'name' => $name, 'type' => $type, 'callstack' => System::callstack(8)]);
        }
 
        /**
@@ -201,7 +205,7 @@ class Tag
                        }
                }
 
-               if (!empty($target) && !empty($tag['url']) && empty($tag['type'])) {
+               if (!empty($target) && !empty($tag['url']) && ($tag['type'] != $target)) {
                        DBA::update('tag', ['type' => $target], ['url' => $url]);
                }
 
@@ -213,32 +217,38 @@ class Tag
        }
 
        /**
-        * Get a tag id for a given tag name and url
+        * Get a tag id for a given tag name and URL
         *
-        * @param string $name
+        * @param string $name Name of tag
         * @param string $url
-        * @param int    $type
-        * @return void
+        * @param int    $type Type of tag
+        * @return int Tag id
         */
-       public static function getID(string $name, string $url = '', int $type = null)
+       public static function getID(string $name, string $url = '', int $type = null): int
        {
                $fields = ['name' => substr($name, 0, 96), 'url' => $url];
 
-               if (!empty($type)) {
-                       $fields['type'] = $type;
-               }
-
-               $tag = DBA::selectFirst('tag', ['id'], $fields);
+               $tag = DBA::selectFirst('tag', ['id', 'type'], $fields);
                if (DBA::isResult($tag)) {
+                       if (empty($tag['type']) && !empty($type)) {
+                               DBA::update('tag', ['type' => $type], $fields);
+                       }
                        return $tag['id'];
                }
 
+               if (!empty($type)) {
+                       $fields['type'] = $type;
+               }
+
                DBA::insert('tag', $fields, Database::INSERT_IGNORE);
                $tid = DBA::lastInsertId();
                if (!empty($tid)) {
                        return $tid;
                }
 
+               // Also log type
+               $fields['type'] = $type;
+
                Logger::error('No tag id created', $fields);
                return 0;
        }
@@ -246,20 +256,20 @@ class Tag
        /**
         * Store tag/mention elements
         *
-        * @param integer $uriid
+        * @param integer $uriId
         * @param string $hash
         * @param string $name
         * @param string $url
-        * @param boolean $probing
+        * @return void
         */
-       public static function storeByHash(int $uriid, string $hash, string $name, string $url = '', $probing = true)
+       public static function storeByHash(int $uriId, string $hash, string $name, string $url = '')
        {
                $type = self::getTypeForHash($hash);
                if ($type == self::UNKNOWN) {
                        return;
                }
 
-               self::store($uriid, $type, $name, $url, $probing);
+               self::store($uriId, $type, $name, $url);
        }
 
        /**
@@ -270,10 +280,10 @@ class Tag
         *
         * @return array Tag list
         */
-       public static function getFromBody(string $body, string $tags = null)
+       public static function getFromBody(string $body, string $tags = null): array
        {
                if (is_null($tags)) {
-                       $tags =  self::TAG_CHARACTER[self::HASHTAG] . self::TAG_CHARACTER[self::MENTION] . self::TAG_CHARACTER[self::EXCLUSIVE_MENTION];
+                       $tags = self::TAG_CHARACTER[self::HASHTAG] . self::TAG_CHARACTER[self::MENTION] . self::TAG_CHARACTER[self::EXCLUSIVE_MENTION];
                }
 
                if (!preg_match_all("/([" . $tags . "])\[url\=([^\[\]]*)\]([^\[\]]*)\[\/url\]/ism", $body, $result, PREG_SET_ORDER)) {
@@ -286,24 +296,43 @@ class Tag
        /**
         * Store tags and mentions from the body
         *
-        * @param integer $uriid   URI-Id
+        * @param integer $uriId   URI-Id
         * @param string  $body    Body of the post
         * @param string  $tags    Accepted tags
-        * @param boolean $probing Perform a probing for contacts, adding them if needed
+        * @return void
+        */
+       public static function storeFromBody(int $uriId, string $body, string $tags = null)
+       {
+               $item = ['uri-id' => $uriId, 'body' => $body, 'quote-uri-id' => null];
+               self::storeFromArray($item, $tags);
+       }
+
+       /**
+        * Store tags and mentions from the item array
+        *
+        * @param array   $item    Item array
+        * @param string  $tags    Accepted tags
+        * @return void
         */
-       public static function storeFromBody(int $uriid, string $body, string $tags = null, $probing = true)
+       public static function storeFromArray(array $item, string $tags = null)
        {
-               Logger::info('Check for tags', ['uri-id' => $uriid, 'hash' => $tags, 'callstack' => System::callstack()]);
+               Logger::info('Check for tags', ['uri-id' => $item['uri-id'], 'hash' => $tags, 'callstack' => System::callstack()]);
 
-               $result = self::getFromBody($body, $tags);
-               if (empty($result)) {
-                       return;
+               if (is_null($tags)) {
+                       $tags = self::TAG_CHARACTER[self::HASHTAG] . self::TAG_CHARACTER[self::MENTION] . self::TAG_CHARACTER[self::EXCLUSIVE_MENTION];
                }
 
-               Logger::info('Found tags', ['uri-id' => $uriid, 'hash' => $tags, 'result' => $result]);
+               foreach (self::getFromBody($item['body'], $tags) as $tag) {
+                       self::storeByHash($item['uri-id'], $tag[1], $tag[3], $tag[2]);
+               }
 
-               foreach ($result as $tag) {
-                       self::storeByHash($uriid, $tag[1], $tag[3], $tag[2], $probing);
+               $shared = DI::contentItem()->getSharedPost($item, ['uri-id']);
+
+               // Search for hashtags in the shared body (but only if hashtags are wanted)
+               if (!empty($shared) && (strpos($tags, self::TAG_CHARACTER[self::HASHTAG]) !== false)) {
+                       foreach (self::getByURIId($shared['post']['uri-id'], [self::HASHTAG]) as $tag) {
+                               self::store($item['uri-id'], $tag['type'], $tag['name'], $tag['url']);
+                       }
                }
        }
 
@@ -312,50 +341,52 @@ class Tag
         * This function is needed in the intermediate phase.
         * Later we can call item::setHashtags in advance to have all tags converted.
         *
-        * @param integer $uriid URI-Id
+        * @param integer $uriId URI-Id
         * @param string  $body   Body of the post
+        * @return void
         */
-       public static function storeRawTagsFromBody(int $uriid, string $body)
+       public static function storeRawTagsFromBody(int $uriId, string $body)
        {
-               Logger::info('Check for tags', ['uri-id' => $uriid, 'callstack' => System::callstack()]);
+               Logger::info('Check for tags', ['uri-id' => $uriId, 'callstack' => System::callstack()]);
 
                $result = BBCode::getTags($body);
                if (empty($result)) {
                        return;
                }
 
-               Logger::info('Found tags', ['uri-id' => $uriid, 'result' => $result]);
+               Logger::info('Found tags', ['uri-id' => $uriId, 'result' => $result]);
 
                foreach ($result as $tag) {
                        if (substr($tag, 0, 1) != self::TAG_CHARACTER[self::HASHTAG]) {
                                continue;
                        }
-                       self::storeByHash($uriid, substr($tag, 0, 1), substr($tag, 1));
+                       self::storeByHash($uriId, substr($tag, 0, 1), substr($tag, 1));
                }
        }
 
        /**
         * Checks for stored hashtags and mentions for the given post
         *
-        * @param integer $uriid
+        * @param integer $uriId
         * @return bool
         */
-       public static function existsForPost(int $uriid)
+       public static function existsForPost(int $uriId): bool
        {
-               return DBA::exists('post-tag', ['uri-id' => $uriid, 'type' => [self::HASHTAG, self::MENTION, self::EXCLUSIVE_MENTION, self::IMPLICIT_MENTION]]);
+               return DBA::exists('post-tag', ['uri-id' => $uriId, 'type' => [self::HASHTAG, self::MENTION, self::EXCLUSIVE_MENTION, self::IMPLICIT_MENTION]]);
        }
 
        /**
         * Remove tag/mention
         *
-        * @param integer $uriid
-        * @param integer $type
-        * @param string $name
-        * @param string $url
+        * @param integer $uriId
+        * @param integer $type Type
+        * @param string $name Name
+        * @param string $url URL
+        * @return void
         */
-       public static function remove(int $uriid, int $type, string $name, string $url = '')
+       public static function remove(int $uriId, int $type, string $name, string $url = '')
        {
-               $condition = ['uri-id' => $uriid, 'type' => $type, 'url' => $url];
+               $condition = ['uri-id' => $uriId, 'type' => $type, 'url' => $url];
                if ($type == self::HASHTAG) {
                        $condition['name'] = $name;
                }
@@ -365,35 +396,36 @@ class Tag
                        return;
                }
 
-               Logger::info('Removing tag/mention', ['uri-id' => $uriid, 'tid' => $tag['tid'], 'name' => $name, 'url' => $url, 'callstack' => System::callstack(8)]);
-               DBA::delete('post-tag', ['uri-id' => $uriid, 'type' => $type, 'tid' => $tag['tid'], 'cid' => $tag['cid']]);
+               Logger::debug('Removing tag/mention', ['uri-id' => $uriId, 'tid' => $tag['tid'], 'name' => $name, 'url' => $url, 'callstack' => System::callstack(8)]);
+               DBA::delete('post-tag', ['uri-id' => $uriId, 'type' => $type, 'tid' => $tag['tid'], 'cid' => $tag['cid']]);
        }
 
        /**
         * Remove tag/mention
         *
-        * @param integer $uriid
+        * @param integer $uriId
         * @param string $hash
         * @param string $name
         * @param string $url
+        * @return void
         */
-       public static function removeByHash(int $uriid, string $hash, string $name, string $url = '')
+       public static function removeByHash(int $uriId, string $hash, string $name, string $url = '')
        {
                $type = self::getTypeForHash($hash);
                if ($type == self::UNKNOWN) {
                        return;
                }
 
-               self::remove($uriid, $type, $name, $url);
+               self::remove($uriId, $type, $name, $url);
        }
 
        /**
         * Get the type for the given hash
         *
         * @param string $hash
-        * @return integer type
+        * @return integer Tag type
         */
-       private static function getTypeForHash(string $hash)
+       private static function getTypeForHash(string $hash): int
        {
                if ($hash == self::TAG_CHARACTER[self::MENTION]) {
                        return self::MENTION;
@@ -411,22 +443,23 @@ class Tag
        /**
         * Create implicit mentions for a given post
         *
-        * @param integer $uri_id
-        * @param integer $parent_uri_id
+        * @param integer $uriId
+        * @param integer $parentUriId
+        * @return void
         */
-       public static function createImplicitMentions(int $uri_id, int $parent_uri_id)
+       public static function createImplicitMentions(int $uriId, int $parentUriId)
        {
                // Always mention the direct parent author
-               $parent = Post::selectFirst(['author-link', 'author-name'], ['uri-id' => $parent_uri_id]);
-               self::store($uri_id, self::IMPLICIT_MENTION, $parent['author-name'], $parent['author-link']);
+               $parent = Post::selectFirst(['author-link', 'author-name'], ['uri-id' => $parentUriId]);
+               self::store($uriId, self::IMPLICIT_MENTION, $parent['author-name'], $parent['author-link']);
 
                if (DI::config()->get('system', 'disable_implicit_mentions')) {
                        return;
                }
 
-               $tags = DBA::select('tag-view', ['name', 'url'], ['uri-id' => $parent_uri_id, 'type' => [self::MENTION, self::EXCLUSIVE_MENTION, self::IMPLICIT_MENTION]]);
+               $tags = DBA::select('tag-view', ['name', 'url'], ['uri-id' => $parentUriId, 'type' => [self::MENTION, self::EXCLUSIVE_MENTION, self::IMPLICIT_MENTION]]);
                while ($tag = DBA::fetch($tags)) {
-                       self::store($uri_id, self::IMPLICIT_MENTION, $tag['name'], $tag['url']);
+                       self::store($uriId, self::IMPLICIT_MENTION, $tag['name'], $tag['url']);
                }
                DBA::close($tags);
        }
@@ -434,30 +467,49 @@ class Tag
        /**
         * Retrieves the terms from the provided type(s) associated with the provided item ID.
         *
-        * @param int       $item_id
-        * @param int|array $type
-        * @return array
+        * @param int   $uriId
+        * @param array $type Tag type(s)
+        * @return array|bool Array on success, false on error
         * @throws \Exception
         */
-       public static function getByURIId(int $uri_id, array $type = [self::HASHTAG, self::MENTION, self::EXCLUSIVE_MENTION, self::IMPLICIT_MENTION])
+       public static function getByURIId(int $uriId, array $type = [self::HASHTAG, self::MENTION, self::EXCLUSIVE_MENTION, self::IMPLICIT_MENTION])
        {
-               $condition = ['uri-id' => $uri_id, 'type' => $type];
+               $condition = ['uri-id' => $uriId, 'type' => $type];
                return DBA::selectToArray('tag-view', ['type', 'name', 'url', 'tag-type'], $condition);
        }
 
+       /**
+        * Checks if the given url is mentioned in the post
+        *
+        * @param integer $uriId
+        * @param string $url
+        * @param array $type
+        *
+        * @return boolean
+        */
+       public static function isMentioned(int $uriId, string $url, array $type = [self::MENTION, self::EXCLUSIVE_MENTION, self::AUDIENCE]): bool
+       {
+               $tags = self::getByURIId($uriId, $type);
+               foreach ($tags as $tag) {
+                       if (Strings::compareLink($url, $tag['url'])) {
+                               return true;
+                       }
+               }
+               return false;
+       }
+
        /**
         * Return a string with all tags and mentions
         *
-        * @param integer $uri_id
-        * @param array   $type
+        * @param integer $uriId
+        * @param array   $type Tag type(s)
         * @return string tags and mentions
         * @throws \Exception
         */
-       public static function getCSVByURIId(int $uri_id, array $type = [self::HASHTAG, self::MENTION, self::EXCLUSIVE_MENTION, self::IMPLICIT_MENTION])
+       public static function getCSVByURIId(int $uriId, array $type = [self::HASHTAG, self::MENTION, self::EXCLUSIVE_MENTION, self::IMPLICIT_MENTION]): string
        {
                $tag_list = [];
-               $tags = self::getByURIId($uri_id, $type);
-               foreach ($tags as $tag) {
+               foreach (self::getByURIId($uriId, $type) as $tag) {
                        $tag_list[] = self::TAG_CHARACTER[$tag['type']] . '[url=' . $tag['url'] . ']' . $tag['name'] . '[/url]';
                }
 
@@ -473,7 +525,7 @@ class Tag
         * @throws \Friendica\Network\HTTPException\InternalServerErrorException
         * @throws \ImagickException
         */
-       public static function populateFromItem(&$item)
+       public static function populateFromItem(array &$item): array
        {
                $return = [
                        'tags' => [],
@@ -482,10 +534,13 @@ class Tag
                        'implicit_mentions' => [],
                ];
 
-               $searchpath = DI::baseUrl() . "/search?tag=";
+               $searchpath = DI::baseUrl() . '/search?tag=';
 
-               $taglist = DBA::select('tag-view', ['type', 'name', 'url', 'cid'],
-                       ['uri-id' => $item['uri-id'], 'type' => [self::HASHTAG, self::MENTION, self::EXCLUSIVE_MENTION, self::IMPLICIT_MENTION]]);
+               $taglist = DBA::select(
+                       'tag-view',
+                       ['type', 'name', 'url', 'cid'],
+                       ['uri-id' => $item['uri-id'], 'type' => [self::HASHTAG, self::MENTION, self::EXCLUSIVE_MENTION, self::IMPLICIT_MENTION]]
+               );
                while ($tag = DBA::fetch($taglist)) {
                        if ($tag['url'] == '') {
                                $tag['url'] = $searchpath . rawurlencode($tag['name']);
@@ -494,7 +549,7 @@ class Tag
                        $orig_tag = $tag['url'];
 
                        $prefix = self::TAG_CHARACTER[$tag['type']];
-                       switch($tag['type']) {
+                       switch ($tag['type']) {
                                case self::HASHTAG:
                                        if ($orig_tag != $tag['url']) {
                                                $item['body'] = str_replace($orig_tag, $tag['url'], $item['body']);
@@ -503,6 +558,7 @@ class Tag
                                        $return['hashtags'][] = '<bdi>' . $prefix . '<a href="' . $tag['url'] . '" target="_blank" rel="noopener noreferrer">' . htmlspecialchars($tag['name']) . '</a></bdi>';
                                        $return['tags'][] = '<bdi>' . $prefix . '<a href="' . $tag['url'] . '" target="_blank" rel="noopener noreferrer">' . htmlspecialchars($tag['name']) . '</a></bdi>';
                                        break;
+
                                case self::MENTION:
                                case self::EXCLUSIVE_MENTION:
                                        if (!empty($tag['cid'])) {
@@ -513,9 +569,13 @@ class Tag
                                        $return['mentions'][] = '<bdi>' . $prefix . '<a href="' . $tag['url'] . '" target="_blank" rel="noopener noreferrer">' . htmlspecialchars($tag['name']) . '</a></bdi>';
                                        $return['tags'][] = '<bdi>' . $prefix . '<a href="' . $tag['url'] . '" target="_blank" rel="noopener noreferrer">' . htmlspecialchars($tag['name']) . '</a></bdi>';
                                        break;
+
                                case self::IMPLICIT_MENTION:
                                        $return['implicit_mentions'][] = $prefix . $tag['name'];
                                        break;
+
+                               default:
+                                       Logger::warning('Unknown tag type found', $tag);
                        }
                }
                DBA::close($taglist);
@@ -530,11 +590,13 @@ class Tag
         * @param integer $uid
         * @return integer number of posts
         */
-       public static function countByTag(string $search, int $uid = 0)
+       public static function countByTag(string $search, int $uid = 0): int
        {
                $condition = ["`name` = ? AND (`uid` = ? OR (`uid` = ? AND NOT `global`))
                        AND (`network` IN (?, ?, ?, ?) OR (`uid` = ? AND `uid` != ?))",
-                       $search, 0, $uid, Protocol::ACTIVITYPUB, Protocol::DFRN, Protocol::DIASPORA, Protocol::OSTATUS, $uid, 0];
+                       $search, 0, $uid,
+                       Protocol::ACTIVITYPUB, Protocol::DFRN, Protocol::DIASPORA, Protocol::OSTATUS, $uid, 0,
+               ];
 
                return DBA::count('tag-search-view', $condition);
        }
@@ -542,18 +604,20 @@ class Tag
        /**
         * Search posts for given tag
         *
-        * @param string $search
-        * @param integer $uid
-        * @param integer $start
-        * @param integer $limit
+        * @param string $search Tag to search for
+        * @param integer $uid User Id
+        * @param integer $start Starting record
+        * @param integer $limit Maximum count of records
         * @param integer $last_uriid
         * @return array with URI-ID
         */
-       public static function getURIIdListByTag(string $search, int $uid = 0, int $start = 0, int $limit = 100, int $last_uriid = 0)
+       public static function getURIIdListByTag(string $search, int $uid = 0, int $start = 0, int $limit = 100, int $last_uriid = 0): array
        {
                $condition = ["`name` = ? AND (`uid` = ? OR (`uid` = ? AND NOT `global`))
                        AND (`network` IN (?, ?, ?, ?) OR (`uid` = ? AND `uid` != ?))",
-                       $search, 0, $uid, Protocol::ACTIVITYPUB, Protocol::DFRN, Protocol::DIASPORA, Protocol::OSTATUS, $uid, 0];
+                       $search, 0, $uid,
+                       Protocol::ACTIVITYPUB, Protocol::DFRN, Protocol::DIASPORA, Protocol::OSTATUS, $uid, 0,
+               ];
 
                if (!empty($last_uriid)) {
                        $condition = DBA::mergeConditions($condition, ["`uri-id` < ?", $last_uriid]);
@@ -566,13 +630,13 @@ class Tag
 
                $tags = DBA::select('tag-search-view', ['uri-id'], $condition, $params);
 
-               $uriids = [];
+               $uriIds = [];
                while ($tag = DBA::fetch($tags)) {
-                       $uriids[] = $tag['uri-id'];
+                       $uriIds[] = $tag['uri-id'];
                }
                DBA::close($tags);
 
-               return $uriids;
+               return $uriIds;
        }
 
        /**
@@ -580,25 +644,25 @@ class Tag
         *
         * @param int $period Period in hours to consider posts
         * @param int $limit  Number of returned tags
+        * @param int $offset  Page offset in results
         * @return array
         * @throws \Exception
         */
-       public static function getGlobalTrendingHashtags(int $period, $limit = 10)
+       public static function getGlobalTrendingHashtags(int $period, int $limit = 10, int $offset = 0): array
        {
-               $tags = DI::cache()->get('global_trending_tags-' . $period . '-' . $limit);
-               if (!empty($tags)) {
-                       return $tags;
-               } else {
-                       return self::setGlobalTrendingHashtags($period, $limit);
+               $tags = DI::cache()->get("global_trending_tags-$period");
+               if (empty($tags)) {
+                       $tags = self::setGlobalTrendingHashtags($period, 1000);
                }
+               return array_slice($tags, $offset, $limit);
        }
 
        /**
         * Fetch the blocked tags as SQL
         *
-        * @return string
+        * @return string SQL for blocked tag names or empty string
         */
-       private static function getBlockedSQL()
+       private static function getBlockedSQL(): string
        {
                $blocked_txt = DI::config()->get('system', 'blocked_tags');
                if (empty($blocked_txt)) {
@@ -606,8 +670,10 @@ class Tag
                }
 
                $blocked = explode(',', $blocked_txt);
-               array_walk($blocked, function(&$value) { $value = "'" . DBA::escape(trim($value)) . "'";});
-               return " AND NOT `name` IN (" . implode(',', $blocked) . ")";
+               array_walk($blocked, function (&$value) {
+                       $value = "'" . DBA::escape(trim($value)) . "'";
+               });
+               return ' AND NOT `name` IN (' . implode(',', $blocked) . ')';
        }
 
        /**
@@ -618,27 +684,38 @@ class Tag
         * @return array
         * @throws \Exception
         */
-       public static function setGlobalTrendingHashtags(int $period, int $limit = 10)
+       public static function setGlobalTrendingHashtags(int $period, int $limit = 10): array
        {
-               // Get a uri-id that is at least X hours old.
-               // We use the uri-id in the query for the hash tags since this is much faster
-               $post = Post::selectFirstThread(['uri-id'], ["`uid` = ? AND `received` < ?", 0, DateTimeFormat::utc('now - ' . $period . ' hour')],
-                       ['order' => ['received' => true]]);
+               /*
+               * Get a uri-id that is at least X hours old.
+               * We use the uri-id in the query for the hash tags since this is much faster
+               */
+               $post = Post::selectFirstThread(
+                       ['uri-id'],
+                       ["`uid` = ? AND `received` < ?", 0, DateTimeFormat::utc('now - ' . $period . ' hour')],
+                       ['order' => ['received' => true]]
+               );
+
                if (empty($post['uri-id'])) {
                        return [];
                }
 
                $block_sql = self::getBlockedSQL();
 
-               $tagsStmt = DBA::p("SELECT `name` AS `term`, COUNT(*) AS `score`, COUNT(DISTINCT(`author-id`)) as `authors`
+               $tagsStmt = DBA::p(
+                       "SELECT `name` AS `term`, COUNT(*) AS `score`, COUNT(DISTINCT(`author-id`)) as `authors`
                        FROM `tag-search-view`
                        WHERE `private` = ? AND `uid` = ? AND `uri-id` > ? $block_sql
                        GROUP BY `term` ORDER BY `authors` DESC, `score` DESC LIMIT ?",
-                       Item::PUBLIC, 0, $post['uri-id'], $limit);
+                       Item::PUBLIC,
+                       0,
+                       $post['uri-id'],
+                       $limit
+               );
 
                if (DBA::isResult($tagsStmt)) {
                        $tags = DBA::toArray($tagsStmt);
-                       DI::cache()->set('global_trending_tags-' . $period . '-' . $limit, $tags, Duration::DAY);
+                       DI::cache()->set("global_trending_tags-$period", $tags, Duration::HOUR);
                        return $tags;
                }
 
@@ -650,17 +727,17 @@ class Tag
         *
         * @param int $period Period in hours to consider posts
         * @param int $limit  Number of returned tags
+        * @param int $offset  Page offset in results
         * @return array
         * @throws \Exception
         */
-       public static function getLocalTrendingHashtags(int $period, $limit = 10)
+       public static function getLocalTrendingHashtags(int $period, $limit = 10, int $offset = 0): array
        {
-               $tags = DI::cache()->get('local_trending_tags-' . $period . '-' . $limit);
-               if (!empty($tags)) {
-                       return $tags;
-               } else {
-                       return self::setLocalTrendingHashtags($period, $limit);
+               $tags = DI::cache()->get("local_trending_tags-$period");
+               if (empty($tags)) {
+                       $tags = self::setLocalTrendingHashtags($period, 1000);
                }
+               return array_slice($tags, $offset, $limit);
        }
 
        /**
@@ -671,27 +748,34 @@ class Tag
         * @return array
         * @throws \Exception
         */
-       public static function setLocalTrendingHashtags(int $period, int $limit = 10)
+       public static function setLocalTrendingHashtags(int $period, int $limit = 10): array
        {
                // Get a uri-id that is at least X hours old.
                // We use the uri-id in the query for the hash tags since this is much faster
-               $post = Post::selectFirstThread(['uri-id'], ["`uid` = ? AND `received` < ?", 0, DateTimeFormat::utc('now - ' . $period . ' hour')],
-                       ['order' => ['received' => true]]);
+               $post = Post::selectFirstThread(
+                       ['uri-id'],
+                       ["`uid` = ? AND `received` < ?", 0, DateTimeFormat::utc('now - ' . $period . ' hour')],
+                       ['order' => ['received' => true]]
+               );
                if (empty($post['uri-id'])) {
                        return [];
                }
 
                $block_sql = self::getBlockedSQL();
 
-               $tagsStmt = DBA::p("SELECT `name` AS `term`, COUNT(*) AS `score`, COUNT(DISTINCT(`author-id`)) as `authors`
+               $tagsStmt = DBA::p(
+                       "SELECT `name` AS `term`, COUNT(*) AS `score`, COUNT(DISTINCT(`author-id`)) as `authors`
                        FROM `tag-search-view`
                        WHERE `private` = ? AND `wall` AND `origin` AND `uri-id` > ? $block_sql
                        GROUP BY `term` ORDER BY `authors` DESC, `score` DESC LIMIT ?",
-                       Item::PUBLIC, $post['uri-id'], $limit);
+                       Item::PUBLIC,
+                       $post['uri-id'],
+                       $limit
+               );
 
                if (DBA::isResult($tagsStmt)) {
                        $tags = DBA::toArray($tagsStmt);
-                       DI::cache()->set('local_trending_tags-' . $period . '-' . $limit, $tags, Duration::DAY);
+                       DI::cache()->set("local_trending_tags-$period", $tags, Duration::HOUR);
                        return $tags;
                }
 
@@ -701,11 +785,11 @@ class Tag
        /**
         * Check if the provided tag is of one of the provided term types.
         *
-        * @param string $tag
+        * @param string $tag Tag name
         * @param int    ...$types
         * @return bool
         */
-       public static function isType($tag, ...$types)
+       public static function isType(string $tag, ...$types): bool
        {
                $tag_chars = [];
                foreach ($types as $type) {
@@ -723,7 +807,7 @@ class Tag
         * @param string $tag
         * @return array User list
         */
-       private static function getUIDListByTag(string $tag)
+       private static function getUIDListByTag(string $tag): array
        {
                $uids = [];
                $searches = DBA::select('search', ['uid'], ['term' => $tag]);
@@ -738,13 +822,13 @@ class Tag
        /**
         * Fetch user who subscribed to the tags of the given item
         *
-        * @param integer $uri_id
+        * @param integer $uriId
         * @return array User list
         */
-       public static function getUIDListByURIId(int $uri_id)
+       public static function getUIDListByURIId(int $uriId): array
        {
                $uids = [];
-               $tags = self::getByURIId($uri_id, [self::HASHTAG]);
+               $tags = self::getByURIId($uriId, [self::HASHTAG]);
 
                foreach ($tags as $tag) {
                        $uids = array_merge($uids, self::getUIDListByTag(self::TAG_CHARACTER[self::HASHTAG] . $tag['name']));