fix(files_metadata): Don't run generation job on every upgrade

Signed-off-by: Louis Chemineau <louis@chmn.me>
nextcloud · Jul 17, 2024 · 71247f2 · 71247f2
1 parent 4689f3d
commit 71247f2
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/core/BackgroundJobs/GenerateMetadataJob.php b/core/BackgroundJobs/GenerateMetadataJob.php
@@ -53,9 +53,14 @@ public function __construct(
 	}
 
 	protected function run(mixed $argument): void {
-		$users = $this->userManager->search('');
+		if ($this->config->getAppValue('core', 'metadataGenerationDone', 'false') !== 'false') {
+			return;
+		}
+
 		$lastHandledUser = $this->config->getAppValue('core', 'metadataGenerationLastHandledUser', '');
 
+		$users = $this->userManager->search('');
+
 		// we'll only start timer once we have found a valid user to handle
 		// meaning NOW if we have not handled any user from a previous run
 		$startTime = ($lastHandledUser === '') ? time() : null;
@@ -79,8 +84,8 @@ protected function run(mixed $argument): void {
 			}
 		}
 
-		$this->jobList->remove(GenerateMetadataJob::class);
 		$this->config->deleteAppValue('core', 'metadataGenerationLastHandledUser');
+		$this->config->setAppValue('core', 'metadataGenerationDone', 'true');
 	}
 
 	private function scanFilesForUser(string $userId): void {