Use the cached activity function

[friendica.git] / src / Model / Tag.php

diff --git a/src/Model/Tag.php b/src/Model/Tag.php
index c7dbbb336b42c637f9564f403c69cfe4c5919d44..435dfda7cbf2711c3775e83321a4e474b92c177f 100644 (file)
--- a/src/Model/Tag.php
+++ b/src/Model/Tag.php
@@ -73,7 +73,7 @@ class Tag
        /**
         * Store tag/mention elements
         *
-        * @param integer $uriId URI id
+        * @param integer $uriId
         * @param integer $type Tag type
         * @param string  $name Tag name
         * @param string  $url Contact URL (optional)
@@ -156,7 +156,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)]);
        }
 
        /**
@@ -217,7 +217,7 @@ class Tag
         * Get a tag id for a given tag name and URL
         *
         * @param string $name Name of tag
-        * @param string $url URL
+        * @param string $url
         * @param int    $type Type of tag
         * @return int Tag id
         */
@@ -243,9 +243,8 @@ class Tag
                        return $tid;
                }
 
-               // Also log type and tag id
+               // Also log type
                $fields['type'] = $type;
-               $fields['tid'] = $tid;
 
                Logger::error('No tag id created', $fields);
                return 0;
@@ -254,10 +253,10 @@ class Tag
        /**
         * Store tag/mention elements
         *
-        * @param integer $uriId URI id
-        * @param string $hash Hash
-        * @param string $name Name
-        * @param string $url URL
+        * @param integer $uriId
+        * @param string $hash
+        * @param string $name
+        * @param string $url
         * @param boolean $probing Whether probing is active
         * @return void
         */
@@ -372,7 +371,7 @@ class Tag
        /**
         * Remove tag/mention
         *
-        * @param integer $uriId URI id
+        * @param integer $uriId
         * @param integer $type Type
         * @param string $name Name
         * @param string $url URL
@@ -390,17 +389,17 @@ class Tag
                        return;
                }
 
-               Logger::info('Removing tag/mention', ['uri-id' => $uriId, 'tid' => $tag['tid'], 'name' => $name, 'url' => $url, 'callstack' => System::callstack(8)]);
+               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 URI id
-        * @param string $hash Hash
-        * @param string $name Name
-        * @param string $url URL
+        * @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 = '')
@@ -416,7 +415,7 @@ class Tag
        /**
         * Get the type for the given hash
         *
-        * @param string $hash Hash
+        * @param string $hash
         * @return integer Tag type
         */
        private static function getTypeForHash(string $hash): int
@@ -437,8 +436,8 @@ class Tag
        /**
         * Create implicit mentions for a given post
         *
-        * @param integer $uriId URI Id
-        * @param integer $parentUriId Parent URI id
+        * @param integer $uriId
+        * @param integer $parentUriId
         * @return void
         */
        public static function createImplicitMentions(int $uriId, int $parentUriId)
@@ -461,7 +460,7 @@ class Tag
        /**
         * Retrieves the terms from the provided type(s) associated with the provided item ID.
         *
-        * @param int       $uriId URI id
+        * @param int   $uriId
         * @param array $type Tag type(s)
         * @return array|bool Array on success, false on error
         * @throws \Exception
@@ -472,10 +471,30 @@ class Tag
                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]): 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 $uriId URI id
+        * @param integer $uriId
         * @param array   $type Tag type(s)
         * @return string tags and mentions
         * @throws \Exception
@@ -546,7 +565,7 @@ class Tag
                                        break;
 
                                default:
-                                       Logger:warning('Unknown tag type found', $tag);
+                                       Logger::warning('Unknown tag type found', $tag);
                        }
                }
                DBA::close($taglist);
@@ -565,15 +584,8 @@ class Tag
        {
                $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);
@@ -593,15 +605,8 @@ class Tag
        {
                $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)) {
@@ -686,9 +691,7 @@ class Tag
                        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'],
+                       Item::PUBLIC, 0, $post['uri-id'],
                        $limit
                );
 
@@ -743,8 +746,7 @@ class Tag
                        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'],
+                       Item::PUBLIC, $post['uri-id'],
                        $limit
                );
 
@@ -797,7 +799,7 @@ class Tag
        /**
         * Fetch user who subscribed to the tags of the given item
         *
-        * @param integer $uriId URI Id
+        * @param integer $uriId
         * @return array User list
         */
        public static function getUIDListByURIId(int $uriId): array