diff options
author | Jacek Centkowski <jcentkowski@digital.ai> | 2020-11-04 13:05:28 +0100 |
---|---|---|
committer | Jacek Centkowski <jcentkowski@digital.ai> | 2020-11-05 16:21:50 +0100 |
commit | 3991d53b30a02c0ffa6fc34385c963257de23349 (patch) | |
tree | c85ee7d75fd9f08024cce095d91dc0ad1b6cd846 | |
parent | f25727a949121dd6709264c22c7e12fa8fb30f75 (diff) |
CreateProjectTask.java: use interface instead of implementationv3.2.5.1
ReplicationDestinations interface should be used in CreateProjectTask
so that specific implementation can be materialized by Guice (DI).
Change-Id: If7cb21adff5c3feeeea7568c504e8e37d5c08f9e
-rw-r--r-- | src/main/java/com/googlesource/gerrit/plugins/replication/CreateProjectTask.java | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/src/main/java/com/googlesource/gerrit/plugins/replication/CreateProjectTask.java b/src/main/java/com/googlesource/gerrit/plugins/replication/CreateProjectTask.java index 424648e..2599b9b 100644 --- a/src/main/java/com/googlesource/gerrit/plugins/replication/CreateProjectTask.java +++ b/src/main/java/com/googlesource/gerrit/plugins/replication/CreateProjectTask.java @@ -31,7 +31,7 @@ public class CreateProjectTask { } private final RemoteConfig config; - private final DestinationsCollection destinations; + private final ReplicationDestinations destinations; private final DynamicItem<AdminApiFactory> adminApiFactory; private final Project.NameKey project; private final String head; @@ -39,7 +39,7 @@ public class CreateProjectTask { @Inject CreateProjectTask( RemoteConfig config, - DestinationsCollection destinations, + ReplicationDestinations destinations, DynamicItem<AdminApiFactory> adminApiFactory, @Assisted Project.NameKey project, @Assisted String head) { @@ -51,8 +51,10 @@ public class CreateProjectTask { } public boolean create() { - return destinations.getURIs(Optional.of(config.getName()), project, FilterType.PROJECT_CREATION) - .values().stream() + return destinations + .getURIs(Optional.of(config.getName()), project, FilterType.PROJECT_CREATION) + .values() + .stream() .map(u -> createProject(u, project, head)) .reduce(true, (a, b) -> a && b); } |