[IMP] l10n_ar_tax: agregado de etiquetas y secuencias a impuestos de 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.

Author: pablohmontenegro

l10n_ar_tax/models/account_chart_template.py

@@ -71,9 +71,9 @@ def _add_wh_taxes(self, company):
         ]
         for tax_ref, state_ref in tax_state_tupples:
             # Identificamos el impuesto al que se le va a agregar la/s etiqueta/s
-            tax = self.env.ref("account.%s_%s" % (company.id, tax_ref), raise_if_not_found=False)
-            if tax:
-                tax.l10n_ar_state_id = self.env.ref(state_ref).id
+            if tax := self.env.ref("account.%s_%s" % (company.id, tax_ref), raise_if_not_found=False):
+                if not tax.l10n_ar_state_id:
+                    tax.l10n_ar_state_id = self.env.ref(state_ref).id
 
         # creacion de secuencias y agregado de etiquetas para liquidación de impuestos
         withholdings_domain = [
@@ -85,31 +85,33 @@ def _add_wh_taxes(self, company):
         non_profits_domain = withholdings_domain + [("l10n_ar_tax_type", "not in", ["earnings", "earnings_scale"])]
 
         for tax in self.env["account.tax"].with_context(active_test=False).search(non_profits_domain):
+            if not tax.l10n_ar_withholding_sequence_id:
+                sequence = self.env["ir.sequence"].create(
+                    {
+                        "name": tax.invoice_label or tax.name,
+                        "prefix": "%(year)s-",
+                        "padding": 8,
+                        "number_increment": 1,
+                        "implementation": "standard",
+                        "company_id": company.id,
+                    }
+                )
+                tax.l10n_ar_withholding_sequence_id = sequence.id
+
+        profits_domain = withholdings_domain + [("l10n_ar_tax_type", "in", ["earnings", "earnings_scale"])]
+        profits_taxes = self.env["account.tax"].with_context(active_test=False).search(profits_domain)
+        if not profits_taxes.mapped("l10n_ar_withholding_sequence_id"):
             sequence = self.env["ir.sequence"].create(
                 {
-                    "name": tax.invoice_label or tax.name,
+                    "name": "Retención de Ganancias",
                     "prefix": "%(year)s-",
                     "padding": 8,
                     "number_increment": 1,
                     "implementation": "standard",
                     "company_id": company.id,
                 }
             )
-            tax.l10n_ar_withholding_sequence_id = sequence.id
-
-        profits_domain = withholdings_domain + [("l10n_ar_tax_type", "in", ["earnings", "earnings_scale"])]
-        sequence = self.env["ir.sequence"].create(
-            {
-                "name": "Retención de Ganancias",
-                "prefix": "%(year)s-",
-                "padding": 8,
-                "number_increment": 1,
-                "implementation": "standard",
-                "company_id": company.id,
-            }
-        )
-        profits_taxes = self.env["account.tax"].with_context(active_test=False).search(profits_domain)
-        profits_taxes.l10n_ar_withholding_sequence_id = sequence.id
+            profits_taxes.l10n_ar_withholding_sequence_id = sequence.id
 
         # agregado de etiquetas para liquidacion de impuestos sicore
         sicore_taxes = profits_taxes
@@ -121,9 +123,11 @@ def _add_wh_taxes(self, company):
                 tax = self.env.ref(xml_id_percep, raise_if_not_found=False)
                 if tax:
                     sicore_taxes += tax
-            self.env["account.tax.repartition.line"].search(
+            sicore_tags = self.env["account.tax.repartition.line"].search(
                 [("tax_id", "in", sicore_taxes.ids), ("repartition_type", "=", "tax")]
-            ).tag_ids = [Command.link(tag.id)]
+            )
+            sicore_tags_to_update = sicore_tags.filtered(lambda line: tag not in line.tag_ids)
+            sicore_tags_to_update.tag_ids = [Command.link(tag.id)]
 
         # agregado de etiquetas para liquidacion de impuestos pago IIBB a cuenta (sifere web)
         # consideramos de IIBB a todo lo que tiene 10n_ar_state_id
@@ -140,7 +144,9 @@ def _add_wh_taxes(self, company):
                 ("tax_id.type_tax_use", "=", "none"),
                 ("tax_id.l10n_ar_withholding_payment_type", "=", "customer"),
             ]
-            self.env["account.tax.repartition.line"].search(domain).tag_ids = [Command.link(tag.id)]
+            repartition_lines = self.env["account.tax.repartition.line"].search(domain)
+            if repartition_lines_without_tag := repartition_lines.filtered(lambda line: tag not in line.tag_ids):
+                repartition_lines_without_tag.tag_ids = [Command.link(tag.id)]
 
     def _load(self, template_code, company, install_demo, force_create=True):
         """Luego de que creen los impuestos del archivo account.tax-ar_ri.csv de l10n_ar al instalar el plan de cuentas en la nueva compañìa argentina agregamos en este método las etiquetas que correspondan en los repartition lines."""