Author: ajith
Date: Mon Dec  3 11:25:15 2007
New Revision: 600638

URL: http://svn.apache.org/viewvc?rev=600638&view=rev
Log:
1. Fixing the issue  WSCOMMONS-274. The min/max occurances were populated 
correctly but not "0" was ignored at serialization

Modified:
    
webservices/commons/branches/modules/XmlSchema/1.3.3/src/main/java/org/apache/ws/commons/schema/XmlSchemaSerializer.java

Modified: 
webservices/commons/branches/modules/XmlSchema/1.3.3/src/main/java/org/apache/ws/commons/schema/XmlSchemaSerializer.java
URL: 
http://svn.apache.org/viewvc/webservices/commons/branches/modules/XmlSchema/1.3.3/src/main/java/org/apache/ws/commons/schema/XmlSchemaSerializer.java?rev=600638&r1=600637&r2=600638&view=diff
==============================================================================
--- 
webservices/commons/branches/modules/XmlSchema/1.3.3/src/main/java/org/apache/ws/commons/schema/XmlSchemaSerializer.java
 (original)
+++ 
webservices/commons/branches/modules/XmlSchema/1.3.3/src/main/java/org/apache/ws/commons/schema/XmlSchemaSerializer.java
 Mon Dec  3 11:25:15 2007
@@ -1000,7 +1000,8 @@
             sequence.setAttribute("id", sequenceObj.id);
 
 
-        if (sequenceObj.maxOccurs < Long.MAX_VALUE && sequenceObj.maxOccurs > 
1)
+        if (sequenceObj.maxOccurs < Long.MAX_VALUE &&
+                (sequenceObj.maxOccurs > 1 || sequenceObj.maxOccurs == 0))
             sequence.setAttribute("maxOccurs",
                     sequenceObj.maxOccurs + "");
         else if (sequenceObj.maxOccurs == Long.MAX_VALUE)
@@ -1008,8 +1009,9 @@
                     "unbounded");
         //else not serialized
 
-
-        if (sequenceObj.minOccurs > 1)
+        //1 is the default and hence not serialized
+        //there is no valid case where min occurs can be unbounded!
+        if (sequenceObj.minOccurs > 1 || sequenceObj.minOccurs == 0)
             sequence.setAttribute("minOccurs",
                     sequenceObj.minOccurs + "");
 



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to