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 b82479bf1 [MNG-7689] Check if execution lifecycle phase is not null 
(#996)
b82479bf1 is described below

commit b82479bf111e4d0dc806d9be6d87f1fec4c2ba40
Author: Alexey Venderov <[email protected]>
AuthorDate: Wed Feb 15 08:49:39 2023 +0100

    [MNG-7689] Check if execution lifecycle phase is not null (#996)
---
 maven-core/src/main/java/org/apache/maven/ReactorReader.java | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)

diff --git a/maven-core/src/main/java/org/apache/maven/ReactorReader.java 
b/maven-core/src/main/java/org/apache/maven/ReactorReader.java
index 6228dd090..900b5d575 100644
--- a/maven-core/src/main/java/org/apache/maven/ReactorReader.java
+++ b/maven-core/src/main/java/org/apache/maven/ReactorReader.java
@@ -318,11 +318,13 @@ class ReactorReader implements MavenWorkspaceReader {
         switch (event.getType()) {
             case MojoStarted:
                 String phase = event.getMojoExecution().getLifecyclePhase();
-                Deque<String> phases = getLifecycles(project);
-                if (!Objects.equals(phase, phases.peekLast())) {
-                    phases.addLast(phase);
-                    if ("clean".equals(phase)) {
-                        cleanProjectLocalRepository(project);
+                if (phase != null) {
+                    Deque<String> phases = getLifecycles(project);
+                    if (!Objects.equals(phase, phases.peekLast())) {
+                        phases.addLast(phase);
+                        if ("clean".equals(phase)) {
+                            cleanProjectLocalRepository(project);
+                        }
                     }
                 }
                 break;

Reply via email to