[maven] branch master updated: [MNG-7720] Simplify code (#1063)

[gnodet]

This is an automated email from the ASF dual-hosted git repository.

gnodet pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/maven.git


The following commit(s) were added to refs/heads/master by this push:
     new ae933fe68 [MNG-7720] Simplify code (#1063)
ae933fe68 is described below

commit ae933fe68d1b2dc8cbe1e22b14781024e0540d26
Author: Guillaume Nodet <gno...@gmail.com>
AuthorDate: Thu Mar 23 06:31:49 2023 +0100

    [MNG-7720] Simplify code (#1063)
---
 .../lifecycle/internal/LifecycleDependencyResolver.java   | 15 +++------------
 1 file changed, 3 insertions(+), 12 deletions(-)

diff --git 
a/maven-core/src/main/java/org/apache/maven/lifecycle/internal/LifecycleDependencyResolver.java
 
b/maven-core/src/main/java/org/apache/maven/lifecycle/internal/LifecycleDependencyResolver.java
index 62cd8c8ce..f7a4d484f 100644
--- 
a/maven-core/src/main/java/org/apache/maven/lifecycle/internal/LifecycleDependencyResolver.java
+++ 
b/maven-core/src/main/java/org/apache/maven/lifecycle/internal/LifecycleDependencyResolver.java
@@ -31,7 +31,6 @@ import java.util.List;
 import java.util.Map;
 import java.util.Set;
 import java.util.stream.Collectors;
-import java.util.stream.Stream;
 
 import org.apache.maven.RepositoryUtils;
 import org.apache.maven.artifact.Artifact;
@@ -90,8 +89,9 @@ public class LifecycleDependencyResolver {
 
     public static List<MavenProject> getProjects(MavenProject project, 
MavenSession session, boolean aggregator) {
         if (aggregator && project.getCollectedProjects() != null) {
-            List<MavenProject> projectAndSubmodules =
-                    
getProjectAndSubModules(project).collect(Collectors.toList()); // not sorted 
but what we need
+            // get the unsorted list of wanted projects
+            Set<MavenProject> projectAndSubmodules = new 
HashSet<>(project.getCollectedProjects());
+            projectAndSubmodules.add(project);
             return session.getProjects().stream() // sorted all
                     .filter(projectAndSubmodules::contains)
                     .collect(Collectors.toList()); // sorted and filtered to 
what we need
@@ -100,15 +100,6 @@ public class LifecycleDependencyResolver {
         }
     }
 
-    private static Stream<MavenProject> getProjectAndSubModules(MavenProject 
project) {
-        return Stream.concat(
-                Stream.of(project),
-                project.getCollectedProjects() == null
-                        ? Stream.empty()
-                        : project.getCollectedProjects().stream()
-                                
.flatMap(LifecycleDependencyResolver::getProjectAndSubModules));
-    }
-
     public void resolveProjectDependencies(
             MavenProject project,
             Collection<String> scopesToCollect,