Skip to content

BookingPool 48018: Redirect to Post-Booking Information#11737

Merged
thojou merged 1 commit into
ILIAS-eLearning:release_11from
matheuszych:bt/48018
Jul 20, 2026
Merged

BookingPool 48018: Redirect to Post-Booking Information#11737
thojou merged 1 commit into
ILIAS-eLearning:release_11from
matheuszych:bt/48018

Conversation

@matheuszych

Copy link
Copy Markdown
Contributor

See: https://mantis.ilias.de/view.php?id=48018

After booking from the bookable items table, users were not sent to post-booking information when an item had post text or a file attached. BookableItemTableBookAction now collects reservation ids and redirects to displayPostInfo when bookings succeed. ProcessUtilGUI::displayPostInfo() was refactored to resolve periods per object, render one info block per bookable item with booking information, and support multiple objects via book_post_booking_information_for. Added BookingsTableBookingInformationAction for the bookings table and handled returnCmd render in back().

/cc @thojou

See: https://mantis.ilias.de/view.php?id=48018

After booking from the bookable items table, users were not sent to post-booking information when an item had post text or a file attached. `BookableItemTableBookAction` now collects reservation ids and redirects to `displayPostInfo` when bookings succeed. `ProcessUtilGUI::displayPostInfo()` was refactored to resolve periods per object, render one info block per bookable item with booking information, and support multiple objects via `book_post_booking_information_for`. Added `BookingsTableBookingInformationAction` for the bookings table and handled `returnCmd` `render` in `back()`.
@thojou
thojou merged commit 9b8fe51 into ILIAS-eLearning:release_11 Jul 20, 2026
5 checks passed
@thojou

thojou commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Hey @matheuszych,

thanks for the PR.
I've integrated the changes into release_11 and trunk.

Kind Regards,
@thojou

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants