[FIX] l10n_ar_tax_settlement_backward_comp: backward tax compatibility #881
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
|
|
There was a problem hiding this comment.
Pull request overview
Este PR corrige un bug crítico en la compatibilidad hacia atrás de impuestos argentinos al migrar de versión 16 a 18. El problema ocurría cuando un impuesto tomaba la alícuota del contacto pero no tenía definida una "fecha hasta" (to_date): el sistema no detectaba correctamente el impuesto equivalente en la nueva versión.
Changes:
- Corrección de la condición lógica en el filtrado de impuestos de contacto para evaluar correctamente el campo
to_datecuando esNone
Ticket Adhoc: 108632 Cuando migramos por ejemplo de versión 16 a 18 y hay impuestos que toman alícuota del contacto, si dicho impuesto no tenía "fecha hasta" en el contacto entonces cuando hacemos el backward compatibility (que consiste en buscar el impuesto equivalente en la nueva versión) no se estaba detectando el nuevo impuesto en el contacto en la versión 18, detectaba el impuesto viejo de la versión anterior. Este pr arregla eso. Más info en este video https://drive.google.com/file/d/1ov9S-TPKjda-HgQBsdzmUrClJDwiI_hj/view
pablohmontenegro
force-pushed
the
18.0-h-108632-pam
branch
from
c7834c1 to
8c7b7df
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Ticket Adhoc: 108632
Cuando migramos por ejemplo de versión 16 a 18 y hay impuestos que toman alícuota del contacto, si dicho impuesto no tenía "fecha hasta" en el contacto entonces cuando hacemos el backward compatibility (que consiste en buscar el impuesto equivalente en la nueva versión) no se estaba detectando el nuevo impuesto en el contacto en la versión 18, detectaba el impuesto viejo de la versión anterior. Este pr arregla eso.
Más info en este video: https://drive.google.com/file/d/1ov9S-TPKjda-HgQBsdzmUrClJDwiI_hj/view