GitLab

added Feature label

changed target branch from master to feat_odoo_15

Buenas tardes @jesus.lomas te dejo mis comentarios sobre tu MR:

1. Por favor, no hagas git commit --ammend, eso sólo se hace cuando vas a corregir un cambio no cuando estás haciendo una corrección diferente. La idea es que cada commit tenga una corrección para que sea más fácil hacer el seguimiento y la corrección en el futuro. Recurda que el código siempre va a ser leído más veces de las que va a ser escrito.

2. El mensaje de commit. Lo que va entre paréntesis debe ser el objeto que estas modificando, account.move en este caso, y lo que va en el título no es una descripción de los cambios que estás haciendo sino un resumen de la corrección que estás realizando. Nota que describir los cambios es para la persona que está escribiendo el código, mientras que el resumen de la corrección es para la persona que está leyendo el código. Nuevamente, el código siempre va a ser leído más veces de las que va a ser escrito.

Una sugerencia para este commit sería lo siguiente: feat(account.move): agrega soporte para múltiples impuestos en la factura

added 1 commit

• b24a3058 - feat(cambios a la función impuestos): se realizaron cambios a la función impuestos

Compare with previous version

Buenos días @agb80

Con base a la información que me proporciono en su anterior comentario, la función _get_tax_grouping_key_from_tax_line los parámetros que se le están mandando a la función anteriormente mencionada son:

        for line in self.invoice_line_ids:
            for tax in line.export_invoice_line_for_xml().taxes:
                grouping_dict = self._get_tax_grouping_key_from_tax_line(line)

