From 6cd01fb0d28ee2068fd4c8bfdc08bf610d281b8e Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Dar=C3=ADo=20Lodeiros?= Date: Sun, 31 Jan 2021 10:12:51 +0100 Subject: [PATCH] [FIX]pms: change unlink checkins partner autocreate by write --- pms/models/pms_checkin_partner.py | 3 ++- pms/tests/test_pms_checkin_partner.py | 11 +++++------ 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/pms/models/pms_checkin_partner.py b/pms/models/pms_checkin_partner.py index d979451c6..d74195b18 100644 --- a/pms/models/pms_checkin_partner.py +++ b/pms/models/pms_checkin_partner.py @@ -208,7 +208,8 @@ class PmsCheckinPartner(models.Model): lambda c: c.state == "draft" ) if len(draft_checkins) > 0 and vals.get("partner_id"): - draft_checkins[0].sudo().unlink() + draft_checkins[0].write(vals) + return draft_checkins[0] if vals.get("identifier", _("New")) == _("New") or "identifier" not in vals: pms_property_id = ( self.env.user.get_active_property_ids()[0] diff --git a/pms/tests/test_pms_checkin_partner.py b/pms/tests/test_pms_checkin_partner.py index c3c647f44..15a0664a7 100644 --- a/pms/tests/test_pms_checkin_partner.py +++ b/pms/tests/test_pms_checkin_partner.py @@ -348,7 +348,9 @@ class TestPmsCheckinPartner(TestHotel): freezer.start() PmsReservation.auto_no_show() - no_show_reservations = self.folio_1.filtered(lambda r: r.state == "no_show") + no_show_reservations = self.folio_1.reservation_ids.filtered( + lambda r: r.state == "no_show" + ) # ASSERT self.assertEqual( @@ -370,14 +372,11 @@ class TestPmsCheckinPartner(TestHotel): freezer.start() PmsReservation.auto_no_checkout() - no_checkout_reservations = self.folio_1.filtered( - lambda r: r.state == "no_checkout" - ) freezer.stop() # ASSERT self.assertEqual( - len(no_checkout_reservations), - 1, + self.reservation_1.state, + "no_checkout", "Reservations not set like No checkout", )