Skip to content

[IMP] pms: charge non per-day services on current date for on board reservations#418

Merged
OCA-git-bot merged 1 commit into
OCA:16.0from
commitsun:16.0-imp-pms-onboard-service-current-date
Jun 26, 2026
Merged

[IMP] pms: charge non per-day services on current date for on board reservations#418
OCA-git-bot merged 1 commit into
OCA:16.0from
commitsun:16.0-imp-pms-onboard-service-current-date

Conversation

@DarioLodeiros

Copy link
Copy Markdown
Member

Description

A non per-day service added to a reservation is always dated on the reservation checkout date. When the service is added while the guest is already on board (e.g. a drink, a parking spot or any extra charged at the front desk during the stay), this places the charge on the last day of the stay instead of the day it is actually consumed.

Change

In pms.service._compute_service_line_ids, when a non per-day product is added to a reservation whose state is onboard, the auto-created service line is now dated on the current date (property timezone) instead of the checkout date.

Behavior left unchanged:

  • Per-day services (per_day = True) still create one line per night.
  • Services consumed on check-in (consumed_on = "checkin") still use the check-in date.
  • Any other case (reservation not on board) keeps using the checkout date.

…eservations

A non per-day service added to a reservation was always dated on the
reservation checkout date. When the service is added while the guest is
already on board (e.g. a drink or a parking charged at the front desk),
this placed the charge on the last day of the stay instead of the day it
is actually consumed.

Charge non per-day services on the current date, in the property
timezone, whenever the reservation is already on board. Per-day services
and services consumed on check-in keep their previous behavior.
@DarioLodeiros

Copy link
Copy Markdown
Member Author

/ocabot merge minor

@OCA-git-bot

Copy link
Copy Markdown
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 16.0-ocabot-merge-pr-418-by-DarioLodeiros-bump-minor, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 8cbd39c into OCA:16.0 Jun 26, 2026
5 of 7 checks passed
@OCA-git-bot

Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at 15785aa. Thanks a lot for contributing to OCA. ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants