feat(RUP):"Arreglos en la funcionalidad"

Author: nicolasarana

modules/rup/routes/prestacion.ts

@@ -590,62 +590,63 @@ router.post('/prestaciones', async (req, res, next) => {
  * @param registrosNuevos - Registros que vienen en el request
  * @returns Registros mergeados con auditoría preservada
  */
-function mergeRegistrosPreservandoAuditoria(registrosExistentes: any[], registrosNuevos: any[]): any[] {
-    if (!registrosExistentes || registrosExistentes.length === 0) {
+
+function mergeRegistrosPreservandoAuditoria(
+    registrosExistentes: any[],
+    registrosNuevos: any[]
+): any[] {
+    if (!registrosExistentes?.length) {
         return registrosNuevos;
     }
-
-    if (!registrosNuevos || registrosNuevos.length === 0) {
+    if (!registrosNuevos?.length) {
         return registrosExistentes;
     }
 
-    const registrosExistentesMap = new Map();
-    registrosExistentes.forEach(reg => {
-        if (reg._id) {
-            registrosExistentesMap.set(reg._id.toString(), reg);
-        }
-    });
+    const existentesMap = new Map<string, any>(
+        registrosExistentes
+            .filter(r => r._id)
+            .map(r => [r._id.toString(), r])
+    );
 
-    const registrosMergeados = registrosNuevos.map(regNuevo => {
-        const idNuevo = regNuevo._id ? regNuevo._id.toString() : null;
+    return registrosNuevos.map(regNuevo => {
+        const idNuevo = regNuevo._id?.toString() || regNuevo.id?.toString();
+        const regExistente = idNuevo ? existentesMap.get(idNuevo) : null;
 
-        if (idNuevo && registrosExistentesMap.has(idNuevo)) {
-            const regExistente = registrosExistentesMap.get(idNuevo);
+        // 🔹 Registro completamente nuevo
+        if (!regExistente) {
+            return regNuevo;
+        }
 
-            const registroMergeado = {
-                ...regNuevo,
-                createdAt: regExistente.createdAt,
-                createdBy: regExistente.createdBy,
-            };
+        // 🔹 Comparación profunda para detectar si hubo cambios reales
+        const valorIgual =
+            JSON.stringify(regExistente.valor) === JSON.stringify(regNuevo.valor);
 
-            if (regNuevo.registros && regNuevo.registros.length > 0) {
-                registroMergeado.registros = mergeRegistrosPreservandoAuditoria(
-                    regExistente.registros || [],
-                    regNuevo.registros
-                );
-            }
+        const subRegistrosIguales =
+            JSON.stringify(regExistente.registros || []) ===
+            JSON.stringify(regNuevo.registros || []);
 
-            return registroMergeado;
-        } else {
-            if (regNuevo.registros && regNuevo.registros.length > 0) {
-                const registrosAnidadosExistentes = [];
-                registrosExistentes.forEach(regEx => {
-                    if (regEx.registros && regEx.registros.length > 0) {
-                        registrosAnidadosExistentes.push(...regEx.registros);
-                    }
-                });
+        // 🔹 No hubo cambios → devolver registro original (preserva auditoría)
+        if (valorIgual && subRegistrosIguales) {
+            return regExistente;
+        }
 
-                regNuevo.registros = mergeRegistrosPreservandoAuditoria(
-                    registrosAnidadosExistentes,
-                    regNuevo.registros
-                );
-            }
+        // 🔹 Hubo cambios → merge preservando auditoría original
+        const registroMergeado: any = {
+            ...regNuevo,
+            createdAt: regExistente.createdAt,
+            createdBy: regExistente.createdBy
+        };
 
-            return regNuevo;
+        // 🔹 Merge recursivo de subregistros
+        if (regNuevo.registros?.length && regExistente.registros?.length) {
+            registroMergeado.registros = mergeRegistrosPreservandoAuditoria(
+                regExistente.registros,
+                regNuevo.registros
+            );
         }
-    });
 
-    return registrosMergeados;
+        return registroMergeado;
+    });
 }
 
 /**
@@ -654,17 +655,17 @@ function mergeRegistrosPreservandoAuditoria(registrosExistentes: any[], registro
  * @param prestacion - Prestación a actualizar
  * @param profesional - Profesional actual que está registrando
  */
-function actualizarProfesionalesQueRegistran(prestacion: any, profesional: any) {
-    if (!prestacion.profesionalesQueRegistran) {
-        prestacion.profesionalesQueRegistran = [];
+function actualizarProfesionalesRegistrantes(prestacion: any, profesional: any) {
+    if (!prestacion.profesionalesRegistrantes) {
+        prestacion.profesionalesRegistrantes = [];
     }
 
-    const yaExiste = prestacion.profesionalesQueRegistran.some(
+    const yaExiste = prestacion.profesionalesRegistrantes.some(
         (prof: any) => prof.id && prof.id.toString() === profesional.id.toString()
     );
 
     if (!yaExiste) {
-        prestacion.profesionalesQueRegistran.push({
+        prestacion.profesionalesRegistrantes.push({
             id: profesional.id,
             nombreCompleto: profesional.nombreCompleto,
             nombre: profesional.nombre,
@@ -738,7 +739,7 @@ router.patch('/prestaciones/:id', (req: Request, res, next) => {
                     // Actualizar lista de profesionales que registran
                     const profesionalQueRegistra = Auth.getProfesional(req);
                     if (profesionalQueRegistra) {
-                        actualizarProfesionalesQueRegistran(data, profesionalQueRegistra);
+                        actualizarProfesionalesRegistrantes(data, profesionalQueRegistra);
                     }
                 }
                 if (req.body.ejecucion?.fecha) {
@@ -787,7 +788,7 @@ router.patch('/prestaciones/:id', (req: Request, res, next) => {
                     // Actualizar lista de profesionales que registran
                     const profesionalQueRegistra = Auth.getProfesional(req);
                     if (profesionalQueRegistra) {
-                        actualizarProfesionalesQueRegistran(data, profesionalQueRegistra);
+                        actualizarProfesionalesRegistrantes(data, profesionalQueRegistra);
                     }
 
                     if (req.body.solicitud) {
@@ -874,7 +875,7 @@ router.patch('/prestaciones/:id', (req: Request, res, next) => {
                 // Actualizar lista de profesionales que registran
                 const profesionalActual = Auth.getProfesional(req);
                 if (profesionalActual) {
-                    actualizarProfesionalesQueRegistran(data, profesionalActual);
+                    actualizarProfesionalesRegistrantes(data, profesionalActual);
                 }
                 break;
             default:

modules/rup/schemas/prestacion.ts

@@ -168,7 +168,7 @@ export const PrestacionSchema = new Schema({
 
     },
 
-    profesionalesQueRegistran: [
+    profesionalesRegistrantes: [
         {
             _id: false,
             id: Schema.Types.ObjectId,