You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ode.apache.org by mi...@apache.org on 2009/02/21 09:25:47 UTC

svn commit: r746452 - /ode/branches/APACHE_ODE_1.X/axis2/src/main/java/org/apache/ode/axis2/util/SoapMessageConverter.java

Author: milinda
Date: Sat Feb 21 08:25:46 2009
New Revision: 746452

URL: http://svn.apache.org/viewvc?rev=746452&view=rev
Log:
Fixing Reply SOAP Header handling problem occurred due to use of wrong WSDL operation message. Previous code has used input operation wsdl message for output operation wsdl message. This cause headerdef.getMessage().equals(msgdef.getQName()) which resides inside handleSoapHeaderDef to return false.

Modified:
    ode/branches/APACHE_ODE_1.X/axis2/src/main/java/org/apache/ode/axis2/util/SoapMessageConverter.java

Modified: ode/branches/APACHE_ODE_1.X/axis2/src/main/java/org/apache/ode/axis2/util/SoapMessageConverter.java
URL: http://svn.apache.org/viewvc/ode/branches/APACHE_ODE_1.X/axis2/src/main/java/org/apache/ode/axis2/util/SoapMessageConverter.java?rev=746452&r1=746451&r2=746452&view=diff
==============================================================================
--- ode/branches/APACHE_ODE_1.X/axis2/src/main/java/org/apache/ode/axis2/util/SoapMessageConverter.java (original)
+++ ode/branches/APACHE_ODE_1.X/axis2/src/main/java/org/apache/ode/axis2/util/SoapMessageConverter.java Sat Feb 21 08:25:46 2009
@@ -197,7 +197,7 @@
         }
 
         if (message.getHeaderParts().size() > 0 || getSOAPHeaders(bo).size() > 0)
-            createSoapHeaders(soapEnv, getSOAPHeaders(bo), op.getInput().getMessage(), message.getHeaderParts());
+            createSoapHeaders(soapEnv, getSOAPHeaders(bo), op.getOutput().getMessage(), message.getHeaderParts());
 
         SOAPBody soapBody = getSOAPBody(bo);
         if (soapBody != null) {



Re: svn commit: r746452 - /ode/branches/APACHE_ODE_1.X/axis2/src/main/java/org/apache/ode/axis2/util/SoapMessageConverter.java

Posted by Matthieu Riou <ma...@gmail.com>.
On Sat, Feb 21, 2009 at 12:46 AM, Tammo van Lessen <tv...@gmail.com>wrote:

> Milinda's first commit, if I'm not mistaken. Welcome home, good to see
> around again :)
>

+1 :)

Matthieu


>
> Cheers,
>  Tammo
>
> milinda@apache.org wrote:
> > Author: milinda
> > Date: Sat Feb 21 08:25:46 2009
> > New Revision: 746452
> >
> > URL: http://svn.apache.org/viewvc?rev=746452&view=rev
> > Log:
> > Fixing Reply SOAP Header handling problem occurred due to use of wrong
> WSDL operation message. Previous code has used input operation wsdl message
> for output operation wsdl message. This cause
> headerdef.getMessage().equals(msgdef.getQName()) which resides inside
> handleSoapHeaderDef to return false.
> >
> > Modified:
> >
> ode/branches/APACHE_ODE_1.X/axis2/src/main/java/org/apache/ode/axis2/util/SoapMessageConverter.java
> >
> > Modified:
> ode/branches/APACHE_ODE_1.X/axis2/src/main/java/org/apache/ode/axis2/util/SoapMessageConverter.java
> > URL:
> http://svn.apache.org/viewvc/ode/branches/APACHE_ODE_1.X/axis2/src/main/java/org/apache/ode/axis2/util/SoapMessageConverter.java?rev=746452&r1=746451&r2=746452&view=diff
> >
> ==============================================================================
> > ---
> ode/branches/APACHE_ODE_1.X/axis2/src/main/java/org/apache/ode/axis2/util/SoapMessageConverter.java
> (original)
> > +++
> ode/branches/APACHE_ODE_1.X/axis2/src/main/java/org/apache/ode/axis2/util/SoapMessageConverter.java
> Sat Feb 21 08:25:46 2009
> > @@ -197,7 +197,7 @@
> >          }
> >
> >          if (message.getHeaderParts().size() > 0 ||
> getSOAPHeaders(bo).size() > 0)
> > -            createSoapHeaders(soapEnv, getSOAPHeaders(bo),
> op.getInput().getMessage(), message.getHeaderParts());
> > +            createSoapHeaders(soapEnv, getSOAPHeaders(bo),
> op.getOutput().getMessage(), message.getHeaderParts());
> >
> >          SOAPBody soapBody = getSOAPBody(bo);
> >          if (soapBody != null) {
> >
> >
>
>
> --
> Tammo van Lessen - http://www.taval.de
>

Re: svn commit: r746452 - /ode/branches/APACHE_ODE_1.X/axis2/src/main/java/org/apache/ode/axis2/util/SoapMessageConverter.java

Posted by Tammo van Lessen <tv...@gmail.com>.
Milinda's first commit, if I'm not mistaken. Welcome home, good to see
around again :)

Cheers,
  Tammo

milinda@apache.org wrote:
> Author: milinda
> Date: Sat Feb 21 08:25:46 2009
> New Revision: 746452
> 
> URL: http://svn.apache.org/viewvc?rev=746452&view=rev
> Log:
> Fixing Reply SOAP Header handling problem occurred due to use of wrong WSDL operation message. Previous code has used input operation wsdl message for output operation wsdl message. This cause headerdef.getMessage().equals(msgdef.getQName()) which resides inside handleSoapHeaderDef to return false.
> 
> Modified:
>     ode/branches/APACHE_ODE_1.X/axis2/src/main/java/org/apache/ode/axis2/util/SoapMessageConverter.java
> 
> Modified: ode/branches/APACHE_ODE_1.X/axis2/src/main/java/org/apache/ode/axis2/util/SoapMessageConverter.java
> URL: http://svn.apache.org/viewvc/ode/branches/APACHE_ODE_1.X/axis2/src/main/java/org/apache/ode/axis2/util/SoapMessageConverter.java?rev=746452&r1=746451&r2=746452&view=diff
> ==============================================================================
> --- ode/branches/APACHE_ODE_1.X/axis2/src/main/java/org/apache/ode/axis2/util/SoapMessageConverter.java (original)
> +++ ode/branches/APACHE_ODE_1.X/axis2/src/main/java/org/apache/ode/axis2/util/SoapMessageConverter.java Sat Feb 21 08:25:46 2009
> @@ -197,7 +197,7 @@
>          }
>  
>          if (message.getHeaderParts().size() > 0 || getSOAPHeaders(bo).size() > 0)
> -            createSoapHeaders(soapEnv, getSOAPHeaders(bo), op.getInput().getMessage(), message.getHeaderParts());
> +            createSoapHeaders(soapEnv, getSOAPHeaders(bo), op.getOutput().getMessage(), message.getHeaderParts());
>  
>          SOAPBody soapBody = getSOAPBody(bo);
>          if (soapBody != null) {
> 
> 


-- 
Tammo van Lessen - http://www.taval.de