diff options
| author | Côme Chilliet <come.chilliet@nextcloud.com> | 2023-01-10 14:59:43 +0100 |
|---|---|---|
| committer | Vincent Petry <vincent@nextcloud.com> | 2023-01-11 09:53:14 +0100 |
| commit | c5eb39b8ffa346a98426452445e2494b417cc86a (patch) | |
| tree | 436c1b1425e4ad99d490258361728941fa53fc15 | |
| parent | c50b3793d4e8df6709762f9950afd856a75c23ea (diff) | |
| download | nextcloud-server-c5eb39b8ffa346a98426452445e2494b417cc86a.tar.gz nextcloud-server-c5eb39b8ffa346a98426452445e2494b417cc86a.zip | |
Fix theming migration repair step by passing correct type for argument
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
| -rw-r--r-- | apps/theming/lib/Jobs/MigrateBackgroundImages.php | 11 |
1 files changed, 5 insertions, 6 deletions
diff --git a/apps/theming/lib/Jobs/MigrateBackgroundImages.php b/apps/theming/lib/Jobs/MigrateBackgroundImages.php index 54c0d591e40..97665d1179f 100644 --- a/apps/theming/lib/Jobs/MigrateBackgroundImages.php +++ b/apps/theming/lib/Jobs/MigrateBackgroundImages.php @@ -68,10 +68,9 @@ class MigrateBackgroundImages extends QueuedJob { $this->logger = $logger; } - protected function run($argument): void { - if (!isset($argument['stage'])) { - // not executed in 25.0.0?! - $argument['stage'] = self::STAGE_PREPARE; + protected function run(mixed $argument): void { + if (!is_array($argument) || !isset($argument['stage'])) { + throw new \Exception('Job '.self::class.' called with wrong argument'); } switch ($argument['stage']) { @@ -99,10 +98,10 @@ class MigrateBackgroundImages extends QueuedJob { $userIds = $result->fetchAll(\PDO::FETCH_COLUMN); $this->storeUserIdsToProcess($userIds); } catch (\Throwable $t) { - $this->jobList->add(self::class, self::STAGE_PREPARE); + $this->jobList->add(self::class, ['stage' => self::STAGE_PREPARE]); throw $t; } - $this->jobList->add(self::class, self::STAGE_EXECUTE); + $this->jobList->add(self::class, ['stage' => self::STAGE_EXECUTE]); } /** |