odoo-addon-l10n-br-fiscal 16.0.11.3.0__py3-none-any.whl → 16.0.11.5.0__py3-none-any.whl

odoo/addons/l10n_br_fiscal/README.rst

@@ -11,7 +11,7 @@ Módulo fiscal brasileiro
    !! This file is generated by oca-gen-addon-readme !!
    !! changes will be overwritten.                   !!
    !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
-   !! source digest: sha256:3612e46c6fb955a94d39c6bced2a46abadbb539336d76697fbd63869b54cdac5
+   !! source digest: sha256:dd9f065758d59e68e7d24a53621956cc9e6911bb9e58b0ac89b2e202085a51dc
    !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
 
 .. |badge1| image:: https://img.shields.io/badge/maturity-Production%2FStable-green.png

odoo/addons/l10n_br_fiscal/__manifest__.py

@@ -10,7 +10,7 @@
     "maintainers": ["renatonlima", "rvalyi"],
     "website": "https://github.com/OCA/l10n-brazil",
     "development_status": "Production/Stable",
-    "version": "16.0.11.3.0",
+    "version": "16.0.11.5.0",
     "depends": [
         "product",
         "uom_alias",

odoo/addons/l10n_br_fiscal/demo/fiscal_document_demo.xml

@@ -47,13 +47,6 @@
         <value eval="[ref('demo_nfe_line_same_state_1-1')]" />
     </function>
 
-    <function
-        model="l10n_br_fiscal.document.line"
-        name="_onchange_fiscal_operation_line_id"
-    >
-        <value eval="[ref('demo_nfe_line_same_state_1-1')]" />
-    </function>
-
     <record id="demo_nfe_line_same_state_1-2" model="l10n_br_fiscal.document.line">
         <field name="document_id" ref="demo_nfe_same_state" />
         <field name="name">Teste</field>
@@ -76,13 +69,6 @@
         <value eval="[ref('demo_nfe_line_same_state_1-2')]" />
     </function>
 
-    <function
-        model="l10n_br_fiscal.document.line"
-        name="_onchange_fiscal_operation_line_id"
-    >
-        <value eval="[ref('demo_nfe_line_same_state_1-2')]" />
-    </function>
-
     <!-- NFe Test - Empresa Contribuinte - Outro Estado -->
     <record id="demo_nfe_other_state" model="l10n_br_fiscal.document">
         <field name="fiscal_operation_id" ref="l10n_br_fiscal.fo_venda" />
@@ -120,13 +106,6 @@
         <value eval="[ref('demo_nfe_line_other_state_1-3')]" />
     </function>
 
-    <function
-        model="l10n_br_fiscal.document.line"
-        name="_onchange_fiscal_operation_line_id"
-    >
-        <value eval="[ref('demo_nfe_line_other_state_1-3')]" />
-    </function>
-
     <record id="demo_nfe_line_other_state_2-3" model="l10n_br_fiscal.document.line">
         <field name="document_id" ref="demo_nfe_other_state" />
         <field name="name">Teste</field>
@@ -149,13 +128,6 @@
         <value eval="[ref('demo_nfe_line_other_state_2-3')]" />
     </function>
 
-    <function
-        model="l10n_br_fiscal.document.line"
-        name="_onchange_fiscal_operation_line_id"
-    >
-        <value eval="[ref('demo_nfe_line_other_state_2-3')]" />
-    </function>
-
     <record id="demo_nfe_line_other_state_3-3" model="l10n_br_fiscal.document.line">
         <field name="document_id" ref="demo_nfe_other_state" />
         <field name="name">Teste</field>
@@ -179,13 +151,6 @@
         <value eval="[ref('demo_nfe_line_other_state_3-3')]" />
     </function>
 
-    <function
-        model="l10n_br_fiscal.document.line"
-        name="_onchange_fiscal_operation_line_id"
-    >
-        <value eval="[ref('demo_nfe_line_other_state_3-3')]" />
-    </function>
-
     <!-- NFe Test - Empresa Contribuinte - Exterior -->
     <record id="demo_nfe_export" model="l10n_br_fiscal.document">
         <field name="company_id" ref="l10n_br_base.empresa_lucro_presumido" />
@@ -225,13 +190,6 @@
         <value eval="[ref('demo_nfe_line_export_3-1')]" />
     </function>
 
-    <function
-        model="l10n_br_fiscal.document.line"
-        name="_onchange_fiscal_operation_line_id"
-    >
-        <value eval="[ref('demo_nfe_line_export_3-1')]" />
-    </function>
-
     <record id="demo_nfe_line_export_3-2" model="l10n_br_fiscal.document.line">
         <field name="document_id" ref="demo_nfe_export" />
         <field name="name">Teste</field>
@@ -254,13 +212,6 @@
         <value eval="[ref('demo_nfe_line_export_3-2')]" />
     </function>
 
-    <function
-        model="l10n_br_fiscal.document.line"
-        name="_onchange_fiscal_operation_line_id"
-    >
-        <value eval="[ref('demo_nfe_line_export_3-2')]" />
-    </function>
-
     <!-- NFe Test - Empresa Contribuinte - Nao Contribuinte PJ -->
     <record id="demo_nfe_nao_contribuinte" model="l10n_br_fiscal.document">
         <field name="fiscal_operation_id" ref="l10n_br_fiscal.fo_venda" />
@@ -312,13 +263,6 @@
         <value eval="[ref('demo_nfe_line_nao_contribuinte_4-1')]" />
     </function>
 
-    <function
-        model="l10n_br_fiscal.document.line"
-        name="_onchange_fiscal_operation_line_id"
-    >
-        <value eval="[ref('demo_nfe_line_nao_contribuinte_4-1')]" />
-    </function>
-
     <record
         id="demo_nfe_line_nao_contribuinte_4-2"
         model="l10n_br_fiscal.document.line"
@@ -350,13 +294,6 @@
         <value eval="[ref('demo_nfe_line_nao_contribuinte_4-2')]" />
     </function>
 
-    <function
-        model="l10n_br_fiscal.document.line"
-        name="_onchange_fiscal_operation_line_id"
-    >
-        <value eval="[ref('demo_nfe_line_nao_contribuinte_4-2')]" />
-    </function>
-
     <!-- NFe Test - Empresa Contribuinte - Nao Contribuinte PJ Tax Benefit -->
     <record id="demo_nfe_tax_benefit" model="l10n_br_fiscal.document">
         <field name="fiscal_operation_id" ref="l10n_br_fiscal.fo_venda" />
@@ -402,13 +339,6 @@
         <value eval="[ref('demo_nfe_tax_benefit_4-1')]" />
     </function>
 
-    <function
-        model="l10n_br_fiscal.document.line"
-        name="_onchange_fiscal_operation_line_id"
-    >
-        <value eval="[ref('demo_nfe_tax_benefit_4-1')]" />
-    </function>
-
     <!-- NFe Test - Empresa Contribuinte - Nao Contribuinte PF -->
     <record id="demo_nfe_nao_contribuinte_pf" model="l10n_br_fiscal.document">
         <field name="fiscal_operation_id" ref="l10n_br_fiscal.fo_venda" />
@@ -449,13 +379,6 @@
         <value eval="[ref('demo_nfe_nao_contribuinte_pf_1-2')]" />
     </function>
 
-    <function
-        model="l10n_br_fiscal.document.line"
-        name="_onchange_fiscal_operation_line_id"
-    >
-        <value eval="[ref('demo_nfe_nao_contribuinte_pf_1-2')]" />
-    </function>
-
     <record id="demo_nfe_nao_contribuinte_pf_2-2" model="l10n_br_fiscal.document.line">
         <field name="document_id" ref="demo_nfe_nao_contribuinte_pf" />
         <field name="name">Teste</field>
@@ -481,13 +404,6 @@
         <value eval="[ref('demo_nfe_nao_contribuinte_pf_2-2')]" />
     </function>
 
-    <function
-        model="l10n_br_fiscal.document.line"
-        name="_onchange_fiscal_operation_line_id"
-    >
-        <value eval="[ref('demo_nfe_nao_contribuinte_pf_2-2')]" />
-    </function>
-
     <!-- NFe Test - Empresa Simples Nacional - Mesmo Estado -->
     <record id="demo_nfe_sn_same_state" model="l10n_br_fiscal.document">
         <field name="company_id" ref="l10n_br_base.empresa_simples_nacional" />
@@ -525,13 +441,6 @@
         <value eval="[ref('demo_nfe_line_sn_same_state_5-1')]" />
     </function>
 
-    <function
-        model="l10n_br_fiscal.document.line"
-        name="_onchange_fiscal_operation_line_id"
-    >
-        <value eval="[ref('demo_nfe_line_sn_same_state_5-1')]" />
-    </function>
-
     <record id="demo_nfe_line_sn_same_state_5-2" model="l10n_br_fiscal.document.line">
         <field name="document_id" ref="demo_nfe_sn_same_state" />
         <field name="name">Teste</field>
@@ -554,13 +463,6 @@
         <value eval="[ref('demo_nfe_line_sn_same_state_5-2')]" />
     </function>
 
-    <function
-        model="l10n_br_fiscal.document.line"
-        name="_onchange_fiscal_operation_line_id"
-    >
-        <value eval="[ref('demo_nfe_line_sn_same_state_5-2')]" />
-    </function>
-
     <!-- NFe Test - Empresa Simples Nacional - Outro Estado -->
     <record id="demo_nfe_sn_other_state" model="l10n_br_fiscal.document">
         <field name="company_id" ref="l10n_br_base.empresa_simples_nacional" />
@@ -598,13 +500,6 @@
         <value eval="[ref('demo_nfe_line_sn_other_state_6-1')]" />
     </function>
 
-    <function
-        model="l10n_br_fiscal.document.line"
-        name="_onchange_fiscal_operation_line_id"
-    >
-        <value eval="[ref('demo_nfe_line_sn_other_state_6-1')]" />
-    </function>
-
     <record id="demo_nfe_line_sn_other_state_6-2" model="l10n_br_fiscal.document.line">
         <field name="document_id" ref="demo_nfe_sn_other_state" />
         <field name="name">Teste</field>
@@ -627,13 +522,6 @@
         <value eval="[ref('demo_nfe_line_sn_other_state_6-2')]" />
     </function>
 
-    <function
-        model="l10n_br_fiscal.document.line"
-        name="_onchange_fiscal_operation_line_id"
-    >
-        <value eval="[ref('demo_nfe_line_sn_other_state_6-2')]" />
-    </function>
-
     <!-- NFe Test - Empresa Simples Nacional - Exterior -->
     <record id="demo_nfe_sn_export" model="l10n_br_fiscal.document">
         <field name="company_id" ref="l10n_br_base.empresa_simples_nacional" />
@@ -671,13 +559,6 @@
         <value eval="[ref('demo_nfe_line_sn_export_7-1')]" />
     </function>
 
-    <function
-        model="l10n_br_fiscal.document.line"
-        name="_onchange_fiscal_operation_line_id"
-    >
-        <value eval="[ref('demo_nfe_line_sn_export_7-1')]" />
-    </function>
-
     <record id="demo_nfe_line_sn_export_7-2" model="l10n_br_fiscal.document.line">
         <field name="document_id" ref="demo_nfe_sn_export" />
         <field name="name">Teste</field>
@@ -700,13 +581,6 @@
         <value eval="[ref('demo_nfe_line_sn_export_7-2')]" />
     </function>
 
-    <function
-        model="l10n_br_fiscal.document.line"
-        name="_onchange_fiscal_operation_line_id"
-    >
-        <value eval="[ref('demo_nfe_line_sn_export_7-2')]" />
-    </function>
-
     <!-- NFe Test - Empresa Simples Nacional - Outro Estado -->
     <record id="demo_nfe_sn_nao_contribuinte" model="l10n_br_fiscal.document">
         <field name="company_id" ref="l10n_br_base.empresa_simples_nacional" />
@@ -750,13 +624,6 @@
         <value eval="[ref('demo_nfe_line_sn_nao_contribuinte_7-1')]" />
     </function>
 
-    <function
-        model="l10n_br_fiscal.document.line"
-        name="_onchange_fiscal_operation_line_id"
-    >
-        <value eval="[ref('demo_nfe_line_sn_nao_contribuinte_7-1')]" />
-    </function>
-
     <record
         id="demo_nfe_line_sn_nao_contribuinte_7-2"
         model="l10n_br_fiscal.document.line"
@@ -785,13 +652,6 @@
         <value eval="[ref('demo_nfe_line_sn_nao_contribuinte_7-2')]" />
     </function>
 
-    <function
-        model="l10n_br_fiscal.document.line"
-        name="_onchange_fiscal_operation_line_id"
-    >
-        <value eval="[ref('demo_nfe_line_sn_nao_contribuinte_7-2')]" />
-    </function>
-
     <!-- NFe Test - Subsequent Operation - Simples Faturamento -->
     <record id="demo_nfe_so_simples_faturamento" model="l10n_br_fiscal.document">
         <field name="company_id" ref="l10n_br_base.empresa_lucro_presumido" />
@@ -843,13 +703,6 @@
         <value eval="[ref('demo_nfe_line_so_simples_faturamento_7-1')]" />
     </function>
 
-    <function
-        model="l10n_br_fiscal.document.line"
-        name="_onchange_fiscal_operation_line_id"
-    >
-        <value eval="[ref('demo_nfe_line_so_simples_faturamento_7-1')]" />
-    </function>
-
     <!-- NFe Compras Test - Empresa Contribuinte - Mesmo Estado -->
     <record id="demo_nfe_purchase_same_state" model="l10n_br_fiscal.document">
         <field name="fiscal_operation_id" ref="l10n_br_fiscal.fo_compras" />
@@ -894,12 +747,4 @@
         <value eval="[ref('demo_nfe_purchase_line_same_state_1-1')]" />
     </function>
 
-    <function
-        model="l10n_br_fiscal.document.line"
-        name="_onchange_fiscal_operation_line_id"
-    >
-        <value eval="[ref('demo_nfe_purchase_line_same_state_1-1')]" />
-    </function>
-
-
 </odoo>

odoo/addons/l10n_br_fiscal/models/document.py

@@ -596,7 +596,6 @@ class Document(models.Model):
 
             new_doc = record.copy()
             new_doc.fiscal_operation_id = fsc_op
-            new_doc._onchange_fiscal_operation_id()
 
             for line in new_doc.fiscal_line_ids:
                 fsc_op_line = line.fiscal_operation_id.return_fiscal_operation_id
@@ -658,14 +657,3 @@ class Document(models.Model):
     def _compute_edoc_purpose(self):
         for record in self:
             record.edoc_purpose = record.fiscal_operation_id.edoc_purpose
-
-    @api.onchange("fiscal_operation_id")
-    def _onchange_fiscal_operation_id(self):
-        result = super()._onchange_fiscal_operation_id()
-        if self.fiscal_operation_id:
-            self.fiscal_operation_type = self.fiscal_operation_id.fiscal_operation_type
-
-        if self.issuer == DOCUMENT_ISSUER_COMPANY and not self.document_type_id:
-            self.document_type_id = self.company_id.document_type_id
-
-        return result

odoo/addons/l10n_br_fiscal/models/document_line.py

@@ -36,6 +36,7 @@ class DocumentLine(models.Model):
         comodel_name="res.company",
         related="document_id.company_id",
         store=True,
+        precompute=True,
         string="Company",
     )
 
