IN - Eliminar ingresos duplicados

[Fabio-Ramirez]

Requerimiento

https://proyectos.andes.gob.ar/browse/IN-598

Funcionalidad desarrollada

1. El script elimina las internaciones con egresos duplicados que quedaron en base.
2. En caso de egreso duplicado, se elige el egreso con la fecha mas reciente.
3. NOTA colocar como parametros fecha requerida, por ejemplo: node jobs/manual.js scripts/IN-598.js 2021-11-31 2021-12-26

UserStories llegó a completarse

• Si
• No

Requiere actualizaciones en la base de datos

• Si
• No

[MarianoCampetella]

A modo de comentario, en la BD tengo cargado un egreso duplicado con fecha del 06/11/2024 pero al momento de ejecutar dentro de mi terminar node jobs/manual.js scripts/IN-598.js 2024-11-01 2024-11-31 no me lo modifica. Si a nadie le sucede lo mismo entonces el PR se encontraría aprobado.

[Fabio-Ramirez]

Compa, utilizando la BD de Demo, no encuentro la internación en la fecha que mencionas, en todo caso si me podés compartir el registro que utilizaste, quedo atento.

[Fabio-Ramirez]

A modo de comentario, en la BD tengo cargado un egreso duplicado con fecha del 06/11/2024 pero al momento de ejecutar dentro de mi terminar node jobs/manual.js scripts/IN-598.js 2024-11-01 2024-11-31 no me lo modifica. Si a nadie le sucede lo mismo entonces el PR se encontraría aprobado.
Compa, utilizando la BD de Demo, no encuentro la internación en la fecha que mencionas, en todo caso si me podés compartir el registro que utilizaste, quedo atento.

[MarianoCampetella]

egresoDuplicado.txt
Fabi ahi te dejo un ejemplo donde pongo el comando que dejaste en la descripcion: node jobs/manual.js scripts/IN-598.js 2024-11-05 2024-11-09 y no me esta borrando el egreso duplicado. Cualquier cosa escribime en privado o si alguien mas lo prueba y le funciona entonces estaría bien!

[Fabio-Ramirez]

egresoDuplicado.txt Fabi ahi te dejo un ejemplo donde pongo el comando que dejaste en la descripcion: node jobs/manual.js scripts/IN-598.js 2024-11-05 2024-11-09 y no me esta borrando el egreso duplicado. Cualquier cosa escribime en privado o si alguien mas lo prueba y le funciona entonces estaría bien!

Compa, gracias por la observación, tenias toda la razón, en este ejemplo en particular q enviaste, sucedía que no estaba contamplando el caso de egresos duplicados que además de la misma fecha, tengan el mismo horario. Además, me sirvio para agregar una modificación donde el script abarque todo el dia que ingresa en el parametro de "fecha fin".

[negro89]

Fabi, estan bien detectados los movimientos duplicados pero dejo un par de sugerencias en el código. Además recordar que el movimiento que debería prevalecer es el que tiene misma fecha que la prestación. Con respecto a esto ultimo..

1. La prestación debería existir, de lo contrario hay un error.
2. Si ninguno de los dos movimientos coincide con la prestación, modificar alguno para que coincida.

❗IMPORTANTE: Al recorrer los estados/movimientos buscando duplicados, tener en cuenta que pueden existir anulados (deletedAt: Date) que deben ser omitidos.

[negro89]

Tanto para start como para end la inicialización la dejaría como moment(paramInicio).startOf('day').toDate() y moment(paramFin).endOf('day').toDate() respectivamente. Y para validar los parametros, previo a la instanciación de las fechas sugiero..

if(!moment(paramInicio, 'YYYY-MM-DD', true).isValid() || !moment(paramFin, 'YYYY-MM-DD', true).isValid()) {
     done();
}

[Fabio-Ramirez]

agregado

[negro89]

Con respecto a la comparación entre fechas, sugiero usar las funciones de moment para hacer el codigo mas conciso:
const fecha = moment(estado.fecha);
.
.
.
const fechaConvertida = moment(e.fecha);
return (fechaConvertida.isSame(fecha) && ...

[Fabio-Ramirez]

Agregado.