diff options
| author | Daniel Kesselberg <mail@danielkesselberg.de> | 2023-11-21 19:46:13 +0100 |
|---|---|---|
| committer | backportbot-nextcloud[bot] <backportbot-nextcloud[bot]@users.noreply.github.com> | 2023-11-22 19:53:52 +0000 |
| commit | e7c97ad72f99230258b01e8dadc8381e6f145602 (patch) | |
| tree | 2b85a4f1b6b9aeae0bef59261baddda3a3e716cf | |
| parent | 81e5453b35ed3307ac92df2c5a60d1d4ab5eb69f (diff) | |
| download | nextcloud-server-e7c97ad72f99230258b01e8dadc8381e6f145602.tar.gz nextcloud-server-e7c97ad72f99230258b01e8dadc8381e6f145602.zip | |
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 <mail@danielkesselberg.de>
| -rw-r--r-- | apps/dav/lib/Migration/Version1027Date20230504122946.php | 2 |
1 files changed, 1 insertions, 1 deletions
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; |