]> git.mxchange.org Git - friendica.git/blobdiff - src/Model/Contact/Relation.php
Merge remote-tracking branch 'upstream/develop' into json-ld
[friendica.git] / src / Model / Contact / Relation.php
index d9a38e8c3e89ef0b41c0f6921f394519661f402d..ed520f25ffea9c915f7b060b98b5da0b572337b6 100644 (file)
@@ -1,6 +1,6 @@
 <?php
 /**
- * @copyright Copyright (C) 2010-2021, the Friendica project
+ * @copyright Copyright (C) 2010-2022, the Friendica project
  *
  * @license GNU AGPL version 3 or any later version
  *
@@ -78,10 +78,12 @@ class Relation
        {
                $contact = Contact::getByURL($url);
                if (empty($contact)) {
+                       Logger::info('Contact not found', ['url' => $url]);
                        return;
                }
 
                if (!self::isDiscoverable($url, $contact)) {
+                       Logger::info('Contact is not discoverable', ['url' => $url]);
                        return;
                }
 
@@ -112,7 +114,7 @@ class Relation
                }
 
                if (empty($followers) && empty($followings)) {
-                       DBA::update('contact', ['last-discovery' => DateTimeFormat::utcNow()], ['id' => $contact['id']]);
+                       Contact::update(['last-discovery' => DateTimeFormat::utcNow()], ['id' => $contact['id']]);
                        Logger::info('The contact does not offer discoverable data', ['id' => $contact['id'], 'url' => $url, 'network' => $contact['network']]);
                        return;
                }
@@ -160,7 +162,7 @@ class Relation
                        DBA::delete('contact-relation', ['cid' => $target, 'follows' => false, 'last-interaction' => DBA::NULL_DATETIME]);
                }
 
-               DBA::update('contact', ['last-discovery' => DateTimeFormat::utcNow()], ['id' => $target]);
+               Contact::update(['last-discovery' => DateTimeFormat::utcNow()], ['id' => $target]);
                Logger::info('Contacts discovery finished', ['id' => $target, 'url' => $url, 'follower' => $follower_counter, 'following' => $following_counter]);
                return;
        }
@@ -172,7 +174,7 @@ class Relation
         * @param array $rel
         * @return array contact list
         */
-       private static function getContacts(int $uid, array $rel)
+       private static function getContacts(int $uid, array $rel): array
        {
                $list = [];
                $profile = Profile::getByUID($uid);
@@ -180,8 +182,15 @@ class Relation
                        return $list;
                }
 
