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 2009/06/16 23:11:34 UTC
svn commit: r785405 -
/maven/components/trunk/maven-core/src/main/java/org/apache/maven/project/MavenProject.java
Author: olamy
Date: Tue Jun 16 21:11:33 2009
New Revision: 785405
URL: http://svn.apache.org/viewvc?rev=785405&view=rev
Log:
fix possible NPE as filter can be null (dependency:tree)
Modified:
maven/components/trunk/maven-core/src/main/java/org/apache/maven/project/MavenProject.java
Modified: maven/components/trunk/maven-core/src/main/java/org/apache/maven/project/MavenProject.java
URL: http://svn.apache.org/viewvc/maven/components/trunk/maven-core/src/main/java/org/apache/maven/project/MavenProject.java?rev=785405&r1=785404&r2=785405&view=diff
==============================================================================
--- maven/components/trunk/maven-core/src/main/java/org/apache/maven/project/MavenProject.java (original)
+++ maven/components/trunk/maven-core/src/main/java/org/apache/maven/project/MavenProject.java Tue Jun 16 21:11:33 2009
@@ -250,10 +250,12 @@
for ( Dependency d : getDependencies() )
{
Artifact dependencyArtifact = repositorySystem.createArtifact( d.getGroupId(), d.getArtifactId(), d.getVersion(), d.getScope(), d.getType() );
-
- if ( filter.include( dependencyArtifact ) )
+ if ( filter != null )
{
- artifacts.add( dependencyArtifact );
+ if ( filter.include( dependencyArtifact ) )
+ {
+ artifacts.add( dependencyArtifact );
+ }
}
}