Author: thomas.diesler(a)jboss.com Date: 2007-09-25 05:15:08 -0400 (Tue, 25 Sep 2007) New Revision: 4618 Modified: stack/native/trunk/src/main/java/javax/xml/soap/SOAPConnection.java stack/native/trunk/src/main/java/javax/xml/soap/SOAPFactory.java Log: Fix invalid commit for JBCTS-312: javax.xml.soap signature tests Modified: stack/native/trunk/src/main/java/javax/xml/soap/SOAPConnection.java =================================================================== --- stack/native/trunk/src/main/java/javax/xml/soap/SOAPConnection.java 2007-09-25 06:42:25 UTC (rev 4617) +++ stack/native/trunk/src/main/java/javax/xml/soap/SOAPConnection.java 2007-09-25 09:15:08 UTC (rev 4618) @@ -63,7 +63,7 @@ */ public SOAPMessage get(Object to) throws SOAPException { - throw new IllegalArgumentException("Should be implemented by concrete implementation of this class"); + throw new IllegalStateException("Should be implemented by concrete implementation of this class"); } /** Closes this SOAPConnection object. Modified: stack/native/trunk/src/main/java/javax/xml/soap/SOAPFactory.java =================================================================== --- stack/native/trunk/src/main/java/javax/xml/soap/SOAPFactory.java 2007-09-25 06:42:25 UTC (rev 4617) +++ stack/native/trunk/src/main/java/javax/xml/soap/SOAPFactory.java 2007-09-25 09:15:08 UTC (rev 4618) @@ -23,11 +23,10 @@ // $Id$ -import org.w3c.dom.Element; - import javax.xml.namespace.QName; -import java.lang.reflect.Method; +import org.w3c.dom.Element; + /** SOAPFactory is a factory for creating various objects that exist in the SOAP XML tree. * * SOAPFactory can be used to create XML fragments that will eventually end up in the SOAP part. @@ -62,12 +61,9 @@ try { String propertyName = "javax.xml.soap.SOAPFactory"; - - Class loaderClass = Class.forName("org.jboss.ws.soap.SAAJFactoryLoader"); - Method m = loaderClass.getMethod("loadFactory", new Class[] {String.class, String.class}); - soapFactory = (SOAPFactory)m.invoke(null, new Object[] {propertyName, null}); + soapFactory = (SOAPFactory)SAAJFactoryLoader.loadFactory(propertyName, null); } - catch (Exception rte) + catch (RuntimeException rte) { throw new SOAPException(rte); } @@ -127,7 +123,7 @@ */ public SOAPElement createElement(Element domElement) throws SOAPException { - throw new IllegalArgumentException("Should be implemented by concrete implementation of this class"); + throw new IllegalStateException("Should be implemented by concrete implementation of this class"); } /** Create a SOAPElement object initialized with the given local name. @@ -168,7 +164,7 @@ */ public SOAPElement createElement(QName qname) throws SOAPException { - throw new IllegalArgumentException("Should be implemented by concrete implementation of this class"); + throw new IllegalStateException("Should be implemented by concrete implementation of this class"); } /**