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 )


Reply via email to