axis-java-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Davanum Srinivas <d...@yahoo.com>
Subject Re: cvs commit: xml-axis/java/src/org/apache/axis/description OperationDesc.java
Date Wed, 15 Jan 2003 22:07:54 GMT
James,

Does this fix the problem? (16141)

Thanks,
dims

--- jmsnell@apache.org 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

Mime
View raw message