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/models/document_mixin.py

@@ -8,7 +8,6 @@ from ..constants.fiscal import (
     DOCUMENT_ISSUER,
     DOCUMENT_ISSUER_COMPANY,
     FINAL_CUSTOMER,
-    FINAL_CUSTOMER_YES,
     FISCAL_COMMENT_DOCUMENT,
     NFE_IND_PRES,
     NFE_IND_PRES_DEFAULT,
@@ -48,10 +47,6 @@ class FiscalDocumentMixin(models.AbstractModel):
     def _date_server_format(self):
         return fields.Datetime.now().strftime(DEFAULT_SERVER_DATETIME_FORMAT)
 
-    @api.model
-    def _default_operation(self):
-        return False
-
     @api.model
     def _operation_domain(self):
         domain = (
@@ -66,7 +61,6 @@ class FiscalDocumentMixin(models.AbstractModel):
         comodel_name="l10n_br_fiscal.operation",
         string="Operation",
         domain=lambda self: self._operation_domain(),
-        default=_default_operation,
     )
 
     operation_name = fields.Char(
@@ -105,14 +99,10 @@ class FiscalDocumentMixin(models.AbstractModel):
         store=True,
     )
 
-    fiscal_additional_data = fields.Text()
-
     manual_fiscal_additional_data = fields.Text(
         help="Fiscal Additional data manually entered by user",
     )
 
-    customer_additional_data = fields.Text()
-
     manual_customer_additional_data = fields.Text(
         help="Customer Additional data manually entered by user",
     )
@@ -120,7 +110,11 @@ class FiscalDocumentMixin(models.AbstractModel):
     ind_final = fields.Selection(
         selection=FINAL_CUSTOMER,
         string="Final Consumption Operation",
-        default=FINAL_CUSTOMER_YES,
+        compute="_compute_ind_final",
+        inverse="_inverse_ind_final",
+        store=True,
+        precompute=True,
+        readonly=False,
     )
 
     currency_id = fields.Many2one(
@@ -135,7 +129,7 @@ class FiscalDocumentMixin(models.AbstractModel):
         help="Amount without discount.",
     )
 
-    amount_untaxed = fields.Monetary(
+    fiscal_amount_untaxed = fields.Monetary(
         compute="_compute_fiscal_amount",
         store=True,
     )
@@ -403,12 +397,12 @@ class FiscalDocumentMixin(models.AbstractModel):
         store=True,
     )
 
-    amount_tax = fields.Monetary(
+    fiscal_amount_tax = fields.Monetary(
         compute="_compute_fiscal_amount",
         store=True,
     )
 
-    amount_total = fields.Monetary(
+    fiscal_amount_total = fields.Monetary(
         compute="_compute_fiscal_amount",
         store=True,
     )
@@ -501,14 +495,12 @@ class FiscalDocumentMixin(models.AbstractModel):
     document_number = fields.Char(
         copy=False,
         index=True,
-        unaccent=False,
     )
 
     document_key = fields.Char(
         string="Key",
         copy=False,
         index=True,
-        unaccent=False,
     )
 
     key_random_code = fields.Char(string="Document Key Random Code")

odoo/addons/l10n_br_fiscal/models/document_mixin_methods.py

@@ -4,8 +4,6 @@
 from odoo import api, models
 
 from ..constants.fiscal import (
-    COMMENT_TYPE_COMMERCIAL,
-    COMMENT_TYPE_FISCAL,
     DOCUMENT_ISSUER_COMPANY,
 )
 
@@ -75,7 +73,8 @@ class FiscalDocumentMixinMethods(models.AbstractModel):
     def _get_amount_fields(self):
         """Get all fields with 'amount_' prefix"""
         fields = self.env["l10n_br_fiscal.document.mixin"]._fields.keys()
-        amount_fields = [f for f in fields if f.startswith("amount_")]
+        prefixes = ("amount_", "fiscal_amount_")
+        amount_fields = [f for f in fields if f.startswith(prefixes)]
         return amount_fields
 
     @api.depends("document_serie_id", "issuer")
@@ -89,7 +88,11 @@ class FiscalDocumentMixinMethods(models.AbstractModel):
     @api.depends("document_type_id", "issuer")
     def _compute_document_serie_id(self):
         for doc in self:
-            if doc.document_type_id and doc.issuer == DOCUMENT_ISSUER_COMPANY:
+            if (
+                not doc.document_serie_id
+                and doc.document_type_id
+                and doc.issuer == DOCUMENT_ISSUER_COMPANY
+            ):
                 doc.document_serie_id = doc.document_type_id.get_document_serie(
                     doc.company_id, doc.fiscal_operation_id
                 )
@@ -159,30 +162,6 @@ class FiscalDocumentMixinMethods(models.AbstractModel):
 
             doc.update(values)
 
-    def __document_comment_vals(self):
-        return {
-            "user": self.env.user,
-            "ctx": self._context,
-            "doc": self,
-        }
-
-    def _document_comment(self):
-        for d in self:
-            # Fiscal Comments
-            d.fiscal_additional_data = d.comment_ids.filtered(
-                lambda c: c.comment_type == COMMENT_TYPE_FISCAL
-            ).compute_message(
-                d.__document_comment_vals(), d.manual_fiscal_additional_data
-            )
-
-            # Commercial Comments
-            d.customer_additional_data = d.comment_ids.filtered(
-                lambda c: c.comment_type == COMMENT_TYPE_COMMERCIAL
-            ).compute_message(
-                d.__document_comment_vals(), d.manual_customer_additional_data
-            )
-            d.fiscal_line_ids._document_comment()
-
     def _get_fiscal_partner(self):
         """
         Hook method to determine the fiscal partner for the document.
@@ -199,14 +178,22 @@ class FiscalDocumentMixinMethods(models.AbstractModel):
         self.ensure_one()
         return self.partner_id
 
-    @api.onchange("partner_id")
-    def _onchange_partner_id_fiscal(self):
-        partner = self._get_fiscal_partner()
-        if partner:
-            self.ind_final = partner.ind_final
-            for line in self._get_amount_lines():
-                # reload fiscal data, operation line, cfop, taxes, etc.
-                line._onchange_fiscal_operation_id()
+    @api.depends("partner_id")
+    def _compute_ind_final(self):
+        for doc in self:
+            partner = doc._get_fiscal_partner()
+            if partner:
+                doc.ind_final = partner.ind_final
+            else:
+                # Default Value
+                doc.ind_final = "1"  # Yes
+
+    @api.onchange("ind_final")
+    def _inverse_ind_final(self):
+        for doc in self:
+            for line in doc._get_amount_lines():
+                if line.ind_final != doc.ind_final:
+                    line.ind_final = doc.ind_final
 
     @api.depends("fiscal_operation_id")
     def _compute_operation_name(self):
@@ -224,152 +211,37 @@ class FiscalDocumentMixinMethods(models.AbstractModel):
             elif doc.comment_ids is None:
                 doc.comment_ids = []
 
-    def _inverse_amount_freight(self):
-        for record in self.filtered(lambda doc: doc._get_product_amount_lines()):
-            if (
+    def _distribute_amount_to_lines(self, amount_field_name, line_field_name):
+        for record in self:
+            if not (
                 record.delivery_costs == "total"
                 or record.force_compute_delivery_costs_by_total
             ):
-                amount_freight_value = record.amount_freight_value
-                if all(record._get_product_amount_lines().mapped("freight_value")):
-                    amount_freight_old = sum(
-                        record._get_product_amount_lines().mapped("freight_value")
-                    )
-                    for line in record._get_product_amount_lines()[:-1]:
-                        line.freight_value = amount_freight_value * (
-                            line.freight_value / amount_freight_old
-                        )
-                    record._get_product_amount_lines()[-1].freight_value = (
-                        amount_freight_value
-                        - sum(
-                            line.freight_value
-                            for line in record._get_product_amount_lines()[:-1]
-                        )
-                    )
-                else:
-                    amount_total = sum(
-                        record._get_product_amount_lines().mapped("price_gross")
-                    )
-                    for line in record._get_product_amount_lines()[:-1]:
-                        if line.price_gross and amount_total:
-                            line.freight_value = amount_freight_value * (
-                                line.price_gross / amount_total
-                            )
-                    record._get_product_amount_lines()[-1].freight_value = (
-                        amount_freight_value
-                        - sum(
-                            line.freight_value
-                            for line in record._get_product_amount_lines()[:-1]
-                        )
+                continue
+            lines = record._get_product_amount_lines()
+            if not lines:
+                continue
+            amount_to_distribute = record[amount_field_name]
+            total_gross = sum(lines.mapped("price_gross"))
+            if total_gross > 0:
+                distributed_amount = 0
+                for line in lines[:-1]:
+                    proportional_amount = record.currency_id.round(
+                        amount_to_distribute * (line.price_gross / total_gross)
                     )
-                for line in record._get_product_amount_lines():
-                    line._onchange_fiscal_taxes()
-                record._fields["amount_total"].compute_value(record)
-                record.write(
-                    {
-                        name: value
-                        for name, value in record._cache.items()
-                        if record._fields[name].compute == "_amount_all"
-                        and not record._fields[name].inverse
-                    }
-                )
+                    line[line_field_name] = proportional_amount
+                    distributed_amount += proportional_amount
+                lines[-1][line_field_name] = amount_to_distribute - distributed_amount
+            else:
+                lines.write({line_field_name: 0.0})
+                if lines:
+                    lines[0][line_field_name] = amount_to_distribute
+
+    def _inverse_amount_freight(self):
+        self._distribute_amount_to_lines("amount_freight_value", "freight_value")
 
     def _inverse_amount_insurance(self):
-        for record in self.filtered(lambda doc: doc._get_product_amount_lines()):
-            if (
-                record.delivery_costs == "total"
-                or record.force_compute_delivery_costs_by_total
-            ):
-                amount_insurance_value = record.amount_insurance_value
-                if all(record._get_product_amount_lines().mapped("insurance_value")):
-                    amount_insurance_old = sum(
-                        record._get_product_amount_lines().mapped("insurance_value")
-                    )
-                    for line in record._get_product_amount_lines()[:-1]:
-                        line.insurance_value = amount_insurance_value * (
-                            line.insurance_value / amount_insurance_old
-                        )
-                    record._get_product_amount_lines()[-1].insurance_value = (
-                        amount_insurance_value
-                        - sum(
-                            line.insurance_value
-                            for line in record._get_product_amount_lines()[:-1]
-                        )
-                    )
-                else:
-                    amount_total = sum(
-                        record._get_product_amount_lines().mapped("price_gross")
-                    )
-                    for line in record._get_product_amount_lines()[:-1]:
-                        if line.price_gross and amount_total:
-                            line.insurance_value = amount_insurance_value * (
-                                line.price_gross / amount_total
-                            )
-                    record._get_product_amount_lines()[-1].insurance_value = (
-                        amount_insurance_value
-                        - sum(
-                            line.insurance_value
-                            for line in record._get_product_amount_lines()[:-1]
-                        )
-                    )
-                for line in record._get_product_amount_lines():
-                    line._onchange_fiscal_taxes()
-                record._fields["amount_total"].compute_value(record)
-                record.write(
-                    {
-                        name: value
-                        for name, value in record._cache.items()
-                        if record._fields[name].compute == "_amount_all"
-                        and not record._fields[name].inverse
-                    }
-                )
+        self._distribute_amount_to_lines("amount_insurance_value", "insurance_value")
 
     def _inverse_amount_other(self):
-        for record in self.filtered(lambda doc: doc._get_product_amount_lines()):
-            if (
-                record.delivery_costs == "total"
-                or record.force_compute_delivery_costs_by_total
-            ):
-                amount_other_value = record.amount_other_value
-                if all(record._get_product_amount_lines().mapped("other_value")):
-                    amount_other_old = sum(
-                        record._get_product_amount_lines().mapped("other_value")
-                    )
-                    for line in record._get_product_amount_lines()[:-1]:
-                        line.other_value = amount_other_value * (
-                            line.other_value / amount_other_old
-                        )
-                    record._get_product_amount_lines()[-1].other_value = (
-                        amount_other_value
-                        - sum(
-                            line.other_value
-                            for line in record._get_product_amount_lines()[:-1]
-                        )
-                    )
-                else:
-                    amount_total = sum(
-                        record._get_product_amount_lines().mapped("price_gross")
-                    )
-                    for line in record._get_product_amount_lines()[:-1]:
-                        if line.price_gross and amount_total:
-                            line.other_value = amount_other_value * (
-                                line.price_gross / amount_total
-                            )
-                    record._get_product_amount_lines()[-1].other_value = (
-                        amount_other_value
-                        - sum(
-                            line.other_value
-                            for line in record._get_product_amount_lines()[:-1]
-                        )
-                    )
-                for line in record._get_product_amount_lines():
-                    line._onchange_fiscal_taxes()
-                record._fields["amount_total"].compute_value(record)
-                record.write(
-                    {
-                        name: value
-                        for name, value in record._cache.items()
-                        if record._fields[name].compute == "_amount_all"
-                        and not record._fields[name].inverse
-                    }
-                )
+        self._distribute_amount_to_lines("amount_other_value", "other_value")

odoo/addons/l10n_br_fiscal/models/document_related.py

@@ -104,7 +104,7 @@ class DocumentRelated(models.Model):
             return False
 
         self.document_type_id = related.document_type_id
-        self.document_total_amount = related.amount_total
+        self.document_total_amount = related.fiscal_amount_total
         self.document_total_weight = related.total_weight
 
         if related.document_type_id.electronic:

odoo/addons/l10n_br_fiscal/models/document_serie.py

@@ -3,11 +3,13 @@
 # License AGPL-3 - See http://www.gnu.org/licenses/agpl-3.0.html
 
 from odoo import _, api, fields, models
+from odoo.exceptions import ValidationError
 
 from ..constants.fiscal import (
     DOCUMENT_ISSUER_COMPANY,
     FISCAL_IN_OUT,
     FISCAL_IN_OUT_DEFAULT,
+    SITUACAO_EDOC_EM_DIGITACAO,
 )
 
 
@@ -16,9 +18,9 @@ class DocumentSerie(models.Model):
     _description = "Fiscal Document Serie"
     _inherit = "l10n_br_fiscal.data.abstract"
 
-    code = fields.Char(size=3, unaccent=False)
+    code = fields.Char(size=3)
 
-    name = fields.Char(required=True, unaccent=False)
+    name = fields.Char(required=True)
 
     active = fields.Boolean(default=True)
 
@@ -55,6 +57,14 @@ class DocumentSerie(models.Model):
         string="Invalidate Number Range",
     )
 
+    _sql_constraints = [
+        (
+            "document_serie_unique",
+            "unique(code, document_type_id, company_id)",
+            "A Fiscal Document Serie already exists for this document type.",
+        )
+    ]
+
     @api.model
     def _create_sequence(self, values):
         """Create new no_gap entry sequence for every
@@ -83,6 +93,29 @@ class DocumentSerie(models.Model):
         for record in self:
             record.display_name = record.name
 
+    def write(self, vals):
+        if "internal_sequence_id" in vals:
+            raise ValidationError(_("You cannot change the internal sequence."))
+        if "code" in vals:
+            for serie in self:
+                if serie.code == vals["code"]:
+                    continue
+                if self.env["l10n_br_fiscal.document"].search_count(
+                    [
+                        ("document_serie_id", "=", serie.id),
+                        ("state_edoc", "not in", [SITUACAO_EDOC_EM_DIGITACAO]),
+                    ],
+                    limit=1,
+                ):
+                    raise ValidationError(
+                        _(
+                            "You cannot change the code of a document "
+                            "serie %(name)s that is already in use.",
+                            name=serie.name,
+                        )
+                    )
+        return super().write(vals)
+
     def _is_invalid_number(self, document_number):
         self.ensure_one()
         is_invalid_number = True

odoo/addons/l10n_br_fiscal/models/document_type.py

@@ -14,7 +14,6 @@ class DocumentType(models.Model):
 
     code = fields.Char(
         size=8,
-        unaccent=False,
     )
 
     name = fields.Char(

odoo/addons/l10n_br_fiscal/models/ibpt.py

@@ -45,7 +45,7 @@ def _request(ws_url, params, ibpt_request_timeout=30):
         elif response.status_code == requests.codes.service_unavailable:
             raise UserError(_("IBPT Service Unavailable - {!r}").format(ws_url))
     except Exception as e:
-        raise UserError(_("Error in the request: {}").format(e)) from e
+        raise UserError(f"Error in the request: {e}") from e
 
 
 def get_ibpt_product(

odoo/addons/l10n_br_fiscal/models/icms_regulation.py

@@ -2101,7 +2101,7 @@ class ICMSRegulation(models.Model):
             company.state_id != partner.state_id
             and partner.ind_ie_dest == NFE_IND_IE_DEST_9
             and operation_line.fiscal_operation_type == FISCAL_OUT
-            or operation_line.fiscal_operation_id.fiscal_type == "return_in"
+            or operation_line.fiscal_operation_id.fiscal_type != "return_in"
             and operation_line.fiscal_operation_type == FISCAL_IN
         ):
             domain = self._build_map_tax_def_domain(

odoo/addons/l10n_br_fiscal/models/invalidate_number.py

@@ -68,8 +68,8 @@ class InvalidateNumber(models.Model):
 
     state = fields.Selection(
         selection=[
-            ("draft", _("Draft")),
-            ("done", _("Done")),
+            ("draft", "Draft"),
+            ("done", "Done"),
         ],
         string="Status",
         readonly=True,
@@ -99,11 +99,10 @@ class InvalidateNumber(models.Model):
     @api.depends("document_type_id", "document_serie_id", "number_start", "number_end")
     def _compute_name(self):
         for record in self:
-            record.name = "{type}/({serie}): {start} - {end}".format(
-                type=record.document_type_id.type,
-                serie=record.document_serie_id.name,
-                start=record.number_start,
-                end=record.number_end,
+            record.name = (
+                f"{record.document_type_id.type}/"
+                f"({record.document_serie_id.name}): "
+                f"{record.number_start} - {record.number_end}"
             )
 
     def unlink(self):

odoo/addons/l10n_br_fiscal/models/legal_nature.py

@@ -15,6 +15,6 @@ class LegalNature(models.Model):
         (
             "fiscal_legal_nature_code_uniq",
             "unique (code)",
-            "Legal Nature already exists with this code !",
+            "Legal Nature already exists with this code!",
         )
     ]

odoo/addons/l10n_br_fiscal/models/nbm.py

@@ -11,9 +11,9 @@ class Nbm(models.Model):
     _inherit = "l10n_br_fiscal.data.product.abstract"
     _description = "NBM"
 
-    code = fields.Char(size=12, unaccent=False)
+    code = fields.Char(size=12)
 
-    code_unmasked = fields.Char(size=10, unaccent=False)
+    code_unmasked = fields.Char(size=10)
 
     product_tmpl_ids = fields.One2many(inverse_name="nbm_id")
 

odoo/addons/l10n_br_fiscal/models/nbs.py

@@ -1,7 +1,7 @@
 # Copyright (C) 2019  Renato Lima - Akretion
 # License AGPL-3 - See http://www.gnu.org/licenses/agpl-3.0.html
 
-from odoo import _, fields, models
+from odoo import fields, models
 
 from .ibpt import get_ibpt_service
 
@@ -15,9 +15,9 @@ class Nbs(models.Model):
     ]
     _description = "NBS"
 
-    code = fields.Char(size=12, unaccent=False)
+    code = fields.Char(size=12)
 
-    code_unmasked = fields.Char(size=10, unaccent=False)
+    code_unmasked = fields.Char(size=10)
 
     tax_estimate_ids = fields.One2many(inverse_name="nbs_id")
 
@@ -27,7 +27,7 @@ class Nbs(models.Model):
         (
             "fiscal_nbs_code_uniq",
             "unique (code)",
-            _("NBS already exists with this code !"),
+            "NBS already exists with this code!",
         )
     ]
 

odoo/addons/l10n_br_fiscal/models/ncm.py

@@ -1,7 +1,7 @@
 # Copyright (C) 2012  Renato Lima - Akretion <renato.lima@akretion.com.br>
 # License AGPL-3 - See http://www.gnu.org/licenses/agpl-3.0.html
 
-from odoo import _, fields, models
+from odoo import fields, models
 
 from ..constants.fiscal import TAX_DOMAIN_II, TAX_DOMAIN_IPI
 from .ibpt import get_ibpt_product
@@ -16,9 +16,9 @@ class Ncm(models.Model):
     ]
     _description = "NCM"
 
-    code = fields.Char(size=10, unaccent=False)
+    code = fields.Char(size=10)
 
-    code_unmasked = fields.Char(size=8, unaccent=False)
+    code_unmasked = fields.Char(size=8)
 
     exception = fields.Char(size=2)
 
@@ -70,7 +70,7 @@ class Ncm(models.Model):
         (
             "fiscal_ncm_code_exception_uniq",
             "unique (code, exception)",
-            _("NCM already exists with this code !"),
+            "NCM already exists with this code!",
         )
     ]
 

odoo/addons/l10n_br_fiscal/models/operation.py

@@ -98,7 +98,7 @@ class Operation(models.Model):
     )
 
     default_price_unit = fields.Selection(
-        selection=[("sale_price", _("Sale Price")), ("cost_price", _("Cost Price"))],
+        selection=[("sale_price", "Sale Price"), ("cost_price", "Cost Price")],
         string="Default Price Unit?",
         default="sale_price",
         readonly=True,
@@ -173,7 +173,7 @@ class Operation(models.Model):
         (
             "fiscal_operation_code_uniq",
             "unique (code)",
-            _("Fiscal Operation already exists with this code !"),
+            "Fiscal Operation already exists with this code!",
         )
     ]
 

odoo/addons/l10n_br_fiscal/models/operation_dashboard.py

@@ -127,7 +127,7 @@ class Operation(models.Model):
         }
 
     def open_action(self):
-        """return action based on type for related journals"""
+        """Return action based on type for related journals"""
 
         _fiscal_type_map = {
             "purchase": "in",
@@ -157,7 +157,8 @@ class Operation(models.Model):
             }
         )
 
-        [action] = self.env.ref("l10n_br_fiscal.%s" % action_name).read()
+        xmlid = f"l10n_br_fiscal.{action_name}"
+        [action] = self.env.ref(xmlid).read()
         action["context"] = ctx
         action["domain"] = self._context.get("use_domain", [])
         action["domain"] += [

odoo/addons/l10n_br_fiscal/models/operation_line.py

@@ -139,7 +139,7 @@ class OperationLine(models.Model):
         (
             "fiscal_operation_name_uniq",
             "unique (name, fiscal_operation_id)",
-            _("Fiscal Operation Line already exists with this name !"),
+            "Fiscal Operation Line already exists with this name!",
         )
     ]
 

odoo/addons/l10n_br_fiscal/models/partner_profile.py

@@ -8,6 +8,7 @@ from odoo.exceptions import ValidationError
 from ..constants.fiscal import (
     NFE_IND_IE_DEST,
     NFE_IND_IE_DEST_DEFAULT,
+    PUBLIC_ENTIRY_TYPE,
     TAX_FRAMEWORK,
     TAX_FRAMEWORK_NORMAL,
 )
@@ -32,6 +33,11 @@ class PartnerProfile(models.Model):
         "other government-controlled organizations.",
     )
 
+    public_entity_type = fields.Selection(
+        selection=PUBLIC_ENTIRY_TYPE,
+        string="Tipo de Entidade Governamental",
+    )
+
     default = fields.Boolean(string="Default Profile", default=True)
 
     ind_ie_dest = fields.Selection(
@@ -63,7 +69,7 @@ class PartnerProfile(models.Model):
         (
             "fiscal_partner_profile_code_uniq",
             "unique (code)",
-            "Fiscal Partner Profile already exists with this code !",
+            "Fiscal Partner Profile already exists with this code!",
         )
     ]
 

odoo/addons/l10n_br_fiscal/models/product_template.py

@@ -83,7 +83,7 @@ class ProductTemplate(models.Model):
         domain="[('internal_type', '=', 'normal')]",
     )
 
-    city_taxation_code_id = fields.Many2many(
+    city_taxation_code_ids = fields.Many2many(
         comodel_name="l10n_br_fiscal.city.taxation.code", string="City Taxation Code"
     )