@@ -46,6 +47,7 @@ class DocumentLine(models.Model):
     partner_id = fields.Many2one(
         related="document_id.partner_id",
         store=True,
+        precompute=True,
     )
 
     quantity = fields.Float(default=1.0)

odoo/addons/l10n_br_fiscal/models/document_line_mixin.py

@@ -197,6 +197,10 @@ class FiscalDocumentLineMixin(models.AbstractModel):
         comodel_name="l10n_br_fiscal.cfop",
         string="CFOP",
         domain="[('type_in_out', '=', fiscal_operation_type)]",
+        compute="_compute_fiscal_tax_ids",
+        store=True,
+        precompute=True,
+        readonly=False,
     )
 
     cfop_destination = fields.Selection(
@@ -239,6 +243,10 @@ class FiscalDocumentLineMixin(models.AbstractModel):
     fiscal_tax_ids = fields.Many2many(
         comodel_name="l10n_br_fiscal.tax",
         string="Fiscal Taxes",
+        compute="_compute_fiscal_tax_ids",
+        store=True,
+        precompute=True,
+        readonly=False,
     )
 
     amount_fiscal = fields.Monetary(
@@ -398,6 +406,10 @@ class FiscalDocumentLineMixin(models.AbstractModel):
             ("is_benefit", "=", True),
             ("tax_domain", "=", TAX_DOMAIN_ICMS),
         ],
+        compute="_compute_fiscal_tax_ids",
+        store=True,
+        precompute=True,
+        readonly=False,
     )
 
     icms_tax_benefit_code = fields.Char(
@@ -614,6 +626,10 @@ class FiscalDocumentLineMixin(models.AbstractModel):
         comodel_name="l10n_br_fiscal.tax.ipi.guideline",
         string="IPI Guideline",
         domain="['|', ('cst_in_id', '=', ipi_cst_id),('cst_out_id', '=', ipi_cst_id)]",
+        compute="_compute_fiscal_tax_ids",
+        store=True,
+        precompute=True,
+        readonly=False,
     )
 
     # IPI Devolvido Fields
