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} */