Only attach images

[friendica.git] / src / Worker / UpdateContacts.php

diff --git a/src/Worker/UpdateContacts.php b/src/Worker/UpdateContacts.php
index ba4f9db41c8f32482c93097f79a151bc0253dee8..7e352a27caf9f683bee524c8205cf8cf3993a5f4 100644 (file)
--- a/src/Worker/UpdateContacts.php
+++ b/src/Worker/UpdateContacts.php
@@ -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
  *
@@ -35,8 +35,6 @@ class UpdateContacts
 {
        public static function execute()
        {
-               $base_condition = ['network' => array_merge(Protocol::FEDERATED, [Protocol::ZOT, Protocol::PHANTOM]), 'self' => false];
-
                $update_limit = DI::config()->get('system', 'contact_update_limit');
                if (empty($update_limit)) {
                        return;
@@ -49,63 +47,25 @@ class UpdateContacts
                        return;
                }
 
-               $condition = DBA::mergeConditions($base_condition,
-                       ["`uid` != ? AND (`last-update` < ? OR (NOT `failed` AND `last-update` < ?))",
-                       0, DateTimeFormat::utc('now - 1 month'), DateTimeFormat::utc('now - 1 week')]);
-               $ids = self::getContactsToUpdate($condition, [], $limit);
-               Logger::info('Fetched federated user contacts', ['count' => count($ids)]);
-
-               $conditions = ["`id` IN (SELECT `author-id` FROM `post-user`)", "`id` IN (SELECT `owner-id` FROM `post-user`)", 
-                       "`id` IN (SELECT `causer-id` FROM `post-user`)", "`id` IN (SELECT `cid` FROM `post-tag`)",
-                       "`id` IN (SELECT `cid` FROM `user-contact`)"];
+               Logger::info('Updating contact', ['count' => $limit]);
 
-               foreach ($conditions as $contact_condition) {
-                       $condition = DBA::mergeConditions($base_condition,
-                               [$contact_condition . " AND (`last-update` < ? OR (NOT `failed` AND `last-update` < ?))",
-                               DateTimeFormat::utc('now - 1 month'), DateTimeFormat::utc('now - 1 week')]);
-                       $ids = self::getContactsToUpdate($condition, $ids, $limit);
-                       Logger::info('Fetched interacting federated contacts', ['count' => count($ids), 'condition' => $contact_condition]);
-               }
+               $condition = ['self' => false];
 
-               if (count($ids) > $limit) {
-                       $ids = array_slice($ids, 0, $limit, true);
-               }
-
-               if (!DI::config()->get('system', 'update_active_contacts')) {
-                       // Add every contact (mostly failed ones) that hadn't been updated for six months
-                       // and every non failed contact that hadn't been updated for a month
-                       $condition = DBA::mergeConditions($base_condition,
-                               ["(`last-update` < ? OR (NOT `failed` AND `last-update` < ?))",
-                                       DateTimeFormat::utc('now - 6 month'), DateTimeFormat::utc('now - 1 month')]);
-                       $previous = count($ids);
-                       $ids = self::getContactsToUpdate($condition, $ids, $limit - $previous);
-                       Logger::info('Fetched federated contacts', ['count' => count($ids) - $previous]);
+               if (DI::config()->get('system', 'update_active_contacts')) {
+                       $condition = array_merge(['local-data' => true], $condition);
                }
 
+               $condition = DBA::mergeConditions(["`next-update` < ?", DateTimeFormat::utcNow()], $condition);
+               $contacts = DBA::select('contact', ['id'], $condition, ['order' => ['next-update'], 'limit' => $limit]);
                $count = 0;
-               foreach ($ids as $id) {
-                       if (Worker::add(PRIORITY_LOW, "UpdateContact", $id)) {
+               while ($contact = DBA::fetch($contacts)) {
+                       if (Worker::add(['priority' => PRIORITY_LOW, 'dont_fork' => true], "UpdateContact", $contact['id'])) {
                                ++$count;
                        }
+                       Worker::coolDown();
                }
+               DBA::close($contacts);
 
                Logger::info('Initiated update for federated contacts', ['count' => $count]);
        }
-
-       /**
-        * Returns contact ids based on a given condition
-        *
-        * @param array $condition
-        * @param array $ids
-        * @return array contact ids
-        */
-       private static function getContactsToUpdate(array $condition, array $ids = [], int $limit)
-       {
-               $contacts = DBA::select('contact', ['id'], $condition, ['limit' => $limit]);
-               while ($contact = DBA::fetch($contacts)) {
-                       $ids[$contact['id']] = $contact['id'];
-               }
-               DBA::close($contacts);
-               return $ids;
-       }
 }