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

[IMP]pms: set automatically preferred_room_id only in complete reservation_lines assigned

Browse Source
This commit is contained in:
Darío Lodeiros
2022-07-23 18:47:16 +02:00
parent 9705e657c7
commit 8be5cf58b4
1 changed files with 12 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
pms/models/pms_reservation.py
View File

@@ -1335,7 +1335,13 @@ class PmsReservation(models.Model):
reservation.preferred_room_id = False reservation.preferred_room_id = False
else: else:
reservation.splitted = False reservation.splitted = False
if room_ids: # Set automatically preferred_room_id if, and only if,
# all nights has the same room
if (
len(room_ids) == 1
and len(reservation.reservation_line_ids)
== (reservation.checkout - reservation.checkin).days
):
reservation.preferred_room_id = room_ids[0] reservation.preferred_room_id = room_ids[0]
@api.depends( @api.depends(
@@ -2045,7 +2051,7 @@ class PmsReservation(models.Model):
self._check_clousure_reason( self._check_clousure_reason(
reservation_type=vals.get("reservation_type"), reservation_type=vals.get("reservation_type"),
clousure_reason=vals.get("clousure_reason_id"), closure_reason_id=vals.get("closure_reason_id"),
) )
# Create the folio in case of need # Create the folio in case of need
@@ -2130,10 +2136,12 @@ class PmsReservation(models.Model):
folio_vals["email"] = reservation_vals.get("email") folio_vals["email"] = reservation_vals.get("email")
elif reservation_vals.get("reservation_type") != "out": elif reservation_vals.get("reservation_type") != "out":
raise ValidationError(_("Partner contact name is required")) raise ValidationError(_("Partner contact name is required"))
if reservation_vals.get("reservation_type"):
folio_vals["reservation_type"] = reservation_vals.get("reservation_type")
return folio_vals return folio_vals
def _check_clousure_reason(self, reservation_type, clousure_reason): def _check_clousure_reason(self, reservation_type, closure_reason_id):
if reservation_type == "out" and not clousure_reason: if reservation_type == "out" and not closure_reason_id:
raise ValidationError( raise ValidationError(
_( _(
"A closure reason is mandatory when reservation" "A closure reason is mandatory when reservation"