You are viewing a plain text version of this content. The canonical link for it is here.
Posted to axis-cvs@ws.apache.org by de...@apache.org on 2006/01/27 06:38:45 UTC

svn commit: r372748 - in /webservices/axis2/trunk/java/modules/core/src/org/apache/axis2/description: AxisService.java AxisServiceGroup.java

Author: deepal
Date: Thu Jan 26 21:38:39 2006
New Revision: 372748

URL: http://svn.apache.org/viewcvs?rev=372748&view=rev
Log:
-fixing engageNotify bugs

Modified:
    webservices/axis2/trunk/java/modules/core/src/org/apache/axis2/description/AxisService.java
    webservices/axis2/trunk/java/modules/core/src/org/apache/axis2/description/AxisServiceGroup.java

Modified: webservices/axis2/trunk/java/modules/core/src/org/apache/axis2/description/AxisService.java
URL: http://svn.apache.org/viewcvs/webservices/axis2/trunk/java/modules/core/src/org/apache/axis2/description/AxisService.java?rev=372748&r1=372747&r2=372748&view=diff
==============================================================================
--- webservices/axis2/trunk/java/modules/core/src/org/apache/axis2/description/AxisService.java (original)
+++ webservices/axis2/trunk/java/modules/core/src/org/apache/axis2/description/AxisService.java Thu Jan 26 21:38:39 2006
@@ -266,6 +266,11 @@
             }
         }
 
+        Module moduleImpl = moduleref.getModule();
+        if (moduleImpl != null) {
+            // notyfying module for service engagement
+            moduleImpl.engageNotify(this);
+        }
         // adding module operations
         addModuleOperations(moduleref, axisConfig);
 
@@ -273,7 +278,6 @@
 
         while (operations.hasNext()) {
             AxisOperation axisOperation = (AxisOperation) operations.next();
-            Module moduleImpl = moduleref.getModule();
             if (moduleImpl != null) {
                 // notyfying module for service engagement
                 moduleImpl.engageNotify(axisOperation);
@@ -311,7 +315,7 @@
         }
     }
 
-    public void printUsingWSDLDefinition(OutputStream out, String serviceURL) throws AxisFault {
+    private void printUsingWSDLDefinition(OutputStream out, String serviceURL) throws AxisFault {
         try {
             Definition wsdlDefinition = getWSDLDefinition();
             Collection services = wsdlDefinition.getServices().values();
@@ -341,7 +345,7 @@
         }
     }
 
-    public void printUsingWOM(OutputStream out, String serviceURL) throws AxisFault {
+    private void printUsingWOM(OutputStream out, String serviceURL) throws AxisFault {
         AxisService2WOM axisService2WOM = new AxisService2WOM(getSchema(),
                 this,
                 targetNamespace,

Modified: webservices/axis2/trunk/java/modules/core/src/org/apache/axis2/description/AxisServiceGroup.java
URL: http://svn.apache.org/viewcvs/webservices/axis2/trunk/java/modules/core/src/org/apache/axis2/description/AxisServiceGroup.java?rev=372748&r1=372747&r2=372748&view=diff
==============================================================================
--- webservices/axis2/trunk/java/modules/core/src/org/apache/axis2/description/AxisServiceGroup.java (original)
+++ webservices/axis2/trunk/java/modules/core/src/org/apache/axis2/description/AxisServiceGroup.java Thu Jan 26 21:38:39 2006
@@ -134,11 +134,6 @@
             // engaging each service
             AxisService axisService = (AxisService) srevice.next();
             try {
-                Module moduleImpl = module.getModule();
-                if (moduleImpl != null) {
-                    // notyfying module for service engagement
-                    moduleImpl.engageNotify(axisService);
-                }
                 axisService.engageModule(module, (AxisConfiguration) getParent());
             } catch (AxisFault axisFault) {
                 log.info(axisFault.getMessage());