Es la variable line es una instancia de account.move.line (que es cada una de las líneas de factura (invoice_line_ids). Esto significa que dentro de la función _get_tax_grouping_key_from_tax_line(),lo que tomará como entrada la línea de la factura (de la cual tome los datos relacionados con el impuesto, el producto, el monto, o cualquier otro detalle necesario para generar la clave de agrupación).

added 1 commit

• ff386591 - feat(cambios a la función impuestos): se realizaron cambios a la función impuestos

Compare with previous version

added 1 commit

• f908e8fa - feat(cambios a la función impuestos): se realizaron cambios a la función impuestos

Compare with previous version

added 1 commit

• 96d59338 - feat(cambios a la función impuestos): se realizaron cambios a la función impuestos

Compare with previous version

added 1 commit

• 78b8e7e3 - feat(cambios a la función impuestos): se realizaron cambios a la función impuestos

Compare with previous version

added 3 commits

• 86410088 - 1 commit from branch feat_odoo_15
• 643b031c - feat(se realizan cambios tipocambio): se realizaron cambios al tipo de cambio
• f802d239 - feat(cambios a la función impuestos): se realizaron cambios a la función impuestos

Compare with previous version

added 1 commit

• eaf9bf71 - feat(account.move): agrega soporte para múltiples impuestos en la factura

Compare with previous version

added 1 commit

• bd2a60ce - feat(account.move): agrega soporte para múltiples impuestos en la factura

Compare with previous version

added 1 commit

• 2223a9f8 - feat(account.move): agrega soporte para múltiples impuestos en la factura

Compare with previous version

added 1 commit

• 394b98db - feat(account.move): agrega soporte para múltiples impuestos en la factura

Compare with previous version

added 1 commit

• e4999666 - feat(account.move): agrega soporte para múltiples impuestos en la factura

Compare with previous version

added 1 commit

• 0e563df7 - feat(account.move): agrega soporte para múltiples impuestos en la factura

Compare with previous version

added 1 commit

• 689b788b - feat(account.move): agrega soporte para múltiples impuestos en la factura

Compare with previous version

@jesus.lomas platicamos el lunes en la sesión de mentoría para hacerte mis comentarios, nada más no se te olvide subir este tema por favor.

added 1 commit

• 855e0d56 - feat(account.move): se realizaron cambios al tipo de cambio

Compare with previous version

added 1 commit

• 68942803 - feat(account.move): se realizaron cambios al tipo de cambio

Compare with previous version

added 1 commit

• bec562b3 - feat(account.move): se realizaron cambios al tipo de cambio

Compare with previous version

Buenas tardes @agb80

Le adjunto la captura de pantalla del nombre del grupo del impuesto que estoy tomando es tax_group_id

added 1 commit

• be5f6a37 - feat(account.move): se realizaron cambios al tipo de cambio

Compare with previous version

added 1 commit

• 60c89a24 - feat(account.move): se realizaron cambios al tipo de cambio

Compare with previous version

Buenas tardes @agb80

Le comparto los cambios que realice a la función impuestos con base a la hipótesis que realice, note que el valor de la variable impuesto_monto lo tomaba como negativo, hice el cambio del campo tax_line.balance por tax_line.price_subtotal para que cierto valor fuera positivo y así se pudiera realizar la clasificación correcta del impuesto de igual manera agregue el campo tax_line.tax_ids.name para tomar el nombre del impuesto

        for tax_line in self.line_ids.filtered(lambda line: line.tax_line_id):
            impuesto_monto = round(abs(tax_line.price_subtotal), 2)
            base = round(tax_line.tax_base_amount, 2)
            xml_amount = "{0:.2f}".format(impuesto_monto)

            tax_data = {
                "xml_amount": xml_amount,
                "base": base,
                "tax_name": tax_line.tax_ids.name,
                "tax_repartition_line_id": tax_line.tax_repartition_line_id.id,
                "group_tax_id": tax_line.group_tax_id.id,
                "account_id": tax_line.account_id.id,
                "currency_id": tax_line.currency_id.id,
                "tax_ids": [(6, 0, tax_line.tax_ids.ids)],
                "tax_tag_ids": [(6, 0, tax_line.tax_tag_ids.ids)],
                "partner_id": tax_line.partner_id.id,
            }

            group_name = tax_line.tax_line_id.tax_group_id.name

Pero sigo presentado el mismo problema que lo clasifica como un impuesto local.

Le adjunto la captura de pantalla de los valores que se toman del impuesto

Quedo a la espera de sus comentarios o sugerencias.

@jesus.lomas eso significa que tu hipótesis no era correcta, qué otras hipótesis tienes que puedas probar?

added 1 commit

• 4422c729 - feat(account.move): se realizaron cambios al tipo de cambio

Compare with previous version

Buen día @agb80

La hipótesis que probé fue que note que el importe no se tomaba en la parte donde se crea el diccionario tax_data lo que realice fue agregarlo tax_line.importe al diccionario para tener los datos necesarios para la correcta clasificación de impuesto.

        for tax_line in self.line_ids.filtered(lambda line: line.tax_line_id):
            impuesto_monto = round(abs(tax_line.balance), 2)
            base = round(tax_line.tax_base_amount, 2)

            for tax in tax_line.tax_ids:
                key = (tax.amount / 100, tax.tax_group_id.name)
                tax_data = {
                    "id": tax.id,
                    "name": tax.name,
                    "amount": impuesto_monto,
                    "base": base,
                    "importe": tax_line.importe,
                    "group": tax.tax_group_id.name,
                    "TasaOCuota": "{0:.6f}".format(tax.amount / 100),
                }

Pero sigo presentado el mismo problema que lo clasifica como un impuesto local.

Le adjunto la captura de pantalla de los valores que se toman del impuesto

Quedo a la espera de sus comentarios o sugerencias.

added 2 commits

• 4dce491d - feat(se realizan cambios tipocambio): se realizaron cambios al tipo de cambio
• 070c0696 - feat(cambios a la función impuestos): se realizaron cambios a la función impuestos

Compare with previous version

added 1 commit

• 92074dba - feat(cambios a la función impuestos): se realizaron cambios a la función impuestos

Compare with previous version

added 1 commit

• 9a383724 - feat(cambios a la función impuestos): se realizaron cambios a la función impuestos

Compare with previous version

added 1 commit

• 5dd80b0a - feat(cambios a la función impuestos): se realizaron cambios a la función impuestos

Compare with previous version

added 1 commit

• 3988dd31 - feat(cambios a la función impuestos): se realizaron cambios a la función impuestos

Compare with previous version

@jesus.lomas haz un rebase para dejar solamente 1 commit y cambia el mensaje de esta forma:

fix(account.move): timbra correctamente la factura cuando hay varios impuestos

added 3 commits

• 86410088 - 1 commit from branch feat_odoo_15
• 1ac16a5c - feat(se realizan cambios tipocambio): se realizaron cambios al tipo de cambio
• d24cd76f - feat(cambios a la función impuestos): se realizaron cambios a la función impuestos

Compare with previous version

added 1 commit

• 53a20303 - feat(account.move): timbra correctamente la factura cuando hay varios impuestos

Compare with previous version

closed

reopened

closed

reopened

added 1 commit

• def0ca19 - feat(account.move): genera el XML en una factura con varios impuestos

Compare with previous version

resolved all threads

merged