You are viewing a plain text version of this content. The canonical link for it is here.
Posted to java-dev@axis.apache.org by da...@apache.org on 2006/02/09 10:59:18 UTC

svn commit: r376244 - in /webservices/axis2/trunk/c/modules/core/description: op.c svc.c

Author: damitha
Date: Thu Feb  9 01:59:17 2006
New Revision: 376244

URL: http://svn.apache.org/viewcvs?rev=376244&view=rev
Log:
When module is already engaged it set the error code and
return AXIS2_SUCCESS. But currently it return AXIS2_SUCCESS


Modified:
    webservices/axis2/trunk/c/modules/core/description/op.c
    webservices/axis2/trunk/c/modules/core/description/svc.c

Modified: webservices/axis2/trunk/c/modules/core/description/op.c
URL: http://svn.apache.org/viewcvs/webservices/axis2/trunk/c/modules/core/description/op.c?rev=376244&r1=376243&r2=376244&view=diff
==============================================================================
--- webservices/axis2/trunk/c/modules/core/description/op.c (original)
+++ webservices/axis2/trunk/c/modules/core/description/op.c Thu Feb  9 01:59:17 2006
@@ -1026,7 +1026,7 @@
         {
             AXIS2_ERROR_SET((*env)->error, 
                 AXIS2_ERROR_MODULE_ALREADY_ENGAGED_TO_OP, AXIS2_FAILURE);
-            return AXIS2_FAILURE;
+            return AXIS2_SUCCESS;
         }
 
     }
@@ -1085,7 +1085,7 @@
         {
             AXIS2_ERROR_SET((*env)->error, 
                 AXIS2_ERROR_MODULE_ALREADY_ENGAGED_TO_OP, AXIS2_FAILURE);
-            return AXIS2_FAILURE;
+            return AXIS2_SUCCESS;
         }
 
     }

Modified: webservices/axis2/trunk/c/modules/core/description/svc.c
URL: http://svn.apache.org/viewcvs/webservices/axis2/trunk/c/modules/core/description/svc.c?rev=376244&r1=376243&r2=376244&view=diff
==============================================================================
--- webservices/axis2/trunk/c/modules/core/description/svc.c (original)
+++ webservices/axis2/trunk/c/modules/core/description/svc.c Thu Feb  9 01:59:17 2006
@@ -1047,12 +1047,19 @@
     
     for(i = 0; i < size; i++)
     {
+        axis2_qname_t *module_d_name = NULL;
+        axis2_qname_t *module_d_name_l = NULL;
+        
         module_desc = (axis2_module_desc_t *) AXIS2_ARRAY_LIST_GET(
             collection_module, env, i);
-        if(AXIS2_QNAME_EQUALS(AXIS2_MODULE_DESC_GET_NAME(module_desc, env), env,
-                AXIS2_MODULE_DESC_GET_NAME(module_name, env)))
+        module_d_name = AXIS2_MODULE_DESC_GET_NAME(module_desc, env);
+        module_d_name_l = AXIS2_MODULE_DESC_GET_NAME(module_name, env);
+            
+        if(AXIS2_QNAME_EQUALS(module_d_name, env, module_d_name_l))
         {
-            return AXIS2_FAILURE;
+            AXIS2_ERROR_SET((*env)->error, 
+                AXIS2_ERROR_MODULE_ALREADY_ENGAGED_TO_SVC, AXIS2_FAILURE);
+            return AXIS2_SUCCESS;
         }
     }
     return AXIS2_ARRAY_LIST_ADD(collection_module, env, module_name);