odoo-addon-l10n-br-fiscal 17.0.1.0.0.11__py3-none-any.whl → 18.0.5.0.0.1__py3-none-any.whl

odoo/addons/l10n_br_fiscal/tests/test_fiscal_document_generic.py

@@ -40,15 +40,10 @@ class TestFiscalDocumentGeneric(TransactionCase):
     def test_nfe_same_state(self):
         """Test NFe same state."""
         for line in self.nfe_same_state.fiscal_line_ids:
-            line._onchange_product_id_fiscal()
-
             # Restore the original price_unit value,
             # 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,
@@ -77,9 +72,10 @@ class TestFiscalDocumentGeneric(TransactionCase):
             # ICMS
             self.assertTrue(
                 is_icms_internal,
-                "Error to mapping ICMS Inernal for {}"
+                "Error to mapping ICMS Inernal for "
+                f"{self.nfe_same_state.partner_id.state_id.name}"
                 " for Venda de Contribuinte Dentro do "
-                "Estado.".format(self.nfe_same_state.partner_id.state_id.name),
+                "Estado.",
             )
             self.assertEqual(
                 line.icms_cst_id.code,
@@ -162,10 +158,6 @@ 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_product_id_fiscal()
-            line._onchange_fiscal_operation_id()
-            line._onchange_fiscal_taxes()
-
             if "Revenda" in line.fiscal_operation_line_id.name:
                 self.assertEqual(
                     line.cfop_id.code,
@@ -277,10 +269,6 @@ 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_product_id_fiscal()
-            line._onchange_fiscal_operation_id()
-            line._onchange_fiscal_taxes()
-
             if "Revenda" in line.fiscal_operation_line_id.name:
                 self.assertEqual(
                     line.cfop_id.code,
@@ -379,10 +367,6 @@ 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_product_id_fiscal()
-            line._onchange_fiscal_operation_id()
-            line._onchange_fiscal_taxes()
-
             if "Revenda" in line.fiscal_operation_line_id.name:
                 self.assertEqual(
                     line.cfop_id.code,
@@ -481,10 +465,6 @@ class TestFiscalDocumentGeneric(TransactionCase):
     def test_nfe_export(self):
         """Test NFe export."""
         for line in self.nfe_export.fiscal_line_ids:
-            line._onchange_product_id_fiscal()
-            line._onchange_fiscal_operation_id()
-            line._onchange_fiscal_taxes()
-
             if "Revenda" in line.fiscal_operation_line_id.name:
                 self.assertEqual(
                     line.cfop_id.code,
@@ -575,8 +555,6 @@ class TestFiscalDocumentGeneric(TransactionCase):
     def test_nfe_sn_same_state(self):
         """Test NFe Simples Nacional same state."""
         for line in self.nfe_sn_same_state.fiscal_line_ids:
-            line._onchange_product_id_fiscal()
-
             # set fake estimate tax
             line.ncm_id.tax_estimate_ids.create(
                 {
@@ -587,9 +565,9 @@ class TestFiscalDocumentGeneric(TransactionCase):
                     "federal_taxes_national": 33.00,
                 }
             )
-
-            line._onchange_fiscal_operation_id()
-            line._onchange_fiscal_taxes()
+            # força o compute, pois não é chamado automaticamente
+            # quando uma informação externa muda.
+            line._compute_tax_fields()
 
             if "Revenda" in line.fiscal_operation_line_id.name:
                 self.assertEqual(
@@ -688,10 +666,6 @@ 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_product_id_fiscal()
-            line._onchange_fiscal_operation_id()
-            line._onchange_fiscal_taxes()
-
             if "Revenda" in line.fiscal_operation_line_id.name:
                 self.assertEqual(
                     line.cfop_id.code,
@@ -786,10 +760,6 @@ 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_product_id_fiscal()
-            line._onchange_fiscal_operation_id()
-            line._onchange_fiscal_taxes()
-
             if "Revenda" in line.fiscal_operation_line_id.name:
                 self.assertEqual(
                     line.cfop_id.code,
@@ -871,10 +841,6 @@ 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_product_id_fiscal()
-            line._onchange_fiscal_operation_id()
-            line._onchange_fiscal_taxes()
-
             if "Revenda" in line.fiscal_operation_line_id.name:
                 self.assertEqual(
                     line.cfop_id.code,
@@ -982,10 +948,16 @@ class TestFiscalDocumentGeneric(TransactionCase):
     def test_nfe_comments(self):
         self.nfe_not_taxpayer._document_comment()
         additional_data = self.nfe_not_taxpayer.fiscal_line_ids[0].additional_data
-        self.assertEqual(
-            additional_data,
-            "manual comment test",
-        )
+        if self.env.ref("base.lang_pt_BR").active:
+            self.assertEqual(
+                additional_data,
+                "manual comment test - Valor Aprox. dos Tributos: R$\xa00,00",
+            )
+        else:  # no pt_BR amount formatting available
+            self.assertEqual(
+                additional_data,
+                "manual comment test - Valor Aprox. dos Tributos: R$\xa00.00",
+            )
 
     def test_fields_freight_insurance_other_costs(self):
         """Test fields Freight, Insurance and Other Costs when

odoo/addons/l10n_br_fiscal/tests/test_fiscal_document_nfse.py

@@ -13,12 +13,7 @@ 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_product_id_fiscal()
-            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_fiscal_document_serie.py

@@ -0,0 +1,60 @@
+# Copyright (C) 2025  Renato Lima - Akretion <renato.lima@akretion.com.br>
+# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
+
+from psycopg2 import IntegrityError
+
+from odoo.exceptions import ValidationError
+from odoo.tests import TransactionCase
+from odoo.tools import mute_logger
+
+
+class TestFiscalDocumentSerie(TransactionCase):
+    @classmethod
+    def setUpClass(cls):
+        super().setUpClass()
+        cls.env = cls.env(context=dict(cls.env.context, tracking_disable=True))
+
+        # Company
+        cls.company_sn = cls.env.ref("l10n_br_base.empresa_simples_nacional")
+
+        # Fiscal Document Type
+        cls.document_type_nfe = cls.env.ref("l10n_br_fiscal.document_55")
+
+        cls.document_serie_nfe_5 = cls.env["l10n_br_fiscal.document.serie"].create(
+            {
+                "code": "5",
+                "name": "Serie 5",
+                "document_type_id": cls.document_type_nfe.id,
+                "company_id": cls.company_sn.id,
+            }
+        )
+
+        # Fiscal Document
+        cls.document = cls.env["l10n_br_fiscal.document"].create(
+            {
+                "company_id": cls.company_sn.id,
+                "document_type_id": cls.document_type_nfe.id,
+                "document_serie_id": cls.document_serie_nfe_5.id,
+                "partner_id": cls.env.ref("l10n_br_base.res_partner_cliente1_sp").id,
+                "state_edoc": "cancelada",
+            }
+        )
+
+    def test_document_serie_duplicated(self):
+        """Test document serie duplicate constraint."""
+        document_serie = self.env["l10n_br_fiscal.document.serie"]
+        document_serie_values = {
+            "code": "10",
+            "name": "Serie 10",
+            "document_type_id": self.document_type_nfe.id,
+            "company_id": self.company_sn.id,
+        }
+
+        with self.assertRaises(IntegrityError), mute_logger("odoo.sql_db"):
+            for _ in range(2):
+                document_serie.create(document_serie_values)
+
+    def test_document_serie_code_in_use(self):
+        """Test document serie code in use constraint."""
+        with self.assertRaises(ValidationError):
+            self.document_serie_nfe_5.write({"code": "7"})

odoo/addons/l10n_br_fiscal/tests/test_tax_benefit.py

@@ -35,15 +35,12 @@ class TestTaxBenefit(TransactionCase):
                 "state": "approved",
             }
         )
+        # force update
+        cls.nfe_tax_benefit.fiscal_line_ids._compute_fiscal_tax_ids()
 
     def test_nfe_tax_benefit(self):
         """Test NFe with tax benefit."""
-
         for line in self.nfe_tax_benefit.fiscal_line_ids:
-            line._onchange_product_id_fiscal()
-            line._onchange_fiscal_operation_id()
-            line._onchange_fiscal_taxes()
-
             self.assertEqual(
                 line.icms_tax_benefit_id,
                 self.tax_benefit,

odoo/addons/l10n_br_fiscal/views/cest_view.xml

@@ -25,12 +25,12 @@
         <field name="name">l10n_br_fiscal.cest.tree</field>
         <field name="model">l10n_br_fiscal.cest</field>
         <field name="arch" type="xml">
-            <tree>
+            <list>
                 <field name="code" />
                 <field name="item" />
                 <field name="name" />
                 <field name="segment" />
-            </tree>
+            </list>
         </field>
     </record>
 

odoo/addons/l10n_br_fiscal/views/cfop_view.xml

@@ -46,7 +46,7 @@
         <field name="name">l10n_br_fiscal.cfop.tree</field>
         <field name="model">l10n_br_fiscal.cfop</field>
         <field name="arch" type="xml">
-            <tree>
+            <list>
                 <field name="code" />
                 <field name="name" />
                 <field name="type_in_out" />
@@ -61,7 +61,7 @@
                 <field name="ind_anula" optional="hide" />
                 <field name="ind_remes" optional="hide" />
                 <field name="ind_comb" optional="hide" />
-            </tree>
+            </list>
         </field>
     </record>
 

odoo/addons/l10n_br_fiscal/views/city_taxation_code.xml

@@ -21,14 +21,14 @@
         <field name="name">city.taxation.code.tree (in l10n_br_fiscal)</field>
         <field name="model">l10n_br_fiscal.city.taxation.code</field>
         <field name="arch" type="xml">
-            <tree>
+            <list>
                 <field name="code" />
                 <field name="name" />
                 <field name="service_type_id" />
                 <field name="state_id" />
                 <field name="city_id" />
                 <field name="cnae_id" />
-            </tree>
+            </list>
         </field>
     </record>
 

odoo/addons/l10n_br_fiscal/views/cnae_view.xml

@@ -23,12 +23,12 @@
         <field name="name">l10n_br_fiscal.cnae.tree</field>
         <field name="model">l10n_br_fiscal.cnae</field>
         <field name="arch" type="xml">
-            <tree>
+            <list>
                 <field name="code" />
                 <field name="name" />
                 <field name="version" />
                 <field name="parent_id" />
-            </tree>
+            </list>
         </field>
     </record>
 

odoo/addons/l10n_br_fiscal/views/comment_view.xml

@@ -22,14 +22,14 @@
         <field name="name">l10n_br_fiscal.comment.tree</field>
         <field name="model">l10n_br_fiscal.comment</field>
         <field name="arch" type="xml">
-            <tree>
+            <list>
                 <field name="sequence" />
                 <field name="name" />
                 <field name="comment_type" />
                 <field name="object" />
                 <field name="date_begin" />
                 <field name="date_end" />
-            </tree>
+            </list>
         </field>
     </record>
 

odoo/addons/l10n_br_fiscal/views/cst_view.xml

@@ -25,12 +25,12 @@
         <field name="name">l10n_br_fiscal.cst.tree</field>
         <field name="model">l10n_br_fiscal.cst</field>
         <field name="arch" type="xml">
-            <tree>
+            <list>
                 <field name="code" />
                 <field name="name" />
                 <field name="cst_type" />
                 <field name="tax_group_id" />
-            </tree>
+            </list>
         </field>
     </record>
 

odoo/addons/l10n_br_fiscal/views/document_line_mixin_view.xml

@@ -37,6 +37,7 @@
                     <field name="service_type_id" force_save="1" invisible="1" />
                     <field name="city_taxation_code_id" force_save="1" invisible="1" />
                     <field name="uot_id" force_save="1" invisible="1" />
+                    <field name="uom_id" force_save="1" invisible="1" />
                     <field name="fiscal_price" force_save="1" invisible="1" />
                     <field name="fiscal_quantity" force_save="1" invisible="1" />
                     <field
@@ -1081,7 +1082,6 @@
                         <group>
                             <field name="comment_ids" widget="many2many_tags" />
                             <field name="manual_additional_data" />
-                            <field name="additional_data" readonly="1" />
                         </group>
                     </page>
                 </notebook>

odoo/addons/l10n_br_fiscal/views/document_line_view.xml

@@ -102,19 +102,21 @@
                     <page name="amounts" string="Amounts">
                         <group>
                             <group>
-                                <field name="amount_untaxed" />
+                                <field name="fiscal_amount_untaxed" />
                                 <field name="amount_fiscal" />
-                                <field name="amount_tax" />
+                                <field name="fiscal_amount_tax" />
                                 <field name="estimate_tax" />
                             </group>
                             <group>
-                                <field name="amount_total" />
+                                <field name="fiscal_amount_total" />
                                 <field name="amount_tax_withholding" />
                                 <field name="amount_taxed" />
                             </group>
                         </group>
                     </page>
-                    <page name="fiscal_line_extra_info" string="Extra Info" />
+                    <page name="fiscal_line_extra_info" string="Extra Info">
+                        <field name="additional_data" readonly="1" />
+                    </page>
                 </notebook>
             </form>
         </field>

odoo/addons/l10n_br_fiscal/views/document_related_view.xml

@@ -4,12 +4,12 @@
         <field name="name">l10n_br_fiscal.document.related.tree</field>
         <field name="model">l10n_br_fiscal.document.related</field>
         <field name="arch" type="xml">
-            <tree>
+            <list>
                 <field name="document_type_id" />
                 <field name="document_key" />
                 <field name="document_serie" />
                 <field name="document_number" />
-            </tree>
+            </list>
         </field>
     </record>
 

odoo/addons/l10n_br_fiscal/views/document_serie_view.xml

@@ -4,13 +4,13 @@
         <field name="name">l10n_br_fiscal.document.serie.tree</field>
         <field name="model">l10n_br_fiscal.document.serie</field>
         <field name="arch" type="xml">
-            <tree>
+            <list>
                 <field name="name" />
                 <field name="document_type_id" />
                 <field name="sequence_number_next" />
                 <field name="company_id" groups="base.group_multi_company" />
                 <field name="active" widget="boolean_toggle" />
-            </tree>
+            </list>
         </field>
     </record>
 

odoo/addons/l10n_br_fiscal/views/document_type_view.xml

@@ -4,11 +4,11 @@
         <field name="name">l10n_br_fiscal.document.type.tree</field>
         <field name="model">l10n_br_fiscal.document.type</field>
         <field name="arch" type="xml">
-            <tree>
+            <list>
                 <field name="code" />
                 <field name="name" />
                 <field name="electronic" />
-            </tree>
+            </list>
         </field>
     </record>
 

odoo/addons/l10n_br_fiscal/views/document_view.xml

@@ -63,7 +63,7 @@
         <field name="name">l10n_br_fiscal.document.tree</field>
         <field name="model">l10n_br_fiscal.document</field>
         <field name="arch" type="xml">
-            <tree>
+            <list>
                 <field name="company_id" groups="base.group_multi_company" />
                 <field name="document_type" />
                 <field name="document_serie" />
@@ -71,9 +71,9 @@
                 <field name="document_date" />
                 <field name="fiscal_operation_id" />
                 <field name="partner_id" />
-                <field name="amount_untaxed" />
-                <field name="amount_tax" />
-                <field name="amount_total" />
+                <field name="fiscal_amount_untaxed" />
+                <field name="fiscal_amount_tax" />
+                <field name="fiscal_amount_total" />
                 <field name="state" column_invisible="True" />
                 <field
                     name="state_edoc"
@@ -85,7 +85,7 @@
                     optional="show"
                 />
                 <field name="state_fiscal" />
-            </tree>
+            </list>
         </field>
     </record>
 
@@ -219,10 +219,10 @@
                         <h1>
                         <field name="document_type_id" readonly="1" />:
                         <field
-                                name="document_number"
-                                readonly="issuer == 'company'"
-                                required="issuer == 'partner'"
-                            /></h1>
+                            name="document_number"
+                            readonly="issuer == 'company'"
+                            required="issuer == 'partner'"
+                        /></h1>
                     </div>
                     <group>
                         <group name="partner_info">
@@ -234,7 +234,10 @@
                             />
                             <field name="partner_shipping_id" />
                         </group>
-                        <group name="l10n_br_fiscal">
+                        <group
+                            name="l10n_br_fiscal"
+                            invisible="document_type in ('57', '08', '09', '10', '11', '26', '67', '8B')"
+                        >
                             <field
                                 name="fiscal_operation_id"
                                 options="{'no_create': True, 'no_create_edit': True}"
@@ -292,21 +295,23 @@
                             />
                             <field name="company_id" invisible="1" />
                         </group>
-                        <group
-                            name="CT-e Info"
-                            invisible="[('document_type_id.code', 'in', ['57', '08', '09', '10', '11', '26', '67', '8B'])]"
-                        >
-                            <field name="transport_modal" />
-                            <field name="service_provider" />
-                        </group>
                     </group>
                     <notebook>
                         <page name="products" string="Products and Services">
                             <field
                                 name="fiscal_line_ids"
-                                context="{'form_view_ref': 'l10n_br_fiscal.document_line_form', 'default_document_id': id, 'default_company_id': company_id, 'default_partner_id': partner_id, 'default_fiscal_operation_type': fiscal_operation_type, 'default_fiscal_operation_id': fiscal_operation_id, 'no_subcall': True}"
+                                context="{
+                                    'form_view_ref': 'l10n_br_fiscal.document_line_form',
+                                    'default_document_id': id,
+                                    'default_company_id': company_id,
+                                    'default_partner_id': partner_id,
+                                    'default_fiscal_operation_type': fiscal_operation_type,
+                                    'default_fiscal_operation_id': fiscal_operation_id,
+                                    'default_ind_final': ind_final,
+                                    'no_subcall': True,
+                                }"
                             >
-                                <tree>
+                                <list>
                                     <field name="product_id" />
                                     <field name="uom_id" />
                                     <field name="price_unit" />
@@ -315,8 +320,8 @@
                                         name="fiscal_tax_ids"
                                         widget="many2many_tags"
                                     />
-                                    <field name="amount_total" sum="Total" />
-                                </tree>
+                                    <field name="fiscal_amount_total" sum="Total" />
+                                </list>
                             </field>
                         </page>
                         <page name="document_related" string="Document Related">
@@ -413,8 +418,8 @@
                                             readonly="delivery_costs == 'line' and not force_compute_delivery_costs_by_total"
                                         />
                                         <field name="amount_estimate_tax" />
-                                        <field name="amount_tax" />
-                                        <field name="amount_total" />
+                                        <field name="fiscal_amount_tax" />
+                                        <field name="fiscal_amount_total" />
                                         <field name="amount_tax_withholding" />
                                         <field name="amount_financial_total" />
                                         <field name="amount_financial_total_gross" />
@@ -434,12 +439,7 @@
                         </page>
                     </notebook>
                 </sheet>
-                <div class="o_attachment_preview" />
-                <div class="oe_chatter">
-                    <field name="message_follower_ids" groups="base.group_user" />
-                    <field name="activity_ids" />
-                    <field name="message_ids" />
-                </div>
+                <chatter />
             </form>
         </field>
     </record>

odoo/addons/l10n_br_fiscal/views/icms_regulation_view.xml

@@ -14,9 +14,9 @@
         <field name="name">l10n_br_fiscal.icms.regulation.tree</field>
         <field name="model">l10n_br_fiscal.icms.regulation</field>
         <field name="arch" type="xml">
-            <tree>
+            <list>
                 <field name="name" />
-            </tree>
+            </list>
         </field>
     </record>
 

odoo/addons/l10n_br_fiscal/views/icms_relief_view.xml

@@ -15,10 +15,10 @@
         <field name="name">l10n_br_fiscal.icms.relief.tree</field>
         <field name="model">l10n_br_fiscal.icms.relief</field>
         <field name="arch" type="xml">
-            <tree>
+            <list>
                 <field name="code" />
                 <field name="name" />
-            </tree>
+            </list>
         </field>
     </record>
 

odoo/addons/l10n_br_fiscal/views/invalidate_number_view.xml

@@ -15,14 +15,14 @@
         <field name="name">l10n_br_fiscal.invalidate.number.tree</field>
         <field name="model">l10n_br_fiscal.invalidate.number</field>
         <field name="arch" type="xml">
-            <tree>
+            <list>
                 <field name="company_id" readonly="state != 'draft'" />
                 <field name="document_type_id" readonly="state != 'draft'" />
                 <field name="document_serie_id" readonly="state != 'draft'" />
                 <field name="number_start" readonly="state != 'draft'" />
                 <field name="number_end" readonly="state != 'draft'" />
                 <field name="state" />
-            </tree>
+            </list>
         </field>
     </record>