From 46932981dbdc13725cca2562245b89401ffc4afe Mon Sep 17 00:00:00 2001
From: braisab <braisterbutalino@gmail.com>
Date: Fri, 10 Sep 2021 16:39:12 +0200
Subject: [PATCH] [REF]Refactor pms_checkin_partner possible_existing_customers

---
 pms/models/pms_checkin_partner.py       | 37 +++++++++++++++++++------
 pms/views/pms_checkin_partner_views.xml | 13 +++++----
 2 files changed, 35 insertions(+), 15 deletions(-)

diff --git a/pms/models/pms_checkin_partner.py b/pms/models/pms_checkin_partner.py
index 43e267065..62dfa2d68 100644
--- a/pms/models/pms_checkin_partner.py
+++ b/pms/models/pms_checkin_partner.py
@@ -213,15 +213,15 @@ class PmsCheckinPartner(models.Model):
         compute="_compute_partner_incongruences",
     )
 
-    is_possible_existing_customer_id = fields.Many2one(
+    possible_existing_customer_ids = fields.One2many(
         string="Possible existing customer",
         readonly=False,
         store=True,
-        compute="_compute_is_possible_existing_customer_id",
+        compute="_compute_possible_existing_customer_ids",
+        comodel_name="res.partner",
+        inverse_name="checkin_partner_possible_customer_id",
     )
 
-    add_possible_customer = fields.Boolean(string="Add possible Customer")
-
     @api.depends("partner_id")
     def _compute_document_number(self):
         for record in self:
@@ -389,7 +389,7 @@ class PmsCheckinPartner(models.Model):
         "firstname",
         "lastname",
         "lastname2",
-        "add_possible_customer",
+        # "add_possible_customer",
     )
     def _compute_partner_id(self):
         for record in self:
@@ -416,13 +416,17 @@ class PmsCheckinPartner(models.Model):
                             }
                             partner = self.env["res.partner"].create(partner_values)
                     record.partner_id = partner
-                elif record.add_possible_customer:
-                    self.env["pms.folio"]._add_customer(record)
 
     @api.depends("email", "mobile")
-    def _compute_is_possible_existing_customer_id(self):
+    def _compute_possible_existing_customer_ids(self):
         for record in self:
-            self.env["pms.folio"]._apply_is_possible_existing_customer_id(record)
+            possible_customer = self.env[
+                "pms.folio"
+            ]._apply_possible_existing_customer_ids(record.email, record.mobile)
+            if possible_customer:
+                record.possible_existing_customer_ids = possible_customer
+            else:
+                record.possible_existing_customer_ids = False
 
     @api.depends(
         "firstname",
@@ -686,3 +690,18 @@ class PmsCheckinPartner(models.Model):
             "target": "new",
             "flags": {"form": {"action_buttons": True}},
         }
+
+    def open_wizard_several_partners(self):
+        ctx = dict(
+            checkin_partner_id=self.id,
+            possible_existing_customer_ids=self.possible_existing_customer_ids.ids,
+        )
+        return {
+            "view_type": "form",
+            "view_mode": "form",
+            "name": "Several Customers",
+            "res_model": "pms.several.partners.wizard",
+            "target": "new",
+            "type": "ir.actions.act_window",
+            "context": ctx,
+        }
diff --git a/pms/views/pms_checkin_partner_views.xml b/pms/views/pms_checkin_partner_views.xml
index 50aa60cfe..d1e72b911 100644
--- a/pms/views/pms_checkin_partner_views.xml
+++ b/pms/views/pms_checkin_partner_views.xml
@@ -42,13 +42,14 @@
                         class="alert alert-warning"
                         role="alert"
                         style="margin-bottom:0px;"
-                        attrs="{'invisible': [('is_possible_existing_customer_id','=',False)]}"
+                        attrs="{'invisible': [('possible_existing_customer_ids','=',[])]}"
                     >
-                        There is a customer with this email or mobile, do you want to add it to the reservation?
-                        <field name="is_possible_existing_customer_id" invisible="1" />
-                        <field
-                            name="add_possible_customer"
-                            attrs="{'invisible': [('is_possible_existing_customer_id','=',False)]}"
+                        A customer/s has this email or mobile, do you want to add it?
+                        <field name="possible_existing_customer_ids" invisible="1" />
+                        <button
+                            name="open_wizard_several_partners"
+                            string="Add customer"
+                            type="object"
                         />
                     </div>
                     <group name="group_top">