Author: bimargulies
Date: Sat Nov  7 18:59:25 2009
New Revision: 833733

URL: http://svn.apache.org/viewvc?rev=833733&view=rev
Log:
Modernization on Redefine and Element.

Modified:
    
webservices/commons/trunk/modules/XmlSchema/src/main/java/org/apache/ws/commons/schema/SchemaBuilder.java
    
webservices/commons/trunk/modules/XmlSchema/src/main/java/org/apache/ws/commons/schema/XmlSchemaElement.java
    
webservices/commons/trunk/modules/XmlSchema/src/main/java/org/apache/ws/commons/schema/XmlSchemaRedefine.java
    
webservices/commons/trunk/modules/XmlSchema/src/main/java/org/apache/ws/commons/schema/XmlSchemaSerializer.java
    
webservices/commons/trunk/modules/XmlSchema/src/test/java/tests/ConstraintsTest.java
    
webservices/commons/trunk/modules/XmlSchema/src/test/java/tests/RedefineTest.java

Modified: 
webservices/commons/trunk/modules/XmlSchema/src/main/java/org/apache/ws/commons/schema/SchemaBuilder.java
URL: 
http://svn.apache.org/viewvc/webservices/commons/trunk/modules/XmlSchema/src/main/java/org/apache/ws/commons/schema/SchemaBuilder.java?rev=833733&r1=833732&r2=833733&view=diff
==============================================================================
--- 
webservices/commons/trunk/modules/XmlSchema/src/main/java/org/apache/ws/commons/schema/SchemaBuilder.java
 (original)
+++ 
webservices/commons/trunk/modules/XmlSchema/src/main/java/org/apache/ws/commons/schema/SchemaBuilder.java
 Sat Nov  7 18:59:25 2009
@@ -1541,26 +1541,25 @@
                 XmlSchemaType type = handleSimpleType(schema, el, schemaEl,
                         false);
 
