Updated: Added occupancy in the functions and their calls in the QloApps core. #1342
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ons HotelRoomTypeFeaturePricing::getRoomTypeTotalPrice(), HotelRoomTypeFeaturePricing::getRoomTypeFeaturePricesPerDay() and in the calls.
There was a problem hiding this comment.
Pull Request Overview
This PR introduces occupancy parameters into various functions and their calls across the QloApps core modules to support occupancy‐based pricing and validations.
- Added occupancy retrieval and propagation into pricing and booking functions.
- Updated validations and pricing function calls to include the new occupancy parameter.
- Adjusted hook execution in Controller to pass values by reference in AJAX responses.
Reviewed Changes
Copilot reviewed 13 out of 13 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| modules/wkhotelfilterblock/wkhotelfilterblock.php | Added occupancy retrieval and passed it to the pricing function. |
| modules/qlopaypalcommerce/classes/WkPaypalCommerceHelper.php | Introduced occupancy array in PayPal order item details processing. |
| modules/hotelreservationsystem/controllers/admin/AdminHotelRoomsBookingController.php | Updated price validation and occupancy parameter when changing room types. |
| modules/hotelreservationsystem/classes/HotelRoomTypeFeaturePricing.php | Modified occupancy parameter handling in pricing calculation functions. |
| modules/hotelreservationsystem/classes/HotelCartBookingData.php | Added occupancy to booking data formatting calls. |
| modules/hotelreservationsystem/classes/HotelBookingDetail.php | Passed occupancy into feature pricing calls for room booking. |
| modules/hotelreservationsystem/classes/HotelAdvancedPayment.php | Propagated occupancy in advanced payment price calculations. |
| modules/blockcart/blockcart.php | Replaced the room request variable with occupancy for price retrieval. |
| controllers/front/ProductController.php | Updated occupancy value retrieval and pricing function calls in the front office. |
| controllers/admin/AdminOrdersController.php | Added occupancy to order booking and updated AJAX response handling. |
| classes/controller/Controller.php | Adjusted hook calls to pass $value by reference in AJAX responses. |
| classes/PaymentModule.php | Introduced occupancy into booking detail price calculations for orders. |
| classes/Cart.php | Updated occupancy parameter usage in cart product pricing functions. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.