Skip to content

Commit 5f6594d

Browse files
committed
cleanup
1 parent 2c62d51 commit 5f6594d

File tree

1 file changed

+7
-4
lines changed

1 file changed

+7
-4
lines changed

components/sbm-core/src/main/java/org/springframework/sbm/build/api/ApplicationModules.java

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -81,22 +81,25 @@ public List<Module> getModules(Module module) {
8181
MavenResolutionResult mavenResolutionResult = MavenBuildFileUtil.findMavenResolution(((OpenRewriteMavenBuildFile) module.getBuildFile()).getSourceFile()).get();
8282
List<MavenResolutionResult> modulesMarker = mavenResolutionResult.getModules();
8383
if (!modulesMarker.isEmpty()) {
84-
return filterModulesContainingMavens(modulesMarker);
84+
return getModulesContainingMavens(modulesMarker);
8585
} else {
8686
return new ArrayList<>();
8787
}
8888
}
8989

90+
/**
91+
* Takes a list of {@code MavenResolutionResult}s and returns the modules with matching {@code groupId:artifactId}.
92+
*/
9093
@NotNull
91-
private List<Module> filterModulesContainingMavens(List<MavenResolutionResult> modulesMarker) {
92-
List<String> collect = modulesMarker.stream()
94+
private List<Module> getModulesContainingMavens(List<MavenResolutionResult> mavens) {
95+
List<String> relevantGroupAndArtifactIds = mavens.stream()
9396
.map(m -> m.getPom().getGroupId() + ":" + m.getPom().getArtifactId())
9497
.collect(Collectors.toList());
9598

9699
return modules.stream()
97100
.filter(module -> {
98101
String groupAndArtifactId = module.getBuildFile().getGroupId() + ":" + module.getBuildFile().getArtifactId();
99-
return collect.contains(groupAndArtifactId);
102+
return relevantGroupAndArtifactIds.contains(groupAndArtifactId);
100103
})
101104
.collect(Collectors.toList());
102105
}

0 commit comments

Comments
 (0)