You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by ol...@apache.org on 2010/09/05 19:48:47 UTC

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

Author: olamy
Date: Sun Sep  5 17:48:47 2010
New Revision: 992829

URL: http://svn.apache.org/viewvc?rev=992829&view=rev
Log:
[MSITE-500] Warning message about missing report plugin version shows null instead of plugin groupId:aritfactId

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=992829&r1=992828&r2=992829&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 Sep  5 17:48:47 2010
@@ -412,11 +412,10 @@ public class DefaultMavenReportExecutor
                                        MavenReportExecutorRequest mavenReportExecutorRequest )
         throws PluginVersionResolutionException
     {
-        String reportPluginKey = null;
+        String reportPluginKey = reportPlugin.getGroupId() + ':' + reportPlugin.getArtifactId();
 
         if ( getLog().isDebugEnabled() )
         {
-            reportPluginKey = reportPlugin.getGroupId() + ':' + reportPlugin.getArtifactId();
             getLog().debug( "resolving version for " + reportPluginKey );
         }
         if ( reportPlugin.getVersion() != null )