audrius/pms
1
0
Fork 1
mirror of https://github.com/OCA/pms.git synced 2025-01-29 00:17:45 +02:00
Code Issues Packages Projects Releases Wiki Activity

[FIX] pms: set possible_existing_customer_ids to false

Browse Source 
When creating a new folio from the interface, odoo
fails to assign possible_existing_customer_ids to
the record. Setting it to False for the cas when there are
no partner_name nor possible_customer fixes the issue.
This commit is contained in:
Robin Keunen
2023-08-03 17:02:34 +02:00
committed by Darío Lodeiros
parent 4aa8814410
commit c3ccc321b0
2 changed files with 6 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
pms/models/pms_folio.py
View File

@@ -1371,14 +1371,13 @@ class PmsFolio(models.Model):
@api.depends("email", "mobile", "partner_name") @api.depends("email", "mobile", "partner_name")
def _compute_possible_existing_customer_ids(self): def _compute_possible_existing_customer_ids(self):
for record in self: for record in self:
record.possible_existing_customer_ids = False
if record.partner_name: if record.partner_name:
possible_customer = self._apply_possible_existing_customer_ids( possible_customer = self._apply_possible_existing_customer_ids(
record.email, record.mobile, record.partner_id record.email, record.mobile, record.partner_id
) )
if possible_customer: if possible_customer:
record.possible_existing_customer_ids = possible_customer record.possible_existing_customer_ids = possible_customer
else:
record.possible_existing_customer_ids = False
@api.depends("reservation_ids", "reservation_ids.checkin") @api.depends("reservation_ids", "reservation_ids.checkin")
def _compute_first_checkin(self): def _compute_first_checkin(self):

6
pms/tests/test_pms_folio.py
View File

@@ -4,7 +4,7 @@ from freezegun import freeze_time
from odoo import fields from odoo import fields
from odoo.exceptions import ValidationError from odoo.exceptions import ValidationError
from odoo.tests import Form
from .common import TestPms from .common import TestPms
@@ -1537,3 +1537,7 @@ class TestPmsFolio(TestPms):
"sale_channel_origin_id of reservations that coincided " "sale_channel_origin_id of reservations that coincided "
"with sale_channel_origin_id of folio de should be updated", "with sale_channel_origin_id of folio de should be updated",
) )
def test_pms_folio_form_creation(self):
folio_form = Form(self.env["pms.folio"])
self.assertFalse(folio_form.possible_existing_customer_ids)