You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by jd...@apache.org on 2007/08/14 21:29:57 UTC
svn commit: r565861 -
/maven/shared/trunk/maven-reporting-impl/src/main/java/org/apache/maven/reporting/AbstractMavenReport.java
Author: jdcasey
Date: Tue Aug 14 12:29:56 2007
New Revision: 565861
URL: http://svn.apache.org/viewvc?view=rev&rev=565861
Log:
[MNG-2188] Check canGenerateReport() before proceeding with generate() execution. If false, print a message to the getLog(), and return.
Modified:
maven/shared/trunk/maven-reporting-impl/src/main/java/org/apache/maven/reporting/AbstractMavenReport.java
Modified: maven/shared/trunk/maven-reporting-impl/src/main/java/org/apache/maven/reporting/AbstractMavenReport.java
URL: http://svn.apache.org/viewvc/maven/shared/trunk/maven-reporting-impl/src/main/java/org/apache/maven/reporting/AbstractMavenReport.java?view=diff&rev=565861&r1=565860&r2=565861
==============================================================================
--- maven/shared/trunk/maven-reporting-impl/src/main/java/org/apache/maven/reporting/AbstractMavenReport.java (original)
+++ maven/shared/trunk/maven-reporting-impl/src/main/java/org/apache/maven/reporting/AbstractMavenReport.java Tue Aug 14 12:29:56 2007
@@ -93,10 +93,16 @@
throw new MavenReportException( "You must specify a sink." );
}
+ if ( !canGenerateReport() )
+ {
+ getLog().info( "This report cannot be generated as part of the current build." );
+ return;
+ }
+
this.sink = sink;
this.sinkFactory = sinkFactory;
-
+
executeReport( locale );
closeReport();