-               $condition = ['rel' => $rel, 'uid' => $uid, 'self' => false, 'deleted' => false,
-                       'hidden' => false, 'archive' => false, 'pending' => false];
+               $condition = [
+                       'rel' => $rel,
+                       'uid' => $uid,
+                       'self' => false,
+                       'deleted' => false,
+                       'hidden' => false,
+                       'archive' => false,
+                       'pending' => false,
+               ];
                $condition = DBA::mergeConditions($condition, ["`url` IN (SELECT `url` FROM `apcontact`)"]);
                $contacts = DBA::select('contact', ['url'], $condition);
                while ($contact = DBA::fetch($contacts)) {
@@ -199,7 +208,7 @@ class Relation
         * @param array  $contact Contact array
         * @return boolean True if contact is discoverable
         */
-       public static function isDiscoverable(string $url, array $contact = [])
+       public static function isDiscoverable(string $url, array $contact = []): bool
        {
                $contact_discovery = DI::config()->get('system', 'contact_discovery');
 
@@ -252,12 +261,14 @@ class Relation
        }
 
        /**
-        * @param int $uid   user
+        * Returns an array of suggested contacts for given user id
+        *
+        * @param int $uid   User id
         * @param int $start optional, default 0
         * @param int $limit optional, default 80
         * @return array
         */
-       static public function getSuggestions(int $uid, int $start = 0, int $limit = 80)
+       static public function getSuggestions(int $uid, int $start = 0, int $limit = 80): array
        {
                $cid = Contact::getPublicIdByUserId($uid);
                $totallimit = $start + $limit;
@@ -270,20 +281,25 @@ class Relation
 
                // The query returns contacts where contacts interacted with whom the given user follows.
                // Contacts who already are in the user's contact table are ignored.
-               $results = DBA::select('contact', [],
-                       ["`id` IN (SELECT `cid` FROM `contact-relation` WHERE `relation-cid` IN
+               $results = DBA::select('contact', [], ["`id` IN (SELECT `cid` FROM `contact-relation` WHERE `relation-cid` IN
                                (SELECT `cid` FROM `contact-relation` WHERE `relation-cid` = ?)
                                        AND NOT `cid` IN (SELECT `id` FROM `contact` WHERE `uid` = ? AND `nurl` IN
                                                (SELECT `nurl` FROM `contact` WHERE `uid` = ? AND `rel` IN (?, ?))))
                        AND NOT `hidden` AND `network` IN (?, ?, ?, ?)",
-                       $cid, 0, $uid, Contact::FRIEND, Contact::SHARING,
-                       Protocol::ACTIVITYPUB, Protocol::DFRN, $diaspora, $ostatus],
-                       ['order' => ['last-item' => true], 'limit' => $totallimit]
+                       $cid,
+                       0,
+                       $uid, Contact::FRIEND, Contact::SHARING,
+                       Protocol::ACTIVITYPUB, Protocol::DFRN, $diaspora, $ostatus,
+                       ], [
+                               'order' => ['last-item' => true],
+                               'limit' => $totallimit,
+                       ]
                );
 
                while ($contact = DBA::fetch($results)) {
                        $contacts[$contact['id']] = $contact;
                }
+
                DBA::close($results);
 
                Logger::info('Contacts of contacts who are followed by the given user', ['uid' => $uid, 'cid' => $cid, 'count' => count($contacts)]);
@@ -363,12 +379,12 @@ class Relation
         * @return int
         * @throws Exception
         */
-       public static function countFollows(int $cid, array $condition = [])
+       public static function countFollows(int $cid, array $condition = []): int
        {
-               $condition = DBA::mergeConditions($condition,
-                       ['`id` IN (SELECT `relation-cid` FROM `contact-relation` WHERE `cid` = ? AND `follows`)', 
-                       $cid]
-               );
+               $condition = DBA::mergeConditions($condition, [
+                       '`id` IN (SELECT `relation-cid` FROM `contact-relation` WHERE `cid` = ? AND `follows`)', 
+                       $cid,
+               ]);
 
                return DI::dba()->count('contact', $condition);
        }
@@ -554,7 +570,7 @@ class Relation
         * @param int   $count
         * @param int   $offset
         * @param bool  $shuffle
-        * @return array
+        * @return array|bool Array on success, false on failure
         * @throws Exception
         */
        public static function listCommon(int $sourceId, int $targetId, array $condition = [], int $count = 30, int $offset = 0, bool $shuffle = false)
@@ -579,7 +595,7 @@ class Relation
         * @return int
         * @throws Exception
         */
-       public static function countCommonFollows(int $sourceId, int $targetId, array $condition = [])
+       public static function countCommonFollows(int $sourceId, int $targetId, array $condition = []): int
        {
                $condition = DBA::mergeConditions($condition,
                        ['`id` IN (SELECT `relation-cid` FROM `contact-relation` WHERE `cid` = ? AND `follows`) 
@@ -599,7 +615,7 @@ class Relation
         * @param int   $count
         * @param int   $offset
         * @param bool  $shuffle
-        * @return array
+        * @return array|bool Array on success, false on failure
         * @throws Exception
         */
        public static function listCommonFollows(int $sourceId, int $targetId, array $condition = [], int $count = 30, int $offset = 0, bool $shuffle = false)
@@ -624,7 +640,7 @@ class Relation
         * @return int
         * @throws Exception
         */
-       public static function countCommonFollowers(int $sourceId, int $targetId, array $condition = [])
+       public static function countCommonFollowers(int $sourceId, int $targetId, array $condition = []): int
        {
                $condition = DBA::mergeConditions($condition,
                        ["`id` IN (SELECT `cid` FROM `contact-relation` WHERE `relation-cid` = ? AND `follows`) 
@@ -644,7 +660,7 @@ class Relation
         * @param int   $count
         * @param int   $offset
         * @param bool  $shuffle
-        * @return array
+        * @return array|bool Array on success, false on failure
         * @throws Exception
         */
        public static function listCommonFollowers(int $sourceId, int $targetId, array $condition = [], int $count = 30, int $offset = 0, bool $shuffle = false)