You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by hb...@apache.org on 2010/06/27 23:49:14 UTC
svn commit: r958428 - /maven/plugins/branches/maven-site-plugin-3.x/src/main/java/org/apache/maven/plugins/site/DefaultMavenReportExecutor.java

Author: hboutemy
Date: Sun Jun 27 21:49:14 2010
New Revision: 958428

URL: http://svn.apache.org/viewvc?rev=958428&view=rev
Log:
fixed new MavenMultiPageReport interface detection

Modified:
    maven/plugins/branches/maven-site-plugin-3.x/src/main/java/org/apache/maven/plugins/site/DefaultMavenReportExecutor.java

Modified: maven/plugins/branches/maven-site-plugin-3.x/src/main/java/org/apache/maven/plugins/site/DefaultMavenReportExecutor.java
URL: http://svn.apache.org/viewvc/maven/plugins/branches/maven-site-plugin-3.x/src/main/java/org/apache/maven/plugins/site/DefaultMavenReportExecutor.java?rev=958428&r1=958427&r2=958428&view=diff
==============================================================================
--- maven/plugins/branches/maven-site-plugin-3.x/src/main/java/org/apache/maven/plugins/site/DefaultMavenReportExecutor.java (original)
+++ maven/plugins/branches/maven-site-plugin-3.x/src/main/java/org/apache/maven/plugins/site/DefaultMavenReportExecutor.java Sun Jun 27 21:49:14 2010
@@ -85,6 +85,7 @@ public class DefaultMavenReportExecutor
 
         List<String> imports = new ArrayList<String>();
         imports.add( "org.apache.maven.reporting.MavenReport" );
+        imports.add( "org.apache.maven.reporting.MavenMultiPageReport" );
         imports.add( "org.apache.maven.doxia.siterenderer.Renderer" );
         imports.add( "org.apache.maven.doxia.sink.SinkFactory" );
         imports.add( "org.codehaus.doxia.sink.Sink" );
@@ -185,8 +186,8 @@ public class DefaultMavenReportExecutor
                         }
                         if ( getLog().isDebugEnabled() )
                         {
-                            getLog().debug( "mojoExecution mergedConfiguration " + mergedConfiguration );
-                            getLog().debug( "mojoExecution cleanedConfiguration " + cleanedConfiguration );
+                            getLog().debug( "mojoExecution mergedConfiguration: " + mergedConfiguration );
+                            getLog().debug( "mojoExecution cleanedConfiguration: " + cleanedConfiguration );
                         }
 
                         mojoExecution.setConfiguration( cleanedConfiguration );
@@ -299,7 +300,7 @@ public class DefaultMavenReportExecutor
             if ( getLog().isDebugEnabled() && mojoDescriptor != null && mojoDescriptor.getImplementationClass() != null )
             {
                 getLog().debug(
-                                "class " + mojoDescriptor.getImplementationClass().getName() + " isMavenReport "
+                                "class " + mojoDescriptor.getImplementationClass().getName() + " isMavenReport: "
                                     + isMavenReport );
             }
             if ( !isMavenReport )
@@ -414,7 +415,7 @@ public class DefaultMavenReportExecutor
         }
 
 
-        logger.warn( "report plugin " + reportPluginKey + " has an empty version" );
+        logger.warn( "Report plugin " + reportPluginKey + " has an empty version." );
         logger.warn( "" );
         logger.warn( "It is highly recommended to fix these problems"
             + " because they threaten the stability of your build." );