You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ode.apache.org by Tammo van Lessen <tv...@gmail.com> on 2009/02/21 09:46:29 UTC
Re: svn commit: r746452 - /ode/branches/APACHE_ODE_1.X/axis2/src/main/java/org/apache/ode/axis2/util/SoapMessageConverter.java
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
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
>