You are viewing a plain text version of this content. The canonical link for it is here.
Posted to axis-cvs@ws.apache.org by ke...@apache.org on 2008/04/07 07:21:06 UTC
svn commit: r645377 -
/webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/WSDL11ToAxisServiceBuilder.java
Author: keithc
Date: Sun Apr 6 22:21:05 2008
New Revision: 645377
URL: http://svn.apache.org/viewvc?rev=645377&view=rev
Log:
Fixing Axis2-3704 in trunk
Modified:
webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/WSDL11ToAxisServiceBuilder.java
Modified: webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/WSDL11ToAxisServiceBuilder.java
URL: http://svn.apache.org/viewvc/webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/WSDL11ToAxisServiceBuilder.java?rev=645377&r1=645376&r2=645377&view=diff
==============================================================================
--- webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/WSDL11ToAxisServiceBuilder.java (original)
+++ webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/WSDL11ToAxisServiceBuilder.java Sun Apr 6 22:21:05 2008
@@ -2340,10 +2340,7 @@
axisBindingOperation
.setProperty(WSDL2Constants.ATTR_WSOAP_ACTION, soapActionURI);
axisBindingOperation.getAxisOperation().setSoapAction(soapActionURI);
- if (isServerSide) {
- axisBindingOperation.getAxisOperation().setSoapAction(soapActionURI);
- }
- else {
+ if (!isServerSide) {
axisBindingOperation.getAxisOperation().setOutputAction(soapActionURI);
}
@@ -2368,10 +2365,8 @@
if (soapAction != null && !soapAction.equals("")) {
axisBindingOperation.setProperty(WSDL2Constants.ATTR_WSOAP_ACTION, soapAction);
- if (isServerSide) {
- axisBindingOperation.getAxisOperation().setSoapAction(soapAction);
- }
- else {
+ axisBindingOperation.getAxisOperation().setSoapAction(soapAction);
+ if (!isServerSide) {
axisBindingOperation.getAxisOperation().setOutputAction(soapAction);
}
---------------------------------------------------------------------
To unsubscribe, e-mail: axis-cvs-unsubscribe@ws.apache.org
For additional commands, e-mail: axis-cvs-help@ws.apache.org
Re: svn commit: r645377 - /webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/WSDL11ToAxisServiceBuilder.java
Posted by David Illsley <da...@gmail.com>.
Sorry, ignore me, I misread the diff...
David
---------------------------------------------------------------------
To unsubscribe, e-mail: axis-dev-unsubscribe@ws.apache.org
For additional commands, e-mail: axis-dev-help@ws.apache.org
Re: svn commit: r645377 - /webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/WSDL11ToAxisServiceBuilder.java
Posted by David Illsley <da...@gmail.com>.
Keith, can you explain this...? The soapAction really shouldn't be the
output action server side.
Cheers,
David
On Mon, Apr 7, 2008 at 6:21 AM, <ke...@apache.org> wrote:
> Author: keithc
> Date: Sun Apr 6 22:21:05 2008
> New Revision: 645377
>
> URL: http://svn.apache.org/viewvc?rev=645377&view=rev
> Log:
> Fixing Axis2-3704 in trunk
>
>
> Modified:
> webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/WSDL11ToAxisServiceBuilder.java
>
> Modified: webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/WSDL11ToAxisServiceBuilder.java
> URL: http://svn.apache.org/viewvc/webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/WSDL11ToAxisServiceBuilder.java?rev=645377&r1=645376&r2=645377&view=diff
> ==============================================================================
> --- webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/WSDL11ToAxisServiceBuilder.java (original)
> +++ webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/WSDL11ToAxisServiceBuilder.java Sun Apr 6 22:21:05 2008
> @@ -2340,10 +2340,7 @@
> axisBindingOperation
> .setProperty(WSDL2Constants.ATTR_WSOAP_ACTION, soapActionURI);
> axisBindingOperation.getAxisOperation().setSoapAction(soapActionURI);
> - if (isServerSide) {
> - axisBindingOperation.getAxisOperation().setSoapAction(soapActionURI);
> - }
> - else {
> + if (!isServerSide) {
> axisBindingOperation.getAxisOperation().setOutputAction(soapActionURI);
> }
>
> @@ -2368,10 +2365,8 @@
>
> if (soapAction != null && !soapAction.equals("")) {
> axisBindingOperation.setProperty(WSDL2Constants.ATTR_WSOAP_ACTION, soapAction);
> - if (isServerSide) {
> - axisBindingOperation.getAxisOperation().setSoapAction(soapAction);
> - }
> - else {
> + axisBindingOperation.getAxisOperation().setSoapAction(soapAction);
> + if (!isServerSide) {
> axisBindingOperation.getAxisOperation().setOutputAction(soapAction);
> }
>
>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: axis-cvs-unsubscribe@ws.apache.org
> For additional commands, e-mail: axis-cvs-help@ws.apache.org
>
>
--
David Illsley - IBM Web Services Development
---------------------------------------------------------------------
To unsubscribe, e-mail: axis-dev-unsubscribe@ws.apache.org
For additional commands, e-mail: axis-dev-help@ws.apache.org