Author: bimargulies
Date: Tue Jan  6 15:12:26 2009
New Revision: 732168

URL: http://svn.apache.org/viewvc?rev=732168&view=rev
Log:
Fix schema generation problem with missing import in Aegis. Remove JDOM usage 
from Javascript Aegis test
rather than track down very muddy change in behavior from old world. Remove the 
unwanted ant tasks.

Removed:
    cxf/trunk/tools/anttask/
Modified:
    
cxf/trunk/common/common/src/main/java/org/apache/cxf/common/xmlschema/XmlSchemaUtils.java
    
cxf/trunk/rt/databinding/aegis/src/main/java/org/apache/cxf/aegis/databinding/AegisDatabinding.java
    
cxf/trunk/rt/databinding/aegis/src/main/java/org/apache/cxf/aegis/type/AbstractTypeCreator.java
    
cxf/trunk/rt/databinding/aegis/src/main/java/org/apache/cxf/aegis/type/basic/ArrayType.java
    
cxf/trunk/rt/databinding/aegis/src/main/java/org/apache/cxf/aegis/type/basic/BeanType.java
    
cxf/trunk/rt/javascript/src/test/java/org/apache/cxf/javascript/AegisTest.java
    
cxf/trunk/rt/javascript/src/test/java/org/apache/cxf/javascript/fortest/AegisService.java
    
cxf/trunk/rt/javascript/src/test/java/org/apache/cxf/javascript/fortest/AegisServiceImpl.java
    cxf/trunk/rt/javascript/src/test/resources/AegisBeans.xml
    cxf/trunk/tools/pom.xml

Modified: 
cxf/trunk/common/common/src/main/java/org/apache/cxf/common/xmlschema/XmlSchemaUtils.java
URL: 
http://svn.apache.org/viewvc/cxf/trunk/common/common/src/main/java/org/apache/cxf/common/xmlschema/XmlSchemaUtils.java?rev=732168&r1=732167&r2=732168&view=diff
==============================================================================
--- 
cxf/trunk/common/common/src/main/java/org/apache/cxf/common/xmlschema/XmlSchemaUtils.java
 (original)
+++ 
cxf/trunk/common/common/src/main/java/org/apache/cxf/common/xmlschema/XmlSchemaUtils.java
 Tue Jan  6 15:12:26 2009
@@ -202,7 +202,9 @@
      */
     public static void addImportIfNeeded(XmlSchema schema, String 
namespaceUri) {
         // no need to import nothing or the XSD schema.
-        if ("".equals(namespaceUri) || 
XmlSchemaConstants.XSD_NAMESPACE_URI.equals(namespaceUri)) {
+        if ("".equals(namespaceUri) 
+            || XmlSchemaConstants.XSD_NAMESPACE_URI.equals(namespaceUri)
+            || schema.getTargetNamespace().equals(namespaceUri)) {
             return;
         }
             

Modified: 
cxf/trunk/rt/databinding/aegis/src/main/java/org/apache/cxf/aegis/databinding/AegisDatabinding.java
URL: 
http://svn.apache.org/viewvc/cxf/trunk/rt/databinding/aegis/src/main/java/org/apache/cxf/aegis/databinding/AegisDatabinding.java?rev=732168&r1=732167&r2=732168&view=diff
==============================================================================
--- 
cxf/trunk/rt/databinding/aegis/src/main/java/org/apache/cxf/aegis/databinding/AegisDatabinding.java
 (original)
+++ 
cxf/trunk/rt/databinding/aegis/src/main/java/org/apache/cxf/aegis/databinding/AegisDatabinding.java
 Tue Jan  6 15:12:26 2009
@@ -501,9 +501,8 @@
                 si.addSchema(schemaInfo);
             }
         }
-        
     }
-
+    
     private boolean schemaImportsXmime(XmlSchema schema) {
         return XmlSchemaUtils.schemaImportsNamespace(schema, 
AbstractXOPType.XML_MIME_NS);
     }

