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: change check_adults constrain by manual check in create/write methods

Browse Source
This commit is contained in:
Darío Lodeiros
2022-04-23 15:15:55 +02:00
parent 9163f3a618
commit e5fc07f50c
1 changed files with 12 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

21
pms/models/pms_reservation.py
View File

@@ -1624,6 +1624,13 @@ class PmsReservation(models.Model):
) )
) )
def _check_capacity(self):
for record in self:
if record.reservation_type != "out":
self.env["pms.room"]._check_adults(
record, record.service_ids.service_line_ids
)
@api.constrains("reservation_line_ids") @api.constrains("reservation_line_ids")
def checkin_checkout_consecutive_dates(self): def checkin_checkout_consecutive_dates(self):
""" """
@@ -1711,14 +1718,6 @@ class PmsReservation(models.Model):
if record.agency_id and not record.agency_id.is_agency: if record.agency_id and not record.agency_id.is_agency:
raise ValidationError(_("booking agency with wrong configuration: ")) raise ValidationError(_("booking agency with wrong configuration: "))
@api.constrains("check_adults")
def _check_capacity(self):
for record in self:
if record.reservation_type != "out":
self.env["pms.room"]._check_adults(
record, record.service_ids.service_line_ids
)
@api.constrains("closure_reason_id") @api.constrains("closure_reason_id")
def _check_closure_reason_id(self): def _check_closure_reason_id(self):
for record in self: for record in self:
@@ -1917,8 +1916,8 @@ class PmsReservation(models.Model):
vals["reservation_type"] = ( vals["reservation_type"] = (
folio.reservation_type if folio.reservation_type else "normal" folio.reservation_type if folio.reservation_type else "normal"
) )
record = super(PmsReservation, self).create(vals) record = super(PmsReservation, self).create(vals)
record._check_capacity()
if record.preconfirm and record.state == "draft": if record.preconfirm and record.state == "draft":
record.confirm() record.confirm()
@@ -1929,6 +1928,10 @@ class PmsReservation(models.Model):
def write(self, vals): def write(self, vals):
asset = super(PmsReservation, self).write(vals) asset = super(PmsReservation, self).write(vals)
self._check_services(vals) self._check_services(vals)
# Only check if adult to avoid to check capacity in intermediate states (p.e. flush)
# that not take access to possible extra beds service in vals
if "adults" in vals:
self._check_capacity()
return asset return asset
def _check_services(self, vals): def _check_services(self, vals):