From b7338774744bcebace0866e22405107ed4f872ee Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Dar=C3=ADo=20Lodeiros?= <dario@commitsun.com>
Date: Tue, 28 May 2024 17:25:25 +0200
Subject: [PATCH] [DEL]pms_l10n_es: delete reference to document type
 aeat_identification_type

---
 pms_l10n_es/data/pms_data.xml             | 31 ------------------
 pms_l10n_es/models/pms_checkin_partner.py | 38 -----------------------
 2 files changed, 69 deletions(-)

diff --git a/pms_l10n_es/data/pms_data.xml b/pms_l10n_es/data/pms_data.xml
index 541759efd..7d31144f9 100644
--- a/pms_l10n_es/data/pms_data.xml
+++ b/pms_l10n_es/data/pms_data.xml
@@ -62,37 +62,6 @@ else:
 <!--            <field name="has_unique_numbers">True</field>-->
         </record>
 
-        <!-- aeat document type mapped -->
-        <!-- disabled because this is an override of an existe record in this file -->
-        <!-- pylint: disable=redundant-modulename-xml -->
-        <!-- oca-hooks:disable=xml-redundant-module-name -->
-        <record id="pms_l10n_es.document_type_dni" model="res.partner.id_category">
-            <field name="aeat_identification_type">02</field>
-        </record>
-        <record
-        id="pms_l10n_es.document_type_spanish_residence"
-        model="res.partner.id_category"
-    >
-            <field name="aeat_identification_type">05</field>
-        </record>
-        <record
-        id="pms.document_type_european_residence"
-        model="res.partner.id_category"
-    >
-            <field name="aeat_identification_type">05</field>
-        </record>
-        <record id="pms.document_type_passport" model="res.partner.id_category">
-            <field name="aeat_identification_type">03</field>
-        </record>
-        <record id="pms.document_type_driving_license" model="res.partner.id_category">
-            <field name="aeat_identification_type">06</field>
-        </record>
-        <record
-        id="pms.document_type_identification_document"
-        model="res.partner.id_category"
-    >
-            <field name="aeat_identification_type">06</field>
-        </record>
         <record id="base.es" model="res.country">
             <field name="priority">10</field>
         </record>
diff --git a/pms_l10n_es/models/pms_checkin_partner.py b/pms_l10n_es/models/pms_checkin_partner.py
index 3fd7bf3a8..0119f7faa 100644
--- a/pms_l10n_es/models/pms_checkin_partner.py
+++ b/pms_l10n_es/models/pms_checkin_partner.py
@@ -1,5 +1,4 @@
 import logging
-import re
 
 from odoo import api, fields, models
 
@@ -64,40 +63,3 @@ class PmsCheckinPartner(models.Model):
         manual_fields = super(PmsCheckinPartner, self)._checkin_manual_fields(depends)
         manual_fields.extend(["support_number"])
         return manual_fields
-
-    def _get_partner_by_document(self, document_number, document_type):
-        # if not find partner by documents (super method) then search by
-        # partner fields, VAT, or aeat_identification equivalent
-        partner = super(PmsCheckinPartner, self)._get_partner_by_document(
-            document_number, document_type
-        )
-        document_number = re.sub(r"[^a-zA-Z0-9]", "", document_number).upper()
-        if not partner and document_number and document_type:
-            search_field_name = False
-            if document_type.aeat_identification_type in ["03", "05", "06"]:
-                search_field_name = "aeat_identification"
-                search_comparison = "="
-            elif document_type.aeat_identification_type in ["02", "04"]:
-                # If we have a NIF, we search by VAT (with or without country code)
-                search_field_name = "vat"
-                search_comparison = "=like"
-                document_number = "%" + document_number
-            if search_field_name:
-                partner = self.env["res.partner"].search(
-                    [
-                        ("is_company", "=", False),
-                        ("is_agency", "=", False),
-                        (search_field_name, search_comparison, document_number),
-                    ],
-                    limit=1,
-                )
-                if (
-                    partner
-                    and partner.vat
-                    and len(partner.vat) - len(document_number) > 2
-                ):
-                    # The country code length is 2, so if the difference is greater than 2
-                    # the partner is not the same
-                    # TODO: this method need pass country code to search
-                    partner = False
-        return partner