From e2c6a37d3207e59c631e8b4686ff9de0f6de9a66 Mon Sep 17 00:00:00 2001
From: "Benjamin Frantzen (bfr)" <bfr@odoo.com>
Date: Thu, 6 Aug 2020 08:51:04 +0000
Subject: [REF] l10n_mx_edi: Refactoring of l10n_mx_edi depending account_edi

closes odoo/odoo#52407

Task: 2247368
Related: odoo/enterprise#12226
Related: odoo/upgrade#1571
Signed-off-by: oco-odoo <oco-odoo@users.noreply.github.com>
---
 addons/l10n_mx/data/account_tax_data.xml | 12 ++++++++++
 addons/l10n_mx/models/__init__.py        |  1 +
 addons/l10n_mx/models/account_tax.py     | 38 ++++++++++++++++++++++++++++++++
 3 files changed, 51 insertions(+)
 create mode 100644 addons/l10n_mx/models/account_tax.py

diff --git a/addons/l10n_mx/data/account_tax_data.xml b/addons/l10n_mx/data/account_tax_data.xml
index 0b3280f6ee3..40be0d8a2a2 100644
--- a/addons/l10n_mx/data/account_tax_data.xml
+++ b/addons/l10n_mx/data/account_tax_data.xml
@@ -71,6 +71,7 @@
         <field name="tax_group_id" ref="tax_group_iva_0"/>
         <field name="tax_exigibility">on_payment</field>
         <field name="cash_basis_transition_account_id" ref="cuenta209_01"/>