@@ -920,6 +936,10 @@ class FiscalDocumentLineMixin(models.AbstractModel):
         comodel_name="l10n_br_fiscal.comment",
         string="Comments",
         domain=[("object", "=", FISCAL_COMMENT_LINE)],
+        compute="_compute_fiscal_tax_ids",
+        store=True,
+        precompute=True,
+        readonly=False,
     )
 
     additional_data = fields.Text()

odoo/addons/l10n_br_fiscal/models/document_line_mixin_methods.py

@@ -210,7 +210,7 @@ class FiscalDocumentLineMixinMethods(models.AbstractModel):
     def _compute_taxes(self, taxes, cst=None):
         self.ensure_one()
         return taxes.compute_taxes(
-            company=self.company_id,
+            company=self._get_fiscal_company(),
             partner=self._get_fiscal_partner(),
             product=self.product_id,
             price_unit=self.price_unit,
@@ -234,7 +234,7 @@ class FiscalDocumentLineMixinMethods(models.AbstractModel):
             icmssn_range=self.icmssn_range_id,
             icms_origin=self.icms_origin,
             icms_cst_id=self.icms_cst_id,
-            ind_final=self.ind_final,
+            ind_final=self._get_ind_final(),
             icms_relief_id=self.icms_relief_id,
         )
 
