Browse Source

Improving debugging msgs see BT#4882

Julio Montoya 12 years ago
parent
commit
c8f715c52e
2 changed files with 28 additions and 28 deletions
  1. 14 17
      tests/migrate/migration.class.php
  2. 14 11
      tests/migrate/migration.custom.class.php

+ 14 - 17
tests/migrate/migration.class.php

@@ -181,10 +181,7 @@ class Migration {
      * @param array Variables to be passed as params to the function
      * @return array Results as returned by the SOAP call
      */
-    static function soap_call($web_service_params, $function_name, $params = array()) {
-        
-        //$web_service_params = $this->web_service_connection_info;
-        
+    static function soap_call($web_service_params, $function_name, $params = array()) {        
         // Create the client instance
         $url = $web_service_params['url'];        
         try {
@@ -196,13 +193,13 @@ class Migration {
         
         $client->debug_flag = true;     
         try {            
-            $data = $client->$function_name($params);            
+            $data = $client->$function_name($params);
         } catch (SoapFault $fault) {
             $error = 2;
             //die("Problem querying service - $function_name");
             return array(
                 'error' => true,
-                'message' => "Problem querying service - $function_name in URL $url ",
+                'message' => "Problem querying service - $function_name in URL $url with params: ".print_r($params, 1),
                 'status_id' => 0
             );
         }
@@ -213,7 +210,7 @@ class Migration {
         } else {
             return array(
                 'error' => true,
-                'message' => 'No Data found',
+                'message' => "No data found when calling $function_name in URL $url with params: ".print_r($params, 1),
                 'status_id' => 0
             );
         }        
@@ -418,9 +415,9 @@ class Migration {
                 'status_id' => 0
             ),
              array(
-                //'action' => 'horario_eliminar',
-                 'transaction_id' => 1012,
+                //'action' => 'horario_eliminar',                 
                 'action' => 14,
+                 'transaction_id' => 1012,
                 'item_id' =>  'E395895A-B480-456F-87F2-36B3A1EBB81C',
                 'orig_id' => '0',
                 'dest_id' => null,
@@ -457,9 +454,9 @@ class Migration {
             ),
              * */
             array(
-                //'action' => 'sede_agregar',
-                'transaction_id' => 1013,
+                //'action' => 'sede_agregar',                
                 'action' => 19,
+                'transaction_id' => 1013,
                 'item_id' =>  '7379A7D3-6DC5-42CA-9ED4-97367519F1D9',
                 'orig_id' => '0',
                 'branch_id' => 1,
@@ -467,9 +464,9 @@ class Migration {
                 'status_id' => 0
             ),
             array(
-                //'action' => 'sede_editar',
-                'transaction_id' => 1014,
+                //'action' => 'sede_editar',                
                 'action' => 21,
+                'transaction_id' => 1014,
                 'item_id' =>  '7379A7D3-6DC5-42CA-9ED4-97367519F1D9',
                 'orig_id' => '0',
                 'branch_id' => 1,
@@ -478,8 +475,8 @@ class Migration {
             ),
             array(
                 //'action' => 'sede_eliminar',
-                'transaction_id' => 1015,
                 'action' => 20,
+                'transaction_id' => 1015,
                 'item_id' =>  '7379A7D3-6DC5-42CA-9ED4-97367519F1D9',
                 'orig_id' => '0',
                 'branch_id' => 1,
@@ -487,9 +484,9 @@ class Migration {
                 'status_id' => 0
             ),
             array(
-                //'action' => 'frecuencia_agregar',
-                'transaction_id' => 1016,
+                //'action' => 'frecuencia_agregar',                
                 'action' => 22,
+                'transaction_id' => 1016,
                 'item_id' =>  '0091CD3B-F042-11D7-B338-0050DAB14015',
                 'orig_id' => '0',
                 'branch_id' => 1,
@@ -550,7 +547,7 @@ class Migration {
         
         foreach( $transaction_harcoded as  $transaction) {
             $transaction['branch_id'] = 2;
-            if ($transaction['action'] != 8) {
+            if ($transaction['action'] != 21) {
                 continue;
             }
             self::add_transaction($transaction);

+ 14 - 11
tests/migrate/migration.custom.class.php

@@ -936,7 +936,7 @@ class MigrationCustom {
     //const TRANSACTION_TYPE_ADD_COURSE  =  5;
     static function transaction_5($data, $web_service_details) {
         $uidCursoId = $data['item_id'];          
-        $course_info = Migration::soap_call($web_service_details, 'cursoDetalles', array('intIdSede'=> $data['branch_id'], 'uididcurso' => $uidCursoId));         
+        $course_info = Migration::soap_call($web_service_details, 'cursoDetalles', array('intIdSede'=> $data['branch_id'], 'uididcurso' => $uidCursoId));    
         if ($course_info['error'] == false) {
             unset($course_info['error']);
             $course_info = CourseManager::create_course($course_info);
@@ -1295,7 +1295,10 @@ class MigrationCustom {
         $extra_field_option_info = $extra_field_option->get_field_option_by_field_and_option($extra_field_info['id'], $original_data['item_id']);
         
         $function_name = $extra_field_variable."Detalles";
-        $data = Migration::soap_call($web_service_details, $function_name, array('intIdSede'=> $original_data['branch_id'], "uidid".$extra_field_variable => $original_data['item_id']));          
+        
+        $params = array('intIdSede'=> $original_data['branch_id'], "uidid".$extra_field_variable => $original_data['item_id']);
+        
+        $data = Migration::soap_call($web_service_details, $function_name, $params);          
         if ($data['error'] == false) {
             
             //Update 1 item
@@ -1665,7 +1668,7 @@ class MigrationCustom {
         } else {            
             return array(
                 'error' => true,
-                'message' => "No data when calling $result_name with data: ".print_r($data, 1),
+                'message' => "No data when calling $result_name result:".print_r($data, 1)." with params: ".print_r($params, 1),
                 'status_id' => MigrationCustom::TRANSACTION_STATUS_FAILED,
             );
         }
@@ -1821,8 +1824,8 @@ class MigrationCustom {
       ["name"]=>
       string(8) "Sabatino"
     }*/
-    static function faseDetalles($data) {
-        $result = self::genericDetalles($data, __FUNCTION__);
+    static function faseDetalles($data, $params) {
+        $result = self::genericDetalles($data, __FUNCTION__, $params);
         if ($result['error'] == true) {
             return $result;
         }        
@@ -1830,7 +1833,7 @@ class MigrationCustom {
     }
     
     static function frecuenciaDetalles($data, $params) {
-        $result = self::genericDetalles($data, __FUNCTION__); 
+        $result = self::genericDetalles($data, __FUNCTION__, $params); 
         if ($result['error'] == true) {
             return $result;
         }        
@@ -1844,7 +1847,7 @@ class MigrationCustom {
     }*/
 
     static function intensidadDetalles($data, $params) {
-        $result = self::genericDetalles($data, __FUNCTION__);
+        $result = self::genericDetalles($data, __FUNCTION__, $params);
         if ($result['error'] == true) {
             return $result;
         }        
@@ -1859,7 +1862,7 @@ class MigrationCustom {
       string(3) "4  "
     }*/
     static function mesesDetalles($data, $params) {
-        $result = self::genericDetalles($data, __FUNCTION__);
+        $result = self::genericDetalles($data, __FUNCTION__, $params);
         if ($result['error'] == true) {
             return $result;
         }        
@@ -1873,7 +1876,7 @@ class MigrationCustom {
     }*/
 
     static function sedeDetalles($data, $params) {
-        $result = self::genericDetalles($data, __FUNCTION__);
+        $result = self::genericDetalles($data, __FUNCTION__, $params);
         if ($result['error'] == true) {
             return $result;
         }        
@@ -1891,7 +1894,7 @@ class MigrationCustom {
       string(2) "62"
     }*/
     static function horarioDetalles($data, $params) {         
-        $result = self::genericDetalles($data, __FUNCTION__);        
+        $result = self::genericDetalles($data, __FUNCTION__, $params);        
         if ($result['error'] == true) {
             return $result;
         }
@@ -1903,4 +1906,4 @@ class MigrationCustom {
         unset($result['end']);
         return $result;
     }
-}
+}