svn commit: r421968 - /maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/MavenProject.java

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

svn commit: r421968 - /maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/MavenProject.java

Carlos Sanchez-4
Author: carlos
Date: Fri Jul 14 10:55:57 2006
New Revision: 421968

URL: http://svn.apache.org/viewvc?rev=421968&view=rev
Log:
Removed code duplication

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=421968&r1=421967&r2=421968&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 Fri Jul 14 10:55:57 2006
@@ -474,21 +474,7 @@
                 if ( Artifact.SCOPE_COMPILE.equals( a.getScope() ) || Artifact.SCOPE_PROVIDED.equals( a.getScope() ) ||
                     Artifact.SCOPE_SYSTEM.equals( a.getScope() ) )
                 {
-                    String refId = getProjectReferenceId( a.getGroupId(), a.getArtifactId(), a.getVersion() );
-                    MavenProject project = (MavenProject) projectReferences.get( refId );
-                    if ( project != null )
-                    {
-                        list.add( project.getBuild().getOutputDirectory() );
-                    }
-                    else
-                    {
-                        File file = a.getFile();
-                        if ( file == null )
-                        {
-                            throw new DependencyResolutionRequiredException( a );
-                        }
-                        list.add( file.getPath() );
-                    }
+                    addArtifactPath( a, list );
                 }
             }
         }
@@ -745,21 +731,7 @@
                 // TODO: let the scope handler deal with this
                 if ( Artifact.SCOPE_SYSTEM.equals( a.getScope() ) )
                 {
-                    String refId = getProjectReferenceId( a.getGroupId(), a.getArtifactId(), a.getVersion() );
-                    MavenProject project = (MavenProject) projectReferences.get( refId );
-                    if ( project != null )
-                    {
-                        list.add( project.getBuild().getOutputDirectory() );
-                    }
-                    else
-                    {
-                        File file = a.getFile();
-                        if ( file == null )
-                        {
-                            throw new DependencyResolutionRequiredException( a );
-                        }
-                        list.add( file.getPath() );
-                    }
+                    addArtifactPath( a, list );
                 }
             }
         }
@@ -1625,5 +1597,24 @@
             }
         }
         return pluginArtifact;
+    }
+    
+    private void addArtifactPath(Artifact a, List list) throws DependencyResolutionRequiredException
+    {
+        String refId = getProjectReferenceId( a.getGroupId(), a.getArtifactId(), a.getVersion() );
+        MavenProject project = (MavenProject) projectReferences.get( refId );
+        if ( project != null )
+        {
+            list.add( project.getBuild().getOutputDirectory() );
+        }
+        else
+        {
+            File file = a.getFile();
+            if ( file == null )
+            {
+                throw new DependencyResolutionRequiredException( a );
+            }
+            list.add( file.getPath() );
+        }
     }
 }