diff --git a/apps/dav/lib/UserMigration/CalendarMigrator.php b/apps/dav/lib/UserMigration/CalendarMigrator.php index f560b4853b2b9..37a8de36d8db5 100644 --- a/apps/dav/lib/UserMigration/CalendarMigrator.php +++ b/apps/dav/lib/UserMigration/CalendarMigrator.php @@ -414,7 +414,7 @@ function (array $componentNames, VObjectComponent $component) { * @throws CalendarMigratorException */ public function import(IUser $user, IImportSource $importSource, OutputInterface $output): void { - if ($importSource->getMigratorVersion(static::class) === null) { + if ($importSource->getMigratorVersion($this->getId()) === null) { $output->writeln('No version for ' . static::class . ', skipping import…'); return; } diff --git a/apps/dav/lib/UserMigration/ContactsMigrator.php b/apps/dav/lib/UserMigration/ContactsMigrator.php index cfdc20bfdf355..99eea2700a5bc 100644 --- a/apps/dav/lib/UserMigration/ContactsMigrator.php +++ b/apps/dav/lib/UserMigration/ContactsMigrator.php @@ -306,7 +306,7 @@ private function getAddressBookImports(array $importFiles): array { * @throws ContactsMigratorException */ public function import(IUser $user, IImportSource $importSource, OutputInterface $output): void { - if ($importSource->getMigratorVersion(static::class) === null) { + if ($importSource->getMigratorVersion($this->getId()) === null) { $output->writeln('No version for ' . static::class . ', skipping import…'); return; } diff --git a/apps/files_trashbin/lib/UserMigration/TrashbinMigrator.php b/apps/files_trashbin/lib/UserMigration/TrashbinMigrator.php index c7832ec5bc883..95ed25088b163 100644 --- a/apps/files_trashbin/lib/UserMigration/TrashbinMigrator.php +++ b/apps/files_trashbin/lib/UserMigration/TrashbinMigrator.php @@ -93,7 +93,7 @@ public function export(IUser $user, IExportDestination $exportDestination, Outpu * {@inheritDoc} */ public function import(IUser $user, IImportSource $importSource, OutputInterface $output): void { - if ($importSource->getMigratorVersion(static::class) === null) { + if ($importSource->getMigratorVersion($this->getId()) === null) { $output->writeln('No version for ' . static::class . ', skipping import…'); return; } diff --git a/apps/settings/lib/UserMigration/AccountMigrator.php b/apps/settings/lib/UserMigration/AccountMigrator.php index a39c510813f73..e4218d72a76ee 100644 --- a/apps/settings/lib/UserMigration/AccountMigrator.php +++ b/apps/settings/lib/UserMigration/AccountMigrator.php @@ -95,7 +95,7 @@ public function export(IUser $user, IExportDestination $exportDestination, Outpu * {@inheritDoc} */ public function import(IUser $user, IImportSource $importSource, OutputInterface $output): void { - if ($importSource->getMigratorVersion(static::class) === null) { + if ($importSource->getMigratorVersion($this->getId()) === null) { $output->writeln('No version for ' . static::class . ', skipping import…'); return; } diff --git a/apps/settings/tests/UserMigration/AccountMigratorTest.php b/apps/settings/tests/UserMigration/AccountMigratorTest.php index c668bb9e6af15..b7826c20e60c5 100644 --- a/apps/settings/tests/UserMigration/AccountMigratorTest.php +++ b/apps/settings/tests/UserMigration/AccountMigratorTest.php @@ -98,7 +98,7 @@ public function testImportExportAccount(string $userId, array $importData): void $this->importSource ->expects($this->once()) ->method('getMigratorVersion') - ->with(AccountMigrator::class) + ->with($this->migrator->getId()) ->willReturn(1); $this->importSource diff --git a/lib/public/UserMigration/IImportSource.php b/lib/public/UserMigration/IImportSource.php index d34f2752549e1..3816afdd033f8 100644 --- a/lib/public/UserMigration/IImportSource.php +++ b/lib/public/UserMigration/IImportSource.php @@ -91,7 +91,7 @@ public function getMigratorVersions(): array; /** * @return ?int Version for this migrator from the export archive. Null means migrator missing. * - * @param class-string $migrator + * @param string $migrator Migrator id (as returned by IMigrator::getId) * * @since 24.0.0 */ diff --git a/lib/public/UserMigration/IMigrator.php b/lib/public/UserMigration/IMigrator.php index 7c89e041f3437..d02e5df06836e 100644 --- a/lib/public/UserMigration/IMigrator.php +++ b/lib/public/UserMigration/IMigrator.php @@ -89,7 +89,7 @@ public function getVersion(): int; /** * Checks whether it is able to import a version of the export format for this migrator - * Use $importSource->getMigratorVersion(static::class) to get the version from the archive + * Use $importSource->getMigratorVersion($this->getId()) to get the version from the archive * * @since 24.0.0 */ diff --git a/lib/public/UserMigration/TMigratorBasicVersionHandling.php b/lib/public/UserMigration/TMigratorBasicVersionHandling.php index 20a30a243076a..9010d14a173b4 100644 --- a/lib/public/UserMigration/TMigratorBasicVersionHandling.php +++ b/lib/public/UserMigration/TMigratorBasicVersionHandling.php @@ -50,7 +50,7 @@ public function getVersion(): int { public function canImport( IImportSource $importSource ): bool { - $version = $importSource->getMigratorVersion(static::class); + $version = $importSource->getMigratorVersion($this->getId()); if ($version === null) { return !$this->mandatory; }