@@ -321,7 +321,7 @@ class FiscalDocumentLineMixinMethods(models.AbstractModel):
 
     def _update_fiscal_taxes(self):
         for line in self:
-            compute_result = self._compute_taxes(line.fiscal_tax_ids)
+            compute_result = line._compute_taxes(line.fiscal_tax_ids)
             to_update = {
                 "amount_tax_included": compute_result.get("amount_included", 0.0),
                 "amount_tax_not_included": compute_result.get(
@@ -332,7 +332,7 @@ class FiscalDocumentLineMixinMethods(models.AbstractModel):
             }
             to_update.update(line._prepare_tax_fields(compute_result))
 
-            in_draft_mode = self != self._origin
+            in_draft_mode = line != line._origin
             if in_draft_mode:
                 line.update(to_update)
             else:
@@ -389,6 +389,24 @@ class FiscalDocumentLineMixinMethods(models.AbstractModel):
         self.ensure_one()
         return self.partner_id
 
+    def _get_fiscal_company(self):
+        """
+        Meant to be overriden in account.move.line.
+        Indeed when using this mixin in a NewID fiscal document line behind an aml,
+        self.company_id is None but it can be retrived from the aml.
+        """
+        self.ensure_one()
+        return self.company_id
+
+    def _get_ind_final(self):
+        """
+        Meant to be overriden in account.move.line.
+        Indeed when using this mixin in a NewID fiscal document line behind an aml,
+        self.ind_final is None but it can be retrived from the aml.
+        """
+        self.ensure_one()
+        return self.ind_final
+
     @api.onchange(
         "fiscal_operation_id", "ncm_id", "nbs_id", "cest_id", "service_type_id"
     )
@@ -401,43 +419,62 @@ class FiscalDocumentLineMixinMethods(models.AbstractModel):
                 partner=self._get_fiscal_partner(),
                 product=self.product_id,
             )
-            self._onchange_fiscal_operation_line_id()
-
-    @api.onchange("fiscal_operation_line_id")
-    def _onchange_fiscal_operation_line_id(self):
-        # Reset Taxes
-        self._remove_all_fiscal_tax_ids()
-        if self.fiscal_operation_line_id:
-            mapping_result = self.fiscal_operation_line_id.map_fiscal_taxes(
-                company=self.company_id,
-                partner=self._get_fiscal_partner(),
-                product=self.product_id,
-                ncm=self.ncm_id,
-                nbm=self.nbm_id,
-                nbs=self.nbs_id,
-                cest=self.cest_id,
-                city_taxation_code=self.city_taxation_code_id,
-                service_type=self.service_type_id,
-                ind_final=self.ind_final,
-            )
+            self._compute_fiscal_tax_ids()  # sadly seems required for composite move
 
-            self.cfop_id = mapping_result["cfop"]
-            if self._is_imported():
-                return
-            self._process_fiscal_mapping(mapping_result)
-
-        if not self.fiscal_operation_line_id:
-            self.cfop_id = False
-
-    def _process_fiscal_mapping(self, mapping_result):
-        self.ipi_guideline_id = mapping_result["ipi_guideline"]
-        self.icms_tax_benefit_id = mapping_result["icms_tax_benefit_id"]
-        taxes = self.env["l10n_br_fiscal.tax"]
-        for tax in mapping_result["taxes"].values():
-            taxes |= tax
-        self.fiscal_tax_ids = taxes
-        self._update_fiscal_taxes()
-        self.comment_ids = self.fiscal_operation_line_id.comment_ids
+    def _get_fiscal_tax_ids_dependencies(self):
+        """
+        Dynamically get the list of fields dependencies, overriden in l10n_br_purchase.
+        """
+        return [
+            "company_id",
+            "partner_id",
+            "fiscal_operation_line_id",
+            "product_id",
+            "ncm_id",
+            "nbs_id",
+            "nbm_id",
+            "cest_id",
+            "city_taxation_code_id",
+            "service_type_id",
+            "ind_final",
+        ]
+
+    @api.depends(lambda self: self._get_fiscal_tax_ids_dependencies())
+    def _compute_fiscal_tax_ids(self):
+        """
+        Among the dependencies, company_id, partner_id and ind_final are related
+        to the fiscal document/line container. When called from account.move.line
+        via _inherits on newID records, we read these values from the related aml
+        to work around and _inherits/precompute limitation.
+        """
+        for line in self:
+            line._remove_all_fiscal_tax_ids()
+            if line.fiscal_operation_line_id:
+                mapping_result = line.fiscal_operation_line_id.map_fiscal_taxes(
+                    company=line._get_fiscal_company(),
+                    partner=line._get_fiscal_partner(),
+                    product=line.product_id,
+                    ncm=line.ncm_id,
+                    nbm=line.nbm_id,
+                    nbs=line.nbs_id,
+                    cest=line.cest_id,
+                    city_taxation_code=line.city_taxation_code_id,
+                    service_type=line.service_type_id,
+                    ind_final=line._get_ind_final(),
+                )
+                line.cfop_id = mapping_result["cfop"]
+                if line._is_imported():
+                    return
+                line.ipi_guideline_id = mapping_result["ipi_guideline"]
+                line.icms_tax_benefit_id = mapping_result["icms_tax_benefit_id"]
+                taxes = line.env["l10n_br_fiscal.tax"]
+                for tax in mapping_result["taxes"].values():
+                    taxes |= tax
+                line.fiscal_tax_ids = taxes
+                line._update_fiscal_taxes()
+                line.comment_ids = line.fiscal_operation_line_id.comment_ids
+            else:
+                line.cfop_id = False
 
     @api.onchange("product_id")
     def _onchange_product_id_fiscal(self):

