odoo-addon-l10n-es-aeat-sii-oca 16.0.1.8.5.1__py3-none-any.whl → 16.0.2.0.0__py3-none-any.whl

odoo/addons/l10n_es_aeat_sii_oca/models/sii_mixin.py

@@ -7,26 +7,16 @@
 import json
 import logging
 
-from requests import Session
-
 from odoo import _, api, exceptions, fields, models
 from odoo.exceptions import UserError, ValidationError
 from odoo.modules.registry import Registry
 from odoo.tools.float_utils import float_compare
 
-_logger = logging.getLogger(__name__)
+from odoo.addons.l10n_es_aeat.models.aeat_mixin import round_by_keys
 
-try:
-    from zeep import Client
-    from zeep.plugins import HistoryPlugin
-    from zeep.transports import Transport
-except (ImportError, IOError) as err:
-    _logger.debug(err)
+_logger = logging.getLogger(__name__)
 
 SII_STATES = [
-    ("not_sent", "Not sent"),
-    ("sent", "Sent"),
-    ("sent_w_errors", "Accepted with errors"),
     ("sent_modified", "Registered in SII but last modifications not sent"),
     ("cancelled", "Cancelled"),
     ("cancelled_modified", "Cancelled in SII but last modifications not sent"),
@@ -36,24 +26,9 @@ SII_MACRODATA_LIMIT = 100000000.0
 SII_DATE_FORMAT = "%d-%m-%Y"
 
 
-def round_by_keys(elem, search_keys, prec=2):
-    """This uses ``round`` method directly as if has been tested that Odoo's
-    ``float_round`` still returns incorrect amounts for certain values. Try
-    3 units x 3,77 €/unit with 10% tax and you will be hit by the error
-    (on regular x86 architectures)."""
-    if isinstance(elem, dict):
-        for key, value in elem.items():
-            if key in search_keys:
-                elem[key] = round(elem[key], prec)
-            else:
-                round_by_keys(value, search_keys)
-    elif isinstance(elem, list):
-        for value in elem:
-            round_by_keys(value, search_keys)
-
-
 class SiiMixin(models.AbstractModel):
     _name = "sii.mixin"
+    _inherit = "aeat.mixin"
     _description = "SII Mixin"
 
     company_id = fields.Many2one(
@@ -68,38 +43,11 @@ class SiiMixin(models.AbstractModel):
         readonly=False,
         copy=False,
     )
-    sii_state = fields.Selection(
-        selection=SII_STATES,
-        string="SII send state",
-        default="not_sent",
-        readonly=True,
-        copy=False,
-        help="Indicates the state of this document in relation with the "
-        "presentation at the SII",
+    aeat_state = fields.Selection(
+        selection_add=SII_STATES,
     )
     sii_csv = fields.Char(string="SII CSV", copy=False, readonly=True)
     sii_return = fields.Text(string="SII Return", copy=False, readonly=True)
-    sii_header_sent = fields.Text(
-        string="SII last header sent",
-        copy=False,
-        readonly=True,
-    )
-    sii_content_sent = fields.Text(
-        string="SII last content sent",
-        copy=False,
-        readonly=True,
-    )
-    sii_send_error = fields.Text(
-        string="SII Send Error",
-        readonly=True,
-        copy=False,
-    )
-    sii_send_failed = fields.Boolean(
-        string="SII send failed",
-        copy=False,
-        help="Indicates that the last attempt to communicate this document to "
-        "the SII has failed. See SII return for details",
-    )
     sii_refund_type = fields.Selection(
         selection=[
             # ('S', 'By substitution'), - Removed as not fully supported
@@ -214,17 +162,10 @@ class SiiMixin(models.AbstractModel):
                 >= 0
             )
 
-    def _sii_get_partner(self):
-        raise NotImplementedError
-
-    def _get_sii_country_code(self):
-        self.ensure_one()
-        return self._sii_get_partner()._parse_aeat_vat_info()[0]
-
     def _filter_sii_unlink_not_possible(self):
         """Filter records that we do not allow to be deleted, all those
         that are not in not_sent sii status."""
-        return self.filtered(lambda rec: rec.sii_state != "not_sent")
+        return self.filtered(lambda rec: rec.aeat_state != "not_sent")
 
     @api.ondelete(at_uninstall=False)
     def _unlink_except_sii(self):
@@ -235,7 +176,7 @@ class SiiMixin(models.AbstractModel):
             )
 
     @api.model
-    def _get_sii_taxes_map(self, codes, date):
+    def _get_aeat_taxes_map(self, codes, date):
         """Return the codes that correspond to that sii map line codes.
 
         :param codes: List of code strings to get the mapping.
@@ -257,12 +198,7 @@ class SiiMixin(models.AbstractModel):
         tax_templates = sii_map.map_lines.filtered(lambda x: x.code in codes).taxes
         return self.company_id.get_taxes_from_templates(tax_templates)
 
-    def _change_date_format(self, date):
-        datetimeobject = fields.Date.to_date(date)
-        new_date = datetimeobject.strftime(SII_DATE_FORMAT)
-        return new_date
-
-    def _get_sii_header(self, tipo_comunicacion=False, cancellation=False):
+    def _get_aeat_header(self, tipo_comunicacion=False, cancellation=False):
         """Builds SII send header
 
         :param tipo_comunicacion String 'A0': new reg, 'A1': modification
@@ -297,9 +233,6 @@ class SiiMixin(models.AbstractModel):
                 queue.unlink()
         return True
 
-    def _get_valid_document_states(self):
-        raise NotImplementedError()
-
     def send_sii(self):
         """General public method for filtering out of the starting recordset the records
         that shouldn't be sent to the SII:
@@ -316,7 +249,7 @@ class SiiMixin(models.AbstractModel):
             if (
                 not document.sii_enabled
                 or document.state not in valid_states
-                or document.sii_state in ["sent", "cancelled"]
+                or document.aeat_state in ["sent", "cancelled"]
             ):
                 continue
             job_states = document.sudo().mapped(jobs_field_name).mapped("state")
@@ -339,20 +272,20 @@ class SiiMixin(models.AbstractModel):
                 new_delay = (
                     record.sudo()
                     .with_context(company_id=company.id)
-                    .with_delay(eta=eta if not record.sii_send_failed else False)
+                    .with_delay(eta=eta if not record.aeat_send_failed else False)
                     .confirm_one_document()
                 )
                 job = queue_obj.search([("uuid", "=", new_delay.uuid)], limit=1)
                 setattr(record.sudo(), self._get_sii_jobs_field_name(), [(4, job.id)])
 
-    def _bind_sii(self, client, port_name, address=None):
+    def _bind_service(self, client, port_name, address=None):
         self.ensure_one()
         service = client._get_service("siiService")
         port = client._get_port(service, port_name)
         address = address or port.binding_options["address"]
         return client.create_service(port.binding.name, address)
 
-    def _connect_params_sii(self, mapping_key):
+    def _connect_params_aeat(self, mapping_key):
         self.ensure_one()
         agency = self.company_id.tax_agency_id
         if not agency:
@@ -363,19 +296,6 @@ class SiiMixin(models.AbstractModel):
             agency = self.env.ref("l10n_es_aeat.aeat_tax_agency_spain")
         return agency._connect_params_sii(mapping_key, self.company_id)
 
-    def _connect_sii(self, mapping_key):
-        self.ensure_one()
-        public_crt, private_key = self.env["l10n.es.aeat.certificate"].get_certificates(
-            company=self.company_id
-        )
-        params = self._connect_params_sii(mapping_key)
-        session = Session()
-        session.cert = (public_crt, private_key)
-        transport = Transport(session=session)
-        history = HistoryPlugin()
-        client = Client(wsdl=params["wsdl"], transport=transport, plugins=[history])
-        return self._bind_sii(client, params["port_name"], params["address"])
-
     def _get_sii_gen_type(self):
         """Make a choice for general invoice type
 
@@ -394,39 +314,29 @@ class SiiMixin(models.AbstractModel):
             res = 1
         return res
 
-    def _is_sii_simplified_invoice(self):
+    def _is_aeat_simplified_invoice(self):
         """Inheritable method to allow control when an
         invoice are simplified or normal"""
-        partner = self._sii_get_partner()
-        return partner.sii_simplified_invoice
+        partner = self._aeat_get_partner()
+        return partner.aeat_simplified_invoice
 
-    def _sii_check_exceptions(self):
+    def _aeat_check_exceptions(self):
         """Inheritable method for exceptions control when sending SII invoices."""
-        self.ensure_one()
-        gen_type = self._get_sii_gen_type()
-        partner = self._sii_get_partner()
-        country_code = self._get_sii_country_code()
-        is_simplified_invoice = self._is_sii_simplified_invoice()
-        if (
-            (gen_type != 3 or country_code == "ES")
-            and not partner.vat
-            and not is_simplified_invoice
-        ):
-            raise UserError(_("The partner has not a VAT configured."))
-        if not self.company_id.chart_template_id:
-            raise UserError(
-                _("You have to select what account chart template use this" " company.")
-            )
-        if not self.company_id.sii_enabled:
-            raise UserError(_("This company doesn't have SII enabled."))
-        if not self.sii_enabled:
-            raise UserError(_("This invoice is not SII enabled."))
-
-    def _get_mapping_key(self):
-        raise NotImplementedError()
-
-    def _get_document_date(self):
-        raise NotImplementedError()
+        res = super()._aeat_check_exceptions()
+        if self.company_id.sii_enabled:
+            gen_type = self._get_sii_gen_type()
+            partner = self._aeat_get_partner()
+            country_code = self._get_aeat_country_code()
+            is_simplified_invoice = self._is_aeat_simplified_invoice()
+            if (
+                (gen_type != 3 or country_code == "ES")
+                and not partner.vat
+                and not is_simplified_invoice
+            ):
+                raise UserError(_("The partner has not a VAT configured."))
+            if not self.sii_enabled:
+                raise UserError(_("This invoice is not SII enabled."))
+        return res
 
     def _get_document_fiscal_date(self):
         raise NotImplementedError()
@@ -437,9 +347,6 @@ class SiiMixin(models.AbstractModel):
     def _get_document_period(self):
         return "%02d" % fields.Date.to_date(self._get_document_fiscal_date()).month
 
-    def _get_document_serial_number(self):
-        raise NotImplementedError()
-
     def _get_document_product_exempt(self, applied_taxes):
         raise NotImplementedError()
 
@@ -524,7 +431,7 @@ class SiiMixin(models.AbstractModel):
         self.ensure_one()
         if "DesgloseFactura" not in taxes_dict:
             return False
-        country_code = self._get_sii_country_code()
+        country_code = self._get_aeat_country_code()
         sii_gen_type = self._get_sii_gen_type()
         if "DesgloseTipoOperacion" in taxes_dict:
             # DesgloseTipoOperacion and DesgloseFactura are Exclusive
@@ -537,7 +444,7 @@ class SiiMixin(models.AbstractModel):
             # DesgloseTipoOperacion required for national operations
             # with 'IDOtro' in the SII identifier block
             return True
-        elif sii_gen_type == 1 and (self._sii_get_partner().vat or "").startswith(
+        elif sii_gen_type == 1 and (self._aeat_get_partner().vat or "").startswith(
             "ESN"
         ):
             # DesgloseTipoOperacion required if customer's country is Spain and
@@ -553,19 +460,19 @@ class SiiMixin(models.AbstractModel):
         self.ensure_one()
         taxes_dict = {}
         date = self._get_document_fiscal_date()
-        taxes_sfesb = self._get_sii_taxes_map(["SFESB"], date)
-        taxes_sfesbe = self._get_sii_taxes_map(["SFESBE"], date)
-        taxes_sfesisp = self._get_sii_taxes_map(["SFESISP"], date)
+        taxes_sfesb = self._get_aeat_taxes_map(["SFESB"], date)
+        taxes_sfesbe = self._get_aeat_taxes_map(["SFESBE"], date)
+        taxes_sfesisp = self._get_aeat_taxes_map(["SFESISP"], date)
         # taxes_sfesisps = self._get_taxes_map(['SFESISPS'])
-        taxes_sfens = self._get_sii_taxes_map(["SFENS"], date)
-        taxes_sfess = self._get_sii_taxes_map(["SFESS"], date)
-        taxes_sfesse = self._get_sii_taxes_map(["SFESSE"], date)
-        taxes_sfesns = self._get_sii_taxes_map(["SFESNS"], date)
-        taxes_not_in_total = self._get_sii_taxes_map(["NotIncludedInTotal"], date)
-        taxes_not_in_total_neg = self._get_sii_taxes_map(
+        taxes_sfens = self._get_aeat_taxes_map(["SFENS"], date)
+        taxes_sfess = self._get_aeat_taxes_map(["SFESS"], date)
+        taxes_sfesse = self._get_aeat_taxes_map(["SFESSE"], date)
+        taxes_sfesns = self._get_aeat_taxes_map(["SFESNS"], date)
+        taxes_not_in_total = self._get_aeat_taxes_map(["NotIncludedInTotal"], date)
+        taxes_not_in_total_neg = self._get_aeat_taxes_map(
             ["NotIncludedInTotalNegative"], date
         )
-        base_not_in_total = self._get_sii_taxes_map(["BaseNotIncludedInTotal"], date)
+        base_not_in_total = self._get_aeat_taxes_map(["BaseNotIncludedInTotal"], date)
         not_in_amount_total = 0
         exempt_cause = self._get_sii_exempt_cause(taxes_sfesbe + taxes_sfesse)
         tax_lines = self._get_tax_info()
@@ -667,9 +574,6 @@ class SiiMixin(models.AbstractModel):
             del taxes_dict["DesgloseFactura"]
         return taxes_dict, not_in_amount_total
 
-    def _get_document_amount_total(self):
-        raise NotImplementedError()
-
     def _get_sii_invoice_type(self):
         raise NotImplementedError()
 
@@ -683,7 +587,7 @@ class SiiMixin(models.AbstractModel):
             country_code,
             identifier_type,
             identifier,
-        ) = self._sii_get_partner()._parse_aeat_vat_info()
+        ) = self._aeat_get_partner()._parse_aeat_vat_info()
         # Limpiar alfanum
         if identifier:
             identifier = "".join(e for e in identifier if e.isalnum()).upper()
@@ -691,7 +595,7 @@ class SiiMixin(models.AbstractModel):
             identifier = "NO_DISPONIBLE"
             identifier_type = "06"
         if gen_type == 1:
-            if "1117" in (self.sii_send_error or ""):
+            if "1117" in (self.aeat_send_error or ""):
                 return {
                     "IDOtro": {
                         "CodigoPais": country_code,
@@ -707,8 +611,8 @@ class SiiMixin(models.AbstractModel):
                         "CodigoPais": country_code,
                         "IDType": identifier_type,
                         "ID": country_code + identifier
-                        if self._sii_get_partner()._map_aeat_country_code(country_code)
-                        in self._sii_get_partner()._get_aeat_europe_codes()
+                        if self._aeat_get_partner()._map_aeat_country_code(country_code)
+                        in self._aeat_get_partner()._get_aeat_europe_codes()
                         else identifier,
                     },
                 }
@@ -730,7 +634,7 @@ class SiiMixin(models.AbstractModel):
         elif gen_type == 3:
             return {"NIF": identifier}
 
-    def _get_sii_invoice_dict_out(self, cancel=False):
+    def _get_aeat_invoice_dict_out(self, cancel=False):
         """Build dict with data to send to AEAT WS for document types:
         out_invoice and out_refund.
 
@@ -740,11 +644,11 @@ class SiiMixin(models.AbstractModel):
         """
         self.ensure_one()
         document_date = self._change_date_format(self._get_document_date())
-        partner = self._sii_get_partner()
+        partner = self._aeat_get_partner()
         company = self.company_id
         fiscal_year = self._get_document_fiscal_year()
         period = self._get_document_period()
-        is_simplified_invoice = self._is_sii_simplified_invoice()
+        is_simplified_invoice = self._is_aeat_simplified_invoice()
         serial_number = self._get_document_serial_number()
         inv_dict = {
             "IDFactura": {
@@ -782,7 +686,7 @@ class SiiMixin(models.AbstractModel):
                 exp_dict["Contraparte"].update(self._get_sii_identifier())
         return inv_dict
 
-    def _get_sii_invoice_dict_in(self, cancel=False):
+    def _get_aeat_invoice_dict_in(self, cancel=False):
         """Build dict with data to send to AEAT WS for invoice types:
         in_invoice and in_refund.
 
@@ -792,15 +696,15 @@ class SiiMixin(models.AbstractModel):
         """
         raise NotImplementedError()
 
-    def _get_sii_invoice_dict(self):
+    def _get_aeat_invoice_dict(self):
         self.ensure_one()
-        self._sii_check_exceptions()
+        self._aeat_check_exceptions()
         inv_dict = {}
         mapping_key = self._get_mapping_key()
         if mapping_key in ["out_invoice", "out_refund"]:
-            inv_dict = self._get_sii_invoice_dict_out()
+            inv_dict = self._get_aeat_invoice_dict_out()
         elif mapping_key in ["in_invoice", "in_refund"]:
-            inv_dict = self._get_sii_invoice_dict_in()
+            inv_dict = self._get_aeat_invoice_dict_in()
         round_by_keys(
             inv_dict,
             [
@@ -835,24 +739,24 @@ class SiiMixin(models.AbstractModel):
         for document in self.filtered(
             lambda i: i.state in self._get_valid_document_states()
         ):
-            if document.sii_state == "not_sent":
+            if document.aeat_state == "not_sent":
                 tipo_comunicacion = "A0"
             else:
                 tipo_comunicacion = "A1"
-            header = document._get_sii_header(tipo_comunicacion)
+            header = document._get_aeat_header(tipo_comunicacion)
             doc_vals = {
-                "sii_header_sent": json.dumps(header, indent=4),
+                "aeat_header_sent": json.dumps(header, indent=4),
             }
-            # add this extra try except in case _get_sii_invoice_dict fails
+            # add this extra try except in case _get_aeat_invoice_dict fails
             # if not, get the value doc_dict for the next try and except below
             try:
-                inv_dict = document._get_sii_invoice_dict()
+                inv_dict = document._get_aeat_invoice_dict()
             except Exception as fault:
                 raise ValidationError(fault) from fault
             try:
                 mapping_key = document._get_mapping_key()
-                serv = document._connect_sii(mapping_key)
-                doc_vals["sii_content_sent"] = json.dumps(inv_dict, indent=4)
+                serv = document._connect_aeat(mapping_key)
+                doc_vals["aeat_content_sent"] = json.dumps(inv_dict, indent=4)
                 if mapping_key in ["out_invoice", "out_refund"]:
                     res = serv.SuministroLRFacturasEmitidas(header, inv_dict)
                 elif mapping_key in ["in_invoice", "in_refund"]:
@@ -866,9 +770,9 @@ class SiiMixin(models.AbstractModel):
                 if res["EstadoEnvio"] == "Correcto":
                     doc_vals.update(
                         {
-                            "sii_state": "sent",
+                            "aeat_state": "sent",
                             "sii_csv": res["CSV"],
-                            "sii_send_failed": False,
+                            "aeat_send_failed": False,
                         }
                     )
                 elif (
@@ -877,15 +781,15 @@ class SiiMixin(models.AbstractModel):
                 ):
                     doc_vals.update(
                         {
-                            "sii_state": "sent_w_errors",
+                            "aeat_state": "sent_w_errors",
                             "sii_csv": res["CSV"],
-                            "sii_send_failed": True,
+                            "aeat_send_failed": True,
                         }
                     )
                 else:
-                    doc_vals["sii_send_failed"] = True
+                    doc_vals["aeat_send_failed"] = True
                 if (
-                    "sii_state" in doc_vals
+                    "aeat_state" in doc_vals
                     and not document.sii_account_registration_date
                     and mapping_key[:2] == "in"
                 ):
@@ -899,7 +803,7 @@ class SiiMixin(models.AbstractModel):
                         str(res_line["CodigoErrorRegistro"]),
                         str(res_line["DescripcionErrorRegistro"])[:60],
                     )
-                doc_vals["sii_send_error"] = send_error
+                doc_vals["aeat_send_error"] = send_error
                 document.write(doc_vals)
             except Exception as fault:
                 new_cr = Registry(self.env.cr.dbname).cursor()
@@ -907,10 +811,10 @@ class SiiMixin(models.AbstractModel):
                 document = env[document._name].browse(document.id)
                 doc_vals.update(
                     {
-                        "sii_send_failed": True,
-                        "sii_send_error": repr(fault)[:60],
+                        "aeat_send_failed": True,
+                        "aeat_send_error": repr(fault)[:60],
                         "sii_return": repr(fault),
-                        "sii_content_sent": json.dumps(inv_dict, indent=4),
+                        "aeat_content_sent": json.dumps(inv_dict, indent=4),
                     }
                 )
                 document.write(doc_vals)

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

@@ -367,7 +367,7 @@ ul.auto-toc {
 !! This file is generated by oca-gen-addon-readme !!
 !! changes will be overwritten.                   !!
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
-!! source digest: sha256:86d695960bc6ec21d8e704e119dc663acfaa8ce909155d0334ae2d4510ad88d5
+!! source digest: sha256:10113ecb0d04bfe1b6ece65a39892141bff2e511f72909f4cac9e9bb7d86d0ed
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! -->
 <p><a class="reference external image-reference" href="https://odoo-community.org/page/development-status"><img alt="Mature" src="https://img.shields.io/badge/maturity-Mature-brightgreen.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/licence-AGPL--3-blue.png" /></a> <a class="reference external image-reference" href="https://github.com/OCA/l10n-spain/tree/16.0/l10n_es_aeat_sii_oca"><img alt="OCA/l10n-spain" src="https://img.shields.io/badge/github-OCA%2Fl10n--spain-lightgray.png?logo=github" /></a> <a class="reference external image-reference" href="https://translation.odoo-community.org/projects/l10n-spain-16-0/l10n-spain-16-0-l10n_es_aeat_sii_oca"><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-spain&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>Módulo para la presentación inmediata del IVA

odoo/addons/l10n_es_aeat_sii_oca/tests/test_l10n_es_aeat_sii.py

@@ -108,7 +108,7 @@ class TestL10nEsAeatSiiBase(TestL10nEsAeatModBase, TestL10nEsAeatCertificateBase
         if extra_vals:
             vals.update(extra_vals)
         invoice = self.env["account.move"].create(vals)
-        result_dict = invoice._get_sii_invoice_dict()
+        result_dict = invoice._get_aeat_invoice_dict()
         path = get_resource_path(module, "tests/json", json_file)
         if not path:
             raise Exception("Incorrect JSON file: %s" % json_file)
@@ -211,7 +211,7 @@ class TestL10nEsAeatSii(TestL10nEsAeatSiiBase):
             {"partner_id": eu_customer.id, "fiscal_position_id": fp_extra.id}
         )
         invoice.action_post()
-        sii_info = invoice._get_sii_invoice_dict()
+        sii_info = invoice._get_aeat_invoice_dict()
         self.assertEqual(
             sii_info["FacturaExpedida"]["Contraparte"],
             {
@@ -367,7 +367,7 @@ class TestL10nEsAeatSii(TestL10nEsAeatSiiBase):
         company = invoice.company_id
         tax_agency = company.tax_agency_id
         self.sii_cert.company_id.tax_agency_id = tax_agency
-        proxy = invoice._connect_sii(invoice.move_type)
+        proxy = invoice._connect_aeat(invoice.move_type)
         address = proxy._binding_options["address"]
         self.assertTrue(address)
         if company.sii_test and tax_agency:
@@ -421,13 +421,13 @@ class TestL10nEsAeatSii(TestL10nEsAeatSiiBase):
         invoice.move_type = "out_refund"
         self.assertEqual(invoice.sii_refund_type, "I")
 
-    def test_is_sii_simplified_invoice(self):
-        self.assertFalse(self.invoice._is_sii_simplified_invoice())
-        self.partner.sii_simplified_invoice = True
-        self.assertTrue(self.invoice._is_sii_simplified_invoice())
+    def test_is_aeat_simplified_invoice(self):
+        self.assertFalse(self.invoice._is_aeat_simplified_invoice())
+        self.partner.aeat_simplified_invoice = True
+        self.assertTrue(self.invoice._is_aeat_simplified_invoice())
 
-    def test_sii_check_exceptions_case_supplier_simplified(self):
-        self.partner.sii_simplified_invoice = True
+    def test_aeat_check_exceptions_case_supplier_simplified(self):
+        self.partner.aeat_simplified_invoice = True
         invoice = self.env["account.move"].create(
             {
                 "partner_id": self.partner.id,
@@ -436,9 +436,9 @@ class TestL10nEsAeatSii(TestL10nEsAeatSiiBase):
             }
         )
         with self.assertRaises(exceptions.UserError):
-            invoice._sii_check_exceptions()
+            invoice._aeat_check_exceptions()
 
-    def test_sii_check_exceptions_case_supplier_on_post(self):
+    def test_aeat_check_exceptions_case_supplier_on_post(self):
         """Check sii exceptions when posting supplier bills"""
         supplier = self.supplier.copy()
         supplier.country_id = self.env.ref("base.es")
@@ -473,14 +473,14 @@ class TestL10nEsAeatSii(TestL10nEsAeatSiiBase):
         self.assertFalse(draft_invoice.exists())
 
     def test_unlink_invoice_when_sent_to_sii(self):
-        self.invoice.sii_state = "sent"
+        self.invoice.aeat_state = "sent"
         self.invoice.button_draft()  # Convert to draft to check only SII exception
         with self.assertRaises(exceptions.UserError):
             self.invoice.unlink()
 
     def test_account_move_sii_write_exceptions(self):
         # out_invoice
-        self.invoice.sii_state = "sent"
+        self.invoice.aeat_state = "sent"
         with self.assertRaises(exceptions.UserError):
             self.invoice.write({"invoice_date": "2022-01-01"})
         with self.assertRaises(exceptions.UserError):
@@ -488,7 +488,7 @@ class TestL10nEsAeatSii(TestL10nEsAeatSiiBase):
         # in_invoice
         in_invoice = self._create_invoice_for_sii("in_invoice")
         in_invoice.ref = "REF"
-        in_invoice.sii_state = "sent"
+        in_invoice.aeat_state = "sent"
         partner = self.partner.copy()
         with self.assertRaises(exceptions.UserError):
             in_invoice.write({"partner_id": partner.id})

odoo/addons/l10n_es_aeat_sii_oca/views/account_fiscal_position_view.xml

@@ -16,26 +16,26 @@
                     name="group_sii"
                     attrs="{'invisible': [('sii_enabled', '=', False)]}"
                 >
-                    <field name="sii_active" />
+                    <field name="aeat_active" />
                     <field
                         name="sii_registration_key_sale"
-                        attrs="{'invisible': [('sii_active', '!=', True)]}"
+                        attrs="{'invisible': [('aeat_active', '!=', True)]}"
                     />
                     <field
                         name="sii_registration_key_purchase"
-                        attrs="{'invisible': [('sii_active', '!=', True)]}"
+                        attrs="{'invisible': [('aeat_active', '!=', True)]}"
                     />
                     <field
                         name="sii_no_taxable_cause"
-                        attrs="{'invisible': [('sii_active', '!=', True)]}"
+                        attrs="{'invisible': [('aeat_active', '!=', True)]}"
                     />
                     <field
                         name="sii_exempt_cause"
-                        attrs="{'invisible': [('sii_active', '!=', True)]}"
+                        attrs="{'invisible': [('aeat_active', '!=', True)]}"
                     />
                     <field
                         name="sii_partner_identification_type"
-                        attrs="{'invisible': [('sii_active', '!=', True)]}"
+                        attrs="{'invisible': [('aeat_active', '!=', True)]}"
                     />
                 </group>
             </xpath>