Author: jkaputin
Date: Sat Nov 11 16:42:36 2006
New Revision: 473840

URL: http://svn.apache.org/viewvc?view=rev&rev=473840
Log:
WODEN-69 modified the getTranferCoding() method
to reflect Section 6.8.2 in Part 2 spec and instead
of trying to retrieve transfer coding default from
parent binding operation, just return null if the
binding message reference's transfer coding property
is null. This fixes the NPE seen on the MessageTest-1G
test case.

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

Modified: 
incubator/woden/trunk/java/src/org/apache/woden/internal/wsdl20/extensions/http/HTTPBindingMessageReferenceExtensionsImpl.java
URL: 
http://svn.apache.org/viewvc/incubator/woden/trunk/java/src/org/apache/woden/internal/wsdl20/extensions/http/HTTPBindingMessageReferenceExtensionsImpl.java?view=diff&rev=473840&r1=473839&r2=473840
==============================================================================
--- 
incubator/woden/trunk/java/src/org/apache/woden/internal/wsdl20/extensions/http/HTTPBindingMessageReferenceExtensionsImpl.java
 (original)
+++ 
incubator/woden/trunk/java/src/org/apache/woden/internal/wsdl20/extensions/http/HTTPBindingMessageReferenceExtensionsImpl.java
 Sat Nov 11 16:42:36 2006
@@ -44,21 +44,8 @@
     {
         StringAttr tfrCoding = (StringAttr) ((WSDLElement)fParent)
             .getExtensionAttribute(HTTPConstants.Q_ATTR_TRANSFER_CODING);
-        if(tfrCoding != null) {
-            return tfrCoding.getString();
-        }
         
-        //Part 2, 6.8.2 Relationship to WSDL Component Model, default to
-        //BindingOperation {http transfer coding default}
-        
-        BindingOperation bindOper = (BindingOperation) 
-                ((BindingMessageReference)fParent).getParent();
-        HTTPBindingOperationExtensions httpBindOperExts = 
(HTTPBindingOperationExtensions)
-            
bindOper.getComponentExtensionsForNamespace(HTTPConstants.NS_URI_HTTP);
-        
-        //No need to null check httpBindOperExts because BindingOperation has 
-        //REQUIRED http extension properties, so it must be present. 
-        return httpBindOperExts.getHttpTransferCodingDefault();
+        return (tfrCoding != null ? tfrCoding.getString() : null);
     }
 
     /* (non-Javadoc)



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

Reply via email to