feat(RUP):"Arreglos en la funcionalidad"

Author: nicolasarana

modules/rup/routes/prestacion.ts

@@ -590,81 +590,51 @@ 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) {
-        return registrosNuevos;
-    }
-
-    if (!registrosNuevos || registrosNuevos.length === 0) {
-        return registrosExistentes;
-    }
-
-    const registrosExistentesMap = new Map();
-    registrosExistentes.forEach(reg => {
-        if (reg._id) {
-            registrosExistentesMap.set(reg._id.toString(), reg);
-        }
-    });
-
-    const registrosMergeados = registrosNuevos.map(regNuevo => {
-        const idNuevo = regNuevo._id ? regNuevo._id.toString() : null;
-
-        if (idNuevo && registrosExistentesMap.has(idNuevo)) {
-            const regExistente = registrosExistentesMap.get(idNuevo);
-
+function mergeRegistrosPreservandoAuditoria(registrosExistentes, registrosNuevos) {
+    if (!registrosExistentes || registrosExistentes.length === 0) { return registrosNuevos; }
+    if (!registrosNuevos || registrosNuevos.length === 0) { return registrosExistentes; }
+    const existentesMap = new Map(registrosExistentes.filter(r => r._id).map(r => [r._id.toString(), r]));
+    return registrosNuevos.map(regNuevo => {
+        const idNuevo = regNuevo._id?.toString() || regNuevo.id?.toString();
+        const regExistente: any = existentesMap.get(idNuevo);
+        if (regExistente) {
             const registroMergeado = {
                 ...regNuevo,
                 createdAt: regExistente.createdAt,
                 createdBy: regExistente.createdBy,
+                updatedAt: regExistente.updatedAt || regExistente.createdAt,
+                updatedBy: regExistente.updatedBy || regExistente.createdBy
             };
-
-            if (regNuevo.registros && regNuevo.registros.length > 0) {
+            if (regNuevo.registros?.length && regExistente.registros?.length) {
                 registroMergeado.registros = mergeRegistrosPreservandoAuditoria(
-                    regExistente.registros || [],
+                    regExistente.registros,
                     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);
-                    }
-                });
-
-                regNuevo.registros = mergeRegistrosPreservandoAuditoria(
-                    registrosAnidadosExistentes,
-                    regNuevo.registros
-                );
-            }
-
-            return regNuevo;
         }
+        return regNuevo;
     });
-
-    return registrosMergeados;
 }
 
+
 /**
  * Actualiza la lista de profesionales que han registrado en esta prestación
  *
  * @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 +708,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 +757,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 +844,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,