uso de registerLog en lugar de Log::error y correciones al updateVisitStatus

sistema-mantenimiento-back/app/Http/Controllers/PreventiveMaintenanceController.php

@@ -2234,7 +2234,6 @@ class PreventiveMaintenanceController extends Controller
     // Visitas técnicas no programadas (Preventivas)
     public function newUnprogrammedOrder(Request $request)
     {
-        Log::info($request->all());
         DB::enableQueryLog();
 
         $validator = Validator::make($request->all(), [
@@ -2643,7 +2642,6 @@ class PreventiveMaintenanceController extends Controller
             'id_user' => 'required|string',
             'linea' => 'required|integer',
             'id_visit' => 'required|string',
-            'comments' => 'required|string|min:15',
             'status' => 'required|string|in:VA,EP,CP,CE,P,C,R,A,F'
         ]);
 
@@ -2761,14 +2759,6 @@ class PreventiveMaintenanceController extends Controller
             }
         }
 
-        // Actualizar comentarios
-        $commentsArr = json_decode($visit->RVTN_COME, true);
-        if (!is_array($commentsArr)) {
-            $commentsArr = [];
-        }
-        $commentsArr["C$newStatus"] = $form['comments'];
-        $commentsStr = json_encode($commentsArr);
-
         // Obtener y actualizar historial de estados
         $statusHistoryArr = json_decode($visit->RVTN_HIES, true);
         if (!is_array($statusHistoryArr)) {
@@ -2789,7 +2779,6 @@ class PreventiveMaintenanceController extends Controller
         // Preparar actualización
         $updateArr = [
             'RVTN_ESTA' => $newStatus,
-            'RVTN_COME' => $commentsStr,
             'RVTN_HIES' => $statusHistoryStr,
             'RVTN_USMO' => $idUser,
             'RVTN_FEMO' => $nowStr,
@@ -2863,7 +2852,7 @@ class PreventiveMaintenanceController extends Controller
 
         $statusAction = $statusMessages[$newStatus] ?? 'Cambio de estado';
         $notificationTitle = "Visita Técnica No Programada #$idVisit";
-        $notificationMessage = "La visita técnica no programada #$idVisit ha sido {$statusAction}.";
+        $notificationMessage = "La visita técnica no programada #$idVisit ha sido actulizada a estado:{$statusAction}.";
 
         // Notificación general para todos los cambios de estado (A, R, C, EP, F, VA, CE, CP)
         if (in_array($newStatus, ['A', 'R', 'C', 'EP', 'F', 'VA', 'CE', 'CP']) && !empty($audience)) {
@@ -3234,7 +3223,25 @@ class PreventiveMaintenanceController extends Controller
             // Revertir transacción en caso de error
             DB::rollBack();
 
-            Log::error('Error en registerOperatorClosingComment: ' . $e->getMessage());
+            $actions = DB::getQueryLog();
+            $now = $this->functionsController->now();
+            $nowStr = $now->toDateTimeString();
+            $name = $this->functionsController->joinName($usr->USUA_NOMB ?? '', $usr->USUA_APPA ?? '', $usr->USUA_APMA ?? '');
+
+            $idac = $this->functionsController->registerActivity(
+                $form['linea'],
+                'S002V01M10GMPR',
+                'S002V01F11RVTP',
+                'S002V01P01REVI',
+                'Error',
+                "Error al registrar comentario de finalización para la visita no programada #$idVisit: " . $e->getMessage(),
+                $idUser,
+                $nowStr,
+                'S002V01S02AOTR'
+            );
+
+            $this->functionsController->registerLog($actions, $idUser, $nowStr, $idac, $form['linea']);
+
             return $this->responseController->makeResponse(
                 true,
                 'Ocurrió un error al registrar el comentario de finalización.',
@@ -5930,7 +5937,25 @@ class PreventiveMaintenanceController extends Controller
             // Revertir transacción en caso de error
             DB::rollBack();
 
-            Log::error('Error en assignOperariosToPreventiveVisit: ' . $e->getMessage());
+            $actions = DB::getQueryLog();
+            $now = $this->functionsController->now();
+            $nowStr = $now->toDateTimeString();
+            $name = $this->functionsController->joinName($usr->USUA_NOMB ?? '', $usr->USUA_APPA ?? '', $usr->USUA_APMA ?? '');
+
+            $idac = $this->functionsController->registerActivity(
+                $form['linea'],
+                'S002V01M10GMPR',
+                'S002V01F11RVTP',
+                'S002V01P01REVI',
+                'Error',
+                "Error al asignar operarios a la visita preventiva #$idVisit: " . $e->getMessage(),
+                $idUser,
+                $nowStr,
+                'S002V01S02AOTR'
+            );
+
+            $this->functionsController->registerLog($actions, $idUser, $nowStr, $idac, $form['linea']);
+
             return $this->responseController->makeResponse(
                 true,
                 'Ocurrió un error al asignar operarios a la visita preventiva.',
@@ -6538,7 +6563,25 @@ class PreventiveMaintenanceController extends Controller
             // Revertir transacción en caso de error
             DB::rollBack();
 
-            Log::error('Error en attendPreventiveVisit: ' . $e->getMessage());
+            $actions = DB::getQueryLog();
+            $now = $this->functionsController->now();
+            $nowStr = $now->toDateTimeString();
+            $userName = $this->functionsController->joinName($usr->USUA_NOMB ?? '', $usr->USUA_APPA ?? '', $usr->USUA_APMA ?? '');
+
+            $idac = $this->functionsController->registerActivity(
+                $form['linea'],
+                'S002V01M10GMPR',
+                'S002V01F11RVTP',
+                'S002V01P01REVI',
+                'Error',
+                "Error al procesar la respuesta de la visita preventiva #$idVisit: " . $e->getMessage(),
+                $idUser,
+                $nowStr,
+                'S002V01S02AOTR'
+            );
+
+            $this->functionsController->registerLog($actions, $idUser, $nowStr, $idac, $form['linea']);
+
             return $this->responseController->makeResponse(
                 true,
                 'Ocurrió un error al procesar la respuesta de la visita.',

sistema-mantenimiento-back/routes/api.php

@@ -186,8 +186,6 @@ Route::post("/update-work-order",
 Route::post("/end-order-execution",                                                 "App\Http\Controllers\PreventiveMaintenanceController@endOrderExecution");          //Registra una nueva orden de trabajo
 Route::post("/copy-work-order",                                                     "App\Http\Controllers\PreventiveMaintenanceController@copyWorkOrder");              //Registra una nueva orden de trabajo
 Route::post("/new-unprogrammed-order",                                              "App\Http\Controllers\PreventiveMaintenanceController@newUnprogrammedOrder");       //Registra una nueva orden de trabajo
-Route::post("/update-visit-status",                                                 "App\Http\Controllers\PreventiveMaintenanceController@updateVisitStatus");          //Registra una nueva orden de trabajo
-Route::post("/update-visit-status",                                                 "App\Http\Controllers\PreventiveMaintenanceController@updateVisitStatus");          //Registra una nueva orden de trabajo
 Route::post("/update-order-status",                                                 "App\Http\Controllers\PreventiveMaintenanceController@updateOrderStatus");          //Registra una nueva orden de trabajo
 Route::post("/set-budget-analysis",                                                 "App\Http\Controllers\PreventiveMaintenanceController@setBudgetAnalysis");          //Registra una nueva orden de trabajo
 Route::post("/update-order-with-activator",                                         "App\Http\Controllers\PreventiveMaintenanceController@updateOrderWithActivator");   //Registra una nueva orden de trabajo
@@ -201,6 +199,7 @@ Route::post("/attend-preventive-visit",
 Route::get("/get-visit-attendance/{idVisit}/{idUser}/{line}",                       "App\Http\Controllers\PreventiveMaintenanceController@getVisitAttendance");          //Obtiene la información de una visita preventiva
 Route::get("/get-visit-staff/{idVisit}/{idUser}/{line}",                            "App\Http\Controllers\PreventiveMaintenanceController@getVisitStaff");                //Obtiene el personal de una visita preventiva
 Route::post("/register-operator-closing-comment",                                    "App\Http\Controllers\PreventiveMaintenanceController@registerOperatorClosingComment"); //Registra comentario de finalización de operario en estado CP
+Route::post("/update-visit-status",                                                 "App\Http\Controllers\PreventiveMaintenanceController@updateVisitStatus");          //Registra una nueva orden de trabajo
 
 //Módulo contadores y activadores
 Route::get("/activator/consult/{idUser}/{line}",                                    "App\Http\Controllers\CountersActivatorsController@getActivators");