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

odoo/addons/l10n_br_fiscal/models/document_line_mixin_methods.py

@@ -5,11 +5,10 @@ from copy import deepcopy
 
 from lxml import etree
 
-from odoo import api, models
+from odoo import Command, api, models
 
 from ..constants.fiscal import CFOP_DESTINATION_EXPORT, FISCAL_IN
 from ..constants.icms import ICMS_BASE_TYPE_DEFAULT, ICMS_ST_BASE_TYPE_DEFAULT
-from .tax import TAX_DICT_VALUES
 
 FISCAL_TAX_ID_FIELDS = [
     "cofins_tax_id",
@@ -207,37 +206,6 @@ class FiscalDocumentLineMixinMethods(models.AbstractModel):
                 record.financial_total_gross = record.financial_total = 0.0
                 record.financial_discount_value = 0.0
 
-    def _compute_taxes(self, taxes, cst=None):
-        self.ensure_one()
-        return taxes.compute_taxes(
-            company=self._get_fiscal_company(),
-            partner=self._get_fiscal_partner(),
-            product=self.product_id,
-            price_unit=self.price_unit,
-            quantity=self.quantity,
-            uom_id=self.uom_id,
-            fiscal_price=self.fiscal_price,
-            fiscal_quantity=self.fiscal_quantity,
-            uot_id=self.uot_id,
-            discount_value=self.discount_value,
-            insurance_value=self.insurance_value,
-            ii_customhouse_charges=self.ii_customhouse_charges,
-            ii_iof_value=self.ii_iof_value,
-            other_value=self.other_value,
-            freight_value=self.freight_value,
-            ncm=self.ncm_id,
-            nbs=self.nbs_id,
-            nbm=self.nbm_id,
-            cest=self.cest_id,
-            operation_line=self.fiscal_operation_line_id,
-            cfop=self.cfop_id,
-            icmssn_range=self.icmssn_range_id,
-            icms_origin=self.icms_origin,
-            icms_cst_id=self.icms_cst_id,
-            ind_final=self._get_ind_final(),
-            icms_relief_id=self.icms_relief_id,
-        )
-
     @api.depends("tax_icms_or_issqn", "partner_is_public_entity")
     def _compute_allow_csll_irpj(self):
         """Calculates the possibility of 'CSLL' and 'IRPJ' tax charges."""
@@ -265,78 +233,197 @@ class FiscalDocumentLineMixinMethods(models.AbstractModel):
             return {f"default_{k}": vals[k] for k in vals.keys()}
         return vals
 
-    def _get_all_tax_id_fields(self):
-        self.ensure_one()
-        taxes = self.env["l10n_br_fiscal.tax"]
+    @api.onchange("fiscal_operation_id", "company_id", "partner_id", "product_id")
+    def _onchange_fiscal_operation_id(self):
+        if self.fiscal_operation_id:
+            self.fiscal_operation_line_id = self.fiscal_operation_id.line_definition(
+                company=self.company_id,
+                partner=self._get_fiscal_partner(),
+                product=self.product_id,
+            )
 
-        for fiscal_tax_field in FISCAL_TAX_ID_FIELDS:
-            taxes |= self[fiscal_tax_field]
+    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",
+        ]
 
-        return taxes
+    @api.depends(lambda self: self._get_fiscal_tax_ids_dependencies())
+    def _compute_fiscal_tax_ids(self):
+        """
+        Use fiscal_operation_line_id to map and compute the applicable Brazilian taxes.
 
-    def _remove_all_fiscal_tax_ids(self):
-        if self._is_imported():
+        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.
+        """
+        if self._context.get("skip_compute_fiscal_tax_ids"):
             return
         for line in self:
