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 )