Modified: 
cxf/trunk/rt/databinding/aegis/src/main/java/org/apache/cxf/aegis/type/AbstractTypeCreator.java
URL: 
http://svn.apache.org/viewvc/cxf/trunk/rt/databinding/aegis/src/main/java/org/apache/cxf/aegis/type/AbstractTypeCreator.java?rev=732168&r1=732167&r2=732168&view=diff
==============================================================================
--- 
cxf/trunk/rt/databinding/aegis/src/main/java/org/apache/cxf/aegis/type/AbstractTypeCreator.java
 (original)
+++ 
cxf/trunk/rt/databinding/aegis/src/main/java/org/apache/cxf/aegis/type/AbstractTypeCreator.java
 Tue Jan  6 15:12:26 2009
@@ -41,6 +41,8 @@
  * @author Hani Suleiman Date: Jun 14, 2005 Time: 11:59:57 PM
  */
 public abstract class AbstractTypeCreator implements TypeCreator {
+    public static final String HTTP_CXF_APACHE_ORG_ARRAYS = 
"http://cxf.apache.org/arrays";;
+
     protected TypeMapping tm;
 
     protected AbstractTypeCreator nextCreator;
@@ -308,7 +310,7 @@
             ns = tm.getMappingIdentifierURI();
         }
         if (WSDLConstants.NS_SCHEMA_XSD.equals(ns)) {
-            ns = "http://cxf.apache.org/arrays";;
+            ns = HTTP_CXF_APACHE_ORG_ARRAYS;
         }
 
         String first = type.getSchemaType().getLocalPart().substring(0, 1);

Modified: 
cxf/trunk/rt/databinding/aegis/src/main/java/org/apache/cxf/aegis/type/basic/ArrayType.java
URL: 
http://svn.apache.org/viewvc/cxf/trunk/rt/databinding/aegis/src/main/java/org/apache/cxf/aegis/type/basic/ArrayType.java?rev=732168&r1=732167&r2=732168&view=diff
==============================================================================
--- 
cxf/trunk/rt/databinding/aegis/src/main/java/org/apache/cxf/aegis/type/basic/ArrayType.java
 (original)
+++ 
cxf/trunk/rt/databinding/aegis/src/main/java/org/apache/cxf/aegis/type/basic/ArrayType.java
 Tue Jan  6 15:12:26 2009
@@ -30,11 +30,13 @@
 import org.apache.commons.logging.LogFactory;
 import org.apache.cxf.aegis.Context;
 import org.apache.cxf.aegis.DatabindingException;
+import org.apache.cxf.aegis.type.AbstractTypeCreator;
 import org.apache.cxf.aegis.type.Type;
 import org.apache.cxf.aegis.type.TypeUtil;
 import org.apache.cxf.aegis.xml.MessageReader;
 import org.apache.cxf.aegis.xml.MessageWriter;
 import org.apache.cxf.common.xmlschema.XmlSchemaConstants;
+import org.apache.cxf.common.xmlschema.XmlSchemaUtils;
 import org.apache.ws.commons.schema.XmlSchema;
 import org.apache.ws.commons.schema.XmlSchemaComplexType;
 import org.apache.ws.commons.schema.XmlSchemaElement;
@@ -286,6 +288,7 @@
         XmlSchemaElement element = new XmlSchemaElement();
         element.setName(componentType.getSchemaType().getLocalPart());
         element.setSchemaTypeName(componentType.getSchemaType());
