You are viewing a plain text version of this content. The canonical link for it is here.
Posted to java-commits@axis.apache.org by nt...@apache.org on 2010/04/30 20:54:00 UTC

svn commit: r939778 - /axis/axis2/java/core/trunk/modules/metadata/src/org/apache/axis2/jaxws/description/impl/EndpointInterfaceDescriptionImpl.java

Author: nthaker
Date: Fri Apr 30 18:53:59 2010
New Revision: 939778

URL: http://svn.apache.org/viewvc?rev=939778&view=rev
Log:
Adding code to fix test case org.apache.axis2.jaxws.description.AnnotationServiceImplDescriptionTests
The test failure was introduced when Legacy @Webmethod changes where committed. 

Modified:
    axis/axis2/java/core/trunk/modules/metadata/src/org/apache/axis2/jaxws/description/impl/EndpointInterfaceDescriptionImpl.java

Modified: axis/axis2/java/core/trunk/modules/metadata/src/org/apache/axis2/jaxws/description/impl/EndpointInterfaceDescriptionImpl.java
URL: http://svn.apache.org/viewvc/axis/axis2/java/core/trunk/modules/metadata/src/org/apache/axis2/jaxws/description/impl/EndpointInterfaceDescriptionImpl.java?rev=939778&r1=939777&r2=939778&view=diff
==============================================================================
--- axis/axis2/java/core/trunk/modules/metadata/src/org/apache/axis2/jaxws/description/impl/EndpointInterfaceDescriptionImpl.java (original)
+++ axis/axis2/java/core/trunk/modules/metadata/src/org/apache/axis2/jaxws/description/impl/EndpointInterfaceDescriptionImpl.java Fri Apr 30 18:53:59 2010
@@ -266,8 +266,8 @@ public class EndpointInterfaceDescriptio
                         log.debug("EID: Just added operation= " + operation.getOperationName());
                     addOperation(operation);
                 }
-                //Since wsdl is not defined add all operations we found.
-                else if (axisOperation == null) {
+                //Since wsdl is not defined add all operations to AxisService and OperationDescriptionList.
+                else if(axisOperation == null) {
                     if(log.isDebugEnabled()){
                         log.debug("wsdl defintion NOT found, we will expose operation using annotations.");
                     }
@@ -278,16 +278,13 @@ public class EndpointInterfaceDescriptio
                         log.debug("EID: Just added operation= " + operation.getOperationName());
                     addOperation(operation);
                 }
-                //This check is to add operations in case an Async binding is used while generating
-                //jax-ws artifacts, So any async operation example invokeAsync is mapped to operation invoke.
-                //However, we will keep an operation Description that holds details of invokeAsync.
-                //this check will ensure Async operations get added to operation description list. 
-                else if(axisOperation!=null && operation.getName().getLocalPart()!=mdc.getMethodName()){
+                //This is the case where wsdl is not defined and AxisOperation is found in Axis Service.
+                //Here we have to ensure that corresponding OperationDescription is added to OperationDescriptionList.
+                else if(getWSDLDefinition()==null && axisOperation!=null){
                     if (log.isDebugEnabled())
                         log.debug("EID: Just added operation= " + operation.getOperationName());
                     addOperation(operation);
                 }
-
             }
            
         }