You are viewing a plain text version of this content. The canonical link for it is here.

Posted to doxia-commits@maven.apache.org by vs...@apache.org on 2008/02/07 13:16:35 UTC

svn commit: r619375 - /maven/doxia/doxia/trunk/doxia-modules/doxia-module-fo/src/main/java/org/apache/maven/doxia/module/fo/FoAggregateSink.java

Author: vsiveton
Date: Thu Feb  7 04:16:28 2008
New Revision: 619375

URL: http://svn.apache.org/viewvc?rev=619375&view=rev
Log:
o fixed potential NPE

Modified:
    maven/doxia/doxia/trunk/doxia-modules/doxia-module-fo/src/main/java/org/apache/maven/doxia/module/fo/FoAggregateSink.java

Modified: maven/doxia/doxia/trunk/doxia-modules/doxia-module-fo/src/main/java/org/apache/maven/doxia/module/fo/FoAggregateSink.java
URL: http://svn.apache.org/viewvc/maven/doxia/doxia/trunk/doxia-modules/doxia-module-fo/src/main/java/org/apache/maven/doxia/module/fo/FoAggregateSink.java?rev=619375&r1=619374&r2=619375&view=diff
==============================================================================
--- maven/doxia/doxia/trunk/doxia-modules/doxia-module-fo/src/main/java/org/apache/maven/doxia/module/fo/FoAggregateSink.java (original)
+++ maven/doxia/doxia/trunk/doxia-modules/doxia-module-fo/src/main/java/org/apache/maven/doxia/module/fo/FoAggregateSink.java Thu Feb  7 04:16:28 2008
@@ -30,6 +30,7 @@
 import org.apache.maven.doxia.document.DocumentTOC;
 import org.apache.maven.doxia.document.DocumentTOCItem;
 import org.apache.maven.doxia.util.HtmlTools;
+import org.codehaus.plexus.util.StringUtils;
 
 /**
  * A Doxia Sink that produces an aggregated FO model.
@@ -193,6 +194,12 @@
      */
     private String getIdName( String name )
     {
+        if ( StringUtils.isEmpty( name ) )
+        {
+            // TODO log this behavior
+            return "";
+        }
+
         String idName = name;
 
         // prepend "./" and strip extension