svn commit: r600128 - /maven/plugins/trunk/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/dependencies/Dependencies.java

[vsiveton]

Author: vsiveton
Date: Sat Dec  1 05:51:14 2007
New Revision: 600128

URL: http://svn.apache.org/viewvc?rev=600128&view=rev
Log:
o improved code legibility

Modified:
    
maven/plugins/trunk/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/dependencies/Dependencies.java

Modified: 
maven/plugins/trunk/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/dependencies/Dependencies.java
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/dependencies/Dependencies.java?rev=600128&r1=600127&r2=600128&view=diff
==============================================================================
--- 
maven/plugins/trunk/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/dependencies/Dependencies.java
 (original)
+++ 
maven/plugins/trunk/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/dependencies/Dependencies.java
 Sat Dec  1 05:51:14 2007
@@ -67,29 +67,6 @@
         mapArtifactFiles( dependencyTree.getRootNode(), projectMap );
     }
 
-    private void mapArtifactFiles( DependencyNode node, Map projectMap )
-    {
-        List childs = node.getChildren();
-        if ( ( childs == null ) || childs.isEmpty() )
-        {
-            return;
-        }
-
-        Iterator it = childs.iterator();
-        while ( it.hasNext() )
-        {
-            DependencyNode anode = (DependencyNode) it.next();
-            String key = ArtifactUtils.versionlessKey( anode.getArtifact() );
-            Artifact projartifact = (Artifact) projectMap.get( key );
-            if ( projartifact != null )
-            {
-                anode.getArtifact().setFile( projartifact.getFile() );
-            }
-
-            mapArtifactFiles( anode, projectMap );
-        }
-    }
-
     public boolean hasDependencies()
     {
         return ( projectDependencies != null ) && ( 
!this.projectDependencies.isEmpty() );
@@ -145,5 +122,32 @@
         }
 
         return jarAnalyzer.getJarData();
+    }
+
+    // ----------------------------------------------------------------------
+    // Private methods
+    // ----------------------------------------------------------------------
+
+    private void mapArtifactFiles( DependencyNode node, Map projectMap )
+    {
+        List childs = node.getChildren();
+        if ( ( childs == null ) || childs.isEmpty() )
+        {
+            return;
+        }
+
+        Iterator it = childs.iterator();
+        while ( it.hasNext() )
+        {
+            DependencyNode anode = (DependencyNode) it.next();
+            String key = ArtifactUtils.versionlessKey( anode.getArtifact() );
+            Artifact projartifact = (Artifact) projectMap.get( key );
+            if ( projartifact != null )
+            {
+                anode.getArtifact().setFile( projartifact.getFile() );
+            }
+
+            mapArtifactFiles( anode, projectMap );
+        }
     }
 }