Cancelación con substitución

cambios para la cancelación y substitucion de facturas y pagos

@OpenPymeMx

#116 (closed)

l10n_mx_ir_attachment_facturae/models/base_cfdi_mixin.py

         "related_cfdi_id",
         "original_cfdi_id",
         "Refund invoices",
+        compute="_compute_related_cfdi_ids",

l10n_mx_ir_attachment_facturae/models/base_cfdi_mixin.py

         "related_cfdi_id",
         "original_cfdi_id",
         "Refund invoices",
+        compute="_compute_related_cfdi_ids",
         readonly=True,
         copy=False,
         help="Original CFDI to which this CFDI is referred to",
     )
 
+    @api.multi
+    def _compute_related_cfdi_ids(self):
+        for record in self:
+            cfdi_related = self.env["cfdi.related"].search(
+                [
+                    ("related_cfdi_id", "=", record.id),
+                    ("type_attachment", "=", self._name),
+                ]
+            )
+            self.update({"related_cfdi_ids": cfdi_related.original_cfdi_id})
+
     @api.one
     def create_cfdi(self):
         """Creates a new cfdi object related with current record"""
         """ Cancels the cfdi related with current record
         return True if cancel proceed or false if must wait
         """
+        res = []
         if self.cfdi_id and self.cfdi_id.state in ["signed", "done", "waiting"]:
-            return self.cfdi_id.action_cancel()
+            res = self.cfdi_id.action_cancel()
+        return res
 
     def consult_cfdi_cancellation_status(self):
         """Consult the status cancel of the cfdi related"""
+        res = []
         if self.cfdi_id and self.cfdi_id.state in ["waiting", "cancel"]:
-            return self.cfdi_id.action_consult_cancellation_status()
+            res =  self.cfdi_id.action_consult_cancellation_status()
+        return res
+
+@api.multi
+    def _cancel_cfdi(self):
+        """General steps needed for cancel a voucher"""
+        # self._origin_cfdi_search()
+        self.ensure_one()
+        # Before cancel original voucher set relation to current related CFDI
+        self.write(
+            {"cfdi_relation_type": self.env.ref("l10n_mx.cfdi_relation_type_04").id}
+        )
+        self.env["cfdi.related"].search(
+            [("related_cfdi_id", "=", self.id), ("type_attachment", "=", self._name)]
+        ).unlink()
+        self.env["cfdi.related"].create(
+            {
+                "related_cfdi_id": self.id,
+                "original_cfdi_id": self.cfdi_id.id,
+                "type_attachment": self._name,
+            }
+        )
+        # We also remove relation for attachments to avoid confusing situations
+        self.cfdi_id.file_xml_sign.res_id = None
+        if self.cfdi_id.file_pdf:
+            self.cfdi_id.file_pdf.res_id = None
+        if self._name == "account.voucher":
+            self.cancel_voucher()
+        else:
+            self.invoice_cancel()

[Agustín Cruz]

Pregunta, probaste usar un dominio en vez de esta función compute¿? Me da la impresión de que con algo como esto podría funcionar:

    related_cfdi_ids = fields.Many2many(
        "ir.attachment.facturae.mx",
        "cfdis_related_rel",
        "related_cfdi_id",
        "original_cfdi_id",
        "Refund invoices",
        domain="[('type_attachment', '=', self._name)]",
        readonly=True,
        copy=False,
        help="Original CFDI to which this CFDI is referred to",
    )

con el parámetro domain se filtrarían los documentos para que no te regrese todos y con eso evitar la función compute.

¿Cómo ves?

[Sotero Toribio Campistrano]

@agb80 Estuve checando esta situación pero el self al ponerlo en el dominio como me lo muestras aparece que no esta disponible.

Encontré una documentación donde se puede hacer una función que genere el dominio (http://learnopenerp.blogspot.com/2018/12/add-domain-on-many2many-field-in-odoo.html), pero de todos modos crearía una función. Entonces creo mejor dejamos la función compute.

O me falta poner algo mas para ocupar el self de la manera que me muestras...??

[Agustín Cruz]

@sotero.toribio tienes razón, como se está poniendo entre comillas, se asume que es un texto plano el que se está pasando y por eso no evalúa el self como variable.

Habría que probar el ponerlo de esta manera: domain=[('type_attachment', '=', self._name)], quitando las comillas, o incluso directamente como función así como lo comentas.

También puede ser que se deje la función compute, ahí te dejo a ti que elijas de que fomra te parece mejor que quede implementado.

Solamente comentarte de que que, en caso de que elijas usar la función compute, creo que convendría quitar esta parte de la función _cancel_cfdi:

        self.env["cfdi.related"].create(
            {
                "related_cfdi_id": self.id,
                "original_cfdi_id": self.cfdi_id.id,
                "type_attachment": self._name,
            }
        )

porque me parece que es redundante o, dicho de otra manera, ¿para qué tenemos una función compute si vamos a escribir directamente el valor de los registros en base de datos?

[Sotero Toribio Campistrano]

@agb80 Te comento que es mejor dejar la función compute, ya que estuve checando el dominio y la función para agregarle el dominio y no salio como se esperaba no encontré la razón de por que me seguía mandando los 2 registros en la variable related_cfdi_ids.

self.env["cfdi.related"].create(
            {
                "related_cfdi_id": self.id,
                "original_cfdi_id": self.cfdi_id.id,
                "type_attachment": self._name,
            }
        )

Sobre este create también lo deje por que al usar la función compute no encuentra ningún registro y no le asigna nada a la variable related_cfdi_ids

@api.multi
    def _compute_related_cfdi_ids(self):
        for record in self:
            cfdi_related = self.env["cfdi.related"].search(
                [
                    ("related_cfdi_id", "=", record.id),
                    ("type_attachment", "=", record._name),
                ]
            )
            record.update({"related_cfdi_ids": cfdi_related.original_cfdi_id})

es mejor dejar las 2 funciones como están por que a qui solo actualiza el valor de related_cfdi_ids

[Agustín Cruz]

@sotero.toribio un rebase por favor!!

[Sotero Toribio Campistrano]

@agb80 Listo