GitLab

changed milestone to %Cancelación con substitución

added Feature label

changed the description

added 1 commit

65350991 - feat(cfdi.related): nuevo objeto para la relación

Compare with previous version

added 1 commit

e4618d8a - feat(cfdi.related): nuevo objeto para la relación

Compare with previous version

added 1 commit

a47ad483 - feat(base.cfdi.mixin): función para la cancelación y substitución

Compare with previous version

added 28 commits

c6e9572b - feat(res.company): se agregan los campos necesarios para cargar la FIEL
f1611f30 - feat(account.config.settings): se crea el modelo account.config.settings
1506ffcc - feat(account.config.settings): se elimina vista y codigo que se agrego en sus respectivos archivos
fe7a84dd - feat(account.config.settings): se agrega grupo de seguridad
4ed825ae - feat(__openerp__): se hacen los cambios necesarios para visualizar la configuracion de la FIEL
7bd5b5c3 - refactor(res.company.facturae.certificate): se agrega al modulo `l10n_mx_ir_attachment_facturae`
be3cceed - refactor(res.company): se fusiona res.company del modulo...
1f2bc99b - refactor(demo): se agrega los xml con la informacion demo
98d6af01 - refactor(security): se fusionan los archivos de seguridad
abbd419c - refactor(l10n_mx_ir_attachment_facturae): se cargan los archivos fusionados
2a17a9b5 - feat(lib): se agrega archivo de utilidad
c6d75b22 - feat(PkiUtils): se crean las funciones para generar la codificacion PEM de los...
26b9a554 - refactor(res.company.facturae.certificate): se refactoriza la funcion...
5ba2e73d - feat(res.company): se agregan los campos en donde se almacenara la key y el cer en formato PEM
5e07f795 - feat(account.config.settings): se agregan los campos relacionados para los archivos en formato PEM
a6276ddf - feat(account.config.settings): se agregan los campos `fiel_cer_pem` y `fiel_key_pem` a la vista
a8157399 - refactor(account.config.settings): eliminamos comentario
fc1599a9 - refactor(l10n_mx_ir_attachment_facturae): code-analysis
318fa8f6 - style(l10n_mx_facturae_mx): code-analysis
47ac4d59 - style(l10n_mx_ir_attachment_facturae): lint-analysis
ae0ad89a - refactor(base.cfdi): se corrige la forma de tratar con R1710(inconsistent-return-statements)
37aebc03 - feat(account.config.settings): modificación en vista
fe65d49a - feat(res.company.facturae.fiel): agregar nuevo objeto para los datos de la FIEL
8e153176 - feat(res.company.facturae.certificate): modifico en la forma de validar certificados de la compañia
bd0deaaf - feat(account.config.settings): modificar la vizualización para FIEL
e2d9e89a - feat(i18n): cargar traducción
46605f55 - feat(cfdi.related): nuevo objeto para la relación
2203eec0 - feat(base.cfdi.mixin): función para la cancelación y substitución

Compare with previous version

unmarked as a Work In Progress

added 2 commits

e9c9efdf - feat(cfdi.related): nuevo objeto para la relación
0d7cf7f4 - feat(base.cfdi.mixin): función para la cancelación y substitución

Compare with previous version

added 1 commit

eaa59c0e - feat(base.cfdi.mixin): función para la cancelación y substitución

Compare with previous version

resolved all threads

added 2 commits

8ce94436 - feat(cfdi.related): nuevo objeto para la relación
3d583cfb - feat(base.cfdi.mixin): función para la cancelación y substitución

Compare with previous version

added 1 commit

38a21773 - feat(base.cfdi.mixin): función para la cancelación y substitución

Compare with previous version

added 1 commit

51f07ee1 - feat(base.cfdi.mixin): función para la cancelación y substitución

Compare with previous version

added 1 commit

db86c0b7 - feat(base.cfdi.mixin): función para la cancelación y substitución

Compare with previous version

added 9 commits

db86c0b7...87bce6b3 - 7 commits from branch l10n_mx:master
441441f2 - feat(cfdi.related): nuevo objeto para la relación
b2c70d79 - feat(base.cfdi.mixin): función para la cancelación y substitución

Compare with previous version

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?

@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...??

@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?

@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})