-            to_update = {"fiscal_tax_ids": False}
-            for fiscal_tax_field in FISCAL_TAX_ID_FIELDS:
-                to_update[fiscal_tax_field] = False
-            tax_methods = [
-                self._prepare_fields_issqn,
-                self._prepare_fields_csll,
-                self._prepare_fields_irpj,
-                self._prepare_fields_inss,
-                self._prepare_fields_icms,
-                self._prepare_fields_icmsfcp,
-                self._prepare_fields_icmsfcpst,
-                self._prepare_fields_icmsst,
-                self._prepare_fields_icmssn,
-                self._prepare_fields_ipi,
-                self._prepare_fields_ii,
-                self._prepare_fields_pis,
-                self._prepare_fields_pisst,
-                self._prepare_fields_cofins,
-                self._prepare_fields_cofinsst,
-                self._prepare_fields_issqn_wh,
-                self._prepare_fields_pis_wh,
-                self._prepare_fields_cofins_wh,
-                self._prepare_fields_csll_wh,
-                self._prepare_fields_irpj_wh,
-                self._prepare_fields_inss_wh,
-            ]
-            for method in tax_methods:
-                prepared_fields = method(TAX_DICT_VALUES)
-                if prepared_fields:
-                    to_update.update(prepared_fields)
-            # Update all fields at once
-            line.update(to_update)
-
-    def _update_fiscal_tax_ids(self, taxes):
-        for line in self:
-            taxes_groups = line.fiscal_tax_ids.mapped("tax_domain")
-            fiscal_taxes = line.fiscal_tax_ids.filtered(
-                lambda ft, taxes_groups=taxes_groups: ft.tax_domain not in taxes_groups
+            if hasattr(line, "account_line_ids") and line.account_line_ids:
+                # it seems Odoo 16 ORM has a limitation when line is an
+                # l10n_br_fiscal.document.line that is edited via an account.move.line
+                # form and when both are a newID, then line relational field might be
+                # empty here. But in this case, we detect it and we wrap it back in the
+                wrapped_line = line.account_line_ids[0]
+            else:
+                wrapped_line = line
+
+            if wrapped_line.fiscal_operation_line_id:
+                mapping_result = wrapped_line.fiscal_operation_line_id.map_fiscal_taxes(
+                    company=wrapped_line.company_id,
+                    partner=wrapped_line._get_fiscal_partner(),
+                    product=wrapped_line.product_id,
+                    ncm=wrapped_line.ncm_id,
+                    nbm=wrapped_line.nbm_id,
+                    nbs=wrapped_line.nbs_id,
+                    cest=wrapped_line.cest_id,
+                    city_taxation_code=wrapped_line.city_taxation_code_id,
+                    service_type=wrapped_line.service_type_id,
+                    ind_final=wrapped_line.ind_final,
+                )
+                line.cfop_id = mapping_result["cfop"]
+                line.ipi_guideline_id = mapping_result["ipi_guideline"]
+                line.icms_tax_benefit_id = mapping_result["icms_tax_benefit_id"]
+                if wrapped_line._is_imported():
+                    return
+
+                taxes = line.env["l10n_br_fiscal.tax"]
+                for tax in mapping_result["taxes"].values():
+                    taxes |= tax
+                line.fiscal_tax_ids = taxes
+                line.comment_ids = line.fiscal_operation_line_id.comment_ids
+
+            else:
+                line.fiscal_tax_ids = [Command.clear()]
+
+    @api.model
+    def _build_null_mask_dict(self) -> dict:
+        """
+        Build a null values mask dict to reset all fiscal fields.
+        """
+        mask_dict = {
+            f[0]: False
+            for f in filter(
+                lambda f: f[1].compute == "_compute_tax_fields",
+                self.env["l10n_br_fiscal.document.line.mixin"]._fields.items(),
             )
-            line.fiscal_tax_ids = fiscal_taxes + taxes
+        }
+        for fiscal_tax_field in FISCAL_TAX_ID_FIELDS:
+            mask_dict[fiscal_tax_field] = False
+        return mask_dict
 
-    def _update_fiscal_taxes(self):
-        for line in self:
-            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(
-                    "amount_not_included", 0.0
-                ),
-                "amount_tax_withholding": compute_result.get("amount_withholding", 0.0),
-                "estimate_tax": compute_result.get("estimate_tax", 0.0),
-            }
-            to_update.update(line._prepare_tax_fields(compute_result))
+    def _get_tax_fields_dependencies(self):
+        """
+        Dynamically get the list of fields dependencies, overriden in l10n_br_purchase.
+        """
+        # IMPORTANT NOTE: as _compute_fiscal_tax_ids triggers _compute_tax_fields,
+        # we don't put fields that trigger _compute_fiscal_tax_ids as dependencies here.
+        return [
+            "price_unit",
+            "quantity",
+            "uom_id",
+            "fiscal_price",
+            "fiscal_quantity",
+            "uot_id",
+            "discount_value",
+            "insurance_value",
+            "ii_customhouse_charges",
+            "ii_iof_value",
+            "other_value",
+            "freight_value",
+            "cfop_id",
+            "icmssn_range_id",
+            "icms_origin",
+            "icms_cst_id",
+            "icms_relief_id",
+            "fiscal_tax_ids",
+        ]
+
+    @api.depends(lambda self: self._get_tax_fields_dependencies())
+    def _compute_tax_fields(self):
+        """
+        Compute base, percent, value... tax fields for ICMS, IPI, PIS, COFINS... taxes.
+        """
+        if self._context.get("skip_compute_tax_fields"):
+            return
 
