Skip to content

Migration orchestration #27

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
come-nc merged 12 commits into from
Feb 22, 2022
Merged

Migration orchestration #27

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
7d2d8aa
Run exports and imports of registered migrators
Pytal Feb 4, 2022
dffce83
Extend export destination as defined by the public interface
Pytal Feb 9, 2022
ebe7931
Suppress psalm errors
Pytal Feb 15, 2022
c1b931a
Add a stubs file for Coordinator
come-nc Feb 15, 2022
4f7dde7
Update baseline
come-nc Feb 15, 2022
245ad9d
Export migrator versions in the export archive and check compatibilit…
come-nc Feb 14, 2022
474c21b
Moved the trait to server repository
come-nc Feb 14, 2022
cc8393b
Adapt to version handling changes in core interfaces
come-nc Feb 15, 2022
77c4fc8
Trait property default value cannot be overridden
come-nc Feb 15, 2022
60e60ec
Adapt to changes in core
come-nc Feb 21, 2022
b6be1e1
update christophwurst/nextcloud
come-nc Feb 22, 2022
82469c0
Update baseline
come-nc Feb 22, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Suppress psalm errors 
Signed-off-by: Christopher Ng <[email protected]>
  • Loading branch information
@Pytal
Pytal committed Feb 15, 2022
commit ebe7931802427e414556d0d4d90c81f7fb63a66d
16 changes: 16 additions & 0 deletions lib/Service/UserMigrationService.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -59,6 +59,10 @@ class UserMigrationService {

protected ContainerInterface $container;

// Allow use of the private Coordinator class here to get and run registered migrators
/**
* @psalm-suppress UndefinedClass
*/
protected Coordinator $coordinator;

public function __construct(
Expand All @@ -68,6 +72,9 @@ public function __construct(
ITempManager $tempManager,
IUserManager $userManager,
ContainerInterface $container,
/**
* @psalm-suppress UndefinedClass
*/
Coordinator $coordinator
) {
$this->root = $rootFolder;
Expand All @@ -76,6 +83,9 @@ public function __construct(
$this->tempManager = $tempManager;
$this->userManager = $userManager;
$this->container = $container;
/**
* @psalm-suppress UndefinedClass
*/
$this->coordinator = $coordinator;
}

Expand Down Expand Up @@ -126,6 +136,9 @@ public function export(IUser $user, ?OutputInterface $output = null): string {
);

// Run exports of registered migrators
/**
* @psalm-suppress UndefinedClass
*/
$context = $this->coordinator->getRegistrationContext();

if ($context !== null) {
Expand Down Expand Up @@ -156,6 +169,9 @@ public function import(string $path, ?OutputInterface $output = null): void {
$this->importFiles($user, $importSource, $output);

// Run imports of registered migrators
/**
* @psalm-suppress UndefinedClass
*/
$context = $this->coordinator->getRegistrationContext();

if ($context !== null) {
Expand Down