commit 16758cc6cf00fd95dffd719bc3d144e476713e6e
author Herve Boutemy <hboutemy@apache.org> Sat Sep 10 14:48:57 2016 +0000
committer Herve Boutemy <hboutemy@apache.org> Sat Sep 10 14:48:57 2016 +0000
tree 8b4355759ccb0268fcb6deb39dabffed07ac6405
parent 7d6138439f560e132589721ba2dd383b22306288

[MSHARED-589] improved after forked execution

git-svn-id: https://svn.apache.org/repos/asf/maven/shared/trunk@1760178 13f79535-47bb-0310-9956-ffa450edef68

src/main/java/org/apache/maven/reporting/exec/DefaultMavenReportExecutor.java

diff --git a/src/main/java/org/apache/maven/reporting/exec/DefaultMavenReportExecutor.java b/src/main/java/org/apache/maven/reporting/exec/DefaultMavenReportExecutor.java
index 99e6254..c339b05 100644
--- a/src/main/java/org/apache/maven/reporting/exec/DefaultMavenReportExecutor.java
+++ b/src/main/java/org/apache/maven/reporting/exec/DefaultMavenReportExecutor.java
@@ -292,28 +292,27 @@
 
             if ( !mojoExecution.getForkedExecutions().isEmpty() )
             {
-                String msg = "preparing '" + report.getGoal() + "' report requires '";
-                boolean isPhase = StringUtils.isNotEmpty( mojoDescriptor.getExecutePhase() );
-                if ( isPhase )
+                String execution;
+                if ( StringUtils.isNotEmpty( mojoDescriptor.getExecutePhase() ) )
                 {
                     // forked phase
-                    String lifecycleId =
-                        StringUtils.isEmpty( mojoDescriptor.getExecuteLifecycle() ) ? ""
-                                        : ( '[' + mojoDescriptor.getExecuteLifecycle() + ']' );
-                    logger.info( msg + lifecycleId + mojoDescriptor.getExecutePhase() + "' forked phase execution" );
+                    execution = "'"
+                        + ( StringUtils.isEmpty( mojoDescriptor.getExecuteLifecycle() ) ? ""
+                                        : ( '[' + mojoDescriptor.getExecuteLifecycle() + ']' ) )
+                        + mojoDescriptor.getExecutePhase() + "' forked phase execution";
                 }
                 else
                 {
                     // forked goal
-                    logger.info( msg + mojoDescriptor.getExecuteGoal() + "' forked goal execution" );
+                    execution = "'" + mojoDescriptor.getExecuteGoal() + "' forked goal execution";
                 }
 
+                logger.info( "preparing '" + report.getGoal() + "' report requires " + execution );
+
                 lifecycleExecutor.executeForkedExecutions( mojoExecution,
                                                            mavenReportExecutorRequest.getMavenSession() );
 
-                logger.info( "" );
-                logger.info( "forked " + ( isPhase ? "phase" : "goal" ) + " execution for '" + report.getGoal()
-                    + "' report preparation done" );
+                logger.info( execution + " for '" + report.getGoal() + "' report preparation done" );
             }
 
             // ok, report is ready to generate