The Axis part of the problem, yes. See the bug detail for full info.
- James Snell
IBM Emerging Technologies
[EMAIL PROTECTED]
(559) 587-1233 (office)
(700) 544-9035 (t/l)
Programming Web Services With SOAP
O'Reilly & Associates, ISBN 0596000952
Have I not commanded you? Be strong and courageous.
Do not be terrified, do not be discouraged, for the Lord your
God will be with you whereever you go. - Joshua 1:9
Davanum Srinivas <[EMAIL PROTECTED]>
01/15/2003 02:07 PM
Please respond to axis-dev
To
[EMAIL PROTECTED]
cc
bcc
Subject
Re: cvs commit: xml-axis/java/src/org/apache/axis/description
OperationDesc.java
James,
Does this fix the problem? (16141)
Thanks,
dims
--- [EMAIL PROTECTED] wrote:
> jmsnell 2003/01/15 14:04:39
>
> Modified: java/src/org/apache/axis/description OperationDesc.java
> Log:
> Fixing a problem with the getOutputParamByQName method. See bug
#16141
>
> Revision Changes Path
> 1.31 +13 -9
xml-axis/java/src/org/apache/axis/description/OperationDesc.java
>
> Index: OperationDesc.java
> ===================================================================
> RCS file:
/home/cvs/xml-axis/java/src/org/apache/axis/description/OperationDesc.java,v
> retrieving revision 1.30
> retrieving revision 1.31
> diff -u -r1.30 -r1.31
> --- OperationDesc.java 17 Dec 2002 13:35:03 -0000 1.30
> +++ OperationDesc.java 15 Jan 2003 22:04:38 -0000 1.31
> @@ -359,18 +359,22 @@
> ParameterDesc param = null;
>
> for (Iterator i = parameters.iterator(); i.hasNext();) {
> - param = (ParameterDesc) i.next();
> + ParameterDesc pnext = (ParameterDesc)i.next();
> if (param.getQName().equals(qname) &&
> - param.getMode() != ParameterDesc.IN)
> - return param;
> + param.getMode() != ParameterDesc.IN) {
> + param = pnext;
> + break;
> + }
> }
>
> - if (null == returnDesc.getQName() ){
> - param= new ParameterDesc( returnDesc); //Create copy
> - param.setQName(qname);
> - }
> - else if ( qname.equals(returnDesc.getQName())) {
> - param = returnDesc;
> + if (param == null) {
> + if (null == returnDesc.getQName() ){
> + param= new ParameterDesc( returnDesc); //Create copy
> + param.setQName(qname);
> + }
> + else if ( qname.equals(returnDesc.getQName())) {
> + param = returnDesc;
> + }
> }
>
> return param;
>
>
>
=====
Davanum Srinivas - http://xml.apache.org/~dims/
__________________________________________________
Do you Yahoo!?
Yahoo! Mail Plus - Powerful. Affordable. Sign up now.
http://mailplus.yahoo.com
