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 de...@apache.org on 2007/08/02 19:43:38 UTC

svn commit: r562197 - in /maven/doxia/doxia/trunk/doxia-modules/doxia-module-apt/src: main/java/org/apache/maven/doxia/module/apt/AptSink.java test/java/org/apache/maven/doxia/module/apt/AptSinkTest.java

Author: dennisl
Date: Thu Aug  2 10:43:37 2007
New Revision: 562197

URL: http://svn.apache.org/viewvc?view=rev&rev=562197
Log:
o Do not write out null values in the head section.

Modified:
    maven/doxia/doxia/trunk/doxia-modules/doxia-module-apt/src/main/java/org/apache/maven/doxia/module/apt/AptSink.java
    maven/doxia/doxia/trunk/doxia-modules/doxia-module-apt/src/test/java/org/apache/maven/doxia/module/apt/AptSinkTest.java

Modified: maven/doxia/doxia/trunk/doxia-modules/doxia-module-apt/src/main/java/org/apache/maven/doxia/module/apt/AptSink.java
URL: http://svn.apache.org/viewvc/maven/doxia/doxia/trunk/doxia-modules/doxia-module-apt/src/main/java/org/apache/maven/doxia/module/apt/AptSink.java?view=diff&rev=562197&r1=562196&r2=562197
==============================================================================
--- maven/doxia/doxia/trunk/doxia-modules/doxia-module-apt/src/main/java/org/apache/maven/doxia/module/apt/AptSink.java (original)
+++ maven/doxia/doxia/trunk/doxia-modules/doxia-module-apt/src/main/java/org/apache/maven/doxia/module/apt/AptSink.java Thu Aug  2 10:43:37 2007
@@ -140,11 +140,20 @@
         headerFlag = false;
 
         write( HEADER_START + EOL );
-        write( " " + title + EOL  );
+        if ( title != null )
+        {
+            write( " " + title + EOL );
+        }
         write( HEADER_START + EOL );
-        write( " " + author + EOL  );
+        if ( author != null )
+        {
+            write( " " + author + EOL );
+        }
         write( HEADER_START + EOL );
-        write( " " + date + EOL  );
+        if ( date != null )
+        {
+            write( " " + date + EOL );
+        }
         write( HEADER_START + EOL );
     }
 

Modified: maven/doxia/doxia/trunk/doxia-modules/doxia-module-apt/src/test/java/org/apache/maven/doxia/module/apt/AptSinkTest.java
URL: http://svn.apache.org/viewvc/maven/doxia/doxia/trunk/doxia-modules/doxia-module-apt/src/test/java/org/apache/maven/doxia/module/apt/AptSinkTest.java?view=diff&rev=562197&r1=562196&r2=562197
==============================================================================
--- maven/doxia/doxia/trunk/doxia-modules/doxia-module-apt/src/test/java/org/apache/maven/doxia/module/apt/AptSinkTest.java (original)
+++ maven/doxia/doxia/trunk/doxia-modules/doxia-module-apt/src/test/java/org/apache/maven/doxia/module/apt/AptSinkTest.java Thu Aug  2 10:43:37 2007
@@ -65,8 +65,7 @@
     /** {@inheritDoc} */
     protected String getHeadBlock()
     {
-        return AptMarkup.HEADER_START + " null" + AptMarkup.HEADER_START + " null" + AptMarkup.HEADER_START + " null"
-            + AptMarkup.HEADER_START;
+        return AptMarkup.HEADER_START + AptMarkup.HEADER_START + AptMarkup.HEADER_START + AptMarkup.HEADER_START;
     }
 
     /** {@inheritDoc} */