diff --git a/pms/__manifest__.py b/pms/__manifest__.py index 459e37610..091728350 100644 --- a/pms/__manifest__.py +++ b/pms/__manifest__.py @@ -41,7 +41,7 @@ "data/menus.xml", "wizards/wizard_payment_folio.xml", "wizards/folio_make_invoice_advance_views.xml", - "wizards/wizard_folio.xml", + "wizards/pms_booking_engine_views.xml", "wizards/wizard_folio_changes.xml", "views/pms_amenity_views.xml", "views/pms_amenity_type_views.xml", diff --git a/pms/models/pms_folio.py b/pms/models/pms_folio.py index cfd2a4c8c..80c2d148d 100644 --- a/pms/models/pms_folio.py +++ b/pms/models/pms_folio.py @@ -987,7 +987,7 @@ class PmsFolio(models.Model): def _check_required_partner_name(self): for record in self: if not record.partner_name: - raise models.ValidationError(_("You musyt assign a customer name")) + raise models.ValidationError(_("You must assign a customer name")) @api.model def create(self, vals): diff --git a/pms/models/pms_reservation.py b/pms/models/pms_reservation.py index 55308d575..0a89ab2dd 100644 --- a/pms/models/pms_reservation.py +++ b/pms/models/pms_reservation.py @@ -1518,7 +1518,7 @@ class PmsReservation(models.Model): folio = self.env["pms.folio"].browse(vals["folio_id"]) default_vals = {"pms_property_id": folio.pms_property_id.id} if folio.partner_id: - default_vals["partner_id"] = folio.partner_id + default_vals["partner_id"] = folio.partner_id.id elif folio.partner_name: default_vals["partner_name"] = folio.partner_name default_vals["partner_mobile"] = folio.mobile diff --git a/pms/security/ir.model.access.csv b/pms/security/ir.model.access.csv index 67bc1107a..d313323f6 100644 --- a/pms/security/ir.model.access.csv +++ b/pms/security/ir.model.access.csv @@ -53,7 +53,7 @@ user_access_pms_reservation_split_join_swap_wizard,user_access_pms_reservation_s user_access_pms_wizard_reservation_lines_split,user_access_pms_wizard_reservation_lines_split,model_pms_wizard_reservation_lines_split,pms.group_pms_user,1,1,1,1 user_access_pms_massive_changes_wizard,user_access_pms_massive_changes_wizard,model_pms_massive_changes_wizard,pms.group_pms_user,1,1,1,1 user_access_pms_advanced_filters_wizard,user_access_pms_advanced_filters_wizard,model_pms_advanced_filters_wizard,pms.group_pms_user,1,1,1,1 -user_access_pms_folio_wizard,user_access_pms_folio_wizard,model_pms_folio_wizard,pms.group_pms_user,1,1,1,1 +user_access_pms_booking_engine,user_access_pms_booking_engine,model_pms_booking_engine,pms.group_pms_user,1,1,1,1 user_access_pms_folio_availability_wizard,user_access_pms_folio_availability_wizard,model_pms_folio_availability_wizard,pms.group_pms_user,1,1,1,1 user_access_pms_num_rooms_selection,user_access_pms_num_rooms_selection,model_pms_num_rooms_selection,pms.group_pms_user,1,1,1,1 user_access_pms_folio_sale_line,user_access_pms_folio_sale_line,model_folio_sale_line,pms.group_pms_user,1,0,0,0 diff --git a/pms/static/src/js/pms_list_controller.js b/pms/static/src/js/pms_list_controller.js index 6f0837a95..74db95065 100644 --- a/pms/static/src/js/pms_list_controller.js +++ b/pms/static/src/js/pms_list_controller.js @@ -11,7 +11,7 @@ odoo.define("booking.engine.tree", function (require) { self.do_action({ name: "Booking Engine", type: "ir.actions.act_window", - res_model: "pms.folio.wizard", + res_model: "pms.booking.engine", target: "new", views: [[false, "form"]], context: {is_modal: true}, diff --git a/pms/tests/__init__.py b/pms/tests/__init__.py index fa95794df..e736526ea 100644 --- a/pms/tests/__init__.py +++ b/pms/tests/__init__.py @@ -29,7 +29,7 @@ from . import test_pms_room_type from . import test_pms_room_type_class from . import test_pms_board_service from . import test_pms_wizard_massive_changes -from . import test_pms_wizard_folio +from . import test_pms_booking_engine from . import test_pms_res_users from . import test_pms_amenity from . import test_pms_room diff --git a/pms/tests/test_pms_wizard_folio.py b/pms/tests/test_pms_booking_engine.py similarity index 94% rename from pms/tests/test_pms_wizard_folio.py rename to pms/tests/test_pms_booking_engine.py index 84fb8dcf2..7f4bc3bce 100644 --- a/pms/tests/test_pms_wizard_folio.py +++ b/pms/tests/test_pms_booking_engine.py @@ -202,7 +202,7 @@ class TestPmsWizardMassiveChanges(common.SavepointCase): ] # create folio wizard with partner id => pricelist & start-end dates - wizard_folio = self.env["pms.folio.wizard"].create( + booking_engine = self.env["pms.booking.engine"].create( { "start_date": checkin, "end_date": checkout, @@ -213,7 +213,7 @@ class TestPmsWizardMassiveChanges(common.SavepointCase): ) # force pricelist load - wizard_folio.flush() + booking_engine.flush() # availability items belonging to test property lines_availability_test = self.env["pms.folio.availability.wizard"].search( @@ -234,11 +234,11 @@ class TestPmsWizardMassiveChanges(common.SavepointCase): for discount in discounts: with self.subTest(k=discount): # ACT - wizard_folio.discount = discount["discount"] + booking_engine.discount = discount["discount"] # ASSERT self.assertEqual( - wizard_folio.total_price_folio, + booking_engine.total_price_folio, discount["expected_price"], "The total price calculation is wrong", ) @@ -284,7 +284,7 @@ class TestPmsWizardMassiveChanges(common.SavepointCase): pricelist_item.flush() # create folio wizard with partner id => pricelist & start-end dates - wizard_folio = self.env["pms.folio.wizard"].create( + booking_engine = self.env["pms.booking.engine"].create( { "start_date": checkin, "end_date": checkout, @@ -293,7 +293,7 @@ class TestPmsWizardMassiveChanges(common.SavepointCase): "pms_property_id": self.test_property.id, } ) - wizard_folio.flush() + booking_engine.flush() # availability items belonging to test property lines_availability_test = self.env["pms.folio.availability.wizard"].search( @@ -315,7 +315,7 @@ class TestPmsWizardMassiveChanges(common.SavepointCase): # ASSERT self.assertEqual( - wizard_folio.total_price_folio, + booking_engine.total_price_folio, expected_price_total, "The total price calculation is wrong", ) @@ -355,7 +355,7 @@ class TestPmsWizardMassiveChanges(common.SavepointCase): # pricelist_item.flush() # # create folio wizard with partner id => pricelist & start-end dates - # wizard_folio = self.env["pms.folio.wizard"].create( + # booking_engine = self.env["pms.booking.engine"].create( # { # "start_date": checkin, # "end_date": checkout, @@ -363,7 +363,7 @@ class TestPmsWizardMassiveChanges(common.SavepointCase): # "pricelist_id": self.test_pricelist.id, # } # ) - # wizard_folio.flush() + # booking_engine.flush() # # availability items belonging to test property # lines_availability_test = self.env["pms.folio.availability.wizard"].search( @@ -395,7 +395,7 @@ class TestPmsWizardMassiveChanges(common.SavepointCase): # # ASSERT # self.assertEqual( - # wizard_folio.total_price_folio, + # booking_engine.total_price_folio, # tc["expected_price"], # "The total price calculation is wrong", # ) @@ -413,7 +413,7 @@ class TestPmsWizardMassiveChanges(common.SavepointCase): checkout = fields.date.today() + datetime.timedelta(days=1) # create folio wizard with partner id => pricelist & start-end dates - wizard_folio = self.env["pms.folio.wizard"].create( + booking_engine = self.env["pms.booking.engine"].create( { "start_date": checkin, "end_date": checkout, @@ -422,7 +422,7 @@ class TestPmsWizardMassiveChanges(common.SavepointCase): "pms_property_id": self.test_property.id, } ) - wizard_folio.flush() + booking_engine.flush() # availability items belonging to test property lines_availability_test = self.env["pms.folio.availability.wizard"].search( @@ -440,7 +440,7 @@ class TestPmsWizardMassiveChanges(common.SavepointCase): lines_availability_test[0].num_rooms_selected = value # ACT - wizard_folio.create_folio() + booking_engine.create_folio() # ASSERT folio = self.env["pms.folio"].search_count( @@ -462,7 +462,7 @@ class TestPmsWizardMassiveChanges(common.SavepointCase): checkout = fields.date.today() + datetime.timedelta(days=1) # create folio wizard with partner id => pricelist & start-end dates - wizard_folio = self.env["pms.folio.wizard"].create( + booking_engine = self.env["pms.booking.engine"].create( { "start_date": checkin, "end_date": checkout, @@ -471,7 +471,7 @@ class TestPmsWizardMassiveChanges(common.SavepointCase): "pms_property_id": self.test_property.id, } ) - wizard_folio.flush() + booking_engine.flush() # availability items belonging to test property lines_availability_test = self.env["pms.folio.availability.wizard"].search( @@ -489,10 +489,10 @@ class TestPmsWizardMassiveChanges(common.SavepointCase): lines_availability_test[0].num_rooms_selected = value lines_availability_test[0].value_num_rooms_selected = 2 lines_availability_test.flush() - wizard_folio.flush() + booking_engine.flush() # ACT - wizard_folio.create_folio() + booking_engine.create_folio() folio = self.env["pms.folio"].search([("partner_id", "=", self.partner_id.id)]) folio.flush() @@ -513,7 +513,7 @@ class TestPmsWizardMassiveChanges(common.SavepointCase): checkout = fields.date.today() + datetime.timedelta(days=1) # create folio wizard with partner id => pricelist & start-end dates - wizard_folio = self.env["pms.folio.wizard"].create( + booking_engine = self.env["pms.booking.engine"].create( { "start_date": checkin, "end_date": checkout, @@ -522,7 +522,7 @@ class TestPmsWizardMassiveChanges(common.SavepointCase): "pms_property_id": self.test_property.id, } ) - wizard_folio.flush() + booking_engine.flush() # availability items belonging to test property lines_availability_test = self.env["pms.folio.availability.wizard"].search( @@ -541,7 +541,7 @@ class TestPmsWizardMassiveChanges(common.SavepointCase): lines_availability_test[0].value_num_rooms_selected = 1 # ACT - wizard_folio.create_folio() + booking_engine.create_folio() vals = { "partner_id": self.partner_id.id, "pricelist_id": self.test_pricelist.id, @@ -570,7 +570,7 @@ class TestPmsWizardMassiveChanges(common.SavepointCase): checkout = fields.date.today() + datetime.timedelta(days=1) # create folio wizard with partner id => pricelist & start-end dates - wizard_folio = self.env["pms.folio.wizard"].create( + booking_engine = self.env["pms.booking.engine"].create( { "start_date": checkin, "end_date": checkout, @@ -579,7 +579,7 @@ class TestPmsWizardMassiveChanges(common.SavepointCase): "pms_property_id": self.test_property.id, } ) - wizard_folio.flush() + booking_engine.flush() # availability items belonging to test property lines_availability_test = self.env["pms.folio.availability.wizard"].search( @@ -598,7 +598,7 @@ class TestPmsWizardMassiveChanges(common.SavepointCase): lines_availability_test[0].value_num_rooms_selected = 1 # ACT - wizard_folio.create_folio() + booking_engine.create_folio() folio = self.env["pms.folio"].search([("partner_id", "=", self.partner_id.id)]) @@ -639,7 +639,7 @@ class TestPmsWizardMassiveChanges(common.SavepointCase): discount = 0.5 # create folio wizard with partner id => pricelist & start-end dates - wizard_folio = self.env["pms.folio.wizard"].create( + booking_engine = self.env["pms.booking.engine"].create( { "start_date": checkin, "end_date": checkout, @@ -649,7 +649,7 @@ class TestPmsWizardMassiveChanges(common.SavepointCase): "pms_property_id": self.test_property.id, } ) - wizard_folio.flush() + booking_engine.flush() # availability items belonging to test property lines_availability_test = self.env["pms.folio.availability.wizard"].search( @@ -668,7 +668,7 @@ class TestPmsWizardMassiveChanges(common.SavepointCase): lines_availability_test[0].value_num_rooms_selected = 1 # ACT - wizard_folio.create_folio() + booking_engine.create_folio() folio = self.env["pms.folio"].search([("partner_id", "=", self.partner_id.id)]) @@ -705,7 +705,7 @@ class TestPmsWizardMassiveChanges(common.SavepointCase): ) # create folio wizard with partner id => pricelist & start-end dates - wizard_folio = self.env["pms.folio.wizard"].create( + booking_engine = self.env["pms.booking.engine"].create( { "start_date": checkin, "end_date": checkout, @@ -714,9 +714,9 @@ class TestPmsWizardMassiveChanges(common.SavepointCase): "pms_property_id": self.test_property.id, } ) - wizard_folio.flush() + booking_engine.flush() - room_type_plan_avail = wizard_folio.availability_results.filtered( + room_type_plan_avail = booking_engine.availability_results.filtered( lambda r: r.room_type_id.id == self.test_room_type_double.id ).num_rooms_available @@ -747,7 +747,7 @@ class TestPmsWizardMassiveChanges(common.SavepointCase): ) # create folio wizard with partner id => pricelist & start-end dates - wizard_folio = self.env["pms.folio.wizard"].create( + booking_engine = self.env["pms.booking.engine"].create( { "start_date": checkin, "end_date": checkout, @@ -756,9 +756,9 @@ class TestPmsWizardMassiveChanges(common.SavepointCase): "pms_property_id": self.test_property.id, } ) - wizard_folio.flush() + booking_engine.flush() - room_type_plan_avail = wizard_folio.availability_results.filtered( + room_type_plan_avail = booking_engine.availability_results.filtered( lambda r: r.room_type_id.id == self.test_room_type_double.id ).num_rooms_available diff --git a/pms/views/pms_folio_views.xml b/pms/views/pms_folio_views.xml index 4436691a5..7ecb95b2a 100644 --- a/pms/views/pms_folio_views.xml +++ b/pms/views/pms_folio_views.xml @@ -140,7 +140,7 @@