Author: ropalka Date: 2012-04-17 07:19:23 -0400 (Tue, 17 Apr 2012) New Revision: 16169 Modified: stack/native/trunk/modules/core/src/main/java/org/jboss/ws/core/soap/NameImpl.java stack/native/trunk/modules/core/src/main/java/org/jboss/ws/core/soap/SOAPElementImpl.java stack/native/trunk/modules/core/src/main/java/org/jboss/ws/core/soap/utils/SOAPUtils.java stack/native/trunk/modules/testsuite/native-tests/src/test/java/org/jboss/test/ws/jaxrpc/jbws1303/ServerHandler.java stack/native/trunk/modules/testsuite/native-tests/src/test/java/org/jboss/test/ws/jaxrpc/jbws720/GetProperty.java Log: [JBWS-3465][JBPAPP-8718] removing precondition Native SAAJ impl is always in use - allows multiple SAAJ impls. to coexist at runtime Modified: stack/native/trunk/modules/core/src/main/java/org/jboss/ws/core/soap/NameImpl.java =================================================================== --- stack/native/trunk/modules/core/src/main/java/org/jboss/ws/core/soap/NameImpl.java 2012-04-17 09:12:49 UTC (rev 16168) +++ stack/native/trunk/modules/core/src/main/java/org/jboss/ws/core/soap/NameImpl.java 2012-04-17 11:19:23 UTC (rev 16169) @@ -24,6 +24,8 @@ import javax.xml.namespace.QName; import javax.xml.soap.Name; +import org.jboss.ws.core.soap.utils.SOAPUtils; + /** * An implementation of a Name * <p/> @@ -107,10 +109,10 @@ public boolean equals(Object obj) { - if (!(obj instanceof NameImpl)) return false; + if (!(obj instanceof Name)) return false; if (obj == this) return true; - NameImpl other = (NameImpl)obj; - return qname.equals(other.qname); + Name other = (Name)obj; + return qname.equals(SOAPUtils.toQName(other)); } public QName toQName() Modified: stack/native/trunk/modules/core/src/main/java/org/jboss/ws/core/soap/SOAPElementImpl.java =================================================================== --- stack/native/trunk/modules/core/src/main/java/org/jboss/ws/core/soap/SOAPElementImpl.java 2012-04-17 09:12:49 UTC (rev 16168) +++ stack/native/trunk/modules/core/src/main/java/org/jboss/ws/core/soap/SOAPElementImpl.java 2012-04-17 11:19:23 UTC (rev 16169) @@ -41,6 +41,7 @@ import org.jboss.ws.api.util.BundleUtils; import org.jboss.ws.common.Constants; import org.jboss.ws.common.DOMUtils; +import org.jboss.ws.core.soap.utils.SOAPUtils; import org.w3c.dom.Attr; import org.w3c.dom.DOMException; import org.w3c.dom.Document; @@ -532,7 +533,8 @@ */ public Iterator<SOAPElement> getChildElements(Name name) { - return getChildElements(((NameImpl)name).toQName()); + QName qName = SOAPUtils.toQName(name); + return getChildElements(qName); } public Iterator<SOAPElement> getChildElements(QName qname) Modified: stack/native/trunk/modules/core/src/main/java/org/jboss/ws/core/soap/utils/SOAPUtils.java =================================================================== --- stack/native/trunk/modules/core/src/main/java/org/jboss/ws/core/soap/utils/SOAPUtils.java 2012-04-17 09:12:49 UTC (rev 16168) +++ stack/native/trunk/modules/core/src/main/java/org/jboss/ws/core/soap/utils/SOAPUtils.java 2012-04-17 11:19:23 UTC (rev 16169) @@ -90,4 +90,9 @@ public static Name newName(final QName faultCode, final SOAPEnvelope soapEnvelope) throws SOAPException { return soapEnvelope.createName(faultCode.getLocalPart(), faultCode.getPrefix(), faultCode.getNamespaceURI()); } + + public static QName toQName(final Name name) { + return new QName(name.getURI(), name.getLocalName(), name.getPrefix()); + } + } Modified: stack/native/trunk/modules/testsuite/native-tests/src/test/java/org/jboss/test/ws/jaxrpc/jbws1303/ServerHandler.java =================================================================== --- stack/native/trunk/modules/testsuite/native-tests/src/test/java/org/jboss/test/ws/jaxrpc/jbws1303/ServerHandler.java 2012-04-17 09:12:49 UTC (rev 16168) +++ stack/native/trunk/modules/testsuite/native-tests/src/test/java/org/jboss/test/ws/jaxrpc/jbws1303/ServerHandler.java 2012-04-17 11:19:23 UTC (rev 16169) @@ -59,10 +59,9 @@ SOAPEnvelope soapEnvelope = soapMessage.getSOAPPart().getEnvelope(); soapEnvelope.addNamespaceDeclaration(Constants.PREFIX_XSD, Constants.NS_SCHEMA_XSD); soapEnvelope.addNamespaceDeclaration(Constants.PREFIX_XSI, Constants.NS_SCHEMA_XSI); - SOAPElement bodyElement = soapMessage.getSOAPBody().addChildElement("lastmodResponse"); - bodyElement.setAttribute("xmlns", "http://netid.msu.edu:8080/lastmod.pl"); - SOAPElement soapElement = bodyElement.addChildElement("TimeChanged"); - soapElement.setAttributeNS("xsi", "type", "xsd:string"); + SOAPElement bodyElement = soapMessage.getSOAPBody().addChildElement("lastmodResponse", "foo", "http://netid.msu.edu:8080/lastmod.pl"); + SOAPElement soapElement = bodyElement.addChildElement("TimeChanged", "foo"); + soapElement.setAttributeNS(Constants.NS_SCHEMA_XSI, "type", "xsd:string"); soapElement.setValue("yesterday"); ((SOAPMessageContext)msgContext).setMessage(soapMessage); } Modified: stack/native/trunk/modules/testsuite/native-tests/src/test/java/org/jboss/test/ws/jaxrpc/jbws720/GetProperty.java =================================================================== --- stack/native/trunk/modules/testsuite/native-tests/src/test/java/org/jboss/test/ws/jaxrpc/jbws720/GetProperty.java 2012-04-17 09:12:49 UTC (rev 16168) +++ stack/native/trunk/modules/testsuite/native-tests/src/test/java/org/jboss/test/ws/jaxrpc/jbws720/GetProperty.java 2012-04-17 11:19:23 UTC (rev 16169) @@ -104,7 +104,7 @@ { SOAPElement sel = _any[i]; Name name = sel.getElementName(); - builder.append(name); + builder.append("{" + name.getURI() + "}").append(name.getLocalName()); } } return "[elmt=" + strElement + ",attr=" + strAttr + ",lang=" + lang + ",any=" + builder + "]";