Don't run repair step for every individual user, outsource that to background job

Signed-off-by: Georg Ehrke <developer@georgehrke.com>

1 files changed, 3 insertions, 16 deletions

diff --git a/apps/dav/lib/Migration/RegenerateBirthdayCalendars.php b/apps/dav/lib/Migration/RegenerateBirthdayCalendars.php
index 263e6d00db4..492ae876e8a 100644
--- a/apps/dav/lib/Migration/RegenerateBirthdayCalendars.php
+++ b/apps/dav/lib/Migration/RegenerateBirthdayCalendars.php
@@ -22,19 +22,14 @@
  */
 namespace OCA\DAV\Migration;
 
-use OCA\DAV\BackgroundJob\GenerateBirthdayCalendarBackgroundJob;
+use OCA\DAV\BackgroundJob\RegisterRegenerateBirthdayCalendars;
 use OCP\BackgroundJob\IJobList;
 use OCP\IConfig;
-use OCP\IUser;
-use OCP\IUserManager;
 use OCP\Migration\IOutput;
 use OCP\Migration\IRepairStep;
 
 class RegenerateBirthdayCalendars implements IRepairStep {
 
-	/** @var IUserManager */
-	private $userManager;
-
 	/** @var IJobList */
 	private $jobList;
 
@@ -42,14 +37,11 @@ class RegenerateBirthdayCalendars implements IRepairStep {
 	private $config;
 
 	/**
-	 * @param IUserManager $userManager,
 	 * @param IJobList $jobList
 	 * @param IConfig $config
 	 */
-	public function __construct(IUserManager $userManager,
-								IJobList $jobList,
+	public function __construct(IJobList $jobList,
 								IConfig $config) {
-		$this->userManager = $userManager;
 		$this->jobList = $jobList;
 		$this->config = $config;
 	}
@@ -72,12 +64,7 @@ class RegenerateBirthdayCalendars implements IRepairStep {
 		}
 
 		$output->info('Adding background jobs to regenerate birthday calendar');
-		$this->userManager->callForSeenUsers(function(IUser $user) {
-			$this->jobList->add(GenerateBirthdayCalendarBackgroundJob::class, [
-				'userId' => $user->getUID(),
-				'purgeBeforeGenerating' => true
-			]);
-		});
+		$this->jobList->add(RegisterRegenerateBirthdayCalendars::class);
 
 		// if all were done, no need to redo the repair during next upgrade
 		$this->config->setAppValue('dav', 'regeneratedBirthdayCalendarsForYearFix', 'yes');