You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by vs...@apache.org on 2008/07/09 22:56:46 UTC

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

Author: vsiveton
Date: Wed Jul  9 13:56:46 2008
New Revision: 675341

URL: http://svn.apache.org/viewvc?rev=675341&view=rev
Log:
MPIR-114: Validate all generated reports with validator.w3.org

o fixed dependencies report

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

Modified: maven/plugins/trunk/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/dependencies/renderer/DependenciesRenderer.java
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/dependencies/renderer/DependenciesRenderer.java?rev=675341&r1=675340&r2=675341&view=diff
==============================================================================
--- maven/plugins/trunk/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/dependencies/renderer/DependenciesRenderer.java (original)
+++ maven/plugins/trunk/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/dependencies/renderer/DependenciesRenderer.java Wed Jul  9 13:56:46 2008
@@ -887,7 +887,8 @@
 
         if ( !node.getChildren().isEmpty() )
         {
-            sink.list();
+            boolean toBeIncluded = false;
+            List subList = new ArrayList();
             for ( Iterator deps = node.getChildren().iterator(); deps.hasNext(); )
             {
                 DependencyNode dep = (DependencyNode) deps.next();
@@ -897,9 +898,21 @@
                     continue;
                 }
 
-                printDependencyListing( dep );
+                subList.add( dep );
+                toBeIncluded = true;
+            }
+
+            if ( toBeIncluded )
+            {
+                sink.list();
+                for ( Iterator deps = subList.iterator(); deps.hasNext(); )
+                {
+                    DependencyNode dep = (DependencyNode) deps.next();
+
+                    printDependencyListing( dep );
+                }
+                sink.list_();
             }
-            sink.list_();
         }
 
         sink.listItem_();