gdaniels    02/05/03 05:30:56

  Modified:    java/src/org/apache/axis/description ServiceDesc.java
  Log:
  Create not OperationDescs for certain Methods - for lo, they may
  forsooth be not public!
  
  Fixes:
  http://nagoya.apache.org/bugzilla/show_bug.cgi?id=8585
  
  Revision  Changes    Path
  1.21      +7 -3      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/ServiceDesc.java,v
  retrieving revision 1.20
  retrieving revision 1.21
  diff -u -r1.20 -r1.21
  --- ServiceDesc.java  23 Apr 2002 16:25:38 -0000      1.20
  +++ ServiceDesc.java  3 May 2002 12:30:56 -0000       1.21
  @@ -414,7 +414,8 @@
           Method [] methods = implClass.getDeclaredMethods();
           for (int i = 0; i < methods.length; i++) {
               Method method = methods[i];
  -            if (method.getName().equals(oper.getName())) {
  +            if (Modifier.isPublic(method.getModifiers()) &&
  +                    method.getName().equals(oper.getName())) {
                   // Check params
                   Class [] paramTypes = method.getParameterTypes();
                   if (paramTypes.length != oper.getNumParams())
  @@ -504,7 +505,9 @@
           Method [] methods = implClass.getDeclaredMethods();
   
           for (int i = 0; i < methods.length; i++) {
  -            getSyncedOperationsForName(implClass, methods[i].getName());
  +            if (Modifier.isPublic(methods[i].getModifiers())) {
  +                getSyncedOperationsForName(implClass, methods[i].getName());
  +            }
           }
   
           if (implClass.isInterface()) {
  @@ -643,7 +646,8 @@
   
           for (int i = 0; i < methods.length; i++) {
               Method method = methods[i];
  -            if (method.getName().equals(methodName)) {
  +            if (Modifier.isPublic(method.getModifiers()) &&
  +                    method.getName().equals(methodName)) {
                   createOperationForMethod(method);
               }
           }
  
  
  


Reply via email to