You are viewing a plain text version of this content. The canonical link for it is here.
Posted to woden-dev@ws.apache.org by jk...@apache.org on 2006/07/06 15:44:13 UTC

svn commit: r419549 - /incubator/woden/java/src/org/apache/woden/internal/wsdl20/extensions/http/HTTPBindingOperationExtensionsImpl.java

Author: jkaputin
Date: Thu Jul  6 06:44:13 2006
New Revision: 419549

URL: http://svn.apache.org/viewvc?rev=419549&view=rev
Log:
Modified behaviour of getQuerySeparatorParameter to 
conform to the spec, so that if the underlying attribute
is not present the method will return null, rather than
defaulting to Binding.getQuerySeparatorParameterDefault().

Modified:
    incubator/woden/java/src/org/apache/woden/internal/wsdl20/extensions/http/HTTPBindingOperationExtensionsImpl.java

Modified: incubator/woden/java/src/org/apache/woden/internal/wsdl20/extensions/http/HTTPBindingOperationExtensionsImpl.java
URL: http://svn.apache.org/viewvc/incubator/woden/java/src/org/apache/woden/internal/wsdl20/extensions/http/HTTPBindingOperationExtensionsImpl.java?rev=419549&r1=419548&r2=419549&view=diff
==============================================================================
--- incubator/woden/java/src/org/apache/woden/internal/wsdl20/extensions/http/HTTPBindingOperationExtensionsImpl.java (original)
+++ incubator/woden/java/src/org/apache/woden/internal/wsdl20/extensions/http/HTTPBindingOperationExtensionsImpl.java Thu Jul  6 06:44:13 2006
@@ -204,21 +204,7 @@
         
 		StringAttr separator = (StringAttr) ((WSDLElement) fParent)
 				.getExtensionAttribute(HTTPConstants.Q_ATTR_QUERY_PARAMETER_SEPARATOR);
-		if (separator != null) {
-			return separator.getString();
-		}
-        
-        //As per Part 2, 6.7.2.2.1 Construction of the query string, the
-        //default is Binding {http query parameter separator default}.
-
-		Binding binding = (Binding) ((BindingOperation) fParent).getParent();
-
-		HTTPBindingExtensions httpBindExts = (HTTPBindingExtensions) binding
-				.getComponentExtensionsForNamespace(HTTPConstants.NS_URI_HTTP);
-
-		//No need for null check on httpBindExts because it has REQUIRED
-        //properties, so it must be present.
-		return httpBindExts.getHttpQueryParameterSeparatorDefault();
+        return separator != null ? separator.getString() : null;
 	}
 
 	/*



---------------------------------------------------------------------
To unsubscribe, e-mail: woden-dev-unsubscribe@ws.apache.org
For additional commands, e-mail: woden-dev-help@ws.apache.org