Author: ruchithf
Date: Sat Sep 30 05:12:11 2006
New Revision: 451587

URL: http://svn.apache.org/viewvc?view=rev&rev=451587
Log:
ElementHelper - Aviod adding child elements that are already added
Fix for WSCOMMONS-103
NOTE: The latest Axis2 build passes with these changes


Modified:
    
webservices/commons/trunk/modules/axiom/modules/axiom-api/src/main/java/org/apache/axiom/soap/impl/builder/SOAP11BuilderHelper.java
    
webservices/commons/trunk/modules/axiom/modules/axiom-dom/src/main/java/org/apache/axiom/soap/impl/dom/SOAPHeaderBlockImpl.java
    
webservices/commons/trunk/modules/axiom/modules/axiom-dom/src/main/java/org/apache/axiom/soap/impl/dom/soap11/SOAP11Factory.java
    
webservices/commons/trunk/modules/axiom/modules/axiom-dom/src/main/java/org/apache/axiom/soap/impl/dom/soap11/SOAP11HeaderBlockImpl.java
    
webservices/commons/trunk/modules/axiom/modules/axiom-dom/src/main/java/org/apache/axiom/soap/impl/dom/soap12/SOAP12Factory.java
    
webservices/commons/trunk/modules/axiom/modules/axiom-dom/src/main/java/org/apache/axiom/soap/impl/dom/soap12/SOAP12HeaderBlockImpl.java

Modified: 
webservices/commons/trunk/modules/axiom/modules/axiom-api/src/main/java/org/apache/axiom/soap/impl/builder/SOAP11BuilderHelper.java
URL: 
http://svn.apache.org/viewvc/webservices/commons/trunk/modules/axiom/modules/axiom-api/src/main/java/org/apache/axiom/soap/impl/builder/SOAP11BuilderHelper.java?view=diff&rev=451587&r1=451586&r2=451587
==============================================================================
--- 
webservices/commons/trunk/modules/axiom/modules/axiom-api/src/main/java/org/apache/axiom/soap/impl/builder/SOAP11BuilderHelper.java
 (original)
+++ 
webservices/commons/trunk/modules/axiom/modules/axiom-api/src/main/java/org/apache/axiom/soap/impl/builder/SOAP11BuilderHelper.java
 Sat Sep 30 05:12:11 2006
