Skip to content

Commit

Permalink
Merge pull request #46563 from nextcloud/backport/46450/stable28
Browse files Browse the repository at this point in the history
[stable28] fix(files_metadata): Don't run generation job on every upgrade
  • Loading branch information
nickvergessen authored Jul 25, 2024
2 parents 07b9068 + 71247f2 commit 12a672f
Showing 1 changed file with 7 additions and 2 deletions.
9 changes: 7 additions & 2 deletions core/BackgroundJobs/GenerateMetadataJob.php
Original file line number Diff line number Diff line change
Expand Up @@ -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;
Expand All @@ -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 {
Expand Down

0 comments on commit 12a672f

Please sign in to comment.