]> git.mxchange.org Git - friendica.git/blobdiff - src/Model/Tag.php
Merge pull request #8558 from annando/corrected-view
[friendica.git] / src / Model / Tag.php
index ac2600903391c1cd275bfa0ff417e5f5278862cf..cf453f2431356ec6e9498b3662050cae36ce773b 100644 (file)
@@ -61,10 +61,11 @@ class Tag
         *
         * @param integer $uriid
         * @param integer $type
-        * @param string $name
-        * @param string $url
+        * @param string  $name
+        * @param string  $url
+        * @param boolean $probing
         */
-       public static function store(int $uriid, int $type, string $name, string $url = '')
+       public static function store(int $uriid, int $type, string $name, string $url = '', $probing = true)
        {
                $name = trim($name, "\x00..\x20\xFF#!@");
                if (empty($name)) {
@@ -80,16 +81,32 @@ class Tag
                                return;
                        }
 
-                       Logger::info('Get ID for contact', ['url' => $url]);
+                       if (!$probing) {
+                               $condition = ['nurl' => Strings::normaliseLink($url), 'uid' => 0, 'deleted' => false];
+                               $contact = DBA::selectFirst('contact', ['id'], $condition, ['order' => ['id']]);
+                               if (DBA::isResult($contact)) {
+                                       $cid = $contact['id'];
+                                       Logger::info('Got id for contact url', ['cid' => $cid, 'url' => $url]);
+                               }
 
-                       $condition = ['nurl' => Strings::normaliseLink($url), 'uid' => 0, 'deleted' => false];
-                       $contact = DBA::selectFirst('contact', ['id'], $condition, ['order' => ['id']]);
-                       if (DBA::isResult($contact)) {
-                               $cid = $contact['id'];
+                               if (empty($cid)) {
+                                       $ssl_url = str_replace('http://', 'https://', $url);
+                                       $condition = ['`alias` IN (?, ?, ?) AND `uid` = ? AND NOT `deleted`', $url, Strings::normaliseLink($url), $ssl_url, 0];
+                                       $contact = DBA::selectFirst('contact', ['id'], $condition, ['order' => ['id']]);
+                                       if (DBA::isResult($contact)) {
+                                               $cid = $contact['id'];
+                                               Logger::info('Got id for contact alias', ['cid' => $cid, 'url' => $url]);
+                                       }
+                               }
                        } else {
+                               $cid = Contact::getIdForURL($url, 0, true);
+                               Logger::info('Got id by probing', ['cid' => $cid, 'url' => $url]);
+                       }
+
+                       if (empty($cid)) {
                                // The contact wasn't found in the system (most likely some dead account)
                                // We ensure that we only store a single entry by overwriting the previous name
-                               Logger::info('Update tag', ['url' => $url, 'name' => $name]);
+                               Logger::info('Contact not found, updating tag', ['url' => $url, 'name' => $name]);
                                DBA::update('tag', ['name' => substr($name, 0, 96)], ['url' => $url]);
                        }
                }
@@ -100,7 +117,7 @@ class Tag
                        if (($type != Tag::HASHTAG) && !empty($url) && ($url != $name)) {
                                $fields['url'] = strtolower($url);
                        }
-       
+
                        $tag = DBA::selectFirst('tag', ['id'], $fields);
                        if (!DBA::isResult($tag)) {
                                DBA::insert('tag', $fields, true);
@@ -108,16 +125,27 @@ class Tag
                        } else {
                                $tagid = $tag['id'];
                        }
-       
+
                        if (empty($tagid)) {
                                Logger::error('No tag id created', $fields);
                                return;
                        }
                }
 
-               DBA::insert('post-tag', ['uri-id' => $uriid, 'type' => $type, 'tid' => $tagid, 'cid' => $cid], true);
+               $fields = ['uri-id' => $uriid, 'type' => $type, 'tid' => $tagid, 'cid' => $cid];
 
-               Logger::info('Stored tag/mention', ['uri-id' => $uriid, 'tag-id' => $tagid, 'contact-id' => $cid]);
+               if (in_array($type, [Tag::MENTION, Tag::EXCLUSIVE_MENTION, Tag::IMPLICIT_MENTION])) {
+                       $condition = $fields;
+                       $condition['type'] = [Tag::MENTION, Tag::EXCLUSIVE_MENTION, Tag::IMPLICIT_MENTION];
+                       if (DBA::exists('post-tag', $condition)) {
+                               Logger::info('Tag already exists', $fields);
+                               return;
+                       }
+               }
+
+               DBA::insert('post-tag', $fields, true);
+
+               Logger::info('Stored tag/mention', ['uri-id' => $uriid, 'tag-id' => $tagid, 'contact-id' => $cid, 'name' => $name, 'type' => $type, 'callstack' => System::callstack(8)]);
        }
 
        /**
@@ -127,25 +155,27 @@ class Tag
         * @param string $hash
         * @param string $name
         * @param string $url
+        * @param boolean $probing
         */
-       public static function storeByHash(int $uriid, string $hash, string $name, string $url = '')
+       public static function storeByHash(int $uriid, string $hash, string $name, string $url = '', $probing = true)
        {
                $type = self::getTypeForHash($hash);
                if ($type == self::UNKNOWN) {
                        return;
                }
 
-               self::store($uriid, $type, $name, $url);
+               self::store($uriid, $type, $name, $url, $probing);
        }
 
        /**
         * Store tags and mentions from the body
         * 
-        * @param integer $uriid URI-Id
-        * @param string  $body   Body of the post
-        * @param string  $tags   Accepted tags
+        * @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
         */
-       public static function storeFromBody(int $uriid, string $body, string $tags = null)
+       public static function storeFromBody(int $uriid, string $body, string $tags = null, $probing = true)
        {
                if (is_null($tags)) {
                        $tags =  self::TAG_CHARACTER[self::HASHTAG] . self::TAG_CHARACTER[self::MENTION] . self::TAG_CHARACTER[self::EXCLUSIVE_MENTION];
@@ -160,7 +190,7 @@ class Tag
                Logger::info('Found tags', ['uri-id' => $uriid, 'hash' => $tags, 'result' => $result]);
 
                foreach ($result as $tag) {
-                       self::storeByHash($uriid, $tag[1], $tag[3], $tag[2]);
+                       self::storeByHash($uriid, $tag[1], $tag[3], $tag[2], $probing);
                }
        }
 
@@ -201,13 +231,18 @@ class Tag
         */
        public static function remove(int $uriid, int $type, string $name, string $url = '')
        {
-               $tag = DBA::fetchFirst("SELECT `id` FROM `tag` INNER JOIN `post-tag` ON `post-tag`.`tid` = `tag`.`id`
-                       WHERE `uri-id` = ? AND `type` = ? AND `name` = ? AND `url` = ?", $uriid, $type, $name, $url);
+               $condition = ['uri-id' => $uriid, 'type' => $type, 'url' => $url];
+               if ($type == self::HASHTAG) {
+                       $condition['name'] = $name;
+               }
+
+               $tag = DBA::selectFirst('tag-view', ['tid', 'cid'], $condition);
                if (!DBA::isResult($tag)) {
                        return;
                }
-               Logger::info('Removing tag/mention', ['uri-id' => $uriid, 'tid' => $tag['id'], 'name' => $name, 'url' => $url]);
-               DBA::delete('post-tag', ['uri-id' => $uriid, 'tid' => $tag['id']]);
+
+               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']]);
        }
 
        /**