jochen 2005/08/04 14:33:29
Modified: src/jaxme/org/apache/ws/jaxme/junit Tag: v0_4 JiraTest.java
src/xs/org/apache/ws/jaxme/xs/junit Tag: v0_4 JiraTest.java
. Tag: v0_4 status.xml
src/xs/org/apache/ws/jaxme/xs/impl Tag: v0_4 XSTypeImpl.java
Log:
If a complex type was created by referencing a group, then
the referencing particles maxOccurs and minOccurs values haven't
been inherited.
PR: JAXME-63
Revision Changes Path
No revision
No revision
1.7.2.3 +35 -1
ws-jaxme/src/jaxme/org/apache/ws/jaxme/junit/JiraTest.java
Index: JiraTest.java
===================================================================
RCS file:
/home/cvs/ws-jaxme/src/jaxme/org/apache/ws/jaxme/junit/JiraTest.java,v
retrieving revision 1.7.2.2
retrieving revision 1.7.2.3
diff -u -r1.7.2.2 -r1.7.2.3
--- JiraTest.java 30 Jul 2005 21:04:47 -0000 1.7.2.2
+++ JiraTest.java 4 Aug 2005 21:33:29 -0000 1.7.2.3
@@ -26,6 +26,7 @@
import org.apache.ws.jaxme.test.misc.types.impl.RowImpl;
import org.apache.ws.jaxme.test.misc.xsimport.a.Outer;
import org.xml.sax.InputSource;
+import org.xml.sax.SAXException;
/** Some excerpts from Jira bug reports.
@@ -187,7 +188,6 @@
xml += (char) ('0' + j);
}
xml += "'/>";
- System.out.println("Parsing " + xml);
InputSource isource = new InputSource(new StringReader(xml));
isource.setSystemId("testJAXME62-a-" + i + ".xsd");
EventDetector ed = new EventDetector();
@@ -196,4 +196,38 @@
u.unmarshal(isource);
assertEquals(pSuccess, ed.isSuccess());
}
+
+ /** Test for <a
href="http://issues.apache.org/jira/browse/JAXME-63";>JAXME-63</a>.
+ */
+ public void testJAXME63() throws Exception {
+ final String xml =
+ "<xs:schema\n"
+ + " xmlns:xs='http://www.w3.org/2001/XMLSchema'\n"
+ + " elementFormDefault='qualified'>\n"
+ + " <xs:group name='params'>\n"
+ + " <xs:choice>\n"
+ + " <xs:element name='string' type='xs:string'/>\n"
+ + " <xs:element name='int' type='xs:int'/>\n"
+ + " <xs:element name='boolean'
type='xs:boolean'/>\n"
+ + " </xs:choice>\n"
+ + " </xs:group>\n"
+ + " <xs:element name='call'>\n"
+ + " <xs:complexType>\n"
+ + " <xs:group ref='params'
maxOccurs='unbounded'/>\n"
+ + " </xs:complexType>\n"
+ + " </xs:element>\n"
+ + "</xs:schema>";
+ Generator g = new GeneratorImpl();
+ g.setProperty("jaxme.package.name",
"org.apache.ws.jaxme.test.jira.jaxme63");
+ SchemaReader sr = new JAXBSchemaReader();
+ g.setSchemaReader(sr);
+ sr.setGenerator(g);
+ SchemaSG schema = g.getSchemaReader().parse(new InputSource(new
StringReader(xml)));
+ try {
+ schema.generate();
+ fail("Expected exception");
+ } catch (SAXException e) {
+ assertTrue(e.getMessage().indexOf("Model groups with
maxOccurs > 1 are not yet supported.") != -1);
+ }
+ }
}
No revision
No revision
1.4.2.1 +34 -0
ws-jaxme/src/xs/org/apache/ws/jaxme/xs/junit/JiraTest.java
Index: JiraTest.java
===================================================================
RCS file:
/home/cvs/ws-jaxme/src/xs/org/apache/ws/jaxme/xs/junit/JiraTest.java,v
retrieving revision 1.4
retrieving revision 1.4.2.1
diff -u -r1.4 -r1.4.2.1
--- JiraTest.java 4 Dec 2004 23:22:33 -0000 1.4
+++ JiraTest.java 4 Aug 2005 21:33:29 -0000 1.4.2.1
@@ -13,6 +13,7 @@
import org.apache.ws.jaxme.xs.XSElement;
import org.apache.ws.jaxme.xs.XSEnumeration;
import org.apache.ws.jaxme.xs.XSParser;
+import org.apache.ws.jaxme.xs.XSParticle;
import org.apache.ws.jaxme.xs.XSSchema;
import org.apache.ws.jaxme.xs.XSSimpleContentType;
import org.apache.ws.jaxme.xs.XSSimpleType;
@@ -170,4 +171,37 @@
assertEquals(XSID.getInstance(), idAttr.getType());
assertFalse(idRef.isOptional());
}
+
+ /** Test for <a
href="http://issues.apache.org/jira/browse/JAXME-63";>JAXME-63</a>.
+ */
+ public void testJAXME63() throws Exception {
+ final String xml =
+ "<xs:schema\n"
+ + " xmlns:xs='http://www.w3.org/2001/XMLSchema'\n"
+ + " elementFormDefault='qualified'>\n"
+ + " <xs:group name='params'>\n"
+ + " <xs:choice>\n"
+ + " <xs:element name='string' type='xs:string'/>\n"
+ + " <xs:element name='int' type='xs:int'/>\n"
+ + " <xs:element name='boolean'
type='xs:boolean'/>\n"
+ + " </xs:choice>\n"
+ + " </xs:group>\n"
+ + " <xs:element name='call'>\n"
+ + " <xs:complexType>\n"
+ + " <xs:group ref='params'
maxOccurs='unbounded'/>\n"
+ + " </xs:complexType>\n"
+ + " </xs:element>\n"
+ + "</xs:schema>";
+ XSSchema schema = parse(xml, "jaxme63.xsd");
+ XSElement[] elements = schema.getElements();
+ assertEquals(1, elements.length);
+ XSElement call = elements[0];
+ assertEquals(new XsQName((String) null, "call"), call.getName());
+ XSComplexType type = assertComplexType(call.getType());
+ XSParticle particle = assertComplexContent(type);
+ assertTrue(particle.isGroup());
+ assertTrue(particle.getGroup().isChoice());
+ assertEquals(1, particle.getMinOccurs());
+ assertEquals(-1, particle.getMaxOccurs());
+ }
}
No revision
No revision
1.45.2.20 +5 -0 ws-jaxme/status.xml
Index: status.xml
===================================================================
RCS file: /home/cvs/ws-jaxme/status.xml,v
retrieving revision 1.45.2.19
retrieving revision 1.45.2.20
diff -u -r1.45.2.19 -r1.45.2.20
--- status.xml 30 Jul 2005 21:11:50 -0000 1.45.2.19
+++ status.xml 4 Aug 2005 21:33:29 -0000 1.45.2.20
@@ -75,6 +75,11 @@
is now trapped and converted to a proper ValidationEvent.
(JAXME-62)
</action>
+ <action dev="JW" type="fix" context="xs">
+ If a complex type was created by referencing a group, then
+ the referencing particles maxOccurs and minOccurs values haven't
+ been inherited. (JAXME-63)
+ </action>
</release>
<release version="0.4" date="2005-May-19">
<action dev="JW" type="fix" context="xs">
No revision
No revision
1.16.2.3 +5 -2
ws-jaxme/src/xs/org/apache/ws/jaxme/xs/impl/XSTypeImpl.java
Index: XSTypeImpl.java
===================================================================
RCS file:
/home/cvs/ws-jaxme/src/xs/org/apache/ws/jaxme/xs/impl/XSTypeImpl.java,v
retrieving revision 1.16.2.2
retrieving revision 1.16.2.3
diff -u -r1.16.2.2 -r1.16.2.3
--- XSTypeImpl.java 19 May 2005 20:30:52 -0000 1.16.2.2
+++ XSTypeImpl.java 4 Aug 2005 21:33:29 -0000 1.16.2.3
@@ -455,10 +455,13 @@
public class XSGroupComplexTypeImpl extends XSBasicComplexTypeImpl {
private final XSGroup group;
- public XSGroupComplexTypeImpl(XSType pOwner, XsTComplexType pType,
XSGroup pGroup) throws SAXException {
+ public XSGroupComplexTypeImpl(XSType pOwner, XsTComplexType pType,
XSGroup pGroup,
+ XsTGroupRef pRef)
throws SAXException {
super(pOwner, pType);
group = pGroup;
XSParticleImpl particle = new XSParticleImpl(group);
+ particle.setMinOccurs(pRef.getMinOccurs());
+ particle.setMaxOccurs(pRef.getMaxOccurs());
if (group.getParticles().length == 0) {
setParticle(XsComplexContentType.EMPTY, particle);
} else {
@@ -711,7 +714,7 @@
throw new LocSAXException("Unknown group: " + myName,
getLocator());
}
group.validate();
- myComplexType = new XSGroupComplexTypeImpl(this,
myXsTComplexType, group);
+ myComplexType = new XSGroupComplexTypeImpl(this,
myXsTComplexType, group, groupRef);
} else {
throw new IllegalStateException("Invalid particle: " +
particle.getClass().getName());
}
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