-            in_draft_mode = line != line._origin
+        null_mask = None
+        for line in self.filtered(lambda line: not line._is_imported()):
+            if hasattr(line, "account_line_ids") and line.account_line_ids:
+                # it seems Odoo 16 ORM has a limitation when line is an
+                # l10n_br_fiscal.document.line that is edited via an account.move.line
+                # form and when both are a newID, then line relational field might be
+                # empty here. But in this case, we detect it and we wrap it back in the
+                wrapped_line = line.account_line_ids[0]
+            else:
+                wrapped_line = line
+
+            if null_mask is None:
+                null_mask = self._build_null_mask_dict()
+            to_update = null_mask.copy()
+            if wrapped_line.fiscal_operation_line_id:
+                compute_result = wrapped_line.fiscal_tax_ids.compute_taxes(
+                    company=wrapped_line.company_id,
+                    partner=wrapped_line._get_fiscal_partner(),
+                    product=wrapped_line.product_id,
+                    price_unit=wrapped_line.price_unit,
+                    quantity=wrapped_line.quantity,
+                    uom_id=wrapped_line.uom_id,
+                    fiscal_price=wrapped_line.fiscal_price,
+                    fiscal_quantity=wrapped_line.fiscal_quantity,
+                    uot_id=wrapped_line.uot_id,
+                    discount_value=wrapped_line.discount_value,
+                    insurance_value=wrapped_line.insurance_value,
+                    ii_customhouse_charges=wrapped_line.ii_customhouse_charges,
+                    ii_iof_value=wrapped_line.ii_iof_value,
+                    other_value=wrapped_line.other_value,
+                    freight_value=wrapped_line.freight_value,
+                    ncm=wrapped_line.ncm_id,
+                    nbs=wrapped_line.nbs_id,
+                    nbm=wrapped_line.nbm_id,
+                    cest=wrapped_line.cest_id,
+                    operation_line=wrapped_line.fiscal_operation_line_id,
+                    cfop=wrapped_line.cfop_id,
+                    icmssn_range=wrapped_line.icmssn_range_id,
+                    icms_origin=wrapped_line.icms_origin,
+                    icms_cst_id=wrapped_line.icms_cst_id,
+                    ind_final=wrapped_line.ind_final,
+                    icms_relief_id=wrapped_line.icms_relief_id,
+                )
+                to_update.update(wrapped_line._prepare_tax_fields(compute_result))
+            else:
+                compute_result = {}
+            to_update.update(
+                {
+                    "amount_tax_included": compute_result.get("amount_included", 0.0),
+                    "amount_tax_not_included": compute_result.get(
+                        "amount_not_included", 0.0
+                    ),
+                    "amount_tax_withholding": compute_result.get(
+                        "amount_withholding", 0.0
+                    ),
+                    "estimate_tax": compute_result.get("estimate_tax", 0.0),
+                }
+            )
+            in_draft_mode = wrapped_line != wrapped_line._origin
             if in_draft_mode:
-                line.update(to_update)
+                wrapped_line.update(to_update)
             else:
-                line.write(to_update)
+                wrapped_line.write(to_update)
 
     def _prepare_tax_fields(self, compute_result):
         self.ensure_one()
@@ -356,7 +443,11 @@ class FiscalDocumentLineMixinMethods(models.AbstractModel):
                         tax_values.update(prepared_fields)
         return tax_values
 
-    def _compute_price_unit_fiscal(self):
+    @api.depends(
+        "product_id",
+        "fiscal_operation_id",
+    )
+    def _compute_price_unit_fiscal(self):  # OK when edited from aml?? c-> check
         for line in self:
             line.price_unit = {
                 "sale_price": line.product_id.list_price,
@@ -368,7 +459,7 @@ class FiscalDocumentLineMixinMethods(models.AbstractModel):
         return {
             "user": self.env.user,
             "ctx": self._context,
-            "doc": self.document_id,
+            "doc": self.document_id if hasattr(self, "document_id") else None,
             "item": self,
         }
 
@@ -389,93 +480,6 @@ 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"
-    )
-    def _onchange_fiscal_operation_id(self):
-        if self.fiscal_operation_id:
-            if not self.price_unit:
-                self._compute_price_unit_fiscal()
-            self.fiscal_operation_line_id = self.fiscal_operation_id.line_definition(
-                company=self.company_id,
-                partner=self._get_fiscal_partner(),
-                product=self.product_id,
-            )
-            self._compute_fiscal_tax_ids()  # sadly seems required for composite move
-
-    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):
         if not self.fiscal_operation_id:
