[IMP] l10n_ar_tax: agregado de etiquetas y secuencias a impuestos de ret ganancias.#1326
Open
pablohmontenegro wants to merge 1 commit intoingadhoc:18.0from
Open
[IMP] l10n_ar_tax: agregado de etiquetas y secuencias a impuestos de ret ganancias.#1326pablohmontenegro wants to merge 1 commit intoingadhoc:18.0from
pablohmontenegro wants to merge 1 commit intoingadhoc:18.0from
Conversation
Contributor
|
|
There was a problem hiding this comment.
Pull request overview
El PR ajusta la inicialización de secuencias y etiquetas de impuestos de retenciones para evitar que, en escenarios de migración a Odoo 18, el hook vuelva a asignar (y potencialmente pise) configuración ya creada por la línea de upgrade de retenciones de ganancias.
Changes:
- Condiciona la creación/asignación de la secuencia de retenciones para impuestos de ganancias según si ya existe una secuencia asignada.
- Evita reasignar etiquetas SICORE si ya hay tags presentes en las repartition lines objetivo.
pablohmontenegro
force-pushed
the
18.0-h-111529-pam
branch
from
d279b1d to
9cb1246
Compare
pablohmontenegro
force-pushed
the
18.0-h-111529-pam
branch
from
9cb1246 to
7b2f740
Compare
pablohmontenegro
force-pushed
the
18.0-h-111529-pam
branch
3 times, most recently
from
8151378 to
cfa46ff
Compare
…ret ganancias. Ticket: 111529 El hook _l10n_ar_update_taxes NO se termina corriendo para bases que migran a 18 (por más que sea un módulo nuevo no se termina corriendo porque el un módulo renombrado), por lo que creamos el ul "[RET18] Etiquetas y secuencias en impuestos" (id: 2241) para que se termine corriendo. Pero el hook termina asignando secuencia y etiquetas a los impuestos de retención de ganancias y no queremos que eso suceda cuando se trata de una migración porque el upgrade line "✏️[RET18] Migración retenciones de Ganancias" (id: 1415) crea los impuestos de retención de ganancias con sus respectivas secuencias y etiquetas entonces por eso tenemos que hacer el ajuste de este commit para que no se vuelva a pisar la secuencia de retención de ganancias y las etiquetas.
pablohmontenegro
force-pushed
the
18.0-h-111529-pam
branch
from
cfa46ff to
b93b12f
Compare
Comment on lines
+116
to
+117
| for prof_tax in profits_taxes.filtered(lambda tax: not tax.l10n_ar_withholding_sequence_id): | ||
| prof_tax.l10n_ar_withholding_sequence_id = sequence.id |
There was a problem hiding this comment.
Acá se escribe la secuencia por impuesto en un bucle, lo que genera múltiples writes cuando podría hacerse en una sola operación sobre el recordset filtrado. Para mejorar performance y reducir trazabilidad/locks, asignar el sequence.id directamente al recordset de impuestos sin secuencia (sin iterar).
Suggested change
| for prof_tax in profits_taxes.filtered(lambda tax: not tax.l10n_ar_withholding_sequence_id): | |
| prof_tax.l10n_ar_withholding_sequence_id = sequence.id | |
| profits_taxes.filtered(lambda tax: not tax.l10n_ar_withholding_sequence_id).l10n_ar_withholding_sequence_id = sequence.id |
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
El hook _l10n_ar_update_taxes NO se termina corriendo para bases que migran a 18 (por más que sea un módulo nuevo no se termina corriendo porque el un módulo renombrado), por lo que creamos el ul "[RET18] Etiquetas y secuencias en impuestos" (id: 2241) para que se termine corriendo. Pero el hook termina asignando secuencia y etiquetas a los impuestos de retención de ganancias y no queremos que eso suceda cuando se trata de una migración porque el upgrade line "✏️[RET18] Migración retenciones de Ganancias" (id: 1415) crea los impuestos de retención de ganancias con sus respectivas secuencias y etiquetas entonces por eso tenemos que hacer el ajuste de este commit para que no se vuelva a pisar la secuencia de retención de ganancias y las etiquetas.
Ticket: 111529