You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ws.apache.org by ve...@apache.org on 2012/05/15 22:57:59 UTC

svn commit: r1338886 - in /webservices/axiom/branches/CHECK_PARENT_BUG/modules/axiom-dom/src/main/java/org/apache/axiom/soap/impl/dom: SOAPElement.java SOAPHeaderBlockImpl.java

Author: veithen
Date: Tue May 15 20:57:58 2012
New Revision: 1338886

URL: http://svn.apache.org/viewvc?rev=1338886&view=rev
Log:
Only use checkParent with Axiom methods and be more lenient with DOM methods.

Modified:
    webservices/axiom/branches/CHECK_PARENT_BUG/modules/axiom-dom/src/main/java/org/apache/axiom/soap/impl/dom/SOAPElement.java
    webservices/axiom/branches/CHECK_PARENT_BUG/modules/axiom-dom/src/main/java/org/apache/axiom/soap/impl/dom/SOAPHeaderBlockImpl.java

Modified: webservices/axiom/branches/CHECK_PARENT_BUG/modules/axiom-dom/src/main/java/org/apache/axiom/soap/impl/dom/SOAPElement.java
URL: http://svn.apache.org/viewvc/webservices/axiom/branches/CHECK_PARENT_BUG/modules/axiom-dom/src/main/java/org/apache/axiom/soap/impl/dom/SOAPElement.java?rev=1338886&r1=1338885&r2=1338886&view=diff
==============================================================================
--- webservices/axiom/branches/CHECK_PARENT_BUG/modules/axiom-dom/src/main/java/org/apache/axiom/soap/impl/dom/SOAPElement.java (original)
+++ webservices/axiom/branches/CHECK_PARENT_BUG/modules/axiom-dom/src/main/java/org/apache/axiom/soap/impl/dom/SOAPElement.java Tue May 15 20:57:58 2012
@@ -80,7 +80,7 @@ public abstract class SOAPElement extend
     protected void setParent(OMContainer element, boolean useDomSemantics) {
         super.setParent(element, useDomSemantics);
 
-        if (element instanceof OMElement) {
+        if (!useDomSemantics && element instanceof OMElement) {
             checkParent((OMElement) element);
         }
     }

Modified: webservices/axiom/branches/CHECK_PARENT_BUG/modules/axiom-dom/src/main/java/org/apache/axiom/soap/impl/dom/SOAPHeaderBlockImpl.java
URL: http://svn.apache.org/viewvc/webservices/axiom/branches/CHECK_PARENT_BUG/modules/axiom-dom/src/main/java/org/apache/axiom/soap/impl/dom/SOAPHeaderBlockImpl.java?rev=1338886&r1=1338885&r2=1338886&view=diff
==============================================================================
--- webservices/axiom/branches/CHECK_PARENT_BUG/modules/axiom-dom/src/main/java/org/apache/axiom/soap/impl/dom/SOAPHeaderBlockImpl.java (original)
+++ webservices/axiom/branches/CHECK_PARENT_BUG/modules/axiom-dom/src/main/java/org/apache/axiom/soap/impl/dom/SOAPHeaderBlockImpl.java Tue May 15 20:57:58 2012
@@ -140,7 +140,7 @@ public abstract class SOAPHeaderBlockImp
     protected void setParent(OMContainer element, boolean useDomSemantics) {
         super.setParent(element, useDomSemantics);
     
-        if (element instanceof OMElement) {
+        if (!useDomSemantics && element instanceof OMElement) {
             checkParent((OMElement) element);
         }
     }