-                redefine.schemaTypes.collection.put(type.getQName(), type);
-                redefine.items.add(type);
+                redefine.getSchemaTypes().put(type.getQName(), type);
+                redefine.getItems().add(type);
             } else if (el.getLocalName().equals("complexType")) {
 
                 XmlSchemaType type = handleComplexType(schema, el, schemaEl,
                         true);
 
-                redefine.schemaTypes.collection.put(type.getQName(), type);
-                redefine.items.add(type);
+                redefine.getSchemaTypes().put(type.getQName(), type);
+                redefine.getItems().add(type);
             } else if (el.getLocalName().equals("group")) {
                 XmlSchemaGroup group = handleGroup(schema, el, schemaEl);
-                redefine.groups.collection.put(group.getQName(), group);
-                redefine.items.add(group);
+                redefine.getGroups().put(group.getQName(), group);
+                redefine.getItems().add(group);
             } else if (el.getLocalName().equals("attributeGroup")) {
                 XmlSchemaAttributeGroup group = handleAttributeGroup(schema,
                         el, schemaEl);
 
-                redefine.attributeGroups.collection
-                        .put(group.getQName(), group);
-                redefine.items.add(group);
+                redefine.getAttributeGroups().put(group.getQName(), group);
+                redefine.getItems().add(group);
             } else if (el.getLocalName().equals("annotation")) {
                 XmlSchemaAnnotation annotation = handleAnnotation(el);
                 redefine.setAnnotation(annotation);

Modified: 
webservices/commons/trunk/modules/XmlSchema/src/main/java/org/apache/ws/commons/schema/XmlSchemaElement.java
URL: 
http://svn.apache.org/viewvc/webservices/commons/trunk/modules/XmlSchema/src/main/java/org/apache/ws/commons/schema/XmlSchemaElement.java?rev=833733&r1=833732&r2=833733&view=diff
==============================================================================
--- 
webservices/commons/trunk/modules/XmlSchema/src/main/java/org/apache/ws/commons/schema/XmlSchemaElement.java
 (original)
+++ 
webservices/commons/trunk/modules/XmlSchema/src/main/java/org/apache/ws/commons/schema/XmlSchemaElement.java
 Sat Nov  7 18:59:25 2009
@@ -19,6 +19,10 @@
 
 package org.apache.ws.commons.schema;
 
+import java.util.ArrayList;
+import java.util.Collections;
+import java.util.List;
+
 import javax.xml.namespace.QName;
 
 import org.apache.ws.commons.schema.utils.XmlSchemaNamedWithForm;
@@ -40,7 +44,7 @@
      */
     private XmlSchemaDerivationMethod block;
 
-    private XmlSchemaObjectCollection constraints;
+    private List<XmlSchemaIdentityConstraint> constraints;
 
     /**
      * Provides the default value of the element if its content is a simple 
type or the element's content is
@@ -83,7 +87,7 @@
         namedDelegate.setRefObject(ref);
         ref.setNamedObject(namedDelegate);
 
-        constraints = new XmlSchemaObjectCollection();
+        constraints = Collections.synchronizedList(new 
ArrayList<XmlSchemaIdentityConstraint>());
         abstractElement = false;
         nillable = false;
         finalDerivation = XmlSchemaDerivationMethod.NONE;
@@ -96,7 +100,7 @@
     /**
      * Returns a collection of constraints on the element.
      */
-    public XmlSchemaObjectCollection getConstraints() {
+    public List<XmlSchemaIdentityConstraint> getConstraints() {
         return constraints;
     }
 
@@ -233,13 +237,6 @@
     }
 
     /**
-     * @param constraints The constraints to set.
-     */
-    public void setConstraints(XmlSchemaObjectCollection constraints) {
-        this.constraints = constraints;
-    }
-
-    /**
      * @param finalDerivation The finalDerivation to set.
      */
     public void setFinalDerivation(XmlSchemaDerivationMethod finalDerivation) {

Modified: 
webservices/commons/trunk/modules/XmlSchema/src/main/java/org/apache/ws/commons/schema/XmlSchemaRedefine.java
URL: 
http://svn.apache.org/viewvc/webservices/commons/trunk/modules/XmlSchema/src/main/java/org/apache/ws/commons/schema/XmlSchemaRedefine.java?rev=833733&r1=833732&r2=833733&view=diff
==============================================================================
--- 
webservices/commons/trunk/modules/XmlSchema/src/main/java/org/apache/ws/commons/schema/XmlSchemaRedefine.java
 (original)
+++ 
webservices/commons/trunk/modules/XmlSchema/src/main/java/org/apache/ws/commons/schema/XmlSchemaRedefine.java
 Sat Nov  7 18:59:25 2009
@@ -19,6 +19,14 @@
 
 package org.apache.ws.commons.schema;
 
+import java.util.ArrayList;
+import java.util.Collections;
+import java.util.HashMap;
+import java.util.List;
+import java.util.Map;
+
+import javax.xml.namespace.QName;
+
 /**
  * Allows simple and complex types, groups, and attribute groups from external 
schema files to be redefined in
  * the current schema. This class provides versioning for the schema elements. 
Represents the World Wide Web
@@ -27,36 +35,36 @@
 
 public class XmlSchemaRedefine extends XmlSchemaExternal {
 
-    XmlSchemaObjectTable attributeGroups;
-    XmlSchemaObjectTable groups;
-    XmlSchemaObjectTable schemaTypes;
+    private Map<QName, XmlSchemaAttributeGroup> attributeGroups;
+    private Map<QName, XmlSchemaGroup> groups;
+    private Map<QName, XmlSchemaType> schemaTypes;
 
-    XmlSchemaObjectCollection items;
+    private List<XmlSchemaObject> items;
 
     /**
      * Creates new XmlSchemaRedefine
      */
     public XmlSchemaRedefine(XmlSchema parent) {
         super(parent);
-        items = new XmlSchemaObjectCollection();
-        schemaTypes = new XmlSchemaObjectTable();
-        groups = new XmlSchemaObjectTable();
-        attributeGroups = new XmlSchemaObjectTable();
+        items = Collections.synchronizedList(new ArrayList<XmlSchemaObject>());
+        schemaTypes = Collections.synchronizedMap(new HashMap<QName, 
XmlSchemaType>());
+        groups = Collections.synchronizedMap(new HashMap<QName, 
XmlSchemaGroup>());
+        attributeGroups = Collections.synchronizedMap(new HashMap<QName, 
XmlSchemaAttributeGroup>());
     }
 
-    public XmlSchemaObjectTable getAttributeGroup() {
+    public Map<QName, XmlSchemaAttributeGroup> getAttributeGroups() {
         return attributeGroups;
     }
 
-    public XmlSchemaObjectTable getGroup() {
+    public Map<QName, XmlSchemaGroup> getGroups() {
         return groups;
     }
 
-    public XmlSchemaObjectCollection getItems() {
+    public List<XmlSchemaObject> getItems() {
         return items;
     }
 
-    public XmlSchemaObjectTable getSchemaTypes() {
+    public Map<QName, XmlSchemaType> getSchemaTypes() {
         return schemaTypes;
     }
 

Modified: 
webservices/commons/trunk/modules/XmlSchema/src/main/java/org/apache/ws/commons/schema/XmlSchemaSerializer.java
URL: 
http://svn.apache.org/viewvc/webservices/commons/trunk/modules/XmlSchema/src/main/java/org/apache/ws/commons/schema/XmlSchemaSerializer.java?rev=833733&r1=833732&r2=833733&view=diff
==============================================================================
--- 
webservices/commons/trunk/modules/XmlSchema/src/main/java/org/apache/ws/commons/schema/XmlSchemaSerializer.java
 (original)
+++ 
webservices/commons/trunk/modules/XmlSchema/src/main/java/org/apache/ws/commons/schema/XmlSchemaSerializer.java
 Sat Nov  7 18:59:25 2009
@@ -1070,13 +1070,13 @@
                 serializedEl.appendChild(simpleType);
             }
         }
-        if (elementObj.getConstraints().getCount() > 0) {
-            for (int i = 0; i < elementObj.getConstraints().getCount(); i++) {
+        if (elementObj.getConstraints().size() > 0) {
+            for (int i = 0; i < elementObj.getConstraints().size(); i++) {
                 Element constraint = serializeIdentityConstraint(
                                                                  doc,
                                                                  
(XmlSchemaIdentityConstraint)
                                                                  
elementObj.getConstraints()
-                                                                     
.getItem(i), schema);
+                                                                     .get(i), 
schema);
                 serializedEl.appendChild(constraint);
             }
         }
@@ -1486,9 +1486,9 @@
             Element annotation = serializeAnnotation(doc, 
redefineObj.getAnnotation(), schema);
             redefine.appendChild(annotation);
         }
-        int itemsLength = redefineObj.items.getCount();
+        int itemsLength = redefineObj.getItems().size();
         for (int i = 0; i < itemsLength; i++) {
-            XmlSchemaObject obj = redefineObj.items.getItem(i);
+            XmlSchemaObject obj = redefineObj.getItems().get(i);
             if (obj instanceof XmlSchemaSimpleType) {
                 Element simpleType = serializeSimpleType(doc, 
(XmlSchemaSimpleType)obj, schema);
                 redefine.appendChild(simpleType);

Modified: 
webservices/commons/trunk/modules/XmlSchema/src/test/java/tests/ConstraintsTest.java
URL: 
http://svn.apache.org/viewvc/webservices/commons/trunk/modules/XmlSchema/src/test/java/tests/ConstraintsTest.java?rev=833733&r1=833732&r2=833733&view=diff
==============================================================================
--- 
webservices/commons/trunk/modules/XmlSchema/src/test/java/tests/ConstraintsTest.java
 (original)
+++ 
webservices/commons/trunk/modules/XmlSchema/src/test/java/tests/ConstraintsTest.java
 Sat Nov  7 18:59:25 2009
@@ -22,6 +22,7 @@
 import java.io.FileInputStream;
 import java.io.InputStream;
 import java.util.HashSet;
+import java.util.List;
 import java.util.Set;
 
 import javax.xml.namespace.QName;
@@ -29,6 +30,7 @@
 
 import org.apache.ws.commons.schema.XmlSchemaCollection;
 import org.apache.ws.commons.schema.XmlSchemaElement;
+import org.apache.ws.commons.schema.XmlSchemaIdentityConstraint;
 import org.apache.ws.commons.schema.XmlSchemaKey;
 import org.apache.ws.commons.schema.XmlSchemaKeyref;
 import org.apache.ws.commons.schema.XmlSchemaObjectCollection;
@@ -79,15 +81,15 @@
         assertEquals("constraintTest", elem.getName());
         assertEquals(new QName("http://soapinterop.org/types";, 
"constraintTest"), elem.getQName());
 
-        XmlSchemaObjectCollection c = elem.getConstraints();
-        assertEquals(3, c.getCount());
+        List<XmlSchemaIdentityConstraint> c = elem.getConstraints();
+        assertEquals(3, c.size());
 
         Set<String> s = new HashSet<String>();
         s.add(XmlSchemaKey.class.getName());
         s.add(XmlSchemaKeyref.class.getName());
         s.add(XmlSchemaUnique.class.getName());
-        for (int i = 0; i < c.getCount(); i++) {
-            Object o = c.getItem(i);
+        for (int i = 0; i < c.size(); i++) {
+            Object o = c.get(i);
             if (o instanceof XmlSchemaKey) {
                 XmlSchemaKey key = (XmlSchemaKey)o;
                 assertEquals("keyTest", key.getName());

Modified: 
webservices/commons/trunk/modules/XmlSchema/src/test/java/tests/RedefineTest.java
URL: 
http://svn.apache.org/viewvc/webservices/commons/trunk/modules/XmlSchema/src/test/java/tests/RedefineTest.java?rev=833733&r1=833732&r2=833733&view=diff
==============================================================================
--- 
webservices/commons/trunk/modules/XmlSchema/src/test/java/tests/RedefineTest.java
 (original)
+++ 
webservices/commons/trunk/modules/XmlSchema/src/test/java/tests/RedefineTest.java
 Sat Nov  7 18:59:25 2009
@@ -45,12 +45,12 @@
 import org.apache.ws.commons.schema.XmlSchemaMaxInclusiveFacet;
 import org.apache.ws.commons.schema.XmlSchemaMinInclusiveFacet;
 import org.apache.ws.commons.schema.XmlSchemaObjectCollection;
-import org.apache.ws.commons.schema.XmlSchemaObjectTable;
 import org.apache.ws.commons.schema.XmlSchemaRedefine;
 import org.apache.ws.commons.schema.XmlSchemaSequence;
 import org.apache.ws.commons.schema.XmlSchemaSequenceMember;
 import org.apache.ws.commons.schema.XmlSchemaSimpleType;
 import org.apache.ws.commons.schema.XmlSchemaSimpleTypeRestriction;
+import org.apache.ws.commons.schema.XmlSchemaType;
 
 import org.junit.Assert;
 import org.junit.Test;
@@ -99,16 +99,16 @@
         assertEquals(1, xsoc.size());
 
         XmlSchemaRedefine xsr = (XmlSchemaRedefine)xsoc.get(0);
-        XmlSchemaObjectTable redefTypes = xsr.getSchemaTypes();
-        assertEquals(1, redefTypes.getCount());
+        Map<QName, XmlSchemaType> redefTypes = xsr.getSchemaTypes();
+        assertEquals(1, redefTypes.size());
 
-        for (Iterator i = redefTypes.getNames(); i.hasNext();) {
-            QName qname = (QName)i.next();
+        for (Iterator<QName> i = redefTypes.keySet().iterator(); i.hasNext();) 
{
+            QName qname = i.next();
             assertEquals(new QName("http://soapinterop.org/types";, "person"), 
qname);
         }
 
         XmlSchemaComplexType xsct = null;
-        for (Iterator i = redefTypes.getValues(); i.hasNext();) {
+        for (Iterator<XmlSchemaType> i = redefTypes.values().iterator(); 
i.hasNext();) {
             xsct = (XmlSchemaComplexType)i.next();
         }
         assertNotNull(xsct);
@@ -173,16 +173,16 @@
         assertEquals(1, xsoc.size());
 
         XmlSchemaRedefine xsr = (XmlSchemaRedefine)xsoc.get(0);
-        XmlSchemaObjectTable xsot = xsr.getSchemaTypes();
-        assertEquals(1, xsot.getCount());
+        Map<QName, XmlSchemaType> xsot = xsr.getSchemaTypes();
+        assertEquals(1, xsot.size());
 
-        for (Iterator i = xsot.getNames(); i.hasNext();) {
-            QName qname = (QName)i.next();
+        for (Iterator<QName> i = xsot.keySet().iterator(); i.hasNext();) {
+            QName qname = i.next();
             assertEquals(new QName("http://soapinterop.org/types";, 
"drinksize"), qname);
         }
 
         XmlSchemaSimpleType xsst = null;
-        for (Iterator i = xsot.getValues(); i.hasNext();) {
+        for (Iterator<XmlSchemaType> i = xsot.values().iterator(); 
i.hasNext();) {
             xsst = (XmlSchemaSimpleType)i.next();
         }
         assertNotNull(xsst);
@@ -240,16 +240,16 @@
         assertEquals(1, xsoc.size());
 
         XmlSchemaRedefine xsr = (XmlSchemaRedefine)xsoc.get(0);
-        XmlSchemaObjectTable xsot = xsr.getGroup();
-        assertEquals(1, xsot.getCount());
+        Map<QName, XmlSchemaGroup> xsot = xsr.getGroups();
+        assertEquals(1, xsot.size());
 
-        for (Iterator i = xsot.getNames(); i.hasNext();) {
-            assertEquals("PrologGroup", ((QName)i.next()).getLocalPart());
+        for (Iterator<QName> i = xsot.keySet().iterator(); i.hasNext();) {
+            assertEquals("PrologGroup", i.next().getLocalPart());
         }
 
         XmlSchemaGroup xsg = null;
-        for (Iterator i = xsot.getValues(); i.hasNext();) {
-            xsg = (XmlSchemaGroup)i.next();
+        for (Iterator<XmlSchemaGroup> i = xsot.values().iterator(); 
i.hasNext();) {
+            xsg = i.next();
         }
 
         XmlSchemaSequence xss = (XmlSchemaSequence)xsg.getParticle();
@@ -305,15 +305,15 @@
         assertEquals(1, xsoc.size());
 
         XmlSchemaRedefine xsr = (XmlSchemaRedefine)xsoc.get(0);
-        XmlSchemaObjectTable xsot = xsr.getAttributeGroup();
-        assertEquals(1, xsot.getCount());
+        Map<QName, XmlSchemaAttributeGroup> xsot = xsr.getAttributeGroups();
+        assertEquals(1, xsot.size());
 
-        for (Iterator i = xsot.getNames(); i.hasNext();) {
-            assertEquals("AttribGroup", ((QName)i.next()).getLocalPart());
+        for (Iterator<QName> i = xsot.keySet().iterator(); i.hasNext();) {
+            assertEquals("AttribGroup", (i.next()).getLocalPart());
         }
 
         XmlSchemaAttributeGroup xsag = null;
-        for (Iterator i = xsot.getValues(); i.hasNext();) {
+        for (Iterator<XmlSchemaAttributeGroup> i = xsot.values().iterator(); 
i.hasNext();) {
             xsag = (XmlSchemaAttributeGroup)i.next();
         }
 
@@ -389,16 +389,16 @@
         assertEquals(1, xsoc.size());
 
         XmlSchemaRedefine xsr = (XmlSchemaRedefine)xsoc.get(0);
-        XmlSchemaObjectTable xsot = xsr.getSchemaTypes();
-        assertEquals(1, xsot.getCount());
+        Map<QName, XmlSchemaType> xsot = xsr.getSchemaTypes();
+        assertEquals(1, xsot.size());
 
-        for (Iterator i = xsot.getNames(); i.hasNext();) {
-            QName qname = (QName)i.next();
+        for (Iterator<QName> i = xsot.keySet().iterator(); i.hasNext();) {
+            QName qname = i.next();
             assertEquals(new QName("http://soapinterop.org/types";, "person"), 
qname);
         }
 
         XmlSchemaComplexType xsct = null;
-        for (Iterator i = xsot.getValues(); i.hasNext();) {
+        for (Iterator<XmlSchemaType> i = xsot.values().iterator(); 
i.hasNext();) {
             xsct = (XmlSchemaComplexType)i.next();
         }
         assertNotNull(xsct);


Reply via email to