odoo/addons/l10n_br_fiscal/models/document_mixin.py

@@ -33,7 +33,6 @@ class FiscalDocumentMixin(models.AbstractModel):
     - Inverse methods for distributing header-level costs (freight, insurance)
       to lines.
     - Hooks for customizing data retrieval (e.g., lines, fiscal partner).
-    - Onchange helpers for common fiscal fields.
 
     Models using this mixin are often expected to also include fields defined
     in `l10n_br_fiscal.document.mixin` for methods like
@@ -90,7 +89,6 @@ class FiscalDocumentMixin(models.AbstractModel):
 
     fiscal_operation_type = fields.Selection(
         related="fiscal_operation_id.fiscal_operation_type",
-        readonly=True,
     )
 
     ind_pres = fields.Selection(
@@ -481,6 +479,10 @@ class FiscalDocumentMixin(models.AbstractModel):
 
     document_type_id = fields.Many2one(
         comodel_name="l10n_br_fiscal.document.type",
+        compute="_compute_document_type_id",
+        store=True,
+        precompute=True,
+        readonly=False,
     )
 
     document_serie_id = fields.Many2one(

odoo/addons/l10n_br_fiscal/models/document_mixin_methods.py

@@ -57,13 +57,11 @@ class FiscalDocumentMixinMethods(models.AbstractModel):
                 self.company_id, self.fiscal_operation_id
             )
 
-    @api.onchange("fiscal_operation_id")
-    def _onchange_fiscal_operation_id(self):
-        if self.fiscal_operation_id:
-            self.fiscal_operation_type = self.fiscal_operation_id.fiscal_operation_type
-
-            if self.issuer == DOCUMENT_ISSUER_COMPANY and not self.document_type_id:
-                self.document_type_id = self.company_id.document_type_id
+    @api.depends("fiscal_operation_id")
+    def _compute_document_type_id(self):
+        for doc in self.filtered(lambda doc: doc.fiscal_operation_id):
+            if doc.issuer == DOCUMENT_ISSUER_COMPANY and not doc.document_type_id:
+                doc.document_type_id = doc.company_id.document_type_id
 
     def _get_amount_lines(self):
         """Get object lines instances used to compute fiscal fields"""

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

@@ -372,7 +372,7 @@ ul.auto-toc {
 !! This file is generated by oca-gen-addon-readme !!
 !! changes will be overwritten.                   !!
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
-!! source digest: sha256:3612e46c6fb955a94d39c6bced2a46abadbb539336d76697fbd63869b54cdac5
+!! source digest: sha256:dd9f065758d59e68e7d24a53621956cc9e6911bb9e58b0ac89b2e202085a51dc
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! -->
 <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>

odoo/addons/l10n_br_fiscal/tests/test_fiscal_document_generic.py

@@ -39,9 +39,6 @@ class TestFiscalDocumentGeneric(TransactionCase):
 
     def test_nfe_same_state(self):
         """Test NFe same state."""
-
-        self.nfe_same_state._onchange_fiscal_operation_id()
-
         for line in self.nfe_same_state.fiscal_line_ids:
             line._onchange_product_id_fiscal()
 
@@ -164,9 +161,6 @@ class TestFiscalDocumentGeneric(TransactionCase):
 
     def test_nfe_other_state(self):
         """Test NFe other state."""
-
-        self.nfe_other_state._onchange_fiscal_operation_id()
-
         for line in self.nfe_other_state.fiscal_line_ids:
             line._onchange_product_id_fiscal()
             line._onchange_fiscal_operation_id()
@@ -282,9 +276,6 @@ class TestFiscalDocumentGeneric(TransactionCase):
 
     def test_nfe_not_taxpayer(self):
         """Test NFe not taxpayer."""
-
-        self.nfe_not_taxpayer._onchange_fiscal_operation_id()
-
         for line in self.nfe_not_taxpayer.fiscal_line_ids:
             line._onchange_product_id_fiscal()
             line._onchange_fiscal_operation_id()
@@ -387,9 +378,6 @@ class TestFiscalDocumentGeneric(TransactionCase):
 
     def test_nfe_not_taxpayer_not_company(self):
         """Test NFe not taxpayer not Company."""
-
-        self.nfe_not_taxpayer_pf._onchange_fiscal_operation_id()
-
         for line in self.nfe_not_taxpayer_pf.fiscal_line_ids:
             line._onchange_product_id_fiscal()
             line._onchange_fiscal_operation_id()
@@ -492,9 +480,6 @@ class TestFiscalDocumentGeneric(TransactionCase):
 
     def test_nfe_export(self):
         """Test NFe export."""
-
-        self.nfe_export._onchange_fiscal_operation_id()
-
         for line in self.nfe_export.fiscal_line_ids:
             line._onchange_product_id_fiscal()
             line._onchange_fiscal_operation_id()
@@ -589,9 +574,6 @@ class TestFiscalDocumentGeneric(TransactionCase):
 
     def test_nfe_sn_same_state(self):
         """Test NFe Simples Nacional same state."""
-
-        self.nfe_sn_same_state._onchange_fiscal_operation_id()
-
         for line in self.nfe_sn_same_state.fiscal_line_ids:
             line._onchange_product_id_fiscal()
 
@@ -705,9 +687,6 @@ class TestFiscalDocumentGeneric(TransactionCase):
 
     def test_nfe_sn_other_state(self):
         """Test NFe SN other state."""
-
-        self.nfe_sn_other_state._onchange_fiscal_operation_id()
-
         for line in self.nfe_sn_other_state.fiscal_line_ids:
             line._onchange_product_id_fiscal()
             line._onchange_fiscal_operation_id()
@@ -806,9 +785,6 @@ class TestFiscalDocumentGeneric(TransactionCase):
 
     def test_nfe_sn_not_taxpayer(self):
         """Test NFe SN not taxpayer."""
-
-        self.nfe_sn_not_taxpayer._onchange_fiscal_operation_id()
-
         for line in self.nfe_sn_not_taxpayer.fiscal_line_ids:
             line._onchange_product_id_fiscal()
             line._onchange_fiscal_operation_id()
@@ -894,9 +870,6 @@ class TestFiscalDocumentGeneric(TransactionCase):
 
     def test_nfe_sn_export(self):
         """Test NFe SN export."""
-
-        self.nfe_sn_export._onchange_fiscal_operation_id()
-
         for line in self.nfe_sn_export.fiscal_line_ids:
             line._onchange_product_id_fiscal()
             line._onchange_fiscal_operation_id()

odoo/addons/l10n_br_fiscal/tests/test_fiscal_document_nfse.py

@@ -14,8 +14,6 @@ class TestFiscalDocumentNFSe(TransactionCase):
     def test_nfse_same_state(self):
         """Test NFSe same state."""
 
-        self.nfse_same_state._onchange_fiscal_operation_id()
-
         for line in self.nfse_same_state.fiscal_line_ids:
             line._onchange_product_id_fiscal()
             line._onchange_fiscal_operation_id()

odoo/addons/l10n_br_fiscal/tests/test_tax_benefit.py

@@ -39,8 +39,6 @@ class TestTaxBenefit(TransactionCase):
     def test_nfe_tax_benefit(self):
         """Test NFe with tax benefit."""
 
-        self.nfe_tax_benefit._onchange_fiscal_operation_id()
-
         for line in self.nfe_tax_benefit.fiscal_line_ids:
             line._onchange_product_id_fiscal()
             line._onchange_fiscal_operation_id()

{odoo_addon_l10n_br_fiscal-16.0.11.3.0.dist-info → odoo_addon_l10n_br_fiscal-16.0.11.5.0.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: odoo-addon-l10n_br_fiscal
-Version: 16.0.11.3.0
+Version: 16.0.11.5.0
 Requires-Python: >=3.10
 Requires-Dist: erpbrasil.base>=2.3.0
 Requires-Dist: odoo-addon-l10n_br_base>=16.0dev,<16.1dev
@@ -31,7 +31,7 @@ Módulo fiscal brasileiro
    !! This file is generated by oca-gen-addon-readme !!
    !! changes will be overwritten.                   !!
    !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
-   !! source digest: sha256:3612e46c6fb955a94d39c6bced2a46abadbb539336d76697fbd63869b54cdac5
+   !! source digest: sha256:dd9f065758d59e68e7d24a53621956cc9e6911bb9e58b0ac89b2e202085a51dc
    !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
 
 .. |badge1| image:: https://img.shields.io/badge/maturity-Production%2FStable-green.png

{odoo_addon_l10n_br_fiscal-16.0.11.3.0.dist-info → odoo_addon_l10n_br_fiscal-16.0.11.5.0.dist-info}/RECORD

@@ -1,6 +1,6 @@
-odoo/addons/l10n_br_fiscal/README.rst,sha256=SNSUVGa1x5qhAzVimxpLUAu4FXRUPQP62LoW1Rq-0LQ,13957
+odoo/addons/l10n_br_fiscal/README.rst,sha256=jKHw-vzleAFwqeOUH1HJ-zbjTBownBUkfxpHDwAjC5A,13957
 odoo/addons/l10n_br_fiscal/__init__.py,sha256=BQXfCjW4ehK1W1j0z6k8xN7Q2LoZBCjjvYDkQt6nSkI,2717
-odoo/addons/l10n_br_fiscal/__manifest__.py,sha256=PAvQD_W6EKWV69LoL8UZg6lJ0PXQkI_-YtW6Xb2GQfk,4964
+odoo/addons/l10n_br_fiscal/__manifest__.py,sha256=WxIyOq5PUiqu0iEAKeurN2Pr2LdgM2IQXNanzSnuX3E,4964
 odoo/addons/l10n_br_fiscal/tools.py,sha256=HgaCLkPznFxgLfjb76nH36PJ7bN5W7-fdaqtBvM6XC0,4056
 odoo/addons/l10n_br_fiscal/constants/fiscal.py,sha256=F2a4jYAk2j2Yh1Xw7WeS1GcLfW1DbjeNgEF6l7135wc,14962
 odoo/addons/l10n_br_fiscal/constants/icms.py,sha256=rHlM6jyRvhrIm03sd6guS1xh7SptADLltPmRJ3C_Vh8,3512
@@ -45,7 +45,7 @@ odoo/addons/l10n_br_fiscal/data/uom_data.xml,sha256=bDVsZn9KA5ozkZuPC9cyylPlsBtX
 odoo/addons/l10n_br_fiscal/demo/__init__.py,sha256=D9IO9xsEyTSXTF03sJ836cHDsB3snXXizs1SUA2IvwA,4420
 odoo/addons/l10n_br_fiscal/demo/city_taxation_code_demo.xml,sha256=tVMji5bhxl7cfsf48JjtwHfVVyMOm28WA9H5K6Zr2R0,476
 odoo/addons/l10n_br_fiscal/demo/company_demo.xml,sha256=XPZE-3nHNYdeSGkJ7XamGAWM7Vs5AwhFBl-3yZCc2fA,13429
-odoo/addons/l10n_br_fiscal/demo/fiscal_document_demo.xml,sha256=fvkXSpkyMm5biaCaMAQIr-xgaJW7HbJAQN3EmNRtwG4,38140
+odoo/addons/l10n_br_fiscal/demo/fiscal_document_demo.xml,sha256=IdsvMDxVtxn3PoSvScEaOHGgJngPxRHOZwSGEd69z0c,33787
 odoo/addons/l10n_br_fiscal/demo/fiscal_document_nfse_demo.xml,sha256=5YGmkP1dy-Q2-ZNUDI_dcr3TVMrjxnybnUBH1EQa-Nk,1867
 odoo/addons/l10n_br_fiscal/demo/fiscal_operation_demo.xml,sha256=Va_eLyQwHug8ZVgQ8lPkJyiU7zs8KU-ABqBk0RM4n3o,493
 odoo/addons/l10n_br_fiscal/demo/icms_tax_definition_demo.xml,sha256=38sImUwueccEEBrTnQ8_e4f614U10DIh8r9PYEXYexg,1033
@@ -69,12 +69,12 @@ odoo/addons/l10n_br_fiscal/models/cst.py,sha256=m3KIalTYxOoiydXje4VHni6knmXZVdR3
 odoo/addons/l10n_br_fiscal/models/data_abstract.py,sha256=l46d2ViejGvcjPtKxaA0vn4S0qrn8zqo4bGeQU6oOaU,4599
 odoo/addons/l10n_br_fiscal/models/data_ncm_nbs_abstract.py,sha256=a0K-32ockgmQV5WaljCCSZQYn8YmuYZ0CYqTDct0vLI,6758
 odoo/addons/l10n_br_fiscal/models/data_product_abstract.py,sha256=wbc3exJQp33aYK8IOJ490JvDWBPCpU0NEkMR6YjaLfQ,793
-odoo/addons/l10n_br_fiscal/models/document.py,sha256=4_a07OhC1k5NDluUou1c0-Np-BGm2wwC0XTeEvqg0Ko,19823
-odoo/addons/l10n_br_fiscal/models/document_line.py,sha256=Y87Bevv0wDWP_oRUUvjKsWdow_dvQEKQ-31KEIdZghY,2032
-odoo/addons/l10n_br_fiscal/models/document_line_mixin.py,sha256=1rY8wJ-pTwZeg4kJhDO4E_MNU22ARb_DkIpw6Rk6Tpw,28250
-odoo/addons/l10n_br_fiscal/models/document_line_mixin_methods.py,sha256=FQpYqFC5ywdLul471RlyzUZGkKv5ENdSVsEu31wvx7A,32665
-odoo/addons/l10n_br_fiscal/models/document_mixin.py,sha256=G63Mr1XyX6YXrdCg0bGIjjvrJPmAs8UuzExKYheVOqg,13477
-odoo/addons/l10n_br_fiscal/models/document_mixin_methods.py,sha256=XYrhXQs1blFCEzwgPGToGkWP7uyUnptKXBRwLRvu_FY,16390
+odoo/addons/l10n_br_fiscal/models/document.py,sha256=-0nT613qIHOz2sNPAhgrNM8pcVuw7_Sw7hSOF-nj6QY,19328
+odoo/addons/l10n_br_fiscal/models/document_line.py,sha256=rro8uzmZpAte7N04ycd-KKn2dyGJhin_Q-DjCaKm110,2082
+odoo/addons/l10n_br_fiscal/models/document_line_mixin.py,sha256=qfsjYBJChLyhb_Oy-1BgzduEbQ551bh3tXYUXSIl-OM,28810
+odoo/addons/l10n_br_fiscal/models/document_line_mixin_methods.py,sha256=-i71-iNkhNTv4ji8BLViTG6m-Fyk4pWGBcCpMQbeNJQ,34198
+odoo/addons/l10n_br_fiscal/models/document_mixin.py,sha256=hCi-HEIeOHVqk1CF1irIfOrqztRoHGxvmHaPA14-yCI,13519
+odoo/addons/l10n_br_fiscal/models/document_mixin_methods.py,sha256=jM_lJZC7Nipa6N9GSoUIHKVrS6JmhtxKzmeB7QYqpC4,16326
 odoo/addons/l10n_br_fiscal/models/document_related.py,sha256=SKUToxuoC2eOZAG8pJnBg0o5269JjvLbpvuy0M8zSlQ,4689
 odoo/addons/l10n_br_fiscal/models/document_serie.py,sha256=Spe7kpgMBFGS_LBv3DpiqTC4WuafTK8LRv53GVR2UcM,4128
 odoo/addons/l10n_br_fiscal/models/document_supplement.py,sha256=31oRSHT_SaGSf-dQvriweIxZZiTANxkHHupc5kMYfXA,409
@@ -124,7 +124,7 @@ odoo/addons/l10n_br_fiscal/readme/USAGE.md,sha256=Dw3uZaHJCUtV0xFq6VOQvF2G5fS_oS
 odoo/addons/l10n_br_fiscal/security/fiscal_security.xml,sha256=PxRef68OVjlCVMOJZbhieVxivgKJYJm9r3TiZO4RerA,3362
 odoo/addons/l10n_br_fiscal/security/ir.model.access.csv,sha256=RIVYOOLQ2cOWGp9f8hvKflwglZxAN6exGcOwqCEgDuw,14369
 odoo/addons/l10n_br_fiscal/static/description/icon.png,sha256=Vd1HydYBoGCzNfCqxLlch2i2aeCcyxo-uRxWNp6oMbw,14836
-odoo/addons/l10n_br_fiscal/static/description/index.html,sha256=wHOHgpfVNSGx9lSUYtNyA1qJKZec5Ii96W1FnXhO0pc,26637
+odoo/addons/l10n_br_fiscal/static/description/index.html,sha256=Kf4PBUg1LfcrewsNDdr7DLQNqtLuxPa_irOvxRYEApc,26637
 odoo/addons/l10n_br_fiscal/static/img/fiscal_dashboard.png,sha256=Q0fpqFNqEXh6m6E1aJfzSKV2tQ9lC1Y-ofUt6qxVupc,151668
 odoo/addons/l10n_br_fiscal/static/img/fiscal_line.png,sha256=S4Q4OGSzGnbfm4W5sQVvnD4uUzxS6tbJGT_gs3pB4K0,134276
 odoo/addons/l10n_br_fiscal/static/img/fiscal_operation.png,sha256=2614c1XjxwVznh707e9gujlUXg0ttutKD1ZiSMTqyv8,105871
@@ -133,8 +133,8 @@ odoo/addons/l10n_br_fiscal/static/src/js/list_renderer_with_button.esm.js,sha256
 odoo/addons/l10n_br_fiscal/tests/__init__.py,sha256=83m4BimGb-6EseEmmCVaKTSel5GB_vkzbI3YZi4mStE,393
 odoo/addons/l10n_br_fiscal/tests/test_cnae.py,sha256=mmskSfWaoFSohjmRbnGci_6Euc9fc2zbhyH5FdyeIeA,616
 odoo/addons/l10n_br_fiscal/tests/test_document_edition.py,sha256=w_ANtriwn9IpG3XvMruINMOIcZJyPwUY47iFSgz2bj0,5067
-odoo/addons/l10n_br_fiscal/tests/test_fiscal_document_generic.py,sha256=V8DHVC4sDNlJ_umoIaSkaesJfSEd31wija-bNgrBUZg,40547
-odoo/addons/l10n_br_fiscal/tests/test_fiscal_document_nfse.py,sha256=X5_pHhChyDL6UmAgWTaAhgPuKWiCKZ1tw95hZecSRcY,1600
+odoo/addons/l10n_br_fiscal/tests/test_fiscal_document_generic.py,sha256=nMrevgtmcMgtnZWt-nHZboKLiFO1V_AxfSxIWKd0uLw,39974
+odoo/addons/l10n_br_fiscal/tests/test_fiscal_document_nfse.py,sha256=nEMaaWfv-sb4o48c1-l7mVnSjks3Tue7XxnQb_Q4cqw,1538
 odoo/addons/l10n_br_fiscal/tests/test_fiscal_tax.py,sha256=U6JbuqKTcyyFTiW5FZGqGY5yi_aPKaC6_LWz_3sKdpg,11322
 odoo/addons/l10n_br_fiscal/tests/test_ibpt.py,sha256=vJKit2Aok-zIk_723nRietLTbVKPstD88vznN2PwMZg,5227
 odoo/addons/l10n_br_fiscal/tests/test_ibpt_product.py,sha256=h00TpY-tTw3z8U24yTzzvyWkfnKjHqlCbqetBKJNV94,2794
@@ -144,7 +144,7 @@ odoo/addons/l10n_br_fiscal/tests/test_ncm.py,sha256=legCHGki7r8xIC260jKdPpVUNC_Z
 odoo/addons/l10n_br_fiscal/tests/test_operation.py,sha256=zt3GJh3_z7p4zCFN2VSrUVgCGT7EQJ6GmjMGPIY98cg,480
 odoo/addons/l10n_br_fiscal/tests/test_partner_profile.py,sha256=aMjOQzygxVMQZnvdAEcR9Wun5igpTBzgOegkjsums68,738
 odoo/addons/l10n_br_fiscal/tests/test_service_type.py,sha256=k9Ro0pGVsiHslAa1NcLK-FlNnrFEeh394aw2tOF_M5o,556
-odoo/addons/l10n_br_fiscal/tests/test_tax_benefit.py,sha256=UHC7ZTnHKY12jK1zOlOm00ric8iBBYVL2boifPXOYPM,2053
+odoo/addons/l10n_br_fiscal/tests/test_tax_benefit.py,sha256=qfWnkg03AX42wowtLgwUKw1y3KEKh7DtkCOvGKokrBU,1991
 odoo/addons/l10n_br_fiscal/views/cest_view.xml,sha256=buiOX9AZ4OMHyjHZfoazcQQzjBSE7fT1S0LlYmo_3P4,3162
 odoo/addons/l10n_br_fiscal/views/cfop_view.xml,sha256=eSAo01pTaF2AY5_NfWJqY5H3WnaFmzfaSJ6l0DkyEcY,5018
 odoo/addons/l10n_br_fiscal/views/city_taxation_code.xml,sha256=SnoB8FmZdCzJhqKrEAXG3VVZDBIlDBpIq3oKycbx6uQ,2104
@@ -195,7 +195,7 @@ odoo/addons/l10n_br_fiscal/wizards/base_wizard_mixin.py,sha256=-r25us0vdNCSe11Gn
 odoo/addons/l10n_br_fiscal/wizards/document_import_wizard_mixin.py,sha256=KfbqnzFuVx7WAro__W130rgvMWKkDBvkDFglMwTwgBQ,4115
 odoo/addons/l10n_br_fiscal/wizards/document_import_wizard_mixin.xml,sha256=h19vP1YvC4hLPvCvA6E1goqre0AAGy0CF4a2R6B78Y4,1833
 odoo/addons/l10n_br_fiscal/wizards/document_status_wizard.py,sha256=KsYj5YWWePO7uk0psBsFdnCL71eLWhcyg7_c7J4G6vA,818
-odoo_addon_l10n_br_fiscal-16.0.11.3.0.dist-info/METADATA,sha256=8hS0DDZNvVuFj5WJxPc3iMq6u3MQfUn509lHpgaFrFs,14720
-odoo_addon_l10n_br_fiscal-16.0.11.3.0.dist-info/WHEEL,sha256=ZhOvUsYhy81Dx67gN3TV0RchQWBIIzutDZaJODDg2Vo,81
-odoo_addon_l10n_br_fiscal-16.0.11.3.0.dist-info/top_level.txt,sha256=QE6RBQ0QX5f4eFuUcGgU5Kbq1A_qJcDs-e_vpr6pmfU,4
-odoo_addon_l10n_br_fiscal-16.0.11.3.0.dist-info/RECORD,,
+odoo_addon_l10n_br_fiscal-16.0.11.5.0.dist-info/METADATA,sha256=eaQecQkRORdyybwDttNAjmethP2_797jaxEE7uAet5c,14720
+odoo_addon_l10n_br_fiscal-16.0.11.5.0.dist-info/WHEEL,sha256=ZhOvUsYhy81Dx67gN3TV0RchQWBIIzutDZaJODDg2Vo,81
+odoo_addon_l10n_br_fiscal-16.0.11.5.0.dist-info/top_level.txt,sha256=QE6RBQ0QX5f4eFuUcGgU5Kbq1A_qJcDs-e_vpr6pmfU,4
+odoo_addon_l10n_br_fiscal-16.0.11.5.0.dist-info/RECORD,,