Skip to content

Añadir BalanceRecalculator: recálculo síncrono de saldos de un ejercicio - #2010

Open
lanzalibre wants to merge 1 commit into
NeoRazorX:masterfrom
lanzalibre:feature/balance-recalculator
Open

Añadir BalanceRecalculator: recálculo síncrono de saldos de un ejercicio#2010
lanzalibre wants to merge 1 commit into
NeoRazorX:masterfrom
lanzalibre:feature/balance-recalculator

Conversation

@lanzalibre

@lanzalibre lanzalibre commented Jul 16, 2026

Copy link
Copy Markdown

Descripción

Añade Core/Lib/Accounting/BalanceRecalculator: recálculo síncrono de los saldos en caché de todas las subcuentas y cuentas de un ejercicio a partir de las partidas, en una sola llamada:

BalanceRecalculator::run($codejercicio);

Motivación: los workers PartidaWorker/CuentaWorker recalculan los saldos de forma asíncrona (WorkQueue) y por subcuenta. Las importaciones masivas y los scripts CLI no procesan la WorkQueue, por lo que los saldos en caché quedan desactualizados y ListCuenta muestra saldos a cero o incorrectos. Esta clase permite reconstruirlos todos de golpe, con la misma tolerancia (0.009) y redondeo (Tools::round) que los workers, y acumulando las cuentas hijas en sus cuentas padre igual que CuentaWorker.

Solo se escriben las filas con diferencias, y el SQL es portable (MySQL/MariaDB/PostgreSQL).

Tests unitarios incluidos (Test/Core/Lib/BalanceRecalculatorTest.php)

  1. testRecalculateSubaccountBalances — crea un asiento con dos partidas, corrompe los saldos en caché de las subcuentas por SQL, ejecuta el recálculo y comprueba que debe/haber/saldo vuelven a coincidir con las partidas; después borra el asiento y comprueba que el recálculo las deja a cero.
  2. testRecalculateAccountBalances — corrompe el saldo de una cuenta y comprueba que tras el recálculo la cuenta suma exactamente sus subcuentas y cuentas hijas.
  3. testUnusedSubaccountIsZeroed — pone un saldo falso a una subcuenta sin partidas y comprueba que el recálculo la deja a cero.

¿Cómo has probado los cambios?

  • He revisado mi código antes de enviarlo.
  • He probado que funciona correctamente en mi PC.
  • He probado que funciona correctamente con una base de datos vacía.
  • He ejecutado los tests unitarios (los 3 nuevos pasan; phpcs sin avisos).

Recalcula los saldos en caché de subcuentas y cuentas a partir de las
partidas, en una sola llamada. Complementa a PartidaWorker/CuentaWorker
para importaciones masivas y scripts CLI donde la WorkQueue no se procesa.
@lanzalibre
lanzalibre force-pushed the feature/balance-recalculator branch from 760ce34 to e678a54 Compare July 16, 2026 18:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant