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

odoo/addons/l10n_br_fiscal/models/comment.py

@@ -7,7 +7,7 @@ from datetime import datetime
 from dateutil.relativedelta import relativedelta
 
 from odoo import api, fields, models, tools
-from odoo.osv.expression import AND
+from odoo.osv import expression
 
 from ..constants.fiscal import (
     COMMENT_TYPE,
@@ -62,37 +62,39 @@ class Comment(models.Model):
     )
 
     @api.model
-    def _name_search(
-        self, name, args=None, operator="ilike", limit=100, name_get_uid=None
-    ):
-        args = args or []
+    def _name_search(self, name, domain=None, operator="ilike", limit=None, order=None):
         if name:
-            domain = [
-                "|",
-                ("comment", "ilike", "%" + name + "%"),
-                ("name", operator, name),
-            ]
-            return super()._name_search(
-                args=AND([args, domain]),
-                operator=operator,
+            domain = expression.AND(
+                [
+                    domain or [],
+                    [
+                        "|",
+                        ("comment", "ilike", "%" + name + "%"),
+                        ("name", operator, name),
+                    ],
+                ]
+            )
+            return super()._search(
+                domain,
                 limit=limit,
-                name_get_uid=name_get_uid,
             )
         return super()._name_search(
             name=name,
-            args=args,
+            domain=domain,
             operator=operator,
             limit=limit,
-            name_get_uid=name_get_uid,
+            order=order,
         )
 
-    def name_get(self):
+    @api.depends("comment")
+    def _compute_display_name(self):
         def truncate_name(name):
             if len(name) > 60:
                 name = f"{name[:60]}..."
             return name
 
-        return [(r.id, f"{truncate_name(r.name)}") for r in self]
+        for record in self:
+            record.display_name = truncate_name(record.comment)
 
     # format_amount function for fiscal observation
     # This way we can format numbers in currency template on fiscal observation

odoo/addons/l10n_br_fiscal/models/data_abstract.py

@@ -89,15 +89,13 @@ class DataAbstract(models.AbstractModel):
         return model_view
 
     @api.model
-    def _name_search(
-        self, name, args=None, operator="ilike", limit=100, name_get_uid=None
-    ):
+    def _name_search(self, name, domain=None, operator="ilike", limit=None, order=None):
         if operator == "ilike" and not (name or "").strip():
             domain = []
         elif operator in ("ilike", "like", "=", "=like", "=ilike"):
             domain = expression.AND(
                 [
-                    args or [],
+                    domain or [],
                     [
                         "|",
                         "|",
@@ -108,22 +106,23 @@ class DataAbstract(models.AbstractModel):
                 ]
             )
             return self._search(
-                expression.AND([domain, args]),
+                domain,
                 limit=limit,
-                access_rights_uid=name_get_uid,
             )
 
         return super()._name_search(
-            name, args=args, operator=operator, limit=limit, name_get_uid=name_get_uid
+            name, domain=domain, operator=operator, limit=limit, order=order
         )
 
-    def name_get(self):
+    @api.depends("name", "code")
+    def _compute_display_name(self):
         def truncate_name(name):
             if len(name) > 60:
                 name = f"{name[:60]}..."
             return name
 
-        if self._context.get("show_code_only"):
-            return [(r.id, f"{r.code}") for r in self]
-
-        return [(r.id, f"{r.code} - {truncate_name(r.name)}") for r in self]
+        for record in self:
+            if self._context.get("show_code_only"):
+                record.display_name = record.code
+            else:
+                record.display_name = f"{record.code} - {truncate_name(record.name)}"

odoo/addons/l10n_br_fiscal/models/document.py

@@ -361,11 +361,9 @@ class Document(models.Model):
             )
         return name
 
-    def name_get(self):
-        res = []
+    def _compute_display_name(self):
         for record in self:
-            res.append((record.id, record._compute_document_name()))
-        return res
+            record.display_name = record._compute_document_name()
 
     @api.depends(
         "issuer",

odoo/addons/l10n_br_fiscal/models/document_serie.py

@@ -78,8 +78,10 @@ class DocumentSerie(models.Model):
                 vals.update({"internal_sequence_id": self._create_sequence(vals)})
         return super().create(vals_list)
 
-    def name_get(self):
-        return [(r.id, f"{r.name}") for r in self]
+    @api.depends("name")
+    def _compute_display_name(self):
+        for record in self:
+            record.display_name = record.name
 
     def _is_invalid_number(self, document_number):
         self.ensure_one()

odoo/addons/l10n_br_fiscal/models/invalidate_number.py

@@ -28,14 +28,12 @@ class InvalidateNumber(models.Model):
         readonly=True,
         default=lambda self: self.env.company.id,
         required=True,
-        states={"draft": [("readonly", False)]},
     )
 
     document_type_id = fields.Many2one(
         comodel_name="l10n_br_fiscal.document.type",
         required=True,
         readonly=True,
-        states={"draft": [("readonly", False)]},
     )
 
     document_electronic = fields.Boolean(
@@ -49,27 +47,23 @@ class InvalidateNumber(models.Model):
             ('company_id', '=', company_id)]""",
         required=True,
         readonly=True,
-        states={"draft": [("readonly", False)]},
     )
 
     number_start = fields.Integer(
         string="Initial Number",
         required=True,
         readonly=True,
-        states={"draft": [("readonly", False)]},
     )
 
     number_end = fields.Integer(
         string="End Number",
         required=True,
         readonly=True,
-        states={"draft": [("readonly", False)]},
     )
 
     justification = fields.Char(
         required=True,
         readonly=True,
-        states={"draft": [("readonly", False)]},
     )
 
     state = fields.Selection(

odoo/addons/l10n_br_fiscal/models/operation.py

@@ -72,14 +72,12 @@ class Operation(models.Model):
     code = fields.Char(
         required=True,
         readonly=True,
-        states={"draft": [("readonly", False)]},
         tracking=True,
     )
 
     name = fields.Char(
         required=True,
         readonly=True,
-        states={"draft": [("readonly", False)]},
         tracking=True,
     )
 
@@ -88,7 +86,6 @@ class Operation(models.Model):
         string="Type",
         required=True,
         readonly=True,
-        states={"draft": [("readonly", False)]},
         tracking=True,
     )
 
@@ -97,7 +94,6 @@ class Operation(models.Model):
         string="Finalidade",
         default=EDOC_PURPOSE_NORMAL,
         readonly=True,
-        states={"draft": [("readonly", False)]},
         tracking=True,
     )
 
@@ -106,7 +102,6 @@ class Operation(models.Model):
         string="Default Price Unit?",
         default="sale_price",
         readonly=True,
-        states={"draft": [("readonly", False)]},
         tracking=True,
     )
 
@@ -115,7 +110,6 @@ class Operation(models.Model):
         default=OPERATION_FISCAL_TYPE_DEFAULT,
         required=True,
         readonly=True,
-        states={"draft": [("readonly", False)]},
         tracking=True,
     )
 
@@ -123,7 +117,6 @@ class Operation(models.Model):
         comodel_name="l10n_br_fiscal.operation",
         string="Return Operation",
         readonly=True,
-        states={"draft": [("readonly", False)]},
         domain="[('fiscal_operation_type', '!=', fiscal_operation_type), "
         "('fiscal_type', 'in', {'sale': ['sale_refund'], 'purchase': "
         "['purchase_refund'], 'other': ['return_in', 'return_out'],"
@@ -136,7 +129,6 @@ class Operation(models.Model):
         comodel_name="l10n_br_fiscal.operation",
         string="Inverse Operation",
         readonly=True,
-        states={"draft": [("readonly", False)]},
         tracking=True,
     )
 
@@ -144,7 +136,6 @@ class Operation(models.Model):
         comodel_name="res.company",
         string="Company",
         readonly=True,
-        states={"draft": [("readonly", False)]},
         tracking=True,
     )
 
@@ -162,7 +153,6 @@ class Operation(models.Model):
         inverse_name="fiscal_operation_id",
         string="Operation Document Types",
         readonly=True,
-        states={"draft": [("readonly", False)]},
     )
 
     line_ids = fields.One2many(
@@ -170,7 +160,6 @@ class Operation(models.Model):
         inverse_name="fiscal_operation_id",
         string="Operation Line",
         readonly=True,
-        states={"draft": [("readonly", False)]},
         copy=True,
     )
 

odoo/addons/l10n_br_fiscal/models/tax_definition.py

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

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

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

odoo/addons/l10n_br_fiscal/tests/test_cnae.py

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

odoo/addons/l10n_br_fiscal/tests/test_document_edition.py

@@ -75,42 +75,20 @@ class TestDocumentEdition(TransactionCase):
                 line_form.fiscal_operation_line_id,
                 self.env.ref("l10n_br_fiscal.fo_venda_revenda"),
             )
-            self.assertEqual(
-                line_form.ipi_tax_id, self.env.ref("l10n_br_fiscal.tax_ipi_nt")
-            )
-
-            line_form.fiscal_operation_line_id = self.env.ref(
-                "l10n_br_fiscal.fo_venda_venda"
-            )
-            self.assertEqual(
-                line_form.ipi_tax_id, self.env.ref("l10n_br_fiscal.tax_ipi_3_25")
-            )
 
-            # ensure manually setting a xx_tax_id is properly saved (not recomputed):
-            line_form.icms_tax_id = self.env.ref("l10n_br_fiscal.tax_icms_18")
-            self.assertEqual(line_form.icms_value, 37.17)
-            self.assertEqual(
-                line_form.ipi_tax_id, self.env.ref("l10n_br_fiscal.tax_ipi_3_25")
-            )
+            # line_form.fiscal_operation_line_id = False
+            # self.assertEqual(len(line_form.fiscal_tax_ids), 0)
 
         doc = doc_form.save()
-        line = doc.fiscal_line_ids[0]
-        self.assertEqual(line.price_unit, 100)
-        self.assertEqual(line.fiscal_price, 100)
-        self.assertEqual(line.quantity, 2)
-        self.assertEqual(line.fiscal_quantity, 2)
-        self.assertEqual(len(line.fiscal_tax_ids), 4)
-
-        self.assertEqual(
-            line.fiscal_operation_line_id,
-            self.env.ref("l10n_br_fiscal.fo_venda_venda"),
-        )
+        self.assertEqual(doc.fiscal_line_ids[0].price_unit, 100)
+        self.assertEqual(doc.fiscal_line_ids[0].fiscal_price, 100)
+        self.assertEqual(doc.fiscal_line_ids[0].quantity, 2)
+        self.assertEqual(doc.fiscal_line_ids[0].fiscal_quantity, 2)
+        self.assertEqual(len(doc.fiscal_line_ids[0].fiscal_tax_ids), 4)
         self.assertEqual(
-            line.icms_tax_id.id,
-            self.ref("l10n_br_fiscal.tax_icms_18"),
+            doc.fiscal_line_ids[0].icms_tax_id.id,
+            self.ref("l10n_br_fiscal.tax_icms_12"),
         )
-        self.assertEqual(line.ipi_tax_id, self.env.ref("l10n_br_fiscal.tax_ipi_3_25"))
-        self.assertEqual(line.icms_value, 37.17)
 
     def test_product_fiscal_factor(self):
         doc_form = Form(

odoo/addons/l10n_br_fiscal/tests/test_service_type.py

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

odoo/addons/l10n_br_fiscal/views/cest_view.xml

@@ -1,6 +1,5 @@
 <?xml version="1.0" encoding="utf-8" ?>
 <odoo>
-
     <record id="cest_search" model="ir.ui.view">
         <field name="name">l10n_br_fiscal.cest.search</field>
         <field name="model">l10n_br_fiscal.cest</field>
@@ -10,9 +9,9 @@
                 <field name="name" />
                 <field name="item" />
                 <field name="segment" />
-                <group expand='0' string='Group By...'>
+                <group expand="0" string="Group By...">
                     <filter
-                        string='Segment'
+                        string="Segment"
                         name="segment"
                         domain="[]"
                         context="{'group_by' : 'segment'}"
@@ -48,8 +47,8 @@
                             id="product_tmpl_button"
                             icon="fa-bars"
                             name="%(product.product_template_action_all)d"
-                            context="{'search_default_cest_id': active_id}"
-                            attrs="{'invisible': [('product_tmpl_qty', '=', 0)]}"
+                            context="{'search_default_cest_id': id}"
+                            invisible="product_tmpl_qty == 0"
                         >
                             <field
                                 string="Products"
@@ -79,5 +78,4 @@
             </form>
         </field>
     </record>
-
 </odoo>

odoo/addons/l10n_br_fiscal/views/cfop_view.xml

@@ -1,6 +1,5 @@
 <?xml version="1.0" encoding="utf-8" ?>
 <odoo>
-
     <record id="cfop_search" model="ir.ui.view">
         <field name="name">l10n_br_fiscal.cfop.search</field>
         <field name="model">l10n_br_fiscal.cfop</field>
@@ -25,15 +24,15 @@
                 <field name="code" />
                 <field name="name" />
                 <field name="small_name" />
-                <group expand='0' string='Group By...'>
+                <group expand="0" string="Group By...">
                     <filter
-                        string='Type In Out'
+                        string="Type In Out"
                         name="type_in_out"
                         domain="[]"
                         context="{'group_by' : 'type_in_out'}"
                     />
                     <filter
-                        string='Type Move'
+                        string="Type Move"
                         name="type_move"
                         domain="[]"
                         context="{'group_by' : 'type_move'}"
@@ -116,5 +115,4 @@
             </form>
         </field>
     </record>
-
 </odoo>

odoo/addons/l10n_br_fiscal/views/city_taxation_code.xml

@@ -2,7 +2,6 @@
 <!-- Copyright 2020 KMEE INFORMATICA LTDA
      License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl). -->
 <odoo>
-
     <record id="city_taxation_code_search_view" model="ir.ui.view">
         <field name="name">city.taxation.code.search (in l10n_br_fiscal)</field>
         <field name="model">l10n_br_fiscal.city.taxation.code</field>
@@ -38,8 +37,7 @@
         <field name="model">l10n_br_fiscal.city.taxation.code</field>
         <field name="arch" type="xml">
             <form>
-                <header>
-                </header>
+                <header />
                 <sheet>
                     <group>
                         <field name="code" />
@@ -53,5 +51,4 @@
             </form>
         </field>
     </record>
-
 </odoo>