+      
         seq.getItems().add(element);
 
         if (componentType.isNillable()) {

Modified: 
cxf/trunk/rt/databinding/aegis/src/main/java/org/apache/cxf/aegis/type/basic/BeanType.java
URL: 
http://svn.apache.org/viewvc/cxf/trunk/rt/databinding/aegis/src/main/java/org/apache/cxf/aegis/type/basic/BeanType.java?rev=732168&r1=732167&r2=732168&view=diff
==============================================================================
--- 
cxf/trunk/rt/databinding/aegis/src/main/java/org/apache/cxf/aegis/type/basic/BeanType.java
 (original)
+++ 
cxf/trunk/rt/databinding/aegis/src/main/java/org/apache/cxf/aegis/type/basic/BeanType.java
 Tue Jan  6 15:12:26 2009
@@ -33,6 +33,7 @@
 import org.apache.cxf.aegis.AegisContext;
 import org.apache.cxf.aegis.Context;
 import org.apache.cxf.aegis.DatabindingException;
+import org.apache.cxf.aegis.type.AbstractTypeCreator;
 import org.apache.cxf.aegis.type.Type;
 import org.apache.cxf.aegis.type.TypeMapping;
 import org.apache.cxf.aegis.type.TypeUtil;
@@ -664,6 +665,10 @@
             sequence.getItems().add(element);
 
             Type type = getType(inf, name);
+            if (AbstractTypeCreator.
+                
HTTP_CXF_APACHE_ORG_ARRAYS.equals(type.getSchemaType().getNamespaceURI())) {
+                XmlSchemaUtils.addImportIfNeeded(root, 
AbstractTypeCreator.HTTP_CXF_APACHE_ORG_ARRAYS);
+            }
 
             writeTypeReference(name, element, type);
             needXmime |= type.usesXmime();

Modified: 
cxf/trunk/rt/javascript/src/test/java/org/apache/cxf/javascript/AegisTest.java
URL: 
http://svn.apache.org/viewvc/cxf/trunk/rt/javascript/src/test/java/org/apache/cxf/javascript/AegisTest.java?rev=732168&r1=732167&r2=732168&view=diff
==============================================================================
--- 
cxf/trunk/rt/javascript/src/test/java/org/apache/cxf/javascript/AegisTest.java 
(original)
+++ 
cxf/trunk/rt/javascript/src/test/java/org/apache/cxf/javascript/AegisTest.java 
Tue Jan  6 15:12:26 2009
@@ -22,6 +22,8 @@
 import java.util.Collection;
 import java.util.logging.Logger;
 
+import org.w3c.dom.Document;
+
 import org.apache.cxf.common.logging.LogUtils;
 import org.apache.cxf.javascript.JavascriptTestUtilities.JSRunnable;
 import org.apache.cxf.javascript.JavascriptTestUtilities.Notifier;
@@ -70,7 +72,7 @@
                                 testUtilities.javaToJS(getAddress()));
         implementor.waitForOneWay();
         assertEquals("before items", implementor.getAcceptedString());
-        Collection<org.jdom.Element> something = 
implementor.getAcceptedCollection();
+        Collection<Document> something = implementor.getAcceptedCollection();
         assertNotNull(something);
         return null;
     }
@@ -133,7 +135,6 @@
         return null;
     }
 
