You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by be...@apache.org on 2010/09/02 21:59:15 UTC

svn commit: r992092 - /maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/project/DefaultProjectDependenciesResolver.java

Author: bentmann
Date: Thu Sep  2 19:59:15 2010
New Revision: 992092

URL: http://svn.apache.org/viewvc?rev=992092&view=rev
Log:
o Made project dependency graph reflect partial resolution properly

Modified:
    maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/project/DefaultProjectDependenciesResolver.java

Modified: maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/project/DefaultProjectDependenciesResolver.java
URL: http://svn.apache.org/viewvc/maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/project/DefaultProjectDependenciesResolver.java?rev=992092&r1=992091&r2=992092&view=diff
==============================================================================
--- maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/project/DefaultProjectDependenciesResolver.java (original)
+++ maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/project/DefaultProjectDependenciesResolver.java Thu Sep  2 19:59:15 2010
@@ -162,13 +162,15 @@ public class DefaultProjectDependenciesR
     {
         for ( ArtifactResult ar : results )
         {
+            DependencyNode node = ar.getRequest().getDependencyNode();
             if ( ar.isResolved() )
             {
-                result.addResolvedDependency( ar.getRequest().getDependencyNode().getDependency() );
+                node.setArtifact( ar.getArtifact() );
+                result.addResolvedDependency( node.getDependency() );
             }
             else
             {
-                result.setResolutionErrors( ar.getRequest().getDependencyNode().getDependency(), ar.getExceptions() );
+                result.setResolutionErrors( node.getDependency(), ar.getExceptions() );
             }
         }
     }