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:54:02 UTC

svn commit: r958429 - /maven/plugins/branches/maven-site-plugin-3.x/src/main/java/org/apache/maven/plugins/site/ReportDocumentRenderer.java

Author: hboutemy
Date: Sun Jun 27 21:54:01 2010
New Revision: 958429

URL: http://svn.apache.org/viewvc?rev=958429&view=rev
Log:
added a warning when a report doesn't implement new MavenMultiPageReport interface but only old MavenReport

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

Modified: maven/plugins/branches/maven-site-plugin-3.x/src/main/java/org/apache/maven/plugins/site/ReportDocumentRenderer.java
URL: http://svn.apache.org/viewvc/maven/plugins/branches/maven-site-plugin-3.x/src/main/java/org/apache/maven/plugins/site/ReportDocumentRenderer.java?rev=958429&r1=958428&r2=958429&view=diff
==============================================================================
--- maven/plugins/branches/maven-site-plugin-3.x/src/main/java/org/apache/maven/plugins/site/ReportDocumentRenderer.java (original)
+++ maven/plugins/branches/maven-site-plugin-3.x/src/main/java/org/apache/maven/plugins/site/ReportDocumentRenderer.java Sun Jun 27 21:54:01 2010
@@ -162,8 +162,10 @@ public class ReportDocumentRenderer
             }
             else
             {
+                // old API compatibility
+                log.warn( "\"" + localReportName + "\" report is using old MavenReport API: "
+                    + "please upgrade report to a version using MavenMultiPageReport." );
                 report.generate( sink, locale );
-
             }
         }
         catch ( MavenReportException e )
@@ -174,7 +176,7 @@ public class ReportDocumentRenderer
         {
             StringBuilder stringBuilder =
                 new StringBuilder( " an issue has occured with report " + report.getClass().getName() );
-            stringBuilder.append( ", skip LinkageError " + e.getMessage() + ", please report an issue to maven dev team" );
+            stringBuilder.append( ", skip LinkageError " + e.getMessage() + ", please report an issue to maven dev team." );
             log.warn( stringBuilder.toString(), e );
         }
         finally