-    @org.junit.Ignore
     @Test
     public void callReturnBeanWithAnyTypeArray() {
         testUtilities.runInsideContext(Void.class, new JSRunnable<Void>() {

Modified: 
cxf/trunk/rt/javascript/src/test/java/org/apache/cxf/javascript/fortest/AegisService.java
URL: 
http://svn.apache.org/viewvc/cxf/trunk/rt/javascript/src/test/java/org/apache/cxf/javascript/fortest/AegisService.java?rev=732168&r1=732167&r2=732168&view=diff
==============================================================================
--- 
cxf/trunk/rt/javascript/src/test/java/org/apache/cxf/javascript/fortest/AegisService.java
 (original)
+++ 
cxf/trunk/rt/javascript/src/test/java/org/apache/cxf/javascript/fortest/AegisService.java
 Tue Jan  6 15:12:26 2009
@@ -21,10 +21,12 @@
 
 import java.util.Collection;
 
+import org.w3c.dom.Document;
+
 import org.apache.cxf.javascript.fortest.aegis.BeanWithAnyTypeArray;
 
 public interface AegisService {
-    void acceptAny(String before, Collection<org.jdom.Element> anything);
+    void acceptAny(String before, Collection<Document> anything);
     void acceptObjects(Collection<Object> anything);
     void acceptStrings(Collection<String> someStrings);
     BeanWithAnyTypeArray returnBeanWithAnyTypeArray();

Modified: 
cxf/trunk/rt/javascript/src/test/java/org/apache/cxf/javascript/fortest/AegisServiceImpl.java
URL: 
http://svn.apache.org/viewvc/cxf/trunk/rt/javascript/src/test/java/org/apache/cxf/javascript/fortest/AegisServiceImpl.java?rev=732168&r1=732167&r2=732168&view=diff
==============================================================================
--- 
cxf/trunk/rt/javascript/src/test/java/org/apache/cxf/javascript/fortest/AegisServiceImpl.java
 (original)
+++ 
cxf/trunk/rt/javascript/src/test/java/org/apache/cxf/javascript/fortest/AegisServiceImpl.java
 Tue Jan  6 15:12:26 2009
@@ -22,6 +22,8 @@
 import java.util.Collection;
 import java.util.concurrent.CountDownLatch;
 
+import org.w3c.dom.Document;
+
 import org.apache.cxf.javascript.fortest.aegis.BeanWithAnyTypeArray;
 import org.apache.cxf.javascript.fortest.aegis.Mammal;
 import org.apache.cxf.javascript.fortest.aegis.Vegetable;
@@ -31,7 +33,7 @@
  */
 public class AegisServiceImpl implements AegisService {
     private String acceptedString;
-    private Collection<org.jdom.Element> acceptedCollection;
+    private Collection<Document> acceptedCollection;
     private Collection<String> acceptedStrings;
     private Collection<Object> acceptedObjects;
     private CountDownLatch oneWayLatch;
@@ -46,7 +48,7 @@
     }
     
     /** {...@inheritdoc}*/
-    public void acceptAny(String before, Collection<org.jdom.Element> 
anything) {
+    public void acceptAny(String before, Collection<Document> anything) {
         acceptedString = before;
         acceptedCollection = anything;
         if (oneWayLatch != null) {
@@ -57,7 +59,7 @@
     /**
      * @return Returns the acceptedCollection.
      */
-    public Collection<org.jdom.Element> getAcceptedCollection() {
+    public Collection<Document> getAcceptedCollection() {
         return acceptedCollection;
     }
 

Modified: cxf/trunk/rt/javascript/src/test/resources/AegisBeans.xml
URL: 
http://svn.apache.org/viewvc/cxf/trunk/rt/javascript/src/test/resources/AegisBeans.xml?rev=732168&r1=732167&r2=732168&view=diff
==============================================================================
--- cxf/trunk/rt/javascript/src/test/resources/AegisBeans.xml (original)
+++ cxf/trunk/rt/javascript/src/test/resources/AegisBeans.xml Tue Jan  6 
15:12:26 2009
@@ -44,11 +44,6 @@
                <simple:dataBinding>
                        <bean
                                
class="org.apache.cxf.aegis.databinding.AegisDatabinding">
-                               <property name="aegisContext">
-                                       <bean 
class="org.apache.cxf.aegis.AegisContext">
-                                               <property 
name="enableJDOMMappings" value="true"/>
-                                       </bean>
-                               </property>
                        </bean>
                </simple:dataBinding>
                <simple:serviceFactory>

Modified: cxf/trunk/tools/pom.xml
URL: 
http://svn.apache.org/viewvc/cxf/trunk/tools/pom.xml?rev=732168&r1=732167&r2=732168&view=diff
==============================================================================
--- cxf/trunk/tools/pom.xml (original)
+++ cxf/trunk/tools/pom.xml Tue Jan  6 15:12:26 2009
@@ -37,7 +37,6 @@
         <module>wsdlto</module>
         <module>javato</module>
         <module>corba</module>
-        <module>anttask</module>
     </modules>
 
     <scm>


Reply via email to