Really?  Static's are OK?  Who knew? :-)

OK, then we should back out my fix and filter for the Skeleton interfaces.

I am going on vacation now...
--
Tom Jordahl


-----Original Message-----
From: Glen Daniels [mailto:[EMAIL PROTECTED]]
Sent: Wednesday, July 03, 2002 5:12 PM
To: '[EMAIL PROTECTED]'
Subject: RE: cvs commit: xml-axis/java/src/org/apache/axis/description
Ser viceDesc.java



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: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED]]
> Sent: Wednesday, July 03, 2002 4:57 PM
> To: [EMAIL PROTECTED]
> 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);
>                }
>   
>   
>   
> 

Reply via email to