Análisis de extractos bancarios: errores encontrados y corregidos

Qué pasaba: un crédito de $ 1.000 y otro de U$S 100 daban un total de $ 1.100. Afectaba a las cuentas en dólares de Macro, Nación y Patagonia.

Arreglo: cada movimiento lleva su moneda. Los totales son solo en pesos; los dólares van aparte, con un aviso en pantalla, un bloque propio en el Excel y la columna «Moneda».

Arreglo: la moneda se toma del encabezado CUENTA: de la exportación.

Arreglo: se agrupa también por moneda y los montos en dólares se marcan con (U$S).

Qué pasaba: 1234,56 o 1234,56- no coincidían con el patrón de importe. En «Últimos movimientos» de Macro se perdía el movimiento entero.

Arreglo: el patrón acepta importes con y sin separador de miles.

Qué pasaba: si faltaban un débito y un crédito del mismo importe, se compensaban y el saldo cerraba igual, sin aviso.

Arreglo: también se comparan los créditos y los débitos leídos con los totales que imprime el banco (con y sin la columna de IVA).

Qué pasaba: la detección se guardaba por nombre de archivo; dos extracto.pdf se pisaban y el mismo período se sumaba dos veces sin aviso.

Arreglo: se compara cada archivo por su posición en el lote.

Qué pasaba: la clave se recortaba a 300 caracteres al guardarla, pero se buscaba entera. La app decía «guardado» y el concepto seguía sin clasificar.

Arreglo: la clave se guarda entera (hasta 2000 caracteres).

Qué pasaba: al cambiar de plan, si no se podía cancelar la suscripción anterior, el error se ignoraba y se creaba otra. Quedaban dos débitos y la vieja sin seguimiento.

Arreglo: si la anterior no queda cancelada, no se crea la nueva y se le avisa al usuario.

Qué pasaba: el débito seguía activo y la página de suscripción ya no le mostraba cómo cancelarlo.

Arreglo: primero se cancela el débito; si Mercado Pago no lo cancela, el cambio no se hace.

Qué pasaba: comparaba contra el usuario leído antes de consultar a Mercado Pago. Si mientras tanto el administrador regalaba días, la respuesta vieja los borraba.

Arreglo: el vencimiento solo se alarga, con la condición dentro del UPDATE.

Arreglo: los puestos contratados se aplican solo cuando la suscripción pasa a autorizada, no en cada revisión.

Arreglo: todo lo que escribe la sincronización queda atado al id de la suscripción consultada.

Qué pasaba: si fallaba entre una escritura y la otra, la suscripción quedaba autorizada con los puestos viejos y nunca se corregía.

Arreglo: estado y puestos van en la misma transacción.

Qué pasaba: /mp/aviso respondía 200 aunque hubiera fallado. Mercado Pago no reintentaba y el pago faltaba para siempre en el historial.

Arreglo: ante un error responde 500 y Mercado Pago lo vuelve a mandar.

Qué pasaba: el contenedor corría en UTC y los vencimientos son fechas de Argentina.

Arreglo: TZ=America/Argentina/Buenos_Aires en el contenedor.

Qué pasaba: el acceso y el panel solo miraban si era administrador, antes de mirar el bloqueo.

Arreglo: el bloqueo se revisa primero, también para administradores.

Qué pasaba: un registro con el mismo email, hecho justo cuando el dueño verificaba, reemplazaba la contraseña de una cuenta ya activa y le daba sesión al otro.

Arreglo: el reemplazo solo se aplica si la cuenta sigue sin verificar (condición en el UPDATE).

Qué pasaba: la cuenta quedaba activada con la contraseña del otro.

Arreglo: reemplazar un registro invalida el código, y la verificación exige que la contraseña y el código sigan siendo los leídos.

Qué pasaba: servía para mandar códigos sin límite a una casilla ajena.

Arreglo: la espera de un minuto vale también al registrarse.

Arreglo: el envío se reserva con un UPDATE condicional antes de mandar el correo.

Qué pasaba: con 3 puestos y 2 en uso, dos computadoras nuevas entrando a la vez veían el mismo lugar libre y quedaban registradas las dos.

Arreglo: contar y registrar es una sola operación, con un candado del proceso (la app corre con gunicorn -w 1).

Arreglo: la reducción usa el mismo candado que el registro.

Qué pasaba: confirmando dos enlaces a la vez, los dos veían una liberación disponible.

Arreglo: usar el enlace, contar y liberar van juntos, y un enlace ya usado no vale dos veces.

Qué pasaba: los análisis se guardan en un diccionario compartido por los 4 hilos. Si uno lo limpiaba mientras otro agregaba, Python cortaba con «dictionary changed size during iteration».

Arreglo: todo acceso a los análisis pasa por un candado.

Qué pasaba: los dos pedidos reclasificaban los mismos movimientos al mismo tiempo, así que un total podía salir con una regla y otro con la anterior.

Arreglo: clasificar y totalizar se hace con el mismo candado.

Arreglo: el análisis se valida y se usa sin soltar el candado; si venció, se muestra «El análisis expiró».