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(res_partner.py): use api.model decorator for create method to avoid pylint warning

Browse Source 
🐛 fix(res_partner.py): use values instead of vals in create method to improve semantics
This commit is contained in:
Justin
2023-06-01 01:59:59 +00:00
committed by Darío Lodeiros
parent d7248054a8
commit 5da2d73d6b
2 changed files with 6 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
pms/models/res_partner.py
View File

@@ -697,12 +697,13 @@ class ResPartner(models.Model):
)
return super().unlink()
def create(self, vals):
check_missing_document = self._check_document_partner_required(vals)
@api.model
def create(self, values):
check_missing_document = self._check_document_partner_required(values)
if check_missing_document:
raise ValidationError(_("A document identification is required"))
return super().create(vals)
return super(ResPartner, self).create(values)
def write(self, vals):
check_missing_document = self._check_document_partner_required(

4
pms/wizards/pms_booking_engine.py
View File

@@ -277,8 +277,8 @@ class BookingEngine(models.TransientModel):
folio.reservation_ids.reservation_line_ids.discount = (
record.discount * 100
)
action = self.env.ref("pms.open_pms_folio1_form_tree_all").read()[0]
action["views"] = [(self.env.ref("pms.pms_folio_view_form").id, "form")]
action = self.sudo().env.ref("pms.open_pms_folio1_form_tree_all").read()[0]
action["views"] = [(self.sudo().env.ref("pms.pms_folio_view_form").id, "form")]
action["res_id"] = folio.id
return action