@@ -760,40 +764,18 @@ class FiscalDocumentLineMixinMethods(models.AbstractModel):
             "cofinsst_value": tax_dict.get("tax_value", 0.00),
         }
 
-    @api.onchange(
-        "csll_tax_id",
-        "csll_wh_tax_id",
-        "irpj_tax_id",
-        "irpj_wh_tax_id",
-        "inss_tax_id",
-        "inss_wh_tax_id",
-        "issqn_tax_id",
-        "issqn_wh_tax_id",
-        "icms_tax_id",
-        "icmssn_tax_id",
-        "icmsst_tax_id",
-        "icmsfcp_tax_id",
-        "icmsfcpst_tax_id",
-        "icms_relief_id",
-        "icms_relief_value",
-        "ipi_tax_id",
-        "ii_tax_id",
-        "pis_tax_id",
-        "pis_wh_tax_id",
-        "pisst_tax_id",
-        "cofins_tax_id",
-        "cofins_wh_tax_id",
-        "cofinsst_tax_id",
-        "fiscal_price",
-        "fiscal_quantity",
-        "discount_value",
-        "insurance_value",
-        "other_value",
-        "freight_value",
-    )
+    @api.onchange(*FISCAL_TAX_ID_FIELDS)
     def _onchange_fiscal_taxes(self):
-        self._update_fiscal_tax_ids(self._get_all_tax_id_fields())
-        self._update_fiscal_taxes()
+        taxes = self.env["l10n_br_fiscal.tax"]
+        for fiscal_tax_field in FISCAL_TAX_ID_FIELDS:
+            taxes |= self[fiscal_tax_field]
+
+        for line in self:
+            taxes_groups = line.fiscal_tax_ids.mapped("tax_domain")
+            fiscal_taxes = line.fiscal_tax_ids.filtered(
+                lambda ft, taxes_groups=taxes_groups: ft.tax_domain not in taxes_groups
+            )
+            line.fiscal_tax_ids = fiscal_taxes + taxes
 
     @api.depends("uom_id")
     def _compute_uot_id(self):
@@ -837,15 +819,6 @@ class FiscalDocumentLineMixinMethods(models.AbstractModel):
                 else:
                     line.fiscal_quantity = line.quantity
 
-    @api.onchange("ii_customhouse_charges")
-    def _onchange_ii_customhouse_charges(self):
-        if self.ii_customhouse_charges:
-            self._update_fiscal_taxes()
-
-    @api.onchange("fiscal_tax_ids")
-    def _onchange_fiscal_tax_ids(self):
-        self._update_fiscal_taxes()
-
     @api.onchange("city_taxation_code_id")
     def _onchange_city_taxation_code_id(self):
         if self.city_taxation_code_id:

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:dd9f065758d59e68e7d24a53621956cc9e6911bb9e58b0ac89b2e202085a51dc
+!! source digest: sha256:23217515f72b532774e760de0f276bfcd34336e630330b71ecd27146458063e8
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! -->
 <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_document_edition.py

@@ -24,7 +24,7 @@ class TestDocumentEdition(TransactionCase):
         doc_form.fiscal_operation_id = self.env.ref("l10n_br_fiscal.fo_venda")
         doc_form.ind_final = "1"
         product_id = self.env.ref("product.product_product_6")
-        product_id.list_price = 150  # we will later check we can set priceunit to 100
+        product_id.list_price = 150  # we will later check we can set price_unit to 100
         with doc_form.fiscal_line_ids.new() as line_form:
             original_method = type(
                 self.env["l10n_br_fiscal.operation.line"]
@@ -59,14 +59,58 @@ class TestDocumentEdition(TransactionCase):
                 ind_final="1",
             )
 
-            line_form.price_unit = 100
+            line_form.price_unit = 50
             line_form.quantity = 2
+            self.assertEqual(len(line_form.fiscal_tax_ids), 4)
+            self.assertEqual(
+                line_form.icms_tax_id, self.env.ref("l10n_br_fiscal.tax_icms_12")
+            )
+            self.assertEqual(line_form.icms_value, 12.0)
+            line_form.price_unit = 100
+            self.assertEqual(
+                line_form.icms_tax_id, self.env.ref("l10n_br_fiscal.tax_icms_12")
+            )
+            self.assertEqual(line_form.icms_value, 24.0)
+            self.assertEqual(
+                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")
+            )
 
         doc = doc_form.save()
