diff --git a/pms/models/pms_checkin_partner.py b/pms/models/pms_checkin_partner.py
index 08af80e57..9f6228501 100644
--- a/pms/models/pms_checkin_partner.py
+++ b/pms/models/pms_checkin_partner.py
@@ -606,7 +606,7 @@ class PmsCheckinPartner(models.Model):
def open_partner(self):
""" Utility method used to add an "View Customer" button in folio views """
self.ensure_one()
- partner_form_id = self.env.ref("base.view_partner_address_form").id
+ partner_form_id = self.env.ref("pms.view_partner_data_form").id
return {
"type": "ir.actions.act_window",
"res_model": "res.partner",
diff --git a/pms/models/pms_reservation.py b/pms/models/pms_reservation.py
index 6ad6ccb8a..f11df9899 100644
--- a/pms/models/pms_reservation.py
+++ b/pms/models/pms_reservation.py
@@ -1474,7 +1474,7 @@ class PmsReservation(models.Model):
checkins = self.env["pms.checkin.partner"]
for record in self:
checkins += record.checkin_partner_ids.filtered(
- lambda s: s.state in ("onboard", "done")
+ lambda s: s.state in ("precheckin", "onboard", "done")
)
if checkins:
return self.env.ref("pms.action_traveller_report").report_action(checkins)
diff --git a/pms/tests/test_pms_checkin_partner.py b/pms/tests/test_pms_checkin_partner.py
index f6257edd8..2ea0502ba 100644
--- a/pms/tests/test_pms_checkin_partner.py
+++ b/pms/tests/test_pms_checkin_partner.py
@@ -824,7 +824,7 @@ class TestPmsCheckinPartner(TestPms):
# ASSERT
self.assertTrue(self.host1.mobile, "Partner mobile must be added")
- def _test_partner_id_numbers_created_from_checkin(self):
+ def test_partner_id_numbers_created_from_checkin(self):
"""
Some of the required data of the checkin_partner to create the partner are document_type
and document_number, with them an id_number is created associated with the partner that
@@ -842,6 +842,8 @@ class TestPmsCheckinPartner(TestPms):
}
)
+ checkin.flush()
+
# ASSERT
self.assertTrue(
checkin.partner_id.id_numbers,
@@ -864,12 +866,13 @@ class TestPmsCheckinPartner(TestPms):
"Checkin partner email and partner email shouldn't match",
)
- def _test_partner_modified_previous_checkin_not_modified(self):
+ def test_partner_modified_previous_checkin_not_modified(self):
"""
If a partner modifies any of its fields, these change mustn't be reflected
in the previous checkins associated with it
"""
# ARRANGE
+ self.checkin1.flush()
self.host1.gender = "female"
# ASSERT
self.assertNotEqual(
diff --git a/pms/views/pms_checkin_partner_views.xml b/pms/views/pms_checkin_partner_views.xml
index e7c562442..116113283 100644
--- a/pms/views/pms_checkin_partner_views.xml
+++ b/pms/views/pms_checkin_partner_views.xml
@@ -73,7 +73,7 @@
icon="fa-file-pdf-o"
name="%(action_traveller_report)d"
string="Print in PDF"
- attrs="{'invisible': [('state','not in', ('onboard','done'))]}"
+ attrs="{'invisible': [('state','not in', ('precheckin','onboard','done'))]}"
/>
diff --git a/pms/views/res_partner_views.xml b/pms/views/res_partner_views.xml
index 1d1eae5e4..5ab2c6581 100644
--- a/pms/views/res_partner_views.xml
+++ b/pms/views/res_partner_views.xml
@@ -93,6 +93,44 @@
+
PART OF TRAVELERS ENTRY
-Document number: +
PART OF TRAVELERS ENTRY
+Document number: /
-+
Total amount (Reservation Card):
-TRAVELER'S DOCUMENT
+TRAVELER'S DOCUMENT