[FIX] l10n_es_vat_book: Missed copy dictionary method #1223
Conversation
|
Supongo que es para solucionar #1221? |
|
Pon por favor en el mensaje del commit los pasos para reproducir el problema y por qué esto es la solución |
|
Sí, pero bueno, un copy cuando se va a utilizar un diccionario más de una vez siempre va bien 😄 |
|
Ya digo que no lo he puesto directamente relacionado con el problema porque es algo que ya debería haber hecho cuando se van asignar un diccionario de valores iniciales a varias líneas distintas donde pueden variar esos valores. |
|
Ya, pero eso, luego están los commits tan escuetos y se pierde el sentido de las cosas mirando hacia atrás. Haz por favor el esfuerzo. Ejemplo de mensaje de commit: odoo/odoo@4b99272 Y aunque fuera un error de codificación inicial, el caso es que ha surgido a raíz de la publicación del issue, por lo que sí que debe ir vinculado. |
carlosdauden
force-pushed
the
11.0-FIX-l10n_es_vat_book-balance_zero
branch
from
97ab62e
to
9c98f8d
Compare
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
pedrobaeza
force-pushed
the
11.0-FIX-l10n_es_vat_book-balance_zero
branch
from
9c98f8d
to
a1e8233
Compare
|
/ocabot merge patch |
|
What a great day to merge this nice PR. Let's do it! |
|
Cherry-pick para 12 en 107ae0e |
|
Congratulations, your PR was merged at 0b9937c. Thanks a lot for contributing to OCA. ❤️ |
Resuelve este problema #1221
@Tecnativa