+        <field name="l10n_mx_tax_type">Tasa</field>
         <field name="invoice_repartition_line_ids" eval="[(5, 0, 0),
             (0,0, {
                 'factor_percent': 100,
@@ -110,6 +111,7 @@
         <field name="tax_group_id" ref="tax_group_iva_16"/>
         <field name="tax_exigibility">on_payment</field>
         <field name="cash_basis_transition_account_id" ref="cuenta209_01"/>
+        <field name="l10n_mx_tax_type">Tasa</field>
         <field name="invoice_repartition_line_ids" eval="[(5, 0, 0),
             (0,0, {
                 'factor_percent': 100,
@@ -149,6 +151,7 @@
         <field name="tax_group_id" ref="tax_group_iva_ret_4"/>
         <field name="tax_exigibility">on_payment</field>
         <field name="cash_basis_transition_account_id" ref="cuenta216_10"/>
+        <field name="l10n_mx_tax_type">Tasa</field>
         <field name="invoice_repartition_line_ids" eval="[(5, 0, 0),
             (0,0, {
                 'factor_percent': 100,
@@ -188,6 +191,7 @@
         <field name="tax_group_id" ref="tax_group_iva_ret_10"/>
         <field name="tax_exigibility">on_payment</field>
         <field name="cash_basis_transition_account_id" ref="cuenta216_10"/>
+        <field name="l10n_mx_tax_type">Tasa</field>
         <field name="invoice_repartition_line_ids" eval="[(5, 0, 0),
             (0,0, {
                 'factor_percent': 100,
@@ -225,6 +229,7 @@
         <field name="amount_type">percent</field>
         <field name="type_tax_use">purchase</field>
         <field name="tax_group_id" ref="tax_group_isr_ret_10"/>
+        <field name="l10n_mx_tax_type">Tasa</field>
         <field name="invoice_repartition_line_ids" eval="[(5, 0, 0),
             (0,0, {
                 'factor_percent': 100,
@@ -260,6 +265,7 @@
         <field name="amount_type">percent</field>
         <field name="type_tax_use">purchase</field>
         <field name="tax_group_id" ref="tax_group_isr_ret_10"/>
+        <field name="l10n_mx_tax_type">Tasa</field>
         <field name="invoice_repartition_line_ids" eval="[(5, 0, 0),
             (0,0, {
                 'factor_percent': 100,
@@ -297,6 +303,7 @@
         <field name="tax_group_id" ref="tax_group_iva_ret_1067"/>
         <field name="tax_exigibility">on_payment</field>
         <field name="cash_basis_transition_account_id" ref="cuenta216_10"/>
+        <field name="l10n_mx_tax_type">Tasa</field>
         <field name="invoice_repartition_line_ids" eval="[(5, 0, 0),
             (0,0, {
                 'factor_percent': 100,
@@ -336,6 +343,7 @@
         <field name="tax_group_id" ref="tax_group_iva_ret_1067"/>
         <field name="tax_exigibility">on_payment</field>
         <field name="cash_basis_transition_account_id" ref="cuenta216_10"/>
+        <field name="l10n_mx_tax_type">Tasa</field>
         <field name="invoice_repartition_line_ids" eval="[(5, 0, 0),
             (0,0, {
                 'factor_percent': 100,
@@ -375,6 +383,7 @@
         <field name="tax_group_id" ref="tax_group_iva_0"/>
         <field name="tax_exigibility">on_payment</field>
         <field name="cash_basis_transition_account_id" ref="cuenta119_01"/>
+        <field name="l10n_mx_tax_type">Tasa</field>
         <field name="invoice_repartition_line_ids" eval="[(5, 0, 0),
             (0,0, {
                 'factor_percent': 100,
@@ -412,6 +421,7 @@
         <field name="tax_group_id" ref="tax_group_iva_16"/>
         <field name="tax_exigibility">on_payment</field>
         <field name="cash_basis_transition_account_id" ref="cuenta119_01"/>
+        <field name="l10n_mx_tax_type">Tasa</field>
         <field name="invoice_repartition_line_ids" eval="[(5, 0, 0),
             (0,0, {
                 'factor_percent': 100,
@@ -450,6 +460,7 @@
         <field name="tax_group_id" ref="tax_group_iva_8"/>
         <field name="tax_exigibility">on_payment</field>
         <field name="cash_basis_transition_account_id" ref="cuenta119_01"/>
+        <field name="l10n_mx_tax_type">Tasa</field>
         <field name="sequence" eval="10"/>
         <field name="invoice_repartition_line_ids" eval="[(5, 0, 0),
             (0,0, {
@@ -489,6 +500,7 @@
         <field name="tax_group_id" ref="tax_group_iva_8"/>
         <field name="tax_exigibility">on_payment</field>
         <field name="cash_basis_transition_account_id" ref="cuenta209_01"/>
+        <field name="l10n_mx_tax_type">Tasa</field>
         <field name="sequence" eval="10"/>
         <field name="invoice_repartition_line_ids" eval="[(5, 0, 0),
             (0,0, {
diff --git a/addons/l10n_mx/models/__init__.py b/addons/l10n_mx/models/__init__.py
index e92febf4135..81b9491a806 100644
--- a/addons/l10n_mx/models/__init__.py
+++ b/addons/l10n_mx/models/__init__.py
@@ -3,6 +3,7 @@
 # License LGPL-3.0 or later (http://www.gnu.org/licenses/lgpl).
 
 from . import account
+from . import account_tax
 from . import res_bank
 from . import res_config_settings
 from . import chart_template
diff --git a/addons/l10n_mx/models/account_tax.py b/addons/l10n_mx/models/account_tax.py
new file mode 100644
index 00000000000..a435a9ea25a
--- /dev/null
+++ b/addons/l10n_mx/models/account_tax.py
@@ -0,0 +1,38 @@
+# coding: utf-8
+from odoo import models, fields
+
+
+class AccountTaxTemplate(models.Model):
+    _inherit = 'account.tax.template'
+
+    l10n_mx_tax_type = fields.Selection(
+        selection=[
+            ('Tasa', "Tasa"),
+            ('Cuota', "Cuota"),
+            ('Exento', "Exento"),
+        ],
+        string="Factor Type",
+        default='Tasa',
+        help="The CFDI version 3.3 have the attribute 'TipoFactor' in the tax lines. In it is indicated the factor "
+             "type that is applied to the base of the tax.")
+
+    def _get_tax_vals(self, company, tax_template_to_tax):
+        # OVERRIDE
+        res = super()._get_tax_vals(company, tax_template_to_tax)
+        res['l10n_mx_tax_type'] = self.l10n_mx_tax_type
+        return res
+
+
+class AccountTax(models.Model):
+    _inherit = 'account.tax'
+
+    l10n_mx_tax_type = fields.Selection(
+        selection=[
+            ('Tasa', "Tasa"),
+            ('Cuota', "Cuota"),
+            ('Exento', "Exento"),
+        ],
+        string="Factor Type",
+        default='Tasa',
+        help="The CFDI version 3.3 have the attribute 'TipoFactor' in the tax lines. In it is indicated the factor "
+             "type that is applied to the base of the tax.")
-- 
cgit v1.2.3