RE: cvs commit: xml-axis/java/src/org/apache/axis/description Ser viceDesc.java

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

Hm - static methods are perfectly fine to include in a Web Service...

A better fix would be to notice the Skeleton class while introspecting, and skip it.  I've made this fix in my sandbox, and can check it in if desired.

--Glen

> -----Original Message-----
> From: tomj@apache.org [mailto:tomj@apache.org]
> Sent: Wednesday, July 03, 2002 4:57 PM
> To: xml-axis-cvs@apache.org
> Subject: cvs commit: xml-axis/java/src/org/apache/axis/description
> ServiceDesc.java
> 
> 
> tomj        2002/07/03 13:57:14
> 
>   Modified:    java/src/org/apache/axis/description ServiceDesc.java
>   Log:
>   Fix a problem with Skeletons and WSDL generation:
>   The getOperationDescs() and getOperationDescByName()
>   APIs were showing up in WSDL when skeletons were
>   turned on.
>   
>   Fix: Filter out static methods when creating the
>   method list for a service.
>   
>   NOTE TO IBM: You may want this for Beta 3 if you care
>   about the WSDL generated when using skeletons!
>   
>   Revision  Changes    Path
>   1.37      +1 -0      
> xml-axis/java/src/org/apache/axis/description/ServiceDesc.java
>   
>   Index: ServiceDesc.java
>   ===================================================================
>   RCS file: 
> /home/cvs/xml-axis/java/src/org/apache/axis/description/Servic
> eDesc.java,v
>   retrieving revision 1.36
>   retrieving revision 1.37
>   diff -u -r1.36 -r1.37
>   --- ServiceDesc.java	3 Jul 2002 17:50:21 -0000	1.36
>   +++ ServiceDesc.java	3 Jul 2002 20:57:14 -0000	1.37
>   @@ -717,6 +717,7 @@
>            for (int i = 0; i < methods.length; i++) {
>                Method method = methods[i];
>                if (Modifier.isPublic(method.getModifiers()) &&
>   +                    !Modifier.isStatic(method.getModifiers()) &&
>                        method.getName().equals(methodName)) {
>                    createOperationForMethod(method);
>                }
>   
>   
>   
>