Skip to content

[19.0][MIG] l10n_es_vat_book: Migration to 19.0 - #4884

Closed
AlvaroRamirez10 wants to merge 117 commits into
OCA:19.0from
AlvaroRamirez10:19.0-mig-l10n_es_vat_book
Closed

[19.0][MIG] l10n_es_vat_book: Migration to 19.0#4884
AlvaroRamirez10 wants to merge 117 commits into
OCA:19.0from
AlvaroRamirez10:19.0-mig-l10n_es_vat_book

Conversation

@AlvaroRamirez10

Copy link
Copy Markdown
Contributor

Migration of l10n_es_vat_book module from 18.0 to 19.0

JordiBForgeFlow and others added 30 commits March 4, 2026 11:30
facturas "normales" como rectificativas.
* Translations
* Terms homogeneization
* README by fragments
* Test reports
When a large amount of move lines are processed (hundreds of thousands)
it can lead to a MemoryError when we try to calculate the VAT book
Steps to reproduce the problem:

- Create an invoice with 0 amount, but any line.
- Put in that line a tax "Recargo equivalencia X%".
- Create a VAT book in the interval of the invoice.
- Compute the book
- We get the error

File "/home/odoo/build/OCA/l10n-spain/l10n_es_vat_book/models/l10n_es_vat_book.py", line 265, in upsert_book_line_tax
lambda l: not l['special_tax_group'], implied_lines))
StopIteration

This is due to a reused dictionary in the loop, so we perform a copy of the original dictionary each time.

Fixes OCA#1221
etobella and others added 13 commits March 4, 2026 11:30
…p line includes an account

In previous changes, if there was a mapping including it, it started to fail.
Now it should work specifically, so, If you have some mapping, the exclusion is applied, otherwise, all lines are added.
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: l10n-spain-18.0/l10n-spain-18.0-l10n_es_vat_book
Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-18-0/l10n-spain-18-0-l10n_es_vat_book/
…it is a harmless tax and does not need to be reflected.
Currently translated at 98.2% (164 of 167 strings)

Translation: l10n-spain-18.0/l10n-spain-18.0-l10n_es_vat_book
Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-18-0/l10n-spain-18-0-l10n_es_vat_book/es/
@pedrobaeza

Copy link
Copy Markdown
Member

/ocabot migration l10n_es_vat_book

@OCA-git-bot OCA-git-bot added this to the 19.0 milestone Mar 9, 2026
@OCA-git-bot OCA-git-bot mentioned this pull request Mar 9, 2026
40 tasks
@EmilioPascual

Copy link
Copy Markdown
Contributor

¿@AlvaroRamirez10 puedes revisar los fallos que están dando pre-commit y los tests?
Gracias

@EmilioPascual

Copy link
Copy Markdown
Contributor

Hola @AlvaroRamirez10, ¿por favor puedes revisar los errores de los tests y el pre-commit para poder empezar a revisarlo? Muchas gracias

@HaraldPanten

Copy link
Copy Markdown
Contributor

En este módulo hay que tener en cuenta la incompatibilidad con los grupos de impuestos. Probablemente sea necesario un replanteamiento de ciertos apartados del mismo.

Más info:

#4652
#4971

@pedrobaeza

Copy link
Copy Markdown
Member

En los OCA days se podría plantear esa refactorización de la que hablaba para tener una fuente común de información fiscal a nivel de l10n_es_aeat, y que de ahí bebieran SII, VERI*FACTU y el libro de IVA. Hay piezas ya puestas, pero faltan algunas otras.

@HaraldPanten

Copy link
Copy Markdown
Contributor

En los OCA days se podría plantear esa refactorización de la que hablaba para tener una fuente común de información fiscal a nivel de l10n_es_aeat, y que de ahí bebieran SII, VERI*FACTU y el libro de IVA. Hay piezas ya puestas, pero faltan algunas otras.

Sería perfecto, sí.

@arch-fan

arch-fan commented Jun 4, 2026

Copy link
Copy Markdown

Al final se debatió esto en los OCA Days?

@pedrobaeza

Copy link
Copy Markdown
Member

No hubo ningún contribuidor allí que se dedicara a esto en concreto.

@rafaelbn

rafaelbn commented Jun 6, 2026

Copy link
Copy Markdown
Member

Hola @AlvaroRamirez10 , ¿Vas a continuar con este PR? Muchas gracias! 😄

@HaraldPanten

Copy link
Copy Markdown
Contributor

Se sigue en este PR, ya que no ha habido seguimiento #5049

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.