-        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)
+        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(
+            line.icms_tax_id.id,
+            self.ref("l10n_br_fiscal.tax_icms_18"),
+        )
+        self.assertEqual(line.ipi_tax_id, self.env.ref("l10n_br_fiscal.tax_ipi_3_25"))
+        self.assertEqual(line.icms_value, 37.17)
 
     def test_product_fiscal_factor(self):
         doc_form = Form(
@@ -109,6 +153,10 @@ class TestDocumentEdition(TransactionCase):
             line_form.quantity = 10
             line_form.fiscal_price = 112
             line_form.fiscal_quantity = 5
+            self.assertEqual(line_form.price_unit, 110)
+            self.assertEqual(line_form.fiscal_price, 112)
+            self.assertEqual(line_form.quantity, 10)
+            self.assertEqual(line_form.fiscal_quantity, 5)
 
         doc = doc_form.save()
         self.assertEqual(doc.fiscal_line_ids[0].price_unit, 110)

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

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: odoo-addon-l10n_br_fiscal
-Version: 16.0.11.5.0
+Version: 16.0.12.0.1
 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:dd9f065758d59e68e7d24a53621956cc9e6911bb9e58b0ac89b2e202085a51dc
+   !! source digest: sha256:23217515f72b532774e760de0f276bfcd34336e630330b71ecd27146458063e8
    !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
 
 .. |badge1| image:: https://img.shields.io/badge/maturity-Production%2FStable-green.png

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

@@ -1,6 +1,6 @@
-odoo/addons/l10n_br_fiscal/README.rst,sha256=jKHw-vzleAFwqeOUH1HJ-zbjTBownBUkfxpHDwAjC5A,13957
+odoo/addons/l10n_br_fiscal/README.rst,sha256=80hZrWNk1JA-HCEmgj9LX4-UEfx4JjGyPlR2pI7RbOc,13957
 odoo/addons/l10n_br_fiscal/__init__.py,sha256=BQXfCjW4ehK1W1j0z6k8xN7Q2LoZBCjjvYDkQt6nSkI,2717
-odoo/addons/l10n_br_fiscal/__manifest__.py,sha256=WxIyOq5PUiqu0iEAKeurN2Pr2LdgM2IQXNanzSnuX3E,4964
+odoo/addons/l10n_br_fiscal/__manifest__.py,sha256=FmKjJs9WooHmMSIfxnkWkcgu8D_s1DlUCjvEQpkyrIE,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
@@ -71,8 +71,8 @@ odoo/addons/l10n_br_fiscal/models/data_ncm_nbs_abstract.py,sha256=a0K-32ockgmQV5
 odoo/addons/l10n_br_fiscal/models/data_product_abstract.py,sha256=wbc3exJQp33aYK8IOJ490JvDWBPCpU0NEkMR6YjaLfQ,793
 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_line_mixin.py,sha256=0s5u5_4_gXMs02f1pFtJQxfxqNfNbn1Dvo6sAvvVn10,43906
+odoo/addons/l10n_br_fiscal/models/document_line_mixin_methods.py,sha256=KJkN02mSfR2BjOYXlDPduBZw0Tda-JWEDCznnK417vc,34376
 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
@@ -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=Kf4PBUg1LfcrewsNDdr7DLQNqtLuxPa_irOvxRYEApc,26637
+odoo/addons/l10n_br_fiscal/static/description/index.html,sha256=mpnVGRdEteVBvr_vNQopHW4abO6ZNdxShfSWr6QPeMA,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
@@ -132,7 +132,7 @@ odoo/addons/l10n_br_fiscal/static/img/fiscal_total.png,sha256=MaR6ZoM6ZZEaNZk2yn
 odoo/addons/l10n_br_fiscal/static/src/js/list_renderer_with_button.esm.js,sha256=Ucw7pymMrW2yi_nNrDMvq6ubyub9QMlqH3ZOoaTEHN0,1243
 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_document_edition.py,sha256=aQYSQixnTQGk6xPQJ6-i5qbdbEHlua_-c7-tr-IT0V8,7064
 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
@@ -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.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,,
+odoo_addon_l10n_br_fiscal-16.0.12.0.1.dist-info/METADATA,sha256=n-n23qbzzpN0F4jEeYRy2cuTcxTfBRsh_7NYbCrsWqk,14720
+odoo_addon_l10n_br_fiscal-16.0.12.0.1.dist-info/WHEEL,sha256=ZhOvUsYhy81Dx67gN3TV0RchQWBIIzutDZaJODDg2Vo,81
+odoo_addon_l10n_br_fiscal-16.0.12.0.1.dist-info/top_level.txt,sha256=QE6RBQ0QX5f4eFuUcGgU5Kbq1A_qJcDs-e_vpr6pmfU,4
+odoo_addon_l10n_br_fiscal-16.0.12.0.1.dist-info/RECORD,,