Skip to content

[FIX] Crear lecturas ficticias según los periodos de la tarifa en vez de los periodos facturados#294

Merged
eberloso merged 4 commits intomasterfrom
fix_fake_readings_creation
Jan 23, 2026
Merged

[FIX] Crear lecturas ficticias según los periodos de la tarifa en vez de los periodos facturados#294
eberloso merged 4 commits intomasterfrom
fix_fake_readings_creation

Conversation

@MarcCalvoC
Copy link
Contributor

@MarcCalvoC MarcCalvoC commented Jan 15, 2026

Objetivos

Si hay que crear lecturas ficticias, crearlas para todos los periodos de la tarifa según el tipo.

Comportamiento antiguo

Se creaban tantas lecturas ficticias como periodos facturados por la distribuidora:

Por ejemplo:

Si habia excedentes en P1, P2, P3: Se creaban 3 lecturas ficticias, y posteriormente se aplicaban los ajustes correspondientes.

Si habia excedentes en P1, P3: Se creaban solo 2 lecturas ficticias (P1, P2), provocando que posteriormente no se aplicasen los ajustes correspondientes en P3.

Comportamiento nuevo

Se crean las lecturas ficticias según los periodos de la tarifa:

Por ejemplo: una 2.0TD, tiene 3 periodos de excedentes. Aunque no haya excedentes en algún periodo, se crearan las lecturas a 0 de todos los periodos, y para los periodos que no haya excedentes no se aplicará ningún ajuste.

Se dan de alta dos métodos nuevos en la clase FacturaATR:

  • get_num_periodes_from_tarifa_and_tipus(): Devuelve el número de periodos según la tarifa ATR y el tipo de lectura ('A', 'S', 'R', ...)
  • get_periodes_from_tarifa_and_tipus(): Devuelve un diccionario donde las claves son los periodos (P1, P2, ...), y los valores son listas vacías.

@MarcCalvoC MarcCalvoC self-assigned this Jan 15, 2026
@MarcCalvoC MarcCalvoC requested a review from eberloso January 15, 2026 10:32
@MarcCalvoC MarcCalvoC added the bug label Jan 15, 2026
@MarcCalvoC MarcCalvoC changed the title fix: Crear lecturas fake según los periodos de la tarifa en vez de los periodos facturados [FIX] Crear lecturas ficticias según los periodos de la tarifa en vez de los periodos facturados Jan 15, 2026
eberloso
eberloso previously approved these changes Jan 15, 2026
@eberloso eberloso added the patch label Jan 23, 2026
@eberloso eberloso merged commit 8548f15 into master Jan 23, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants