You are viewing a plain text version of this content. The canonical link for it is here.
Posted to java-dev@axis.apache.org by "Antonio Andrade (JIRA)" <ji...@apache.org> on 2008/10/16 21:25:44 UTC

[jira] Issue Comment Edited: (AXIS2-3964) WSDL20ToAxisServiceBuilder > HTTP Binding: AxisBindingMessage entry is being overwritten in corresponding AxisBindingOperation

    [ https://issues.apache.org/jira/browse/AXIS2-3964?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12640186#action_12640186 ] 

aandrade edited comment on AXIS2-3964 at 10/16/08 12:24 PM:
-------------------------------------------------------------------

Submitting patch for Review: 

Saving AxisBindingOperation children using the corresponding message label as key (as described in the bug's description). Note that since the message label depends on the message direction AND this (the direction) depends on the 'isServerSide' flag, the retrieval of the proper AxisMessage was also updated both for HTTP and SOAP bindings processing. This, by itself,  was a separate issue.

Other: I think that the 'isServerSide' flag usage should be reviewed and a general refactoring should be considered. i.e. There should probably be methods defined in the WSDLToAxisServiceBuilder that deal with the retrieval of isServerSide-sensitive information (messages based on directions, labels, etc) in a more consistent way. The usage of  this flag has caused a lot of duplication of code.





      was (Author: aandrade):
    Saving AxisBindingOperation children using the corresponding message label as key (as described in the bug's description). Note that since the message label depends on the message direction AND this (the direction) depends on the 'isServerSide' flag, the retrieval of the proper AxisMessage was also updated both for HTTP and SOAP bindings processing. This, by itself,  was a separate issue.

Other: I think that the 'isServerSide' flag usage should be reviewed and a general refactoring should be considered. i.e. There should probably be methods defined in the WSDLToAxisServiceBuilder that deal with the retrieval of isServerSide-sensitive information (messages based on directions, labels, etc) in a more consistent way. The usage of  this flag has caused a lot of duplication of code.




  
> WSDL20ToAxisServiceBuilder > HTTP Binding:  AxisBindingMessage entry is being overwritten in corresponding AxisBindingOperation
> -------------------------------------------------------------------------------------------------------------------------------
>
>                 Key: AXIS2-3964
>                 URL: https://issues.apache.org/jira/browse/AXIS2-3964
>             Project: Axis 2.0 (Axis2)
>          Issue Type: Bug
>          Components: kernel
>    Affects Versions: 1.4, nightly
>         Environment: Windows XP.  4GB RAM Intel Xeon dual-core @2.75Ghz
>            Reporter: Antonio Andrade
>            Assignee: Keith Godwin Chapman
>            Priority: Minor
>         Attachments: AXIS2-3964.patch
>
>   Original Estimate: 4h
>  Remaining Estimate: 4h
>
> Observed Behavior:
> Within WSDL20ToAxisServiceBuilder, AxisBindingOperation children are always saved under the WSDLConstants.MESSAGE_LABEL_IN_VALUE key when the HTTP Binding is used. Look at processHTTPBindingExtention method (around line 921).
> Expected Behavior:
> When using the HTTP Binding, AxisBindingMessages should be saved under the corresponding key (should be correctly set into the corresponding AxisBindingOperation):
> - WSDLConstants.MESSAGE_LABEL_IN_VALUE whenever the message has a WSDLConstants.WSDL_MESSAGE_DIRECTION_IN direction
> - WSDLConstants.MESSAGE_LABEL_OUT_VALUE whenever the message has a WSDLConstants.WSDL_MESSAGE_DIRECTION_OUT direction

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


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


Re: why is axis2 checkout missing some code from source bundle?

Posted by keith chapman <ke...@gmail.com>.
As per a discussion we had on the mailing list the transports now reside on
a seperate commons project [1]. Even the http transport related stuff can be
found there.

Thanks,
Keith.

[1]
https://svn.apache.org/repos/asf/webservices/commons/trunk/modules/transport

On Fri, Oct 17, 2008 at 1:52 AM, emiddio-verizon <em...@verizon.net>wrote:

> for example
>
> modules/kernel/src/org/apache/axis2/transport/http/AxisServlet.java
>
> is not in the checkout system; only the src bundle.
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: axis-dev-unsubscribe@ws.apache.org
> For additional commands, e-mail: axis-dev-help@ws.apache.org
>
>


-- 
Keith Chapman
Senior Software Engineer
WSO2 Inc.
Oxygenating the Web Service Platform.
http://wso2.org/

blog: http://www.keith-chapman.org

why is axis2 checkout missing some code from source bundle?

Posted by emiddio-verizon <em...@verizon.net>.
for example

modules/kernel/src/org/apache/axis2/transport/http/AxisServlet.java

is not in the checkout system; only the src bundle.

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