RE: cvs commit: xml-axis/java/src/org/apache/axis/wsdl/fromJava E mitter.java

[Glen Daniels <gd...@macromedia.com>]

I don't think it got removed so much as confused (for instance, it never failed for me).  Allowed methods processing is, as of last night, now all centralized in the ServiceDesc, which should help make things clearer.

--G

> -----Original Message-----
> From: Tom Jordahl [mailto:tomj@macromedia.com]
> Sent: Monday, August 05, 2002 10:03 AM
> To: 'axis-dev@xml.apache.org'
> Subject: RE: cvs commit: 
> xml-axis/java/src/org/apache/axis/wsdl/fromJava
> E mitter.java
> 
> 
> 
> How in gods-green-earth did this functionality get removed?
> 
> --
> Tom Jordahl
> Macromedia
> 
> 
> -----Original Message-----
> From: rubys@apache.org [mailto:rubys@apache.org]
> Sent: Sunday, August 04, 2002 10:52 AM
> To: xml-axis-cvs@apache.org
> Subject: cvs commit: xml-axis/java/src/org/apache/axis/wsdl/fromJava
> Emitter.java
> 
> 
> rubys       2002/08/04 07:52:10
> 
>   Modified:    java/src/org/apache/axis/wsdl/fromJava Emitter.java
>   Log:
>   Emitted WSDL should only contain the allowed methods
>   Submitted by: Stuart Thomson <st...@swtsoftware.com>
>   
>   Revision  Changes    Path
>   1.55      +12 -0     
> xml-axis/java/src/org/apache/axis/wsdl/fromJava/Emitter.java
>   
>   Index: Emitter.java
>   ===================================================================
>   RCS file: 
> /home/cvs/xml-axis/java/src/org/apache/axis/wsdl/fromJava/Emit
> ter.java,v
>   retrieving revision 1.54
>   retrieving revision 1.55
>   diff -u -r1.54 -r1.55
>   --- Emitter.java	1 Aug 2002 18:35:42 -0000	1.54
>   +++ Emitter.java	4 Aug 2002 14:52:09 -0000	1.55
>   @@ -690,6 +690,9 @@
>            ArrayList operations = serviceDesc.getOperations();
>            for (Iterator i = operations.iterator(); i.hasNext();) {
>                OperationDesc thisOper = (OperationDesc)i.next();
>   +            if (!allowedMethod(thisOper.getName())) 
>   +                continue;
>   +
>                BindingOperation bindingOper = writeOperation(def,
>                                                              binding,
>                                                              
> thisOper);
>   @@ -1616,5 +1619,14 @@
>    
>        public void setServiceDesc(ServiceDesc serviceDesc) {
>            this.serviceDesc = serviceDesc;
>   +    }
>   +
>   +    private boolean allowedMethod(String name) {
>   +        boolean allowed = false;
>   +        if (allowedMethods == null || 
> allowedMethods.contains(name))
>   +            allowed = true;
>   +        if (allowed && disallowedMethods != null && 
> disallowedMethods.contains(name)) 
>   +            allowed = false;
>   +        return allowed;
>        }
>    }
>   
>   
>   
>