You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by si...@apache.org on 2008/09/10 07:27:40 UTC
svn commit: r693707 -
/maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/MavenProject.java
Author: sisbell
Date: Tue Sep 9 22:27:39 2008
New Revision: 693707
URL: http://svn.apache.org/viewvc?rev=693707&view=rev
Log:
Fix: MavenProject.pluginArtifacts now returns empty set, rather than null. This was causing maven-project-info-reports-plugin to fail.
Modified:
maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/MavenProject.java
Modified: maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/MavenProject.java
URL: http://svn.apache.org/viewvc/maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/MavenProject.java?rev=693707&r1=693706&r2=693707&view=diff
==============================================================================
--- maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/MavenProject.java (original)
+++ maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/MavenProject.java Tue Sep 9 22:27:39 2008
@@ -1294,7 +1294,7 @@
{
return pluginArtifacts;
}
- Set pa = new HashSet();
+ pluginArtifacts = new HashSet();
if ( artifactFactory != null )
{
List plugins = getBuildPlugins();
@@ -1320,16 +1320,15 @@
}
catch ( InvalidVersionSpecificationException e )
{
- return pa;
+ return pluginArtifacts;
}
if ( artifact != null )
{
- pa.add( artifact );
+ pluginArtifacts.add( artifact );
}
}
}
- pluginArtifacts = pa;
pluginArtifactMap = null;
return pluginArtifacts;
}