odoo-addon-l10n-br-fiscal 16.0.19.4.0__py3-none-any.whl → 17.0.4.0.0.1__py3-none-any.whl

odoo/addons/l10n_br_fiscal/models/operation_line.py

@@ -37,11 +37,6 @@ class OperationLine(models.Model):
 
     document_type_id = fields.Many2one(comodel_name="l10n_br_fiscal.document.type")
 
-    tax_classification_id = fields.Many2one(
-        comodel_name="l10n_br_fiscal.tax.classification",
-        string="Tax Classification",
-    )
-
     cfop_internal_id = fields.Many2one(
         comodel_name="l10n_br_fiscal.cfop",
         string="CFOP Internal",
@@ -172,13 +167,6 @@ class OperationLine(models.Model):
             cfop = self.cfop_export_id
         return cfop
 
-    def _get_tax_classification(self, company):
-        if self.tax_classification_id:
-            return self.tax_classification_id
-        elif company.tax_classification_id:
-            return company.tax_classification_id
-        return self.env["l10n_br_fiscal.tax.classification"]
-
     def _build_mapping_result_ipi(self, mapping_result, tax_definition):
         if tax_definition and tax_definition.ipi_guideline_id:
             mapping_result["ipi_guideline"] = tax_definition.ipi_guideline_id
@@ -258,15 +246,12 @@ class OperationLine(models.Model):
               (l10n_br_fiscal.tax.ipi.guideline).
             - 'icms_tax_benefit_id': The determined ICMS tax benefit record
               ID (l10n_br_fiscal.tax.definition) or False.
-            - 'tax_classification': The determined Tax Classification record
-              (l10n_br_fiscal.tax.classification).
         """
         mapping_result = {
             "taxes": {},
             "cfop": False,
             "ipi_guideline": self.env.ref("l10n_br_fiscal.tax_guideline_999"),
             "icms_tax_benefit_id": False,
-            "tax_classification": False,
         }
 
         self.ensure_one()
@@ -274,9 +259,6 @@ class OperationLine(models.Model):
         # Define CFOP
         mapping_result["cfop"] = self._get_cfop(company, partner)
 
-        # Define Tax Classification
-        mapping_result["tax_classification"] = self._get_tax_classification(company)
-
         # 1 Get Tax Defs from Company
         for tax_definition in company.tax_definition_ids.map_tax_definition(
             company,
@@ -291,16 +273,6 @@ class OperationLine(models.Model):
         ):
             self._build_mapping_result(mapping_result, tax_definition)
 
-        # 1_5 From Tax Classification
-        if mapping_result["tax_classification"]:
-            mapping_result["taxes"][
-                mapping_result["tax_classification"].tax_cbs_id.tax_domain
-            ] = mapping_result["tax_classification"].tax_cbs_id
-
-            mapping_result["taxes"][
-                mapping_result["tax_classification"].tax_ibs_id.tax_domain
-            ] = mapping_result["tax_classification"].tax_ibs_id
-
         # 2 From NCM
         if not ncm and product:
             ncm = product.ncm_id

odoo/addons/l10n_br_fiscal/models/partner_profile.py

@@ -8,7 +8,6 @@ from odoo.exceptions import ValidationError
 from ..constants.fiscal import (
     NFE_IND_IE_DEST,
     NFE_IND_IE_DEST_DEFAULT,
-    PUBLIC_ENTIRY_TYPE,
     TAX_FRAMEWORK,
     TAX_FRAMEWORK_NORMAL,
 )
@@ -33,11 +32,6 @@ class PartnerProfile(models.Model):
         "other government-controlled organizations.",
     )
 
-    public_entity_type = fields.Selection(
-        selection=PUBLIC_ENTIRY_TYPE,
-        string="Tipo de Entidade Governamental",
-    )
-
     default = fields.Boolean(string="Default Profile", default=True)
 
     ind_ie_dest = fields.Selection(

odoo/addons/l10n_br_fiscal/models/product_template.py

@@ -106,10 +106,6 @@ class ProductTemplate(models.Model):
         comodel_name="l10n_br_fiscal.nbs", index=True, string="NBS"
     )
 
-    operation_indicator_id = fields.Many2one(
-        comodel_name="l10n_br_fiscal.operation.indicator",
-    )
-
     cest_id = fields.Many2one(
         comodel_name="l10n_br_fiscal.cest",
         index=True,

odoo/addons/l10n_br_fiscal/models/res_company.py

@@ -14,12 +14,10 @@ from ..constants.fiscal import (
     PROCESSADOR_NENHUM,
     PROFIT_CALCULATION,
     PROFIT_CALCULATION_PRESUMED,
-    TAX_DOMAIN_CBS,
     TAX_DOMAIN_COFINS,
     TAX_DOMAIN_COFINS_WH,
     TAX_DOMAIN_CSLL,
     TAX_DOMAIN_CSLL_WH,
-    TAX_DOMAIN_IBS,
     TAX_DOMAIN_ICMS,
     TAX_DOMAIN_ICMS_SN,
     TAX_DOMAIN_INSS,
@@ -299,12 +297,6 @@ class ResCompany(models.Model):
         domain=[("tax_domain", "=", TAX_DOMAIN_INSS_WH)],
     )
 
-    tax_classification_id = fields.Many2one(
-        comodel_name="l10n_br_fiscal.tax.classification",
-        string="Default Tax Classification",
-        domain=[("tax_ibs_id", "!=", False), ("tax_cbs_id", "!=", False)],
-    )
-
     tax_definition_ids = fields.One2many(
         comodel_name="l10n_br_fiscal.tax.definition",
         inverse_name="company_id",
@@ -385,7 +377,6 @@ class ResCompany(models.Model):
             self.tax_icms_id = False
 
         self._onchange_piscofins_id()
-        self._onchange_tax_classification_id()
         self._onchange_ripi()
         self._onchange_tax_ipi_id()
         self._onchange_tax_icms_id()
@@ -511,12 +502,3 @@ class ResCompany(models.Model):
             self._set_tax_definition(self.tax_inss_wh_id)
         else:
             self._del_tax_definition(TAX_DOMAIN_INSS_WH)
-
-    @api.onchange("tax_classification_id")
-    def _onchange_tax_classification_id(self):
-        if self.tax_classification_id:
-            self._set_tax_definition(self.tax_classification_id.tax_cbs_id)
-            self._set_tax_definition(self.tax_classification_id.tax_ibs_id)
-        else:
-            self._del_tax_definition(TAX_DOMAIN_CBS)
-            self._del_tax_definition(TAX_DOMAIN_IBS)

odoo/addons/l10n_br_fiscal/models/res_partner.py

@@ -10,7 +10,6 @@ from ..constants.fiscal import (
     NFE_IND_IE_DEST,
     NFE_IND_IE_DEST_9,
     NFE_IND_IE_DEST_DEFAULT,
-    PUBLIC_ENTIRY_TYPE,
     TAX_FRAMEWORK,
     TAX_FRAMEWORK_NORMAL,
 )
@@ -77,11 +76,6 @@ class ResPartner(models.Model):
         "other government-controlled organizations.",
     )
 
-    public_entity_type = fields.Selection(
-        selection=PUBLIC_ENTIRY_TYPE,
-        string="Tipo de Entidade Governamental",
-    )
-
     ind_final = fields.Selection(
         selection=FINAL_CUSTOMER,
         string="Final Consumption Operation",
@@ -117,16 +111,6 @@ class ResPartner(models.Model):
         string="RNTRC Code", size=12, unaccent=False, tracking=True
     )
 
-    nif_motive_absence = fields.Selection(
-        selection=[
-            ("0", "Not informed in the origin note"),
-            ("1", "Exemption from NIF"),
-            ("2", "NIF not required"),
-        ],
-        default=False,
-        string="NIF motive absence",
-    )
-
     def _inverse_fiscal_profile(self):
         for p in self:
             p._onchange_fiscal_profile_id()
@@ -143,7 +127,6 @@ class ResPartner(models.Model):
                 p.tax_framework = p.fiscal_profile_id.tax_framework
                 p.ind_ie_dest = p.fiscal_profile_id.ind_ie_dest
                 p.is_public_entity = p.fiscal_profile_id.is_public_entity
-                p.public_entity_type = p.fiscal_profile_id.public_entity_type
 
     @api.onchange("ind_ie_dest")
     def _onchange_ind_ie_dest(self):

odoo/addons/l10n_br_fiscal/models/simplified_tax_range.py

@@ -66,11 +66,3 @@ class SimplifiedTaxRange(models.Model):
     tax_pis_percent = fields.Float(
         string="Tax PIS Percent", digits="Fiscal Tax Percent"
     )
-
-    tax_ibs_percent = fields.Float(
-        string="Tax IBS Percent", digits="Fiscal Tax Percent"
-    )
-
-    tax_cbs_percent = fields.Float(
-        string="Tax CBS Percent", digits="Fiscal Tax Percent"
-    )

odoo/addons/l10n_br_fiscal/models/tax_definition.py

@@ -63,28 +63,13 @@ class TaxDefinition(models.Model):
         for record in self:
             record.display_name = record._get_complete_name()
 
-    @api.depends("tax_group_id", "tax_id", "cst_code")
-    def name_get(self):
-        result = []
-        for record in self:
-            name = record._get_complete_name()
-            result.append((record.id, name))
-        return result
-
-    display_name = fields.Char(compute="_compute_display_name", store=True)
-
     code = fields.Char(
         size=8,
-        states={"draft": [("readonly", False)]},
     )
 
-    name = fields.Char(
-        states={"draft": [("readonly", False)]},
-    )
+    name = fields.Char()
 
-    description = fields.Text(
-        states={"draft": [("readonly", False)]},
-    )
+    description = fields.Text()
 
     type_in_out = fields.Selection(
         selection=FISCAL_IN_OUT,
@@ -92,7 +77,6 @@ class TaxDefinition(models.Model):
         required=True,
         default=FISCAL_OUT,
         readonly=True,
-        states={"draft": [("readonly", False)]},
     )
 
     tax_group_id = fields.Many2one(
@@ -100,19 +84,16 @@ class TaxDefinition(models.Model):
         string="Tax Group",
         required=True,
         readonly=True,
-        states={"draft": [("readonly", False)]},
     )
 
     custom_tax = fields.Boolean(
         readonly=True,
-        states={"draft": [("readonly", False)]},
     )
 
     tax_id = fields.Many2one(
         comodel_name="l10n_br_fiscal.tax",
         string="Tax",
         readonly=True,
-        states={"draft": [("readonly", False)]},
         domain="[('tax_group_id', '=', tax_group_id)]",
     )
 
@@ -133,26 +114,22 @@ class TaxDefinition(models.Model):
         related="tax_group_id.tax_domain",
         store=True,
         string="Tax Domain",
-        states={"draft": [("readonly", False)]},
     )
 
     is_taxed = fields.Boolean(
         string="Taxed?",
         readonly=True,
-        states={"draft": [("readonly", False)]},
     )
 
     is_debit_credit = fields.Boolean(
         string="Debit/Credit?",
         readonly=True,
-        states={"draft": [("readonly", False)]},
     )
 
     company_id = fields.Many2one(
         comodel_name="res.company",
         string="Company",
         readonly=True,
-        states={"draft": [("readonly", False)]},
     )
 
     state_from_id = fields.Many2one(
@@ -170,19 +147,16 @@ class TaxDefinition(models.Model):
     ncms = fields.Text(
         string="NCM List",
         readonly=True,
-        states={"draft": [("readonly", False)]},
     )
 
     ncm_exception = fields.Text(
         string="NCM Exeption",
         readonly=True,
-        states={"draft": [("readonly", False)]},
     )
 
     not_in_ncms = fields.Text(
         string="Not in NCMs",
         readonly=True,
-        states={"draft": [("readonly", False)]},
     )
 
     ncm_ids = fields.Many2many(
@@ -194,7 +168,6 @@ class TaxDefinition(models.Model):
     cests = fields.Text(
         string="CEST List",
         readonly=True,
-        states={"draft": [("readonly", False)]},
     )
 
     cest_ids = fields.Many2many(
@@ -206,13 +179,11 @@ class TaxDefinition(models.Model):
     nbms = fields.Text(
         string="NBM List",
         readonly=True,
-        states={"draft": [("readonly", False)]},
     )
 
     not_in_nbms = fields.Text(
         string="Not in NBMs",
         readonly=True,
-        states={"draft": [("readonly", False)]},
     )
 
     nbm_ids = fields.Many2many(
@@ -245,13 +216,11 @@ class TaxDefinition(models.Model):
     date_start = fields.Datetime(
         string="Start Date",
         readonly=True,
-        states={"draft": [("readonly", False)]},
     )
 
     date_end = fields.Datetime(
         string="End Date",
         readonly=True,
-        states={"draft": [("readonly", False)]},
     )
 
     state = fields.Selection(
@@ -286,12 +255,10 @@ class TaxDefinition(models.Model):
     is_benefit = fields.Boolean(
         string="Benefit?",
         readonly=True,
-        states={"draft": [("readonly", False)]},
     )
 
     benefit_type = fields.Selection(
         selection=ICMS_TAX_BENEFIT_TYPE,
-        states={"draft": [("readonly", False)]},
     )
 
     def _get_search_domain(self, tax_definition):

odoo/addons/l10n_br_fiscal/security/ir.model.access.csv

@@ -38,9 +38,6 @@
 "l10n_br_fiscal_cest_user","Fiscal CEST for User","model_l10n_br_fiscal_cest","l10n_br_fiscal.group_user",1,0,0,0
 "l10n_br_fiscal_cest_manager","Fiscal CEST for Manager","model_l10n_br_fiscal_cest","l10n_br_fiscal.group_manager",1,0,0,0
 "l10n_br_fiscal_cest_maintenance","Fiscal CEST for Maintenance","model_l10n_br_fiscal_cest","l10n_br_fiscal.group_data_maintenance",1,1,1,1
-"l10n_br_fiscal_tax_classification_user","Fiscal Tax Classification for User","model_l10n_br_fiscal_tax_classification","l10n_br_fiscal.group_user",1,0,0,0
-"l10n_br_fiscal_tax_classification_manager","Fiscal Tax Classification for Manager","model_l10n_br_fiscal_tax_classification","l10n_br_fiscal.group_manager",1,0,0,0
-"l10n_br_fiscal_tax_classification_maintenance","Fiscal Tax Classification for Maintenance","model_l10n_br_fiscal_tax_classification","l10n_br_fiscal.group_data_maintenance",1,1,1,1
 "l10n_br_fiscal_product_genre_user","Fiscal Fiscal Product Genre for User","model_l10n_br_fiscal_product_genre","l10n_br_fiscal.group_user",1,0,0,0
 "l10n_br_fiscal_product_genre_manager","Fiscal Fiscal Product Genre for Manager","model_l10n_br_fiscal_product_genre","l10n_br_fiscal.group_manager",1,0,0,0
 "l10n_br_fiscal_product_genre_maintenance","Fiscal Fiscal Product Genre for Maintenance","model_l10n_br_fiscal_product_genre","l10n_br_fiscal.group_data_maintenance",1,1,1,1
@@ -70,9 +67,6 @@
 "l10n_br_fiscal_operation_line_manager","Fiscal Operation Line for Manager","model_l10n_br_fiscal_operation_line","l10n_br_fiscal.group_manager",1,1,1,1
 "l10n_br_fiscal_operation_document_type_user","Fiscal Operation Document Type for User","model_l10n_br_fiscal_operation_document_type","l10n_br_fiscal.group_user",1,0,0,0
 "l10n_br_fiscal_operation_document_type_manager","Fiscal Operation Document Type for Manager","model_l10n_br_fiscal_operation_document_type","l10n_br_fiscal.group_manager",1,1,1,1
-"l10n_br_fiscal_operation_indicator_user","Operation Indicator for User","model_l10n_br_fiscal_operation_indicator","l10n_br_fiscal.group_user",1,0,0,0
-"l10n_br_fiscal_operation_indicator_manager","Operation Indicator for Manager","model_l10n_br_fiscal_operation_indicator","l10n_br_fiscal.group_manager",1,1,1,1
-"l10n_br_fiscal_operation_indicator_maintenance","Operation Indicator for Maintenance","model_l10n_br_fiscal_operation_indicator","l10n_br_fiscal.group_data_maintenance",1,1,1,1
 "l10n_br_fiscal_partner_profile_user","Fiscal Partner Profile for User","model_l10n_br_fiscal_partner_profile","base.group_user",1,0,0,0
 "l10n_br_fiscal_partner_profile_manager","Fiscal Partner Profile for Manager","model_l10n_br_fiscal_partner_profile","l10n_br_fiscal.group_manager",1,1,1,1
 "l10n_br_fiscal_document_user","Fiscal Document for User","model_l10n_br_fiscal_document","l10n_br_fiscal.group_user",1,1,1,0

odoo/addons/l10n_br_fiscal/static/description/index.html

@@ -372,10 +372,10 @@ ul.auto-toc {
 !! This file is generated by oca-gen-addon-readme !!
 !! changes will be overwritten.                   !!
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
-!! source digest: sha256:946adae5ac2b94ccb38002a3dbc6b2464a2145f654bcfc818a9ae7aab3ae4ac5
+!! source digest: sha256:d3d8d4303c70d7d8573dc89abf8f7661a3c19dde6dc541f94971847d33122362
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! -->
-<p><a class="reference external image-reference" href="https://odoo-community.org/page/development-status"><img alt="Production/Stable" src="https://img.shields.io/badge/maturity-Production%2FStable-green.png" /></a> <a class="reference external image-reference" href="http://www.gnu.org/licenses/agpl-3.0-standalone.html"><img alt="License: AGPL-3" src="https://img.shields.io/badge/license-AGPL--3-blue.png" /></a> <a class="reference external image-reference" href="https://github.com/OCA/l10n-brazil/tree/16.0/l10n_br_fiscal"><img alt="OCA/l10n-brazil" src="https://img.shields.io/badge/github-OCA%2Fl10n--brazil-lightgray.png?logo=github" /></a> <a class="reference external image-reference" href="https://translation.odoo-community.org/projects/l10n-brazil-16-0/l10n-brazil-16-0-l10n_br_fiscal"><img alt="Translate me on Weblate" src="https://img.shields.io/badge/weblate-Translate%20me-F47D42.png" /></a> <a class="reference external image-reference" href="https://runboat.odoo-community.org/builds?repo=OCA/l10n-brazil&amp;target_branch=16.0"><img alt="Try me on Runboat" src="https://img.shields.io/badge/runboat-Try%20me-875A7B.png" /></a></p>
-<p><img alt="image" src="https://raw.githubusercontent.com/OCA/l10n-brazil/16.0/l10n_br_fiscal/static/img/fiscal_dashboard.png" /></p>
+<p><a class="reference external image-reference" href="https://odoo-community.org/page/development-status"><img alt="Production/Stable" src="https://img.shields.io/badge/maturity-Production%2FStable-green.png" /></a> <a class="reference external image-reference" href="http://www.gnu.org/licenses/agpl-3.0-standalone.html"><img alt="License: AGPL-3" src="https://img.shields.io/badge/license-AGPL--3-blue.png" /></a> <a class="reference external image-reference" href="https://github.com/OCA/l10n-brazil/tree/17.0/l10n_br_fiscal"><img alt="OCA/l10n-brazil" src="https://img.shields.io/badge/github-OCA%2Fl10n--brazil-lightgray.png?logo=github" /></a> <a class="reference external image-reference" href="https://translation.odoo-community.org/projects/l10n-brazil-17-0/l10n-brazil-17-0-l10n_br_fiscal"><img alt="Translate me on Weblate" src="https://img.shields.io/badge/weblate-Translate%20me-F47D42.png" /></a> <a class="reference external image-reference" href="https://runboat.odoo-community.org/builds?repo=OCA/l10n-brazil&amp;target_branch=17.0"><img alt="Try me on Runboat" src="https://img.shields.io/badge/runboat-Try%20me-875A7B.png" /></a></p>
+<p><img alt="image" src="https://raw.githubusercontent.com/OCA/l10n-brazil/17.0/l10n_br_fiscal/static/img/fiscal_dashboard.png" /></p>
 <div class="section" id="classificacoes-fiscais">
 <h2>Classificações fiscais</h2>
 <p>Primeramente, este módulo traz uma variedade de cadastros fiscais para
@@ -513,8 +513,8 @@ como:</p>
 <li>ST</li>
 <li>retenções</li>
 </ul>
-<p><img alt="image1" src="https://raw.githubusercontent.com/OCA/l10n-brazil/16.0/l10n_br_fiscal/static/img/fiscal_line.png" /></p>
-<p><img alt="image2" src="https://raw.githubusercontent.com/OCA/l10n-brazil/16.0/l10n_br_fiscal/static/img/fiscal_total.png" /></p>
+<p><img alt="image1" src="https://raw.githubusercontent.com/OCA/l10n-brazil/17.0/l10n_br_fiscal/static/img/fiscal_line.png" /></p>
+<p><img alt="image2" src="https://raw.githubusercontent.com/OCA/l10n-brazil/17.0/l10n_br_fiscal/static/img/fiscal_total.png" /></p>
 <p>É notório que o cálculo dos impostos no Brasil é muito especial e muito
 trabalhoso. Geralmente é o motivo pelo qual os ERPs internacionais não
 tem grande fatia de mercado brasileiro.</p>
@@ -543,7 +543,7 @@ Tributária)…</p>
 <div class="section" id="operacoes-fiscais">
 <h2>Operações fiscais</h2>
 <blockquote>
-<img alt="image3" src="https://raw.githubusercontent.com/OCA/l10n-brazil/16.0/l10n_br_fiscal/static/img/fiscal_operation.png" /></blockquote>
+<img alt="image3" src="https://raw.githubusercontent.com/OCA/l10n-brazil/17.0/l10n_br_fiscal/static/img/fiscal_operation.png" /></blockquote>
 <p>No Odoo nativo, o conceito mais parecido com a operação fiscal e o
 <tt class="docutils literal">account.fiscal.position</tt>. E ate a versão 10.0, era o que a gente
 usava. Porém, a posição fiscal do Odoo não resolve muito os nossos
@@ -610,7 +610,7 @@ l10n_br_sale, l10n_br_purchase…</p>
 <p>Bugs are tracked on <a class="reference external" href="https://github.com/OCA/l10n-brazil/issues">GitHub Issues</a>.
 In case of trouble, please check there if your issue has already been reported.
 If you spotted it first, help us to smash it by providing a detailed and welcomed
-<a class="reference external" href="https://github.com/OCA/l10n-brazil/issues/new?body=module:%20l10n_br_fiscal%0Aversion:%2016.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**">feedback</a>.</p>
+<a class="reference external" href="https://github.com/OCA/l10n-brazil/issues/new?body=module:%20l10n_br_fiscal%0Aversion:%2017.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**">feedback</a>.</p>
 <p>Do not contact contributors directly about support or help with technical issues.</p>
 </div>
 <div class="section" id="credits">
@@ -659,7 +659,7 @@ mission is to support the collaborative development of Odoo features and
 promote its widespread use.</p>
 <p>Current <a class="reference external" href="https://odoo-community.org/page/maintainer-role">maintainers</a>:</p>
 <p><a class="reference external image-reference" href="https://github.com/renatonlima"><img alt="renatonlima" src="https://github.com/renatonlima.png?size=40px" /></a> <a class="reference external image-reference" href="https://github.com/rvalyi"><img alt="rvalyi" src="https://github.com/rvalyi.png?size=40px" /></a></p>
-<p>This module is part of the <a class="reference external" href="https://github.com/OCA/l10n-brazil/tree/16.0/l10n_br_fiscal">OCA/l10n-brazil</a> project on GitHub.</p>
+<p>This module is part of the <a class="reference external" href="https://github.com/OCA/l10n-brazil/tree/17.0/l10n_br_fiscal">OCA/l10n-brazil</a> project on GitHub.</p>
 <p>You are welcome to contribute. To learn how please visit <a class="reference external" href="https://odoo-community.org/page/Contribute">https://odoo-community.org/page/Contribute</a>.</p>
 </div>
 </div>

odoo/addons/l10n_br_fiscal/tests/__init__.py

@@ -6,7 +6,6 @@ from . import (
     test_fiscal_document_generic,
     test_fiscal_document_nfse,
     test_fiscal_tax,
-    test_tax_classification,
     test_tax_benefit,
     test_document_edition,
     test_ibpt_product,

odoo/addons/l10n_br_fiscal/tests/test_cnae.py

@@ -6,8 +6,7 @@ from odoo.tests.common import TransactionCase
 
 
 class TestCNAE(TransactionCase):
-    def test_name_get(self):
-        """Test CNAE name_get()"""
+    def test_display_name(self):
         self.cnae = self.env["l10n_br_fiscal.cnae"].create(
             {
                 "code": "TESTE",
@@ -16,4 +15,4 @@ class TestCNAE(TransactionCase):
                 "internal_type": "normal",
             }
         )
-        assert self.cnae.name_get(), "Error with function name_get()"
+        self.assertEqual(self.cnae.display_name, "TESTE - TESTE")

odoo/addons/l10n_br_fiscal/tests/test_document_edition.py

@@ -4,10 +4,9 @@
 from unittest import mock
 
 from odoo.tests import TransactionCase
-from odoo.tests.common import Form, tagged
+from odoo.tests.common import Form
 
 
-@tagged("post_install", "-at_install")
 class TestDocumentEdition(TransactionCase):
     @classmethod
     def setUpClass(cls):
@@ -80,46 +79,20 @@ class TestDocumentEdition(TransactionCase):
                 line_form.fiscal_operation_line_id,
                 self.env.ref("l10n_br_fiscal.fo_venda_revenda"),
             )
-            self.assertEqual(
-                line_form.ipi_tax_id, self.env.ref("l10n_br_fiscal.tax_ipi_nt")
-            )
-
-            line_form.fiscal_operation_line_id = self.env.ref(
-                "l10n_br_fiscal.fo_venda_venda"
-            )
-            self.assertEqual(
-                line_form.ipi_tax_id, self.env.ref("l10n_br_fiscal.tax_ipi_3_25")
-            )
 
-            # ensure manually setting a xx_tax_id is properly saved (not recomputed):
-            line_form.icms_tax_id = self.env.ref("l10n_br_fiscal.tax_icms_18")
-            self.assertEqual(line_form.icms_value, 37.17)
-            self.assertEqual(
-                line_form.ipi_tax_id, self.env.ref("l10n_br_fiscal.tax_ipi_3_25")
-            )
-            line_form.icmsfcp_base = line_form.price_unit
-            line_form.icmsfcp_value = 3  # ensure manually setting FCP value works
+            # line_form.fiscal_operation_line_id = False
+            # self.assertEqual(len(line_form.fiscal_tax_ids), 0)
 
         doc = doc_form.save()
-        line = doc.fiscal_line_ids[0]
-        self.assertEqual(line.price_unit, 100)
-        self.assertEqual(line.fiscal_price, 100)
-        self.assertEqual(line.quantity, 2)
-        self.assertEqual(line.fiscal_quantity, 2)
-        self.assertEqual(len(line.fiscal_tax_ids), 4)
-
-        self.assertEqual(
-            line.fiscal_operation_line_id,
-            self.env.ref("l10n_br_fiscal.fo_venda_venda"),
-        )
+        self.assertEqual(doc.fiscal_line_ids[0].price_unit, 100)
+        self.assertEqual(doc.fiscal_line_ids[0].fiscal_price, 100)
+        self.assertEqual(doc.fiscal_line_ids[0].quantity, 2)
+        self.assertEqual(doc.fiscal_line_ids[0].fiscal_quantity, 2)
+        self.assertEqual(len(doc.fiscal_line_ids[0].fiscal_tax_ids), 4)
         self.assertEqual(
-            line.icms_tax_id.id,
-            self.ref("l10n_br_fiscal.tax_icms_18"),
+            doc.fiscal_line_ids[0].icms_tax_id.id,
+            self.ref("l10n_br_fiscal.tax_icms_12"),
         )
-        self.assertEqual(line.ipi_tax_id, self.env.ref("l10n_br_fiscal.tax_ipi_3_25"))
-        self.assertEqual(line.icms_value, 37.17)
-        self.assertEqual(line.icmsfcp_base, line.price_unit)
-        self.assertEqual(line.icmsfcp_value, 3)
 
     def test_product_fiscal_factor(self):
         doc_form = Form(

odoo/addons/l10n_br_fiscal/tests/test_fiscal_document_generic.py

@@ -44,6 +44,9 @@ class TestFiscalDocumentGeneric(TransactionCase):
             # as the product change might have altered it.
             line.price_unit = 100
 
+            line._onchange_fiscal_operation_id()
+            line._onchange_fiscal_taxes()
+
             if "Revenda" in line.fiscal_operation_line_id.name:
                 self.assertEqual(
                     line.cfop_id.code,
@@ -158,6 +161,9 @@ class TestFiscalDocumentGeneric(TransactionCase):
     def test_nfe_other_state(self):
         """Test NFe other state."""
         for line in self.nfe_other_state.fiscal_line_ids:
+            line._onchange_fiscal_operation_id()
+            line._onchange_fiscal_taxes()
+
             if "Revenda" in line.fiscal_operation_line_id.name:
                 self.assertEqual(
                     line.cfop_id.code,
@@ -269,6 +275,9 @@ class TestFiscalDocumentGeneric(TransactionCase):
     def test_nfe_not_taxpayer(self):
         """Test NFe not taxpayer."""
         for line in self.nfe_not_taxpayer.fiscal_line_ids:
+            line._onchange_fiscal_operation_id()
+            line._onchange_fiscal_taxes()
+
             if "Revenda" in line.fiscal_operation_line_id.name:
                 self.assertEqual(
                     line.cfop_id.code,
@@ -367,6 +376,9 @@ class TestFiscalDocumentGeneric(TransactionCase):
     def test_nfe_not_taxpayer_not_company(self):
         """Test NFe not taxpayer not Company."""
         for line in self.nfe_not_taxpayer_pf.fiscal_line_ids:
+            line._onchange_fiscal_operation_id()
+            line._onchange_fiscal_taxes()
+
             if "Revenda" in line.fiscal_operation_line_id.name:
                 self.assertEqual(
                     line.cfop_id.code,
@@ -465,6 +477,9 @@ class TestFiscalDocumentGeneric(TransactionCase):
     def test_nfe_export(self):
         """Test NFe export."""
         for line in self.nfe_export.fiscal_line_ids:
+            line._onchange_fiscal_operation_id()
+            line._onchange_fiscal_taxes()
+
             if "Revenda" in line.fiscal_operation_line_id.name:
                 self.assertEqual(
                     line.cfop_id.code,
@@ -565,9 +580,9 @@ class TestFiscalDocumentGeneric(TransactionCase):
                     "federal_taxes_national": 33.00,
                 }
             )
-            # força o compute, pois não é chamado automaticamente
-            # quando uma informação externa muda.
-            line._compute_tax_fields()
+
+            line._onchange_fiscal_operation_id()
+            line._onchange_fiscal_taxes()
 
             if "Revenda" in line.fiscal_operation_line_id.name:
                 self.assertEqual(
@@ -666,6 +681,9 @@ class TestFiscalDocumentGeneric(TransactionCase):
     def test_nfe_sn_other_state(self):
         """Test NFe SN other state."""
         for line in self.nfe_sn_other_state.fiscal_line_ids:
+            line._onchange_fiscal_operation_id()
+            line._onchange_fiscal_taxes()
+
             if "Revenda" in line.fiscal_operation_line_id.name:
                 self.assertEqual(
                     line.cfop_id.code,
@@ -760,6 +778,9 @@ class TestFiscalDocumentGeneric(TransactionCase):
     def test_nfe_sn_not_taxpayer(self):
         """Test NFe SN not taxpayer."""
         for line in self.nfe_sn_not_taxpayer.fiscal_line_ids:
+            line._onchange_fiscal_operation_id()
+            line._onchange_fiscal_taxes()
+
             if "Revenda" in line.fiscal_operation_line_id.name:
                 self.assertEqual(
                     line.cfop_id.code,
@@ -841,6 +862,9 @@ class TestFiscalDocumentGeneric(TransactionCase):
     def test_nfe_sn_export(self):
         """Test NFe SN export."""
         for line in self.nfe_sn_export.fiscal_line_ids:
+            line._onchange_fiscal_operation_id()
+            line._onchange_fiscal_taxes()
+
             if "Revenda" in line.fiscal_operation_line_id.name:
                 self.assertEqual(
                     line.cfop_id.code,
@@ -950,7 +974,7 @@ class TestFiscalDocumentGeneric(TransactionCase):
         additional_data = self.nfe_not_taxpayer.fiscal_line_ids[0].additional_data
         self.assertEqual(
             additional_data,
-            "manual comment test - Valor Aprox. dos Tributos: R$\xa00,00",
+            "manual comment test",
         )
 
     def test_fields_freight_insurance_other_costs(self):

odoo/addons/l10n_br_fiscal/tests/test_fiscal_document_nfse.py

@@ -13,7 +13,11 @@ class TestFiscalDocumentNFSe(TransactionCase):
 
     def test_nfse_same_state(self):
         """Test NFSe same state."""
+
         for line in self.nfse_same_state.fiscal_line_ids:
+            line._onchange_fiscal_operation_id()
+            line._onchange_fiscal_taxes()
+
             self.assertEqual(
                 line.fiscal_operation_line_id.name,
                 "Prestação de Serviço",

odoo/addons/l10n_br_fiscal/tests/test_service_type.py

@@ -6,9 +6,8 @@ from odoo.tests.common import TransactionCase
 
 
 class TestServiceType(TransactionCase):
-    def test_name_get(self):
-        """Test Service Type name_get()"""
+    def test_display_name(self):
         self.service_type = self.env["l10n_br_fiscal.service.type"].create(
             {"code": "TESTE", "name": "TESTE", "internal_type": "normal"}
         )
-        assert self.service_type.name_get(), "Error with function name_get()"
+        self.assertEqual(self.service_type.display_name, "TESTE - TESTE")