From 7b74b07046035e3bb0ba82298472a72ce9b287cc Mon Sep 17 00:00:00 2001 From: Daniel Kesselberg Date: Tue, 21 Nov 2023 19:46:13 +0100 Subject: [PATCH] fix: lower threshold for system address book sync - Switch back to countUsers to have the actual value. countSeenUsers is bad if Nextcloud is connected to a larger directory, but only a part is using Nextcloud and therefore the seen count is much lower because the sync is done for all users. - Lower the threshold to 100 for smaller installations. Signed-off-by: Daniel Kesselberg --- apps/dav/lib/Migration/Version1027Date20230504122946.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/apps/dav/lib/Migration/Version1027Date20230504122946.php b/apps/dav/lib/Migration/Version1027Date20230504122946.php index 6d0ac9f6f86..b1aaec0559b 100644 --- a/apps/dav/lib/Migration/Version1027Date20230504122946.php +++ b/apps/dav/lib/Migration/Version1027Date20230504122946.php @@ -49,7 +49,7 @@ class Version1027Date20230504122946 extends SimpleMigrationStep { * @param array $options */ public function postSchemaChange(IOutput $output, Closure $schemaClosure, array $options): void { - if($this->userManager->countSeenUsers() > 1000) { + if($this->userManager->countSeenUsers() > 100 || array_sum($this->userManager->countUsers()) > 100) { $this->config->setAppValue('dav', 'needs_system_address_book_sync', 'yes'); $output->info('Could not sync system address books during update - too many user records have been found. Please call occ dav:sync-system-addressbook manually.'); return; -- 2.39.5