<?php
/**
- * @copyright Copyright (C) 2020, Friendica
+ * @copyright Copyright (C) 2010-2022, the Friendica project
*
* @license GNU AGPL version 3 or any later version
*
return;
}
- // Add every contact our system interacted with and hadn't been updated for a week if unarchived
- // or for a month if archived.
- $condition = DBA::mergeConditions($base_condition, ["(`id` IN (SELECT `author-id` FROM `post-user`) OR
- `id` IN (SELECT `owner-id` FROM `post-user`) OR `id` IN (SELECT `causer-id` FROM `post-user`) OR
- `id` IN (SELECT `cid` FROM `post-tag`) OR `id` IN (SELECT `cid` FROM `user-contact`) OR `uid` != ?) AND
- (`last-update` < ? OR (NOT `archive` AND `last-update` < ?))",
+ $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')]);
- Logger::info('Updatable interacting federated contacts', ['count' => DBA::count('contact', $condition)]);
- $ids = self::getContactsToUpdate($condition, [], $limit);
- Logger::info('Fetched interacting federated contacts', ['count' => count($ids)]);
+ $ids = self::getContactsToUpdate($condition, $limit, []);
+ Logger::info('Fetched federated user contacts', ['count' => count($ids)]);
+
+ $conditions = ["`id` IN (SELECT `author-id` FROM `post` WHERE `author-id` = `contact`.`id`)",
+ "`id` IN (SELECT `owner-id` FROM `post` WHERE `owner-id` = `contact`.`id`)",
+ "`id` IN (SELECT `causer-id` FROM `post` WHERE `causer-id` = `contact`.`id`)",
+ "`id` IN (SELECT `cid` FROM `post-tag` WHERE `cid` = `contact`.`id`)",
+ "`id` IN (SELECT `cid` FROM `user-contact` WHERE `cid` = `contact`.`id`)"];
+
+ 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, $limit, $ids);
+ Logger::info('Fetched interacting federated contacts', ['count' => count($ids), 'condition' => $contact_condition]);
+ }
+
+ 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 `archive` AND `last-update` < ?))",
+ ["(`last-update` < ? OR (NOT `failed` AND `last-update` < ?))",
DateTimeFormat::utc('now - 6 month'), DateTimeFormat::utc('now - 1 month')]);
- Logger::info('Updatable federated contacts', ['count' => DBA::count('contact', $condition)]);
$previous = count($ids);
- $ids = self::getContactsToUpdate($condition, $ids, $limit - $previous);
+ $ids = self::getContactsToUpdate($condition, $limit - $previous, $ids);
Logger::info('Fetched federated contacts', ['count' => count($ids) - $previous]);
}
* Returns contact ids based on a given condition
*
* @param array $condition
+ * @param int $limit
* @param array $ids
* @return array contact ids
*/
- private static function getContactsToUpdate(array $condition, array $ids = [], int $limit)
+ private static function getContactsToUpdate(array $condition, int $limit, array $ids = []): array
{
- $contacts = DBA::select('contact', ['id'], $condition, ['limit' => $limit, 'order' => ['last-update']]);
+ $contacts = DBA::select('contact', ['id'], $condition, ['limit' => $limit]);
while ($contact = DBA::fetch($contacts)) {
- $ids[] = $contact['id'];
+ $ids[$contact['id']] = $contact['id'];
}
DBA::close($contacts);
return $ids;