You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@cocoon.apache.org by an...@apache.org on 2005/05/27 07:35:33 UTC

svn commit: r178756 - /cocoon/blocks/core/forms/trunk/java/org/apache/cocoon/forms/datatype/StaticSelectionList.java

Author: antonio
Date: Thu May 26 22:35:31 2005
New Revision: 178756

URL: http://svn.apache.org/viewcvs?rev=178756&view=rev
Log:
Move non-null case up

Modified:
    cocoon/blocks/core/forms/trunk/java/org/apache/cocoon/forms/datatype/StaticSelectionList.java

Modified: cocoon/blocks/core/forms/trunk/java/org/apache/cocoon/forms/datatype/StaticSelectionList.java
URL: http://svn.apache.org/viewcvs/cocoon/blocks/core/forms/trunk/java/org/apache/cocoon/forms/datatype/StaticSelectionList.java?rev=178756&r1=178755&r2=178756&view=diff
==============================================================================
--- cocoon/blocks/core/forms/trunk/java/org/apache/cocoon/forms/datatype/StaticSelectionList.java (original)
+++ cocoon/blocks/core/forms/trunk/java/org/apache/cocoon/forms/datatype/StaticSelectionList.java Thu May 26 22:35:31 2005
@@ -100,11 +100,11 @@
         public void generateSaxFragment(ContentHandler contentHandler, Locale locale, Convertor.FormatCache formatCache)
         throws SAXException {
             String stringValue;
-            if (this.value == null) {
+            if (this.value != null) {
+                stringValue = datatype.getConvertor().convertToString(this.value, locale, formatCache);
+            } else {
                 // Null value translates into the empty string
                 stringValue = "";
-            } else {
-                stringValue = datatype.getConvertor().convertToString(this.value, locale, formatCache);
             }
 
             AttributesImpl attrs = new AttributesImpl();