@@ -148,8 +148,7 @@
             int token = parser.next();

             while (token != XMLStreamReader.END_ELEMENT) {

                 if (token == XMLStreamReader.CHARACTERS) {

-                    OMText text = factory.createOMText(value, 
parser.getText());

-                    value.addChild(text);

+                   factory.createOMText(value, parser.getText());

                 } else {

                     throw new SOAPProcessingException(

                             "Only Characters are allowed here");


Modified: 
webservices/commons/trunk/modules/axiom/modules/axiom-dom/src/main/java/org/apache/axiom/soap/impl/dom/SOAPHeaderBlockImpl.java
URL: 
http://svn.apache.org/viewvc/webservices/commons/trunk/modules/axiom/modules/axiom-dom/src/main/java/org/apache/axiom/soap/impl/dom/SOAPHeaderBlockImpl.java?view=diff&rev=451587&r1=451586&r2=451587
==============================================================================
--- 
webservices/commons/trunk/modules/axiom/modules/axiom-dom/src/main/java/org/apache/axiom/soap/impl/dom/SOAPHeaderBlockImpl.java
 (original)
+++ 
webservices/commons/trunk/modules/axiom/modules/axiom-dom/src/main/java/org/apache/axiom/soap/impl/dom/SOAPHeaderBlockImpl.java
 Sat Sep 30 05:12:11 2006
@@ -24,6 +24,7 @@
 import org.apache.axiom.om.impl.dom.ElementImpl;
 import org.apache.axiom.om.impl.dom.NamespaceImpl;
 import org.apache.axiom.om.impl.dom.ParentNode;
+import org.apache.axiom.om.impl.dom.factory.OMDOMFactory;
 import org.apache.axiom.soap.SOAPConstants;
 import org.apache.axiom.soap.SOAPFactory;
 import org.apache.axiom.soap.SOAPHeader;
@@ -48,6 +49,12 @@
         this.setNamespace(ns);
     }
 
+    public SOAPHeaderBlockImpl(String localName, OMNamespace ns,
+            SOAPFactory factory) throws SOAPProcessingException {
+        super(((OMDOMFactory)factory).getDocument(), localName,(NamespaceImpl) 
ns, factory);
+        this.setNamespace(ns);
+    }
+    
     /**
      * Constructor SOAPHeaderBlockImpl.
      *

Modified: 
webservices/commons/trunk/modules/axiom/modules/axiom-dom/src/main/java/org/apache/axiom/soap/impl/dom/soap11/SOAP11Factory.java
URL: 
http://svn.apache.org/viewvc/webservices/commons/trunk/modules/axiom/modules/axiom-dom/src/main/java/org/apache/axiom/soap/impl/dom/soap11/SOAP11Factory.java?view=diff&rev=451587&r1=451586&r2=451587
==============================================================================
--- 
webservices/commons/trunk/modules/axiom/modules/axiom-dom/src/main/java/org/apache/axiom/soap/impl/dom/soap11/SOAP11Factory.java
 (original)
+++ 
webservices/commons/trunk/modules/axiom/modules/axiom-dom/src/main/java/org/apache/axiom/soap/impl/dom/soap11/SOAP11Factory.java
 Sat Sep 30 05:12:11 2006
@@ -69,6 +69,11 @@
         return new SOAP11HeaderImpl(envelope, builder, this);
     }
 
+    
+    public SOAPHeaderBlock createSOAPHeaderBlock(String localName, OMNamespace 
ns) throws SOAPProcessingException {
+        return new SOAP11HeaderBlockImpl(localName, ns, this);
+    }
+
     public SOAPHeaderBlock createSOAPHeaderBlock(String localName,
             OMNamespace ns, SOAPHeader parent) throws SOAPProcessingException {
         return new SOAP11HeaderBlockImpl(localName, ns, parent, this);

Modified: 
webservices/commons/trunk/modules/axiom/modules/axiom-dom/src/main/java/org/apache/axiom/soap/impl/dom/soap11/SOAP11HeaderBlockImpl.java
URL: 
http://svn.apache.org/viewvc/webservices/commons/trunk/modules/axiom/modules/axiom-dom/src/main/java/org/apache/axiom/soap/impl/dom/soap11/SOAP11HeaderBlockImpl.java?view=diff&rev=451587&r1=451586&r2=451587
==============================================================================
--- 
webservices/commons/trunk/modules/axiom/modules/axiom-dom/src/main/java/org/apache/axiom/soap/impl/dom/soap11/SOAP11HeaderBlockImpl.java
 (original)
+++ 
webservices/commons/trunk/modules/axiom/modules/axiom-dom/src/main/java/org/apache/axiom/soap/impl/dom/soap11/SOAP11HeaderBlockImpl.java
 Sat Sep 30 05:12:11 2006
@@ -39,6 +39,10 @@
         checkParent(parent);
     }
 
+    public SOAP11HeaderBlockImpl(String localName, OMNamespace ns, SOAPFactory 
factory) {
+        super(localName, ns, factory);
+    }
+    
     /**
      * Constructor SOAPHeaderBlockImpl
      *

Modified: 
webservices/commons/trunk/modules/axiom/modules/axiom-dom/src/main/java/org/apache/axiom/soap/impl/dom/soap12/SOAP12Factory.java
URL: 
http://svn.apache.org/viewvc/webservices/commons/trunk/modules/axiom/modules/axiom-dom/src/main/java/org/apache/axiom/soap/impl/dom/soap12/SOAP12Factory.java?view=diff&rev=451587&r1=451586&r2=451587
==============================================================================
--- 
webservices/commons/trunk/modules/axiom/modules/axiom-dom/src/main/java/org/apache/axiom/soap/impl/dom/soap12/SOAP12Factory.java
 (original)
+++ 
webservices/commons/trunk/modules/axiom/modules/axiom-dom/src/main/java/org/apache/axiom/soap/impl/dom/soap12/SOAP12Factory.java
 Sat Sep 30 05:12:11 2006
@@ -37,6 +37,7 @@
 import org.apache.axiom.soap.SOAPProcessingException;
 import org.apache.axiom.soap.impl.dom.SOAPEnvelopeImpl;
 import org.apache.axiom.soap.impl.dom.factory.DOMSOAPFactory;
+import org.apache.axiom.soap.impl.dom.soap11.SOAP11HeaderBlockImpl;
 
 public class SOAP12Factory extends DOMSOAPFactory {
        
@@ -65,6 +66,10 @@
     public SOAPHeader createSOAPHeader(SOAPEnvelope envelope,
                                        OMXMLParserWrapper builder) {
         return new SOAP12HeaderImpl(envelope, builder, this);
+    }
+    
+    public SOAPHeaderBlock createSOAPHeaderBlock(String localName, OMNamespace 
ns) throws SOAPProcessingException {
+        return new SOAP12HeaderBlockImpl(localName, ns, this);
     }
 
     public SOAPHeaderBlock createSOAPHeaderBlock(String localName,

Modified: 
webservices/commons/trunk/modules/axiom/modules/axiom-dom/src/main/java/org/apache/axiom/soap/impl/dom/soap12/SOAP12HeaderBlockImpl.java
URL: 
http://svn.apache.org/viewvc/webservices/commons/trunk/modules/axiom/modules/axiom-dom/src/main/java/org/apache/axiom/soap/impl/dom/soap12/SOAP12HeaderBlockImpl.java?view=diff&rev=451587&r1=451586&r2=451587
==============================================================================
--- 
webservices/commons/trunk/modules/axiom/modules/axiom-dom/src/main/java/org/apache/axiom/soap/impl/dom/soap12/SOAP12HeaderBlockImpl.java
 (original)
+++ 
webservices/commons/trunk/modules/axiom/modules/axiom-dom/src/main/java/org/apache/axiom/soap/impl/dom/soap12/SOAP12HeaderBlockImpl.java
 Sat Sep 30 05:12:11 2006
@@ -38,6 +38,10 @@
         super(localName, ns, parent, factory);
         checkParent(parent);
     }
+    
+    public SOAP12HeaderBlockImpl(String localName, OMNamespace ns, SOAPFactory 
factory) {
+        super(localName, ns, factory);
+    }
 
     /**
      * Constructor SOAPHeaderBlockImpl



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

Reply via email to