X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=src%2FWorker%2FUpdateContacts.php;h=7e352a27caf9f683bee524c8205cf8cf3993a5f4;hb=0e82f64d71b598a1349ac22973148715f7182fdd;hp=ba4f9db41c8f32482c93097f79a151bc0253dee8;hpb=befc2af5043a3afde251721c0d27df695db1bb7e;p=friendica.git

diff --git a/src/Worker/UpdateContacts.php b/src/Worker/UpdateContacts.php
index ba4f9db41c..7e352a27ca 100644
--- 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;
-	}
 }