https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 Paul Derscheid <paul.derscheid@lmscloud.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Circulation| |Bookings function| | Depends on| |29002 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29002 [Bug 29002] Add ability to book items ahead of time -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #1 from Paul Derscheid <paul.derscheid@lmscloud.de> --- Created attachment 188546 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=188546&action=edit Bug 41129: [DO NOT PUSH] Add BookingModal.vue, remove place_booking.{inc,js} - Migration to Vue - Pinia store integration - Flatpickr calendar integration with constraint highlighting - Multi-step booking form with validation - Availability checking - API adapter pattern (prep for full APIClient migration) Technical notes: The booking adapters currently use fetch() directly but have been structured to facilitate migration to APIClient pattern. Full APIClient migration requires: - BiblioAPIClient for bookable items/pickup locations - CheckoutAPIClient for checkout queries - CirculationRulesAPIClient for rules endpoint - PatronAPIClient.search() method enhancement This work should be completed as a team effort to ensure consistency across the codebase. Test plan: STAFF INTERFACE - Creating Bookings Preparation: - Run yarn js:build to emit the bundles 1. Navigate to a biblio detail page with bookable items 2. Click "Place booking" button 3. Observe the Vue booking modal opens Step 1 - Patron Selection: 4. Type patron name/cardnumber in search field 5. Verify typeahead search returns results 6. Select a patron 7. Verify patron is selected and displays correctly Step 2 - Booking Details: 8. Verify item type dropdown is populated 9. Select an item type 10. Verify pickup location dropdown is populated 11. Select a pickup location 12. If multiple bookable items exist, verify item dropdown appears 13. Select specific item (if applicable) Step 3 - Booking Period: 14. Verify flatpickr calendar opens 15. Verify calendar shows existing bookings, checkouts, and available dates 16. Select a date range 17. Verify date range validation works 18. Verify lead period/trail period restrictions are enforced 19. Verify maximum booking period is enforced 20. Verify constraint highlighting updates correctly when changing item type, specific item, or pickup location Validation Testing: 21. Try to submit without patron - verify error message 22. Try to submit without item type - verify error message 23. Try to submit without pickup location - verify error message 24. Try to submit without date range - verify error message 25. Try to select dates outside allowed range - verify validation Capacity Testing: 26. For item type with limited bookable items, verify capacity warnings 27. Verify zero-capacity items show appropriate message Submitting Bookings: 28. Complete all required fields 29. Click "Place booking" 30. Verify booking is created successfully 31. Verify modal closes 32. Verify booking appears in bookings list STAFF INTERFACE - Editing Bookings 33. Click "Edit" on an existing booking 34. Verify modal opens with pre-filled data (patron, item type, pickup location, date range) 35. Modify the date range 36. Click "Update booking" 37. Verify changes are saved 38. Verify updated booking reflects changes Calendar Navigation: 39. In date picker, navigate between months 40. Verify highlighting persists across navigation 41. Verify availability data loads correctly for new months Error Handling: 42. Disconnect network 43. Try to create a booking 44. Verify appropriate error message displays 45. Restore network and verify subsequent booking attempts work Store State Management: 46. Open booking modal and fill in some fields but don't submit 47. Close modal and re-open it 48. Verify modal state is reset (no data persists inappropriately) Browser Console: 49. Throughout all tests, verify no JavaScript errors in console 50. Verify no warnings about deprecated code Build Process: 51. Run yarn js:build{,:prod} 52. Verify build completes without errors 53. Verify dist files are generated correctly Additional Regression Testing: 54. Test with various system preferences configurations 55. Test with multiple bookable items per biblio 56. Test with various circulation rules configurations 57. Test with different patron categories 58. Test booking constraints at item level vs itemtype level -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #3 from Paul Derscheid <paul.derscheid@lmscloud.de> --- Created attachment 188551 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=188551&action=edit Bug 41129: Use Bootstrap 5 Modal API for booking modal - Use Bootstrap Modal API for show/hide/dispose lifecycle - Replace custom modal classes with Bootstrap native classes - Remove custom backdrop CSS - Remove modal-scroll.mjs polyfill - Unify two watchers on props.open into single watcher - Add focus management to prevent aria-hidden warnings -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 Andrew Auld <andrew.auld@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |andrew.auld@openfifth.co.uk | |, | |jacob.omara@openfifth.co.uk | |, | |matt.blenkinsop@openfifth.c | |o.uk, | |pedro.amorim@openfifth.co.u | |k -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #5 from Paul Derscheid <paul.derscheid@lmscloud.de> --- Created attachment 189350 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=189350&action=edit Bug 41129: Add cypress tests To test: - Run `yarn cypress run --spec t/cypress/integration/Bookings/BookingModal_spec.ts` -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #6 from Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> --- I have a series of fixes, enhancements and most importantly cypress tests submitted in a line down from bug 40134. I intend to rebase the patches here against those to ensure we catch all the cases I fixed there.. many of my pieces are bugfixes so I believe they should go first. Once I've rebased, I'll check the results against the Cypress tests and work on bringing this up to date for, hopefully, an early push in 26.05 -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |37618 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37618 [Bug 37618] Select patron's, then item's home library by default for pickup location when creating bookings -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #7 from Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> --- *** Bug 37151 has been marked as a duplicate of this bug. *** -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |37151 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37151 [Bug 37151] Add detail on hover for bookings event dots -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Depends on|40134 |37707 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37707 [Bug 37707] Lead/Trail times should work in combination https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40134 [Bug 40134] Fix and optimise 'Any item' functionality of bookings -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #10 from Paul Derscheid <paul.derscheid@lmscloud.de> --- Created attachment 193910 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=193910&action=edit Bug 41129: Add initial Cypress coverage for booking modal - Test modal open/close lifecycle from biblio detail page - Test patron search typeahead and selection - Test item type and pickup location dropdowns - Test flatpickr date range selection and validation - Test booking creation via API with success/error paths - Test edit booking flow with pre-filled form state -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #12 from Paul Derscheid <paul.derscheid@lmscloud.de> --- Created attachment 193912 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=193912&action=edit Bug 41129: Fix booking modal runtime and test regressions Runtime: - Add missing use C4::Context in Libraries.pm - Use safe DOM construction instead of innerHTML in external-dependents.mjs (booking count update, success message) - Remove reference to undefined BookingDateRangeConstraint syspref - Stub OPAC fetchHolidays (public endpoint does not exist) - Pass UI visibility flags to useBookingValidation as reactive parameter instead of reading from store Type/test corrections: - Remove duplicate ItemType type declaration in bookings.d.ts - Remove overly broad uncaught:exception handler from e2e.js - Stabilise DatePicker spec against flatpickr navigation races - Correct fencepost in max booking period calculation -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 Paul Derscheid <paul.derscheid@lmscloud.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #193909|0 |1 is obsolete| | Attachment #193910|0 |1 is obsolete| | Attachment #193911|0 |1 is obsolete| | Attachment #193912|0 |1 is obsolete| | Attachment #193913|0 |1 is obsolete| | --- Comment #14 from Paul Derscheid <paul.derscheid@lmscloud.de> --- Created attachment 193925 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=193925&action=edit Bug 41129: Introduce Vue booking modal and Bootstrap 5 modal wiring Replace the jQuery/TT-based place_booking modal with a Vue 3 SFC component mounted via the island architecture pattern. Core components: - Add BookingModal.vue (<script setup lang="ts">) with multi-step form (patron, details, period) - Add Pinia store (stores/bookings.js) for shared booking state - Add flatpickr calendar with constraint highlighting, marker dots, hover feedback bar, and availability computation - Use Bootstrap 5 Modal API (show/hide/dispose) for modal lifecycle - Add island.inc, button-place.inc, button-edit.inc entry points - Wire into cat-toolbar.inc, detail.tt, bookings/list.tt, and all catalogue detail views - Add KohaAlert.vue, dayjs.mjs, and apiErrors.js shared utilities - Add rspack.config.js entry for booking-modal-island bundle Module architecture: - Add API adapter layer (lib/adapters/api/) using fetch(), structured for future APIClient migration - Split calendar adapter into focused modules under lib/adapters/calendar/ (events, highlighting, locale, markers, prevention, visibility) - Add BookingDate class (immutable, timezone-aware date wrapper) - Add availability/ sub-modules (date-change, disabled-dates, period-validators, rules, unavailable-map) - Add interval tree and sweep-line processor for efficient overlap detection and unavailability computation - Add composables for availability, validation, capacity guard, constraint highlighting, rules fetching, flatpickr lifecycle, and form defaults - Add conflict-resolution, constraints, highlighting, markers modules - Add hover-feedback module for calendar feedback bar - Manage error state in Pinia store Behavior: - Add 3-way payload construction for "any item" bookings - Add editable property to timeline booking items - Show transient success message after booking save - Apply server timezone conversion for timeline dates API/embed compatibility: - Add GET /libraries/{id}/holidays endpoint (Libraries.pm, swagger) - Allow "library" embed on GET /patrons/{id} (patrons.yaml) Cypress coverage: - Test modal open/close lifecycle from biblio detail page - Test patron search typeahead and selection - Test item type and pickup location dropdowns - Test flatpickr date range selection and validation - Test booking creation via API with success/error paths - Test edit booking flow with pre-filled form state - Add Vue Select test helpers (vue-select.js) - Validate flatpickr state via instance API instead of DOM classes Technical notes: The booking adapters use fetch() directly but are structured to facilitate migration to the APIClient pattern. Full migration requires BiblioAPIClient, CheckoutAPIClient, CirculationRulesAPI- Client, and PatronAPIClient.search() enhancements — best done as a coordinated effort across the codebase. Test plan: STAFF INTERFACE - Creating Bookings Preparation: - Run yarn js:build to emit the bundles Automated tests: - yarn cypress run --spec 't/cypress/integration/Circulation/bookingsModal*.ts' 1. Navigate to a biblio detail page with bookable items 2. Click "Place booking" button 3. Observe the Vue booking modal opens Step 1 - Patron Selection: 4. Type patron name/cardnumber in search field 5. Verify typeahead search returns results 6. Select a patron 7. Verify patron is selected and displays correctly Step 2 - Booking Details: 8. Verify item type dropdown is populated 9. Select an item type 10. Verify pickup location dropdown is populated 11. Select a pickup location 12. If multiple bookable items exist, verify item dropdown appears 13. Select specific item (if applicable) Step 3 - Booking Period: 14. Verify flatpickr calendar opens 15. Verify calendar shows existing bookings, checkouts, and available dates 16. Select a date range 17. Verify date range validation works 18. Verify lead period/trail period restrictions are enforced 19. Verify maximum booking period is enforced 20. Verify constraint highlighting updates correctly when changing item type, specific item, or pickup location Validation Testing: 21. Try to submit without patron - verify error message 22. Try to submit without item type - verify error message 23. Try to submit without pickup location - verify error message 24. Try to submit without date range - verify error message 25. Try to select dates outside allowed range - verify validation Capacity Testing: 26. For item type with limited bookable items, verify capacity warnings 27. Verify zero-capacity items show appropriate message Submitting Bookings: 28. Complete all required fields 29. Click "Place booking" 30. Verify booking is created successfully 31. Verify modal closes 32. Verify booking appears in bookings list STAFF INTERFACE - Editing Bookings 33. Click "Edit" on an existing booking 34. Verify modal opens with pre-filled data (patron, item type, pickup location, date range) 35. Modify the date range 36. Click "Update booking" 37. Verify changes are saved 38. Verify updated booking reflects changes Calendar Navigation: 39. In date picker, navigate between months 40. Verify highlighting persists across navigation 41. Verify availability data loads correctly for new months Error Handling: 42. Disconnect network 43. Try to create a booking 44. Verify appropriate error message displays 45. Restore network and verify subsequent booking attempts work Store State Management: 46. Open booking modal and fill in some fields but don't submit 47. Close modal and re-open it 48. Verify modal state is reset (no data persists inappropriately) Build Process: 49. Run yarn js:build and yarn js:build:prod 50. Verify build completes without errors 51. Verify dist files are generated correctly Sponsored-by: Büchereizentrale Schleswig-Holstein <https://www.bz-sh.de/> -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #16 from Paul Derscheid <paul.derscheid@lmscloud.de> --- Created attachment 193927 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=193927&action=edit Bug 41129: Clean up unused code and consolidate styles Code: - Remove uncalled SweepLineProcessor methods (~155 lines) - Remove unused constants, exports, and wrapper functions - Remove unused unavailableByDateRef from useAvailability - Simplify fixTargetEndDateAvailability wrapper in highlighting CSS: - Unify two :root blocks and two <style> blocks in BookingModal.vue - Centralise marker dot color variants in BookingModal.vue; remove duplicates from BookingPeriodStep and BookingTooltip - Replace hardcoded #28a745 with --booking-constraint-marker variable Tests: - Add test cases for modal state reset, zero-day circulation rule warning, and error handling (409 conflict, 500 server error) - Remove unused flatpickr helper commands and assertion functions Misc: - Scope _feedbackHideTimer per createOnDayCreate instance - Document catch blocks in staff-interface.js - Upgrade debug console.log to console.warn in list.tt Sponsored-by: Büchereizentrale Schleswig-Holstein <https://www.bz-sh.de/> -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #18 from Paul Derscheid <paul.derscheid@lmscloud.de> --- Created attachment 193929 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=193929&action=edit Bug 41129: Fix booking modal reopen race and add regression test Sponsored-by: Büchereizentrale Schleswig-Holstein <https://www.bz-sh.de/> -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 Paul Derscheid <paul.derscheid@lmscloud.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff Comma delimited| |Büchereizentrale list of Sponsors| |Schleswig-Holstein | |<https://www.bz-sh.de/> Sponsorship status|--- |Sponsored -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 Paul Derscheid <paul.derscheid@lmscloud.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #193926|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 Paul Derscheid <paul.derscheid@lmscloud.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #193928|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #19 from Paul Derscheid <paul.derscheid@lmscloud.de> --- Created attachment 195807 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=195807&action=edit Bug 41129: Introduce Vue booking modal and Bootstrap 5 modal wiring Replace the jQuery/TT-based place_booking modal with a Vue 3 SFC component mounted via the island architecture pattern. Core components: - Add BookingModal.vue (<script setup lang="ts">) with multi-step form (patron, details, period) - Add Pinia store (stores/bookings.js) for shared booking state - Add flatpickr calendar with constraint highlighting, marker dots, hover feedback bar, and availability computation - Use Bootstrap 5 Modal API (show/hide/dispose) for modal lifecycle - Add island.inc, button-place.inc, button-edit.inc entry points - Wire into cat-toolbar.inc, detail.tt, bookings/list.tt, and all catalogue detail views - Add KohaAlert.vue, dayjs.mjs, and apiErrors.js shared utilities - Add rspack.config.js entry for booking-modal-island bundle Module architecture: - Add API adapter layer (lib/adapters/api/) using fetch(), structured for future APIClient migration - Split calendar adapter into focused modules under lib/adapters/calendar/ (events, highlighting, locale, markers, prevention, visibility) - Add BookingDate class (immutable, timezone-aware date wrapper) - Add availability/ sub-modules (date-change, disabled-dates, period-validators, rules, unavailable-map) - Add interval tree and sweep-line processor for efficient overlap detection and unavailability computation - Add composables for availability, validation, capacity guard, constraint highlighting, rules fetching, flatpickr lifecycle, and form defaults - Add conflict-resolution, constraints, highlighting, markers modules - Add hover-feedback module for calendar feedback bar - Manage error state in Pinia store Behavior: - Add 3-way payload construction for "any item" bookings - Add editable property to timeline booking items - Show transient success message after booking save - Apply server timezone conversion for timeline dates API/embed compatibility: - Add GET /libraries/{id}/holidays endpoint (Libraries.pm, swagger) - Allow "library" embed on GET /patrons/{id} (patrons.yaml) Cypress coverage: - Test modal open/close lifecycle from biblio detail page - Test patron search typeahead and selection - Test item type and pickup location dropdowns - Test flatpickr date range selection and validation - Test booking creation via API with success/error paths - Test edit booking flow with pre-filled form state - Add Vue Select test helpers (vue-select.js) - Validate flatpickr state via instance API instead of DOM classes Technical notes: The booking adapters use fetch() directly but are structured to facilitate migration to the APIClient pattern. Full migration requires BiblioAPIClient, CheckoutAPIClient, CirculationRulesAPI- Client, and PatronAPIClient.search() enhancements — best done as a coordinated effort across the codebase. Test plan: STAFF INTERFACE - Creating Bookings Preparation: - Run yarn js:build to emit the bundles Automated tests: - yarn cypress run --spec 't/cypress/integration/Circulation/bookingsModal*.ts' 1. Navigate to a biblio detail page with bookable items 2. Click "Place booking" button 3. Observe the Vue booking modal opens Step 1 - Patron Selection: 4. Type patron name/cardnumber in search field 5. Verify typeahead search returns results 6. Select a patron 7. Verify patron is selected and displays correctly Step 2 - Booking Details: 8. Verify item type dropdown is populated 9. Select an item type 10. Verify pickup location dropdown is populated 11. Select a pickup location 12. If multiple bookable items exist, verify item dropdown appears 13. Select specific item (if applicable) Step 3 - Booking Period: 14. Verify flatpickr calendar opens 15. Verify calendar shows existing bookings, checkouts, and available dates 16. Select a date range 17. Verify date range validation works 18. Verify lead period/trail period restrictions are enforced 19. Verify maximum booking period is enforced 20. Verify constraint highlighting updates correctly when changing item type, specific item, or pickup location Validation Testing: 21. Try to submit without patron - verify error message 22. Try to submit without item type - verify error message 23. Try to submit without pickup location - verify error message 24. Try to submit without date range - verify error message 25. Try to select dates outside allowed range - verify validation Capacity Testing: 26. For item type with limited bookable items, verify capacity warnings 27. Verify zero-capacity items show appropriate message Submitting Bookings: 28. Complete all required fields 29. Click "Place booking" 30. Verify booking is created successfully 31. Verify modal closes 32. Verify booking appears in bookings list STAFF INTERFACE - Editing Bookings 33. Click "Edit" on an existing booking 34. Verify modal opens with pre-filled data (patron, item type, pickup location, date range) 35. Modify the date range 36. Click "Update booking" 37. Verify changes are saved 38. Verify updated booking reflects changes Calendar Navigation: 39. In date picker, navigate between months 40. Verify highlighting persists across navigation 41. Verify availability data loads correctly for new months Error Handling: 42. Disconnect network 43. Try to create a booking 44. Verify appropriate error message displays 45. Restore network and verify subsequent booking attempts work Store State Management: 46. Open booking modal and fill in some fields but don't submit 47. Close modal and re-open it 48. Verify modal state is reset (no data persists inappropriately) Build Process: 49. Run yarn js:build and yarn js:build:prod 50. Verify build completes without errors 51. Verify dist files are generated correctly Sponsored-by: Büchereizentrale Schleswig-Holstein <https://www.bz-sh.de/> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #21 from Paul Derscheid <paul.derscheid@lmscloud.de> --- Created attachment 195809 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=195809&action=edit Bug 41129: Clean up unused code and consolidate styles Code: - Remove uncalled SweepLineProcessor methods (~155 lines) - Remove unused constants, exports, and wrapper functions - Remove unused unavailableByDateRef from useAvailability - Simplify fixTargetEndDateAvailability wrapper in highlighting CSS: - Unify two :root blocks and two <style> blocks in BookingModal.vue - Centralise marker dot color variants in BookingModal.vue; remove duplicates from BookingPeriodStep and BookingTooltip - Replace hardcoded #28a745 with --booking-constraint-marker variable Tests: - Add test cases for modal state reset, zero-day circulation rule warning, and error handling (409 conflict, 500 server error) - Remove unused flatpickr helper commands and assertion functions Misc: - Scope _feedbackHideTimer per createOnDayCreate instance - Document catch blocks in staff-interface.js - Upgrade debug console.log to console.warn in list.tt Sponsored-by: Büchereizentrale Schleswig-Holstein <https://www.bz-sh.de/> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #23 from Paul Derscheid <paul.derscheid@lmscloud.de> --- Created attachment 195811 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=195811&action=edit Bug 41129: Fix booking modal reopen race and add regression test Sponsored-by: Büchereizentrale Schleswig-Holstein <https://www.bz-sh.de/> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 Paul Derscheid <paul.derscheid@lmscloud.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Comma delimited|Büchereizentrale |Büchereizentrale list of Sponsors|Schleswig-Holstein |Schleswig-Holstein |<https://www.bz-sh.de/> |<https://www.bz-sh.de/>, | |Büchereizentrale | |Schleswig-Holstein | |<https://www.bz-sh.de/> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 Paul Derscheid <paul.derscheid@lmscloud.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #195808|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 Paul Derscheid <paul.derscheid@lmscloud.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #195810|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #24 from Paul Derscheid <paul.derscheid@lmscloud.de> --- Created attachment 196436 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=196436&action=edit Bug 41129: Introduce Vue booking modal and Bootstrap 5 modal wiring Replace the jQuery/TT-based place_booking modal with a Vue 3 SFC component mounted via the island architecture pattern. Core components: - Add BookingModal.vue (<script setup lang="ts">) with multi-step form (patron, details, period) - Add Pinia store (stores/bookings.js) for shared booking state - Add flatpickr calendar with constraint highlighting, marker dots, hover feedback bar, and availability computation - Use Bootstrap 5 Modal API (show/hide/dispose) for modal lifecycle - Add island.inc, button-place.inc, button-edit.inc entry points - Wire into cat-toolbar.inc, detail.tt, bookings/list.tt, and all catalogue detail views - Add KohaAlert.vue, dayjs.mjs, and apiErrors.js shared utilities - Add rspack.config.js entry for booking-modal-island bundle Module architecture: - Add API adapter layer (lib/adapters/api/) using fetch(), structured for future APIClient migration - Split calendar adapter into focused modules under lib/adapters/calendar/ (events, highlighting, locale, markers, prevention, visibility) - Add BookingDate class (immutable, timezone-aware date wrapper) - Add availability/ sub-modules (date-change, disabled-dates, period-validators, rules, unavailable-map) - Add interval tree and sweep-line processor for efficient overlap detection and unavailability computation - Add composables for availability, validation, capacity guard, constraint highlighting, rules fetching, flatpickr lifecycle, and form defaults - Add conflict-resolution, constraints, highlighting, markers modules - Add hover-feedback module for calendar feedback bar - Manage error state in Pinia store Behavior: - Add 3-way payload construction for "any item" bookings - Add editable property to timeline booking items - Show transient success message after booking save - Apply server timezone conversion for timeline dates API/embed compatibility: - Add GET /libraries/{id}/holidays endpoint (Libraries.pm, swagger) - Allow "library" embed on GET /patrons/{id} (patrons.yaml) Cypress coverage: - Test modal open/close lifecycle from biblio detail page - Test patron search typeahead and selection - Test item type and pickup location dropdowns - Test flatpickr date range selection and validation - Test booking creation via API with success/error paths - Test edit booking flow with pre-filled form state - Add Vue Select test helpers (vue-select.js) - Validate flatpickr state via instance API instead of DOM classes Technical notes: The booking adapters use fetch() directly but are structured to facilitate migration to the APIClient pattern. Full migration requires BiblioAPIClient, CheckoutAPIClient, CirculationRulesAPI- Client, and PatronAPIClient.search() enhancements — best done as a coordinated effort across the codebase. Test plan: STAFF INTERFACE - Creating Bookings Preparation: - Run yarn js:build to emit the bundles Automated tests: - yarn cypress run --spec 't/cypress/integration/Circulation/bookingsModal*.ts' 1. Navigate to a biblio detail page with bookable items 2. Click "Place booking" button 3. Observe the Vue booking modal opens Step 1 - Patron Selection: 4. Type patron name/cardnumber in search field 5. Verify typeahead search returns results 6. Select a patron 7. Verify patron is selected and displays correctly Step 2 - Booking Details: 8. Verify item type dropdown is populated 9. Select an item type 10. Verify pickup location dropdown is populated 11. Select a pickup location 12. If multiple bookable items exist, verify item dropdown appears 13. Select specific item (if applicable) Step 3 - Booking Period: 14. Verify flatpickr calendar opens 15. Verify calendar shows existing bookings, checkouts, and available dates 16. Select a date range 17. Verify date range validation works 18. Verify lead period/trail period restrictions are enforced 19. Verify maximum booking period is enforced 20. Verify constraint highlighting updates correctly when changing item type, specific item, or pickup location Validation Testing: 21. Try to submit without patron - verify error message 22. Try to submit without item type - verify error message 23. Try to submit without pickup location - verify error message 24. Try to submit without date range - verify error message 25. Try to select dates outside allowed range - verify validation Capacity Testing: 26. For item type with limited bookable items, verify capacity warnings 27. Verify zero-capacity items show appropriate message Submitting Bookings: 28. Complete all required fields 29. Click "Place booking" 30. Verify booking is created successfully 31. Verify modal closes 32. Verify booking appears in bookings list STAFF INTERFACE - Editing Bookings 33. Click "Edit" on an existing booking 34. Verify modal opens with pre-filled data (patron, item type, pickup location, date range) 35. Modify the date range 36. Click "Update booking" 37. Verify changes are saved 38. Verify updated booking reflects changes Calendar Navigation: 39. In date picker, navigate between months 40. Verify highlighting persists across navigation 41. Verify availability data loads correctly for new months Error Handling: 42. Disconnect network 43. Try to create a booking 44. Verify appropriate error message displays 45. Restore network and verify subsequent booking attempts work Store State Management: 46. Open booking modal and fill in some fields but don't submit 47. Close modal and re-open it 48. Verify modal state is reset (no data persists inappropriately) Build Process: 49. Run yarn js:build and yarn js:build:prod 50. Verify build completes without errors 51. Verify dist files are generated correctly Sponsored-by: Büchereizentrale Schleswig-Holstein <https://www.bz-sh.de/> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #26 from Paul Derscheid <paul.derscheid@lmscloud.de> --- Created attachment 196438 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=196438&action=edit Bug 41129: Clean up unused code and consolidate styles Code: - Remove uncalled SweepLineProcessor methods (~155 lines) - Remove unused constants, exports, and wrapper functions - Remove unused unavailableByDateRef from useAvailability - Simplify fixTargetEndDateAvailability wrapper in highlighting CSS: - Unify two :root blocks and two <style> blocks in BookingModal.vue - Centralise marker dot color variants in BookingModal.vue; remove duplicates from BookingPeriodStep and BookingTooltip - Replace hardcoded #28a745 with --booking-constraint-marker variable Tests: - Add test cases for modal state reset, zero-day circulation rule warning, and error handling (409 conflict, 500 server error) - Remove unused flatpickr helper commands and assertion functions Misc: - Scope _feedbackHideTimer per createOnDayCreate instance - Document catch blocks in staff-interface.js - Upgrade debug console.log to console.warn in list.tt Sponsored-by: Büchereizentrale Schleswig-Holstein <https://www.bz-sh.de/> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #28 from Paul Derscheid <paul.derscheid@lmscloud.de> --- Created attachment 196440 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=196440&action=edit Bug 41129: Fix booking modal reopen race and add regression test Sponsored-by: Büchereizentrale Schleswig-Holstein <https://www.bz-sh.de/> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 Paul Derscheid <paul.derscheid@lmscloud.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #196436|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 Paul Derscheid <paul.derscheid@lmscloud.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #196438|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 Paul Derscheid <paul.derscheid@lmscloud.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #196440|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #31 from Paul Derscheid <paul.derscheid@lmscloud.de> --- Created attachment 196450 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=196450&action=edit Bug 41129: Harden runtime paths and test reliability Runtime: - Add missing use C4::Context in Libraries.pm - Use safe DOM construction instead of innerHTML in external-dependents.mjs (booking count update, success message) - Guard against undefined BookingDateRangeConstraint syspref - Stub OPAC fetchHolidays (public endpoint does not exist) - Pass UI visibility flags to useBookingValidation as reactive parameter instead of reading from store - Simplify island.inc template and add missing TT output filters - Remove unused ItemType type definition from bookings.d.ts Tests: - Correct fencepost in max booking period calculation - Guard DatePicker spec against flatpickr navigation races - Validate via flatpickr instance state instead of DOM classes - Remove broad error suppression from e2e.js support file Sponsored-by: Büchereizentrale Schleswig-Holstein <https://www.bz-sh.de/> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #33 from Paul Derscheid <paul.derscheid@lmscloud.de> --- Created attachment 196452 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=196452&action=edit Bug 41129: Allow calendar readiness in "any item" default state - Remove item type and item ID requirement from formPrefilterValid and canProceedToStep3; the "any item" default is valid when bookable items exist - Remove unused store refs and UI flags from useBookingValidation Sponsored-by: Büchereizentrale Schleswig-Holstein <https://www.bz-sh.de/> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #35 from Paul Derscheid <paul.derscheid@lmscloud.de> --- Created attachment 196454 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=196454&action=edit Bug 41129: (follow-up) Migrate staff booking adapter to API clients - Create BiblioAPIClient for biblio sub-resource endpoints (items, bookings, checkouts, pickup_locations) - Create BookingAPIClient for bookings CRUD (create, update) - Create LibraryAPIClient for library holidays endpoint - Create CirculationRulesAPIClient for circulation rules endpoint - Extend PatronAPIClient with headers support on get and new search method for paginated patron lookup - Register all new clients in both API client registries (lazy-loaded main and eager Vue) - Rewrite staff-interface.js adapter to use API clients instead of raw fetch() calls - Keep domain validation (bookingValidation) in adapter layer - OPAC adapter left as-is (uses public endpoints, mostly stubs) Sponsored-by: Büchereizentrale Schleswig-Holstein <https://www.bz-sh.de/> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #37 from Paul Derscheid <paul.derscheid@lmscloud.de> --- Created attachment 196456 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=196456&action=edit Bug 41129: (follow-up) Use insertSamplePatron task in Cypress tests - Replace raw SQL INSERT INTO borrowers with the standard insertSamplePatron Cypress task across all 3 booking specs - Remove manual DELETE FROM borrowers cleanup blocks - Let deleteSampleObjects handle patron cleanup in correct dependency order (patrons before libraries), fixing the FK constraint error on test teardown - Inline patron creation in long tests also migrated to insertSamplePatron with deleteSampleObjects cleanup Sponsored-by: Büchereizentrale Schleswig-Holstein <https://www.bz-sh.de/> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #39 from Paul Derscheid <paul.derscheid@lmscloud.de> --- Created attachment 196458 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=196458&action=edit Bug 41129: (follow-up) Disable OPAC island build output - Set OPAC islands rspack entry to empty object to prevent bundle emission while preserving the config for future use - OPAC booking UI is not part of this patchset Sponsored-by: Büchereizentrale Schleswig-Holstein <https://www.bz-sh.de/> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #41 from Paul Derscheid <paul.derscheid@lmscloud.de> --- Created attachment 196947 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=196947&action=edit Bug 41129: (follow-up) Fix Cypress test cleanup and race conditions - Add booking type to deleteSampleObjects with correct deletion order to prevent FK constraint failures - Track bookings and extra patrons in testData so afterEach cleanup handles them even when tests fail early - Wait for patron API response before asserting vue-select pre-population to fix timing-dependent failures - Remove manual in-test cleanup that was unreachable on failure Sponsored-by: Büchereizentrale Schleswig-Holstein <https://www.bz-sh.de/> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 Paul Derscheid <paul.derscheid@lmscloud.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #196450|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 Paul Derscheid <paul.derscheid@lmscloud.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #196452|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 Paul Derscheid <paul.derscheid@lmscloud.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #196454|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 Paul Derscheid <paul.derscheid@lmscloud.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #196456|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 Paul Derscheid <paul.derscheid@lmscloud.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #196458|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 Paul Derscheid <paul.derscheid@lmscloud.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #196947|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #43 from Paul Derscheid <paul.derscheid@lmscloud.de> --- Created attachment 199039 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=199039&action=edit Bug 41129: Add booking domain library and shared utilities - Add lib/booking/BookingDate.mjs (immutable, timezone-aware date wrapper) - Add lib/booking/availability/ (date-change, disabled-dates, index, period-validators, rules, unavailable-map) and the outer lib/booking/availability.mjs barrel - Add lib/booking/algorithms/ (interval-tree, sweep-line-processor) for efficient overlap detection and unavailability computation - Add lib/booking/ logic modules (conflict-resolution, constants, constraints, id-utils, logger, markers, strategies, validation, validation-messages) - Add lib/adapters/api/staff-interface.js and api/opac.js for the per-environment API surface, plus lib/adapters/{form, patron, globals, external-dependents}.mjs for the DOM/legacy shims - Add lib/ui/ helpers (hover-feedback, marker-labels, selection-message, steps) - Add types/ (bookings.d.ts, dayjs-plugins.d.ts, vue-shims.d.ts) and a Bookings-scoped tsconfig.json with allowJs / skipLibCheck and the @koha-vue path alias so vue-tsc reads JSDoc on the .mjs files directly - Add vue/utils/apiErrors.js, validationErrors.js, dayjs.mjs and functions.mjs (shared across the module) - Add Cypress component specs under t/cypress/component/lib/booking/ for createDisableFunction, findFirstBlockingDate, IntervalTree and buildIntervalTree, buildUnavailableByDateMap with addHolidayMarkers, the period validators (calculateMaxEndDate, lead/trail/range-overlap, getAvailableItemsForPeriod), and the rules helpers (calculateMaxBookingPeriod, deriveEffectiveRules, extractBookingConfiguration); pure-function tests with no Vue mount so an algorithmic regression points here directly instead of cascading through the picker DOM Sponsored-by: Büchereizentrale Schleswig-Holstein <https://www.bz-sh.de/> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #45 from Paul Derscheid <paul.derscheid@lmscloud.de> --- Created attachment 199041 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=199041&action=edit Bug 41129: Add Vue booking modal, Pinia store, and island wiring - Add BookingModal.vue (<script setup lang="ts">) orchestrating the multi-step form, plus the step components (BookingPatronStep, BookingDetailsStep, BookingPeriodStep) and BookingTooltip - Add PatronSearchSelect.vue and KohaAlert.vue shared components - Add the setup-style Pinia store under stores/bookings/, sectioned by responsibility along a one-way dep chain (status → data → draft → availability → validation) with an effects section for cross-cutting reactive bridges and an utils/withErrorHandling HOF wrapping per-action loading/error - Add modals/booking/island.inc and the two trigger snippets (button-place.inc, button-edit.inc); wire them into cat-toolbar.inc, bookings/list.tt, and the six catalogue detail views; drop the legacy modals/place_booking.inc - Pass js-date-format.inc through unchanged for the new modal - Add the booking-modal-island entry to rspack.config.js - Add ambient.d.ts for *.css / bootstrap / vue-select shims and configure tsconfig.json: @koha-vue alias, allowJs for JSDoc-typed .mjs flow into TS SFCs, narrow per-feature include scoped to the booking subtree, plus target/lib/module/noEmit/types: [] so tsc runs clean - Hydrate the island via vue/modules/islands.ts, providing the Pinia store under "bookings" for inject() consumption - Add Cypress integration coverage in bookingsModalBasic / bookingsModalDatePicker / bookingsModalTimezone, the BookingPeriodStep component spec, plus support/e2e and support/flatpickr adjustments and the insertData.js helpers Test plan: STAFF INTERFACE - Creating Bookings Preparation: - Run yarn js:build to emit the bundles Automated tests: - yarn cypress run --spec 't/cypress/integration/Circulation/bookingsModal*.ts' - yarn cypress run --component --spec 't/cypress/component/**/*_spec.ts' (covers BaseFlatpickr, useBookingCalendarMaps, BookingPeriodStep and the lib/booking pure-function specs) Test environment setup (preconditions for the manual steps below; cypress sets up its own data and does not need these): Bookable inventory: - Administration > Item types: mark at least one item type as "Bookable" (or use the per-item "Bookable" toggle on the item edit page) - Have a biblio with multiple bookable copies of that item type so the multi-item / item-selection / capacity flows are exercisable Circulation rules (Administration > Circulation and fines rules, applied to the library + item type scope you will test against or to the wildcard row): - bookings_lead_period: non-zero (e.g. 2) so steps 18 and 25 have a lead-time constraint to validate and the hover feedback emits lead-specific messages - bookings_trail_period: non-zero (e.g. 2) so step 18 has a trail buffer to validate - issuelength: a finite value (e.g. 7) so a maxPeriod is derivable and step 19 has a max-booking-period to enforce in the "issuelength" constraint mode - renewalsallowed + renewalperiod: non-zero values to exercise the "issuelength_with_renewals" constraint mode Calendar / holidays: - Tools > Calendar: add at least one closed day inside the visible calendar range for the pickup library so step 15 has a holiday to render (and the hover-feedback "library closed" path runs) Existing data on the test biblio: - Place one booking ahead of time so step 15 has a conflict to render, step 18 exercises lead/trail markers next to a real booking, and the edit flow (steps 33-38) has something to edit - Check out a bookable copy with a due date inside the visible calendar range so the picker shows a checkout interval and the checkout-vs-booking lead handling (Koha::Item->find_booking) participates 1. Navigate to a biblio detail page with bookable items 2. Click "Place booking" button 3. Observe the Vue booking modal opens Step 1 - Patron Selection: 4. Type patron name/cardnumber in search field 5. Verify typeahead search returns results 6. Select a patron 7. Verify patron is selected and displays correctly Step 2 - Booking Details: 8. Verify item type dropdown is populated 9. Select an item type 10. Verify pickup location dropdown is populated 11. Select a pickup location 12. If multiple bookable items exist, verify item dropdown appears 13. Select specific item (if applicable) Step 3 - Booking Period: 14. Verify flatpickr calendar opens 15. Verify calendar shows existing bookings, checkouts, and available dates 16. Select a date range 17. Verify date range validation works 18. Verify lead period/trail period restrictions are enforced 19. Verify maximum booking period is enforced 20. Verify constraint highlighting updates correctly when changing item type, specific item, or pickup location Validation Testing: 21. Try to submit without patron - verify error message 22. Try to submit without item type - verify error message 23. Try to submit without pickup location - verify error message 24. Try to submit without date range - verify error message 25. Try to select dates outside allowed range - verify validation Capacity Testing: 26. For item type with limited bookable items, verify capacity warnings 27. Verify zero-capacity items show appropriate message Submitting Bookings: 28. Complete all required fields 29. Click "Place booking" 30. Verify booking is created successfully 31. Verify modal closes 32. Verify booking appears in bookings list STAFF INTERFACE - Editing Bookings 33. Click "Edit" on an existing booking 34. Verify modal opens with pre-filled data (patron, item type, pickup location, date range) 35. Modify the date range 36. Click "Update booking" 37. Verify changes are saved 38. Verify updated booking reflects changes Calendar Navigation: 39. In date picker, navigate between months 40. Verify highlighting persists across navigation 41. Verify availability data loads correctly for new months Error Handling: 42. Disconnect network 43. Try to create a booking 44. Verify appropriate error message displays 45. Restore network and verify subsequent booking attempts work Store State Management: 46. Open booking modal and fill in some fields but don't submit 47. Close modal and re-open it 48. Verify modal state is reset (no data persists inappropriately) Build Process: 49. Run yarn js:build and yarn js:build:prod 50. Verify build completes without errors 51. Verify dist files are generated correctly Sponsored-by: Büchereizentrale Schleswig-Holstein <https://www.bz-sh.de/> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #47 from Paul Derscheid <paul.derscheid@lmscloud.de> --- Created attachment 199423 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=199423&action=edit Bug 41129: (follow-up) Fall back to setTimeout when requestIdleCallback is missing - Vue island hydration called window.requestIdleCallback directly, which throws TypeError on iOS Safari < 17.4 and prevents the booking-modal-island custom element from being defined; trigger buttons then silently set a property on a plain HTMLElement - Add a module-level scheduleIdle shim that uses requestIdleCallback when available and falls back to setTimeout(cb, 1) otherwise; the hydrate callback never reads the idle deadline, so the fallback is behaviour-equivalent on engines lacking native support Sponsored-by: Büchereizentrale Schleswig-Holstein <https://www.bz-sh.de/> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 Paul Derscheid <paul.derscheid@lmscloud.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #199039|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 Paul Derscheid <paul.derscheid@lmscloud.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #199041|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 Paul Derscheid <paul.derscheid@lmscloud.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #199423|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #49 from Paul Derscheid <paul.derscheid@lmscloud.de> --- Created attachment 200581 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=200581&action=edit Bug 41129: Add booking domain library and shared utilities - Add lib/booking/BookingDate.js (immutable, timezone-aware date wrapper) - Add lib/booking/availability/ (date-change, disabled-dates, period-validators, rules, unavailable-map) and the outer lib/booking/availability.js barrel - Add lib/booking/algorithms/ (interval-tree, sweep-line-processor) for efficient overlap detection and unavailability computation - Add lib/booking/ logic modules (conflict-resolution, constants, constraints, id-utils, markers, strategies, validation, validation-messages) - Add lib/adapters/api/staff-interface.js and api/opac.js for the per-environment API surface, plus lib/adapters/{form, patron, globals, external-dependents}.js for the DOM/legacy shims - Add lib/ui/ helpers (hover-feedback, marker-labels, selection-message, steps) - Add types/ (bookings.d.ts, dayjs-plugins.d.ts, vue-shims.d.ts) and a Bookings-scoped tsconfig.json with allowJs / skipLibCheck and the @koha-vue path alias so vue-tsc reads JSDoc on the .js files directly - Add vue/utils/apiErrors.js, validationErrors.js, dayjs.js and functions.js (shared across the module) - Add Cypress component specs under t/cypress/component/lib/booking/ for createDisableFunction, findFirstBlockingDate, IntervalTree and buildIntervalTree, buildUnavailableByDateMap with addHolidayMarkers, the period validators (calculateMaxEndDate, lead/trail/range-overlap, getAvailableItemsForPeriod), and the rules helpers (calculateMaxBookingPeriod, deriveEffectiveRules, extractBookingConfiguration); pure-function tests with no Vue mount so an algorithmic regression points here directly instead of cascading through the picker DOM Test plan: 1. Run the pure-function component suites covering this library: npx cypress run --component --browser electron \ --spec 't/cypress/component/lib/booking/**/*_spec.ts' 2. Nothing is wired into a page yet; the booking modal patch later in this series exercises the library end-to-end (see its test plan) Sponsored-by: Büchereizentrale Schleswig-Holstein <https://www.bz-sh.de/> Assisted-by: Opus 4.8 (Anthropic) -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #51 from Paul Derscheid <paul.derscheid@lmscloud.de> --- Created attachment 200583 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=200583&action=edit Bug 41129: Add Vue booking modal, Pinia store, and island wiring - Add BookingModal.vue (<script setup lang="ts">) orchestrating the multi-step form, plus the step components (BookingPatronStep, BookingDetailsStep, BookingPeriodStep) and BookingTooltip - Add PatronSearchSelect.vue and KohaAlert.vue shared components - Add the setup-style Pinia store under stores/bookings/, sectioned by responsibility along a one-way dep chain (status → data → draft → availability → validation) with an effects section for cross-cutting reactive bridges and an utils/withErrorHandling HOF wrapping per-action loading - Add modals/booking/island.inc and the button-place.inc trigger snippet and wire them into cat-toolbar.inc; point the bookings/list.tt edit buttons and timeline at the island (data-booking-modal / window.openBookingModal); drop the legacy modals/place_booking.inc, js/modals/place_booking.js, and the asset include lines in the six catalogue detail views - Pass js-date-format.inc through unchanged for the new modal - Add the booking-modal-island entry to rspack.config.js - Add ambient.d.ts for *.css / bootstrap / vue-select shims and configure tsconfig.json: @koha-vue alias, allowJs for JSDoc-typed .js flow into TS SFCs, narrow per-feature include scoped to the booking subtree, plus target/lib/module/noEmit/types: [] so tsc runs clean - Hydrate the island via vue/modules/islands.ts, providing the Pinia store under "bookingStore" for inject() consumption - Add Cypress integration coverage in bookingsModalBasic / bookingsModalDatePicker / bookingsModalTimezone, the BookingPeriodStep and bookings-store effects component specs, plus support/e2e and support/flatpickr adjustments and the insertData.js helpers Test plan: STAFF INTERFACE - Creating Bookings Preparation: - Run yarn js:build to emit the bundles Automated tests: - yarn cypress run --spec 't/cypress/integration/Circulation/bookingsModal*.ts' - yarn cypress run --component --spec 't/cypress/component/**/*_spec.ts' (covers BookingFlatpickr, useBookingCalendarMaps, BookingPeriodStep and the lib/booking pure-function specs) Test environment setup (preconditions for the manual steps below; cypress sets up its own data and does not need these): Bookable inventory: - Administration > Item types: mark at least one item type as "Bookable" (or use the per-item "Bookable" toggle on the item edit page) - Have a biblio with multiple bookable copies of that item type so the multi-item / item-selection / capacity flows are exercisable Circulation rules (Administration > Circulation and fines rules, applied to the library + item type scope you will test against or to the wildcard row): - bookings_lead_period: non-zero (e.g. 2) so steps 18 and 25 have a lead-time constraint to validate and the hover feedback emits lead-specific messages - bookings_trail_period: non-zero (e.g. 2) so step 18 has a trail buffer to validate - issuelength: a finite value (e.g. 7) so a maxPeriod is derivable and step 19 has a max-booking-period to enforce in the "issuelength" constraint mode - renewalsallowed + renewalperiod: non-zero values to exercise the "issuelength_with_renewals" constraint mode Calendar / holidays: - Tools > Calendar: add at least one closed day inside the visible calendar range for the pickup library so step 15 has a holiday to render (and the hover-feedback "library closed" path runs) Existing data on the test biblio: - Place one booking ahead of time so step 15 has a conflict to render, step 18 exercises lead/trail markers next to a real booking, and the edit flow (steps 33-38) has something to edit - Check out a bookable copy with a due date inside the visible calendar range so the picker shows a checkout interval and the checkout-vs-booking lead handling (Koha::Item->find_booking) participates 1. Navigate to a biblio detail page with bookable items 2. Click "Place booking" button 3. Observe the Vue booking modal opens Step 1 - Patron Selection: 4. Type patron name/cardnumber in search field 5. Verify typeahead search returns results 6. Select a patron 7. Verify patron is selected and displays correctly Step 2 - Booking Details: 8. Verify item type dropdown is populated 9. Select an item type 10. Verify pickup location dropdown is populated 11. Select a pickup location 12. If multiple bookable items exist, verify item dropdown appears 13. Select specific item (if applicable) Step 3 - Booking Period: 14. Verify flatpickr calendar opens 15. Verify calendar shows existing bookings, checkouts, and available dates 16. Select a date range 17. Verify date range validation works 18. Verify lead period/trail period restrictions are enforced 19. Verify maximum booking period is enforced 20. Verify constraint highlighting updates correctly when changing item type, specific item, or pickup location Validation Testing: 21. Try to submit without patron - verify error message 22. Try to submit without item type - verify error message 23. Try to submit without pickup location - verify error message 24. Try to submit without date range - verify error message 25. Try to select dates outside allowed range - verify validation Capacity Testing: 26. For item type with limited bookable items, verify capacity warnings 27. Verify zero-capacity items show appropriate message Submitting Bookings: 28. Complete all required fields 29. Click "Place booking" 30. Verify booking is created successfully 31. Verify modal closes 32. Verify booking appears in bookings list STAFF INTERFACE - Editing Bookings 33. Click "Edit" on an existing booking 34. Verify modal opens with pre-filled data (patron, item type, pickup location, date range) 35. Modify the date range 36. Click "Update booking" 37. Verify changes are saved 38. Verify updated booking reflects changes Calendar Navigation: 39. In date picker, navigate between months 40. Verify highlighting persists across navigation 41. Verify availability data loads correctly for new months Error Handling: 42. Disconnect network 43. Try to create a booking 44. Verify appropriate error message displays 45. Restore network and verify subsequent booking attempts work Store State Management: 46. Open booking modal and fill in some fields but don't submit 47. Close modal and re-open it 48. Verify modal state is reset (no data persists inappropriately) Build Process: 49. Run yarn js:build and yarn js:build:prod 50. Verify build completes without errors 51. Verify dist files are generated correctly AI/LLM usage: The architecture and design here are my own, refined by hand over a long stretch that began with a manual refactor of place_booking.js. I used AI/LLM assistants throughout to help port the original plain JavaScript to Vue, write boilerplate, and draft and refactor parts of the implementation and tests, and I reviewed, corrected, and integrated everything they produced. This spanned the past year and a succession of models: Anthropic's Opus 4.5 through 4.8 and Fable 5, and OpenAI's GPT-5 to 5.2; the trailer records Opus 4.8, used most recently. Sponsored-by: Büchereizentrale Schleswig-Holstein <https://www.bz-sh.de/> Assisted-by: Opus 4.8 (Anthropic) -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #53 from Paul Derscheid <paul.derscheid@lmscloud.de> --- Created attachment 200585 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=200585&action=edit Bug 41129: [DO NOT PUSH] Add dev SQL seed for manual test setup - Marks itemtype BK bookable - Sets bookings_lead_period (2), bookings_trail_period (2), issuelength (7), renewalsallowed (2), renewalperiod (7) at the wildcard circulation_rules scope - Adds a closed day at CPL ten days out for holiday rendering - Idempotent via ON DUPLICATE KEY UPDATE / NOT EXISTS guards; run with: koha-mysql kohadev < bug_41129_seed.sql -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #54 from Paul Derscheid <paul.derscheid@lmscloud.de> --- Hi all, I further stripped things out of the patch set that are currently not needed for the scope of this bug to reduce the review surface. The Bookings dir within the Vue directory is still a bit of code to read but nowhere near what it was a few rounds ago. The store methods should be also relatively easy to follow since I grouped and order them a while back. I am aware that some of this is breaking/extending existing Koha conventions, but just throwing everything into a single giant wall of JS makes maintaining this code a nightmare. That's why I split it aggressively. Also added LLM-use notes which got a little convoluted since so many models were used over time, opted for the model in the trailer that last touched the commits and mentioned the rest in the big feature commit in comment #51. As stated, the architecture is my design, so please feel free to tell me where you think it is bad, or where we should align with existing Koha/Vue-patterns more. It's not any LLMs fault :D -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 Bug 41129 depends on bug 42206, which changed state. Bug 42206 Summary: Add REST endpoint GET /libraries/{library_id}/closed_dates https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42206 What |Removed |Added ---------------------------------------------------------------------------- Status|Needs documenting |RESOLVED Resolution|--- |FIXED -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 Paul Derscheid <paul.derscheid@lmscloud.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #200580|0 |1 is obsolete| | Attachment #200581|0 |1 is obsolete| | Attachment #200582|0 |1 is obsolete| | Attachment #200583|0 |1 is obsolete| | Attachment #200584|0 |1 is obsolete| | Attachment #200585|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #56 from Paul Derscheid <paul.derscheid@lmscloud.de> --- Created attachment 201573 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=201573&action=edit Bug 41129: Add API clients for booking-related endpoints - Add BiblioAPIClient for biblio sub-resources (items, bookings, checkouts, pickup_locations) - Add BookingAPIClient for bookings CRUD (create, update) - Add LibraryAPIClient for /libraries/{id}/closed_dates - Add CirculationRulesAPIClient for circulation rules lookup - Extend PatronAPIClient with a search() method for paginated patron lookup and accept optional headers on get() - Register the new clients in fetch/api-client.js (lazy) and vue/fetch/api-client.js (eager) Test plan: 1. Apply the full series and run: yarn js:build 2. The clients carry no UI of their own; they are exercised end-to-end by the Cypress suites added with the booking modal patch later in this series: npx cypress run --browser electron \ --spec 't/cypress/integration/Circulation/bookingsModalBasic_spec.ts' 3. Regression-check an existing consumer of the shared api-client registries, e.g. open the ERM module and confirm its tables still load Sponsored-by: Büchereizentrale Schleswig-Holstein <https://www.bz-sh.de/> Assisted-by: Opus 4.8 (Anthropic) -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #58 from Paul Derscheid <paul.derscheid@lmscloud.de> --- Created attachment 201575 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=201575&action=edit Bug 41129: Add Vue booking modal, Pinia store, and island wiring - Add BookingModal.vue (<script setup lang="ts">) orchestrating the multi-step form, plus the step components (BookingPatronStep, BookingDetailsStep, BookingPeriodStep) and BookingTooltip - Add PatronSearchSelect.vue and KohaAlert.vue shared components - Add the setup-style Pinia store under stores/bookings/, sectioned by responsibility along a one-way dep chain (status → data → draft → availability → validation) with an effects section for cross-cutting reactive bridges and an utils/withErrorHandling HOF wrapping per-action loading - Add modals/booking/island.inc and the button-place.inc trigger snippet and wire them into cat-toolbar.inc; point the bookings/list.tt edit buttons and timeline at the island (data-booking-modal / window.openBookingModal); drop the legacy modals/place_booking.inc, js/modals/place_booking.js, its calendar styles in staff-global.scss, and the asset include lines in the six catalogue detail views - Pass js-date-format.inc through unchanged for the new modal - Add the booking-modal-island entry to rspack.config.js - Add ambient.d.ts for *.css / bootstrap / vue-select shims and configure tsconfig.json: @koha-vue alias, allowJs for JSDoc-typed .js flow into TS SFCs, narrow per-feature include scoped to the booking subtree, plus target/lib/module/noEmit/types: [] so tsc runs clean - Hydrate the island via vue/modules/islands.ts, providing the Pinia store under "bookingStore" for inject() consumption - Add Cypress integration coverage in bookingsModalBasic / bookingsModalDatePicker / bookingsModalTimezone, the BookingPeriodStep and bookings-store effects and validation component specs, plus support/e2e and support/flatpickr adjustments and the insertData.js helpers Test plan: STAFF INTERFACE - Creating Bookings Preparation: - Run yarn js:build to emit the bundles Automated tests: - yarn cypress run --spec 't/cypress/integration/Circulation/bookingsModal*.ts' - yarn cypress run --component --spec 't/cypress/component/**/*_spec.ts' (covers BookingFlatpickr, useBookingCalendarMaps, BookingPeriodStep and the lib/booking pure-function specs) Test environment setup (preconditions for the manual steps below; cypress sets up its own data and does not need these): Bookable inventory: - Administration > Item types: mark at least one item type as "Bookable" (or use the per-item "Bookable" toggle on the item edit page) - Have a biblio with multiple bookable copies of that item type so the multi-item / item-selection / capacity flows are exercisable Circulation rules (Administration > Circulation and fines rules, applied to the library + item type scope you will test against or to the wildcard row): - bookings_lead_period: non-zero (e.g. 2) so steps 18 and 25 have a lead-time constraint to validate and the hover feedback emits lead-specific messages - bookings_trail_period: non-zero (e.g. 2) so step 18 has a trail buffer to validate - issuelength: a finite value (e.g. 7) so a maxPeriod is derivable and step 19 has a max-booking-period to enforce in the "issuelength" constraint mode - renewalsallowed + renewalperiod: non-zero values to exercise the "issuelength_with_renewals" constraint mode Calendar / holidays: - Tools > Calendar: add at least one closed day inside the visible calendar range for the pickup library so step 15 has a holiday to render (and the hover-feedback "library closed" path runs) Existing data on the test biblio: - Place one booking ahead of time so step 15 has a conflict to render, step 18 exercises lead/trail markers next to a real booking, and the edit flow (steps 33-38) has something to edit - Check out a bookable copy with a due date inside the visible calendar range so the picker shows a checkout interval and the checkout-vs-booking lead handling (Koha::Item->find_booking) participates 1. Navigate to a biblio detail page with bookable items 2. Click "Place booking" button 3. Observe the Vue booking modal opens Step 1 - Patron Selection: 4. Type patron name/cardnumber in search field 5. Verify typeahead search returns results 6. Select a patron 7. Verify patron is selected and displays correctly Step 2 - Booking Details: 8. Verify item type dropdown is populated 9. Select an item type 10. Verify pickup location dropdown is populated 11. Select a pickup location 12. If multiple bookable items exist, verify item dropdown appears 13. Select specific item (if applicable) Step 3 - Booking Period: 14. Verify flatpickr calendar opens 15. Verify calendar shows existing bookings, checkouts, and available dates 16. Select a date range 17. Verify date range validation works 18. Verify lead period/trail period restrictions are enforced 19. Verify maximum booking period is enforced 20. Verify constraint highlighting updates correctly when changing item type, specific item, or pickup location Validation Testing: 21. Try to submit without patron - verify error message 22. Try to submit without item type - verify error message 23. Try to submit without pickup location - verify error message 24. Try to submit without date range - verify error message 25. Try to select dates outside allowed range - verify validation Capacity Testing: 26. For item type with limited bookable items, verify capacity warnings 27. Verify zero-capacity items show appropriate message Submitting Bookings: 28. Complete all required fields 29. Click "Place booking" 30. Verify booking is created successfully 31. Verify modal closes 32. Verify booking appears in bookings list STAFF INTERFACE - Editing Bookings 33. Click "Edit" on an existing booking 34. Verify modal opens with pre-filled data (patron, item type, pickup location, date range) 35. Modify the date range 36. Click "Update booking" 37. Verify changes are saved 38. Verify updated booking reflects changes Calendar Navigation: 39. In date picker, navigate between months 40. Verify highlighting persists across navigation 41. Verify availability data loads correctly for new months Error Handling: 42. Disconnect network 43. Try to create a booking 44. Verify appropriate error message displays 45. Restore network and verify subsequent booking attempts work Store State Management: 46. Open booking modal and fill in some fields but don't submit 47. Close modal and re-open it 48. Verify modal state is reset (no data persists inappropriately) Build Process: 49. Run yarn js:build and yarn js:build:prod 50. Verify build completes without errors 51. Verify dist files are generated correctly AI/LLM usage: The architecture and design here are my own, refined by hand over a long stretch that began with a manual refactor of place_booking.js. I used AI/LLM assistants throughout to help port the original plain JavaScript to Vue, write boilerplate, and draft and refactor parts of the implementation and tests, and I reviewed, corrected, and integrated everything they produced. This spanned the past year and a succession of models: Anthropic's Opus 4.5 through 4.8 and Fable 5, and OpenAI's GPT-5 to 5.2; the trailer records Opus 4.8, used most recently. Sponsored-by: Büchereizentrale Schleswig-Holstein <https://www.bz-sh.de/> Assisted-by: Opus 4.8 (Anthropic) -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #60 from Paul Derscheid <paul.derscheid@lmscloud.de> --- Created attachment 201577 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=201577&action=edit Bug 41129: [DO NOT PUSH] Add dev SQL seed for manual test setup - Marks itemtype BK bookable - Sets bookings_lead_period (2), bookings_trail_period (2), issuelength (7), renewalsallowed (2), renewalperiod (7) at the wildcard circulation_rules scope - Adds a closed day at CPL ten days out for holiday rendering - Idempotent via ON DUPLICATE KEY UPDATE / NOT EXISTS guards; run with: koha-mysql kohadev < bug_41129_seed.sql -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 Paul Derscheid <paul.derscheid@lmscloud.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Depends on| |43091 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43091 [Bug 43091] Staff-only Vue island chunks are emitted into the OPAC dist directory -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 Paul Derscheid <paul.derscheid@lmscloud.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #201572|0 |1 is obsolete| | Attachment #201573|0 |1 is obsolete| | Attachment #201574|0 |1 is obsolete| | Attachment #201575|0 |1 is obsolete| | Attachment #201576|0 |1 is obsolete| | Attachment #201577|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #63 from Paul Derscheid <paul.derscheid@lmscloud.de> --- Created attachment 202174 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=202174&action=edit Bug 41129: Add API clients for booking-related endpoints - Add BiblioAPIClient for biblio sub-resources (items, bookings, checkouts, pickup_locations) - Add BookingAPIClient for bookings CRUD (create, update) - Add LibraryAPIClient for /libraries/{id}/closed_dates - Add CirculationRulesAPIClient for circulation rules lookup - Extend PatronAPIClient with a search() method for paginated patron lookup and accept optional headers on get() - Register the new clients in fetch/api-client.js (lazy) and vue/fetch/api-client.js (eager) Test plan: 1. Apply the full series and run: yarn js:build 2. The clients carry no UI of their own; they are exercised end-to-end by the Cypress suites added with the booking modal patch later in this series: yarn cypress run --browser electron \ --spec 't/cypress/integration/Circulation/bookingsModalBasic_spec.ts' 3. Regression-check an existing consumer of the shared api-client registries, e.g. open the ERM module and confirm its tables still load Sponsored-by: Büchereizentrale Schleswig-Holstein <https://www.bz-sh.de/> Assisted-by: Opus 4.8 (Anthropic) -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #65 from Paul Derscheid <paul.derscheid@lmscloud.de> --- Created attachment 202176 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=202176&action=edit Bug 41129: Add Vue booking modal, Pinia store, and island wiring - Add BookingModal.vue (<script setup lang="ts">) orchestrating the multi-step form, plus the step components (BookingPatronStep, BookingDetailsStep, BookingPeriodStep) and BookingTooltip - Add PatronSearchSelect.vue and KohaAlert.vue shared components - Add the setup-style Pinia store under stores/bookings/, sectioned by responsibility along a one-way dep chain (status → data → draft → availability → validation) with an effects section for cross-cutting reactive bridges and an utils/withErrorHandling HOF wrapping per-action loading - Add modals/booking/island.inc and the button-place.inc trigger snippet and wire them into cat-toolbar.inc; point the bookings/list.tt edit buttons and timeline at the island (data-booking-modal / window.openBookingModal); drop the legacy modals/place_booking.inc, js/modals/place_booking.js, its calendar styles in staff-global.scss, and the asset include lines in the six catalogue detail views - Pass js-date-format.inc through unchanged for the new modal - Add the booking-modal-island entry to rspack.config.js - Add ambient.d.ts for *.css / bootstrap / vue-select shims and configure tsconfig.json: @koha-vue alias, allowJs for JSDoc-typed .js flow into TS SFCs, narrow per-feature include scoped to the booking subtree, plus target/lib/module/noEmit/types: [] so tsc runs clean - Hydrate the island via vue/modules/islands.ts, providing the Pinia store under "bookingStore" for inject() consumption - Add Cypress integration coverage in bookingsModalBasic / bookingsModalDatePicker / bookingsModalTimezone, the BookingPeriodStep and bookings-store effects and validation component specs, plus support/e2e and support/flatpickr adjustments and the insertData.js helpers Test plan: STAFF INTERFACE - Creating Bookings Preparation: - Run yarn js:build to emit the bundles Automated tests: - yarn cypress run --spec 't/cypress/integration/Circulation/bookingsModal*.ts' - yarn cypress run --component --spec 't/cypress/component/**/*_spec.ts' (covers BookingFlatpickr, useBookingCalendarMaps, BookingPeriodStep and the lib/booking pure-function specs) Test environment setup (preconditions for the manual steps below; cypress sets up its own data and does not need these): Bookable inventory: - Administration > Item types: mark at least one item type as "Bookable" (or use the per-item "Bookable" toggle on the item edit page) - Have a biblio with multiple bookable copies of that item type so the multi-item / item-selection / capacity flows are exercisable Circulation rules (Administration > Circulation and fines rules, applied to the library + item type scope you will test against or to the wildcard row): - bookings_lead_period: non-zero (e.g. 2) so steps 18 and 25 have a lead-time constraint to validate and the hover feedback emits lead-specific messages - bookings_trail_period: non-zero (e.g. 2) so step 18 has a trail buffer to validate - issuelength: a finite value (e.g. 7) so a maxPeriod is derivable and step 19 has a max-booking-period to enforce in the "issuelength" constraint mode - renewalsallowed + renewalperiod: non-zero values to exercise the "issuelength_with_renewals" constraint mode Calendar / holidays: - Tools > Calendar: add at least one closed day inside the visible calendar range for the pickup library so step 15 has a holiday to render (and the hover-feedback "library closed" path runs) Existing data on the test biblio: - Place one booking ahead of time so step 15 has a conflict to render, step 18 exercises lead/trail markers next to a real booking, and the edit flow (steps 33-38) has something to edit - Check out a bookable copy with a due date inside the visible calendar range so the picker shows a checkout interval and the checkout-vs-booking lead handling (Koha::Item->find_booking) participates 1. Navigate to a biblio detail page with bookable items 2. Click "Place booking" button 3. Observe the Vue booking modal opens Step 1 - Patron Selection: 4. Type patron name/cardnumber in search field 5. Verify typeahead search returns results 6. Select a patron 7. Verify patron is selected and displays correctly Step 2 - Booking Details: 8. Verify item type dropdown is populated 9. Select an item type 10. Verify pickup location dropdown is populated 11. Select a pickup location 12. If multiple bookable items exist, verify item dropdown appears 13. Select specific item (if applicable) Step 3 - Booking Period: 14. Verify flatpickr calendar opens 15. Verify calendar shows existing bookings, checkouts, and available dates 16. Select a date range 17. Verify date range validation works 18. Verify lead period/trail period restrictions are enforced 19. Verify maximum booking period is enforced 20. Verify constraint highlighting updates correctly when changing item type, specific item, or pickup location Validation Testing: 21. Try to submit without patron - verify error message 22. Try to submit without item type - verify error message 23. Try to submit without pickup location - verify error message 24. Try to submit without date range - verify error message 25. Try to select dates outside allowed range - verify validation Capacity Testing: 26. For item type with limited bookable items, verify capacity warnings 27. Verify zero-capacity items show appropriate message Submitting Bookings: 28. Complete all required fields 29. Click "Place booking" 30. Verify booking is created successfully 31. Verify modal closes 32. Verify booking appears in bookings list STAFF INTERFACE - Editing Bookings 33. Click "Edit" on an existing booking 34. Verify modal opens with pre-filled data (patron, item type, pickup location, date range) 35. Modify the date range 36. Click "Update booking" 37. Verify changes are saved 38. Verify updated booking reflects changes Calendar Navigation: 39. In date picker, navigate between months 40. Verify highlighting persists across navigation 41. Verify availability data loads correctly for new months Error Handling: 42. Disconnect network 43. Try to create a booking 44. Verify appropriate error message displays 45. Restore network and verify subsequent booking attempts work Store State Management: 46. Open booking modal and fill in some fields but don't submit 47. Close modal and re-open it 48. Verify modal state is reset (no data persists inappropriately) Build Process: 49. Run yarn js:build and yarn js:build:prod 50. Verify build completes without errors 51. Verify dist files are generated correctly AI/LLM usage: The architecture and design here are my own, refined by hand over a long stretch that began with a manual refactor of place_booking.js. I used AI/LLM assistants throughout to help port the original plain JavaScript to Vue, write boilerplate, and draft and refactor parts of the implementation and tests, and I reviewed, corrected, and integrated everything they produced. This spanned the past year and a succession of models: Anthropic's Opus 4.5 through 4.8 and Fable 5, and OpenAI's GPT-5 to 5.2; the trailer records Opus 4.8, used most recently. Sponsored-by: Büchereizentrale Schleswig-Holstein <https://www.bz-sh.de/> Assisted-by: Opus 4.8 (Anthropic) -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #67 from Paul Derscheid <paul.derscheid@lmscloud.de> --- Created attachment 202178 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=202178&action=edit Bug 41129: (follow-up) Make ESLint parse TypeScript Vue SFCs The booking components are the first SFCs using <script setup lang="ts">; vue-eslint-parser needs the typescript-eslint parser for their script blocks, otherwise every such component fails with a parsing error and is skipped by ESLint entirely. - scope a languageOptions block to **/*.vue delegating script parsing to ts.parser (typescript-eslint is already a dependency) -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #69 from Paul Derscheid <paul.derscheid@lmscloud.de> --- Created attachment 202180 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=202180&action=edit Bug 41129: [DO NOT PUSH] Add dev SQL seed for manual test setup - Marks itemtype BK bookable - Sets bookings_lead_period (2), bookings_trail_period (2), issuelength (7), renewalsallowed (2), renewalperiod (7) at the wildcard circulation_rules scope - Adds a closed day at CPL ten days out for holiday rendering - Idempotent via ON DUPLICATE KEY UPDATE / NOT EXISTS guards; run with: koha-mysql kohadev < bug_41129_seed.sql -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Depends on|42207 | Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42207 [Bug 42207] Allow library embed on GET /patrons/{patron_id} -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 Paul Derscheid <paul.derscheid@lmscloud.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Patch complexity|Large patch |Medium patch Comma delimited|Büchereizentrale |Büchereizentrale list of Sponsors|Schleswig-Holstein |Schleswig-Holstein |<https://www.bz-sh.de/>, |<https://www.bz-sh.de/>, |Büchereizentrale |Büchereizentrale |Schleswig-Holstein |Schleswig-Holstein |<https://www.bz-sh.de/> |<https://www.bz-sh.de/> --- Comment #70 from Paul Derscheid <paul.derscheid@lmscloud.de> --- Current status: Needs Signoff -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #71 from Paul Derscheid <paul.derscheid@lmscloud.de> --- Created attachment 203650 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=203650&action=edit Bug 41129: Add client-side booking rules and utilities Add the client-side rules used to determine which booking dates and options are available. - evaluate booking, checkout, lead-period, trail-period, holiday, and maximum booking-period restrictions - convert booking availability responses into item and date indexes - filter items, item types, and pickup locations as the booking context changes - convert calendar dates to library-timezone day boundaries for API requests - build calendar markers and accessible descriptions for date restrictions - add booking type declarations and shared ID and debounce utilities - add component tests for rules, constraints, availability maps, and dates Test plan: 1. Run: `yarn cypress run --component --browser electron --spec 't/cypress/component/lib/booking/**/*_spec.ts'` 2. Verify that all booking rule, constraint, availability-map, and timezone tests pass. Sponsored-by: Büchereizentrale Schleswig-Holstein <https://www.bz-sh.de/> Assisted-by: Opus 4.8 (Anthropic) Assisted-by: GPT 5.6 (OpenAI) -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #73 from Paul Derscheid <paul.derscheid@lmscloud.de> --- Created attachment 203652 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=203652&action=edit Bug 41129: Add accessible booking calendar and Vue type checking Add a booking date-range picker that presents availability and booking rules in the calendar. - display bookings, checkouts, holidays, lead and trail periods, and booking limits on Flatpickr dates - recalculate item and date availability when the selected range or calendar month changes - retain existing booking ranges while editing - keep restricted dates keyboard-focusable and announce why they cannot be selected - support keyboard selection and day, week, month, and year navigation - exclude Koha's Flatpickr date-shortcut plugins because they bypass booking range validation - add Cypress support and component tests for the calendar and availability maps - enable TypeScript parsing and vue-tsc checking for booking Vue components Test plan: 1. Open the booking modal for a record with a bookable item. 2. Open the booking period calendar and verify that it does not display the Yesterday, Today, and Tomorrow shortcuts. 3. Verify that past dates and dates occupied by a booking or checkout cannot be selected and display the appropriate marker and explanation. 4. Use the keyboard to open the calendar, select a date range, and navigate by day, week, month, and year. 5. Verify that closing the calendar returns focus to the booking period field. 6. Run: `yarn cypress run --component --browser electron --spec 't/cypress/component/BookingCalendar_spec.ts, t/cypress/component/useBookingCalendarMaps_spec.ts'` Sponsored-by: Büchereizentrale Schleswig-Holstein <https://www.bz-sh.de/> Assisted-by: Opus 4.8 (Anthropic) Assisted-by: GPT 5.6 (OpenAI) -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #75 from Paul Derscheid <paul.derscheid@lmscloud.de> --- Created attachment 203654 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=203654&action=edit Bug 41129: Allow assigned unbookable items when editing bookings An item or its effective item type can become unbookable after the item has been assigned to a booking. The availability endpoint then rejects the item, and updating the booking can report a false capacity clash. Allow the assigned item only when excluded_booking_id identifies its booking. Include that item in the capacity calculation, but exclude bookings and checkouts for items outside the resulting item pool. Return a 400 response when an update has a genuine booking clash. Test plan: 1. Create a booking assigned to a specific item. 2. Make the item unbookable and edit the booking. 3. Verify that the assigned item remains selected and its availability request succeeds. 4. Change the booking period and select Update. 5. Verify that the existing booking is updated and no duplicate is created. 6. Repeat the test after making the item's effective item type unbookable. 7. Verify that another unbookable item cannot be selected for the booking. 8. Create a conflicting booking for the assigned item and verify that the update is rejected as a booking clash. 9. Run: `prove -v t/db_dependent/Koha/Biblio.t t/db_dependent/Koha/Biblio/Availability/Booking.t t/db_dependent/api/v1/biblios.t t/db_dependent/api/v1/bookings.t` Sponsored-by: Büchereizentrale Schleswig-Holstein <https://www.bz-sh.de/> Assisted-by: GPT 5.6 (OpenAI) -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 Tomás Cohen Arazi (tcohen) <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |tomascohen@gmail.com Comma delimited|Büchereizentrale |Büchereizentrale list of Sponsors|Schleswig-Holstein |Schleswig-Holstein |<https://www.bz-sh.de/>, |<https://www.bz-sh.de/>, |Büchereizentrale |Büchereizentrale |Schleswig-Holstein |Schleswig-Holstein |<https://www.bz-sh.de/> |<https://www.bz-sh.de/> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 Bug 41129 depends on bug 37707, which changed state. Bug 37707 Summary: Lead/Trail times should work in combination https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37707 What |Removed |Added ---------------------------------------------------------------------------- Status|Needs documenting |RESOLVED Resolution|--- |FIXED -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #203650|0 |1 is obsolete| | Attachment #203651|0 |1 is obsolete| | Attachment #203652|0 |1 is obsolete| | Attachment #203653|0 |1 is obsolete| | Attachment #203654|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #78 from Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> --- Created attachment 205275 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=205275&action=edit Bug 41129: Add frontend clients for booking workflows Add frontend API clients for the resources needed to create and edit bookings. - add biblio client methods for items, bookings, availability, checkouts, and pickup locations - add clients for circulation rules and library closed dates - add paginated patron search and per-request configuration to existing clients - allow booking create and update requests to receive request configuration - add staff and OPAC adapters for the booking data interface - pass AbortSignal through HttpClient and preserve HTTP status and Koha error codes when a request fails - allow callers that display their own errors to suppress the default dialog - add a translated formatter for errors returned by frontend API clients - document parameters and return values for the added client operations - add component tests for HttpClient and API error formatting Test plan: 1. Run: `yarn cypress run --component --browser electron --spec 't/cypress/component/HttpClient_spec.ts, t/cypress/component/fetch/api-error_spec.ts'` 2. Verify that cancelled requests do not display an error. 3. Verify that failed requests retain their status, code, and message. 4. Verify that suppressing the default dialog does not change the error returned to the caller. 5. Verify the translated messages for unauthorized, forbidden, server, string, and empty errors. Sponsored-by: Büchereizentrale Schleswig-Holstein <https://www.bz-sh.de/> Assisted-by: Opus 4.8 (Anthropic) Assisted-by: GPT 5.6 (OpenAI) Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk> Signed-off-by: Jacob O'Mara <jacob.omara@openfifth.co.uk> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #80 from Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> --- Created attachment 205277 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=205277&action=edit Bug 41129: Replace the staff booking modal with Vue Replace the staff interface's jQuery booking modal with a Vue custom-element island. - add components for patron selection, booking details, the booking period, validation messages, the form, and the Bootstrap modal - add a Pinia store for booking data, validation, calendar state, and create and edit workflows - discard responses from superseded requests and cancel patron searches when the form closes - submit create and update requests through the booking API and retain support for CSRF-protected native form submission - add the booking island to catalogue, circulation, and patron pages - update booking tables and the booking timeline to open the Vue modal - refresh page booking counts, tables, timelines, and success messages after a booking is saved - return focus to the control that opened the modal when it closes - format patron ages and calendar marker details with reorderable translations - remove the old place_booking template, JavaScript, and global calendar styles - add component and end-to-end tests for booking creation and editing Test plan: 1. Enable EnableBooking and sign in as a staff user with permission to manage bookings. 2. Create a record with at least two bookable items of different item types and create a patron who can book them. 3. Open the record detail page and select Place booking. 4. Search for and select the patron. Verify that patron ages use the correct singular or plural wording. 5. Verify that pickup location, item type, and item choices become available and that changing one updates the others. 6. Leave the item and item type empty, select an available period, and save an Any item booking. Verify the success message and booking count. 7. Create another booking for a specific item. Verify that existing bookings, checkouts, holidays, and circulation rules restrict the selectable dates. 8. Verify that calendar conflict details present the marker type and barcode as one complete message. 9. Edit each booking from the bookings table or timeline. Change its details and verify that Update changes the existing booking instead of creating another. 10. Open the modal from the catalogue record, circulation, and patron-detail booking views and verify that create and edit actions use the same form. 11. Close and reopen the modal. Verify that the previous draft is cleared and focus returns to the control that opened it. 12. Repeat create, edit, and cancel using only the keyboard and verify that all fields, errors, date restrictions, and modal controls have accessible names. 13. Run the BookingForm, BookingModal, BookingPatronStep, BookingPeriodStep, booking-store, and three Circulation booking-modal Cypress suites. Sponsored-by: Büchereizentrale Schleswig-Holstein <https://www.bz-sh.de/> Assisted-by: Opus 4.8 (Anthropic) Assisted-by: GPT 5.6 (OpenAI) Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk> Signed-off-by: Jacob O'Mara <jacob.omara@openfifth.co.uk> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #82 from Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> --- Created attachment 205279 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=205279&action=edit Bug 41129: (QA follow-up) Disable the calendar when an availability refresh fails Guard against a stale, still-enabled calendar after an availability refresh call fails, and cover the guard with a test. Co-authored-by: Jacob O'Mara <jacob.omara@openfifth.co.uk> Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk> Signed-off-by: Jacob O'Mara <jacob.omara@openfifth.co.uk> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #84 from Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> --- Created attachment 205281 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=205281&action=edit Bug 41129: (QA follow-up) Harden HttpClient against malformed and unreachable API responses Guard against malformed JSON in http-client responses, keep status and error_code available on the Vue HttpClient, and surface a clear message when the API is unreachable. Co-authored-by: Martin Renvoize <martin.renvoize@openfifth.co.uk> Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk> Signed-off-by: Jacob O'Mara <jacob.omara@openfifth.co.uk> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #86 from Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> --- Created attachment 205283 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=205283&action=edit Bug 41129: (QA follow-up) Improve booking workflow diagnostics Log when the availability window is clamped and when a booking session opens without a biblionumber, to aid support diagnosis. Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk> Signed-off-by: Jacob O'Mara <jacob.omara@openfifth.co.uk> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #88 from Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> --- Created attachment 205285 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=205285&action=edit Bug 41129: (QA follow-up) Add a component spec for BookingDetailsStep Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk> Signed-off-by: Jacob O'Mara <jacob.omara@openfifth.co.uk> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #90 from Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> --- Created attachment 205287 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=205287&action=edit Bug 41129: (QA follow-up) Remove dead code left over from earlier iterations Remove unused rangeState, dead event-dots calendar CSS, and correct check_booking's POD synopsis to include item_id. Co-authored-by: Jacob O'Mara <jacob.omara@openfifth.co.uk> Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk> Signed-off-by: Jacob O'Mara <jacob.omara@openfifth.co.uk> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #92 from Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> --- Created attachment 205289 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=205289&action=edit Bug 41129: (QA follow-up) Rework calendar marker colours and block trail-conflicting start dates Give the trail-period marker its own colour, hide the marker-count grid and multi-marker tooltip mechanism (superseded later by an inline day-details panel), remove the marker dot grid entirely, and simplify the calendar's colour/legend model. Block start dates whose trail window would run into an existing booking - a start whose booking already puts its trail on a conflict has no valid end, so grey those days too. Known limitation: this can end up disabling almost every date under some configurations of long trail periods and dense bookings - not fixed here. Also keep the calendar hint visible with tighter step spacing, and only pre-paint the constrained booking range in end_date_only mode. Co-authored-by: Martin Renvoize <martin.renvoize@openfifth.co.uk> Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk> Signed-off-by: Jacob O'Mara <jacob.omara@openfifth.co.uk> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #94 from Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> --- Created attachment 205291 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=205291&action=edit Bug 41129: (QA follow-up) Restore richer calendar colours with hover-scoped adjacency, and rebalance the info panels Consolidate lead/trail colours and clarify the buffer legend, remove the muted baseline from non-adjacent lead/trail days, and only style booked/checked-out/holiday before hover - richer detail (adjacency, partial availability) now only appears in response to hovering, keeping the at-rest view calm. Merge Booked/Checked out into a single Unavailable state, scope booked/checked-out markers to the current selection, and add a partial-availability dot for days with a mix of free and booked items. Move the constraint-info summary next to the legend it explains, rebalance the two hover-feedback panels so each owns one job, and restore a dual lead/trail hover preview before a start date is chosen. Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk> Signed-off-by: Jacob O'Mara <jacob.omara@openfifth.co.uk> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #96 from Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> --- Created attachment 205293 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=205293&action=edit Bug 41129: (QA follow-up) Rework Unavailable-day rendering, gate clash colours, and fix junction shapes per the UX spec Make consecutive Unavailable days read as one contiguous strip rather than a row of separately-rounded pills, then stop rounding those runs entirely and fix the junction shapes between adjacent coloured segments so the bands read as smooth, continuous strips. Gate the clash colours on a genuine conflict, fix a wrong disabled-reason message caused by a stale maxPeriod, and size the existing-booking lead/trail band from leadDays rather than a per-item tag union. Rework colours, legend, and info panels per the UX spec: dock the legend inside flatpickr's own calendar, only highlight existing- booking lead/trail around Unavailable slots, fix month-nav arrows overlapping the legend, and show the same detail dot on Unavailable days as Partial ones. Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk> Signed-off-by: Jacob O'Mara <jacob.omara@openfifth.co.uk> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #205274|0 |1 is obsolete| | Attachment #205275|0 |1 is obsolete| | Attachment #205276|0 |1 is obsolete| | Attachment #205277|0 |1 is obsolete| | Attachment #205278|0 |1 is obsolete| | Attachment #205279|0 |1 is obsolete| | Attachment #205280|0 |1 is obsolete| | Attachment #205281|0 |1 is obsolete| | Attachment #205282|0 |1 is obsolete| | Attachment #205283|0 |1 is obsolete| | Attachment #205284|0 |1 is obsolete| | Attachment #205285|0 |1 is obsolete| | Attachment #205286|0 |1 is obsolete| | Attachment #205287|0 |1 is obsolete| | Attachment #205288|0 |1 is obsolete| | Attachment #205289|0 |1 is obsolete| | Attachment #205290|0 |1 is obsolete| | Attachment #205291|0 |1 is obsolete| | Attachment #205292|0 |1 is obsolete| | Attachment #205293|0 |1 is obsolete| | Attachment #205294|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #99 from Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> --- Created attachment 205302 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=205302&action=edit Bug 41129: Add frontend clients for booking workflows Add frontend API clients for the resources needed to create and edit bookings. - add biblio client methods for items, bookings, availability, checkouts, and pickup locations - add clients for circulation rules and library closed dates - add paginated patron search and per-request configuration to existing clients - allow booking create and update requests to receive request configuration - add staff and OPAC adapters for the booking data interface - pass AbortSignal through HttpClient and preserve HTTP status and Koha error codes when a request fails - allow callers that display their own errors to suppress the default dialog - add a translated formatter for errors returned by frontend API clients - document parameters and return values for the added client operations - add component tests for HttpClient and API error formatting Test plan: 1. Run: `yarn cypress run --component --browser electron --spec 't/cypress/component/HttpClient_spec.ts, t/cypress/component/fetch/api-error_spec.ts'` 2. Verify that cancelled requests do not display an error. 3. Verify that failed requests retain their status, code, and message. 4. Verify that suppressing the default dialog does not change the error returned to the caller. 5. Verify the translated messages for unauthorized, forbidden, server, string, and empty errors. Sponsored-by: Büchereizentrale Schleswig-Holstein <https://www.bz-sh.de/> Assisted-by: Opus 4.8 (Anthropic) Assisted-by: GPT 5.6 (OpenAI) Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk> Signed-off-by: Jacob O'Mara <jacob.omara@openfifth.co.uk> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #101 from Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> --- Created attachment 205304 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=205304&action=edit Bug 41129: Replace the staff booking modal with Vue Replace the staff interface's jQuery booking modal with a Vue custom-element island. - add components for patron selection, booking details, the booking period, validation messages, the form, and the Bootstrap modal - add a Pinia store for booking data, validation, calendar state, and create and edit workflows - discard responses from superseded requests and cancel patron searches when the form closes - submit create and update requests through the booking API and retain support for CSRF-protected native form submission - add the booking island to catalogue, circulation, and patron pages - update booking tables and the booking timeline to open the Vue modal - refresh page booking counts, tables, timelines, and success messages after a booking is saved - return focus to the control that opened the modal when it closes - format patron ages and calendar marker details with reorderable translations - remove the old place_booking template, JavaScript, and global calendar styles - add component and end-to-end tests for booking creation and editing Test plan: 1. Enable EnableBooking and sign in as a staff user with permission to manage bookings. 2. Create a record with at least two bookable items of different item types and create a patron who can book them. 3. Open the record detail page and select Place booking. 4. Search for and select the patron. Verify that patron ages use the correct singular or plural wording. 5. Verify that pickup location, item type, and item choices become available and that changing one updates the others. 6. Leave the item and item type empty, select an available period, and save an Any item booking. Verify the success message and booking count. 7. Create another booking for a specific item. Verify that existing bookings, checkouts, holidays, and circulation rules restrict the selectable dates. 8. Verify that calendar conflict details present the marker type and barcode as one complete message. 9. Edit each booking from the bookings table or timeline. Change its details and verify that Update changes the existing booking instead of creating another. 10. Open the modal from the catalogue record, circulation, and patron-detail booking views and verify that create and edit actions use the same form. 11. Close and reopen the modal. Verify that the previous draft is cleared and focus returns to the control that opened it. 12. Repeat create, edit, and cancel using only the keyboard and verify that all fields, errors, date restrictions, and modal controls have accessible names. 13. Run the BookingForm, BookingModal, BookingPatronStep, BookingPeriodStep, booking-store, and three Circulation booking-modal Cypress suites. Sponsored-by: Büchereizentrale Schleswig-Holstein <https://www.bz-sh.de/> Assisted-by: Opus 4.8 (Anthropic) Assisted-by: GPT 5.6 (OpenAI) Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk> Signed-off-by: Jacob O'Mara <jacob.omara@openfifth.co.uk> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #103 from Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> --- Created attachment 205306 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=205306&action=edit Bug 41129: (QA follow-up) Disable the calendar when an availability refresh fails Guard against a stale, still-enabled calendar after an availability refresh call fails, and cover the guard with a test. Co-authored-by: Jacob O'Mara <jacob.omara@openfifth.co.uk> Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk> Signed-off-by: Jacob O'Mara <jacob.omara@openfifth.co.uk> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #105 from Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> --- Created attachment 205308 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=205308&action=edit Bug 41129: (QA follow-up) Harden HttpClient against malformed and unreachable API responses Guard against malformed JSON in http-client responses, keep status and error_code available on the Vue HttpClient, and surface a clear message when the API is unreachable. Co-authored-by: Martin Renvoize <martin.renvoize@openfifth.co.uk> Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk> Signed-off-by: Jacob O'Mara <jacob.omara@openfifth.co.uk> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #107 from Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> --- Created attachment 205310 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=205310&action=edit Bug 41129: (QA follow-up) Improve booking workflow diagnostics Log when the availability window is clamped and when a booking session opens without a biblionumber, to aid support diagnosis. Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk> Signed-off-by: Jacob O'Mara <jacob.omara@openfifth.co.uk> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #109 from Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> --- Created attachment 205312 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=205312&action=edit Bug 41129: (QA follow-up) Add a component spec for BookingDetailsStep Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk> Signed-off-by: Jacob O'Mara <jacob.omara@openfifth.co.uk> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #111 from Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> --- Created attachment 205314 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=205314&action=edit Bug 41129: (QA follow-up) Remove dead code left over from earlier iterations Remove unused rangeState, dead event-dots calendar CSS, and correct check_booking's POD synopsis to include item_id. Co-authored-by: Jacob O'Mara <jacob.omara@openfifth.co.uk> Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk> Signed-off-by: Jacob O'Mara <jacob.omara@openfifth.co.uk> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #113 from Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> --- Created attachment 205316 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=205316&action=edit Bug 41129: (QA follow-up) Rework calendar marker colours and block trail-conflicting start dates Give the trail-period marker its own colour, hide the marker-count grid and multi-marker tooltip mechanism (superseded later by an inline day-details panel), remove the marker dot grid entirely, and simplify the calendar's colour/legend model. Block start dates whose trail window would run into an existing booking - a start whose booking already puts its trail on a conflict has no valid end, so grey those days too. Known limitation: this can end up disabling almost every date under some configurations of long trail periods and dense bookings - not fixed here. Also keep the calendar hint visible with tighter step spacing, and only pre-paint the constrained booking range in end_date_only mode. Co-authored-by: Martin Renvoize <martin.renvoize@openfifth.co.uk> Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk> Signed-off-by: Jacob O'Mara <jacob.omara@openfifth.co.uk> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #115 from Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> --- Created attachment 205318 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=205318&action=edit Bug 41129: (QA follow-up) Restore richer calendar colours with hover-scoped adjacency, and rebalance the info panels Consolidate lead/trail colours and clarify the buffer legend, remove the muted baseline from non-adjacent lead/trail days, and only style booked/checked-out/holiday before hover - richer detail (adjacency, partial availability) now only appears in response to hovering, keeping the at-rest view calm. Merge Booked/Checked out into a single Unavailable state, scope booked/checked-out markers to the current selection, and add a partial-availability dot for days with a mix of free and booked items. Move the constraint-info summary next to the legend it explains, rebalance the two hover-feedback panels so each owns one job, and restore a dual lead/trail hover preview before a start date is chosen. Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk> Signed-off-by: Jacob O'Mara <jacob.omara@openfifth.co.uk> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #117 from Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> --- Created attachment 205320 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=205320&action=edit Bug 41129: (QA follow-up) Rework Unavailable-day rendering, gate clash colours, and fix junction shapes per the UX spec Make consecutive Unavailable days read as one contiguous strip rather than a row of separately-rounded pills, then stop rounding those runs entirely and fix the junction shapes between adjacent coloured segments so the bands read as smooth, continuous strips. Gate the clash colours on a genuine conflict, fix a wrong disabled-reason message caused by a stale maxPeriod, and size the existing-booking lead/trail band from leadDays rather than a per-item tag union. Rework colours, legend, and info panels per the UX spec: dock the legend inside flatpickr's own calendar, only highlight existing- booking lead/trail around Unavailable slots, fix month-nav arrows overlapping the legend, and show the same detail dot on Unavailable days as Partial ones. Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk> Signed-off-by: Jacob O'Mara <jacob.omara@openfifth.co.uk> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #119 from Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> --- *** Bug 37618 has been marked as a duplicate of this bug. *** -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 Paul Derscheid <paul.derscheid@lmscloud.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #203655|0 |1 is obsolete| | Attachment #205301|0 |1 is obsolete| | Attachment #205302|0 |1 is obsolete| | Attachment #205303|0 |1 is obsolete| | Attachment #205304|0 |1 is obsolete| | Attachment #205305|0 |1 is obsolete| | Attachment #205306|0 |1 is obsolete| | Attachment #205307|0 |1 is obsolete| | Attachment #205308|0 |1 is obsolete| | Attachment #205309|0 |1 is obsolete| | Attachment #205310|0 |1 is obsolete| | Attachment #205311|0 |1 is obsolete| | Attachment #205312|0 |1 is obsolete| | Attachment #205313|0 |1 is obsolete| | Attachment #205314|0 |1 is obsolete| | Attachment #205315|0 |1 is obsolete| | Attachment #205316|0 |1 is obsolete| | Attachment #205317|0 |1 is obsolete| | Attachment #205318|0 |1 is obsolete| | Attachment #205319|0 |1 is obsolete| | Attachment #205320|0 |1 is obsolete| | Attachment #205321|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #121 from Paul Derscheid <paul.derscheid@lmscloud.de> --- Created attachment 205335 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=205335&action=edit Bug 41129: Add client-side booking rules and utilities Add the client-side rules used to determine which booking dates and options are available. - evaluate booking, checkout, lead-period, trail-period, holiday, and maximum booking-period restrictions - convert booking availability responses into item and date indexes - filter items, item types, and pickup locations as the booking context changes - convert calendar dates to library-timezone day boundaries for API requests - build calendar markers and accessible descriptions for date restrictions - add booking type declarations and shared ID and debounce utilities - add component tests for rules, constraints, availability maps, and dates Test plan: 1. Run: `yarn cypress run --component --browser electron --spec 't/cypress/component/lib/booking/**/*_spec.ts'` 2. Verify that all booking rule, constraint, availability-map, and timezone tests pass. Sponsored-by: Büchereizentrale Schleswig-Holstein <https://www.bz-sh.de/> Assisted-by: Opus 4.8 (Anthropic) Assisted-by: GPT 5.6 (OpenAI) Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk> Signed-off-by: Jacob O'Mara <jacob.omara@openfifth.co.uk> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #123 from Paul Derscheid <paul.derscheid@lmscloud.de> --- Created attachment 205337 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=205337&action=edit Bug 41129: Add accessible booking calendar and Vue type checking Add a booking date-range picker that presents availability and booking rules in the calendar. - display bookings, checkouts, holidays, lead and trail periods, and booking limits on Flatpickr dates - recalculate item and date availability when the selected range or calendar month changes - retain existing booking ranges while editing - keep restricted dates keyboard-focusable and announce why they cannot be selected - support keyboard selection and day, week, month, and year navigation - exclude Koha's Flatpickr date-shortcut plugins because they bypass booking range validation - add Cypress support and component tests for the calendar and availability maps - enable TypeScript parsing and vue-tsc checking for booking Vue components Test plan: 1. Open the booking modal for a record with a bookable item. 2. Open the booking period calendar and verify that it does not display the Yesterday, Today, and Tomorrow shortcuts. 3. Verify that past dates and dates occupied by a booking or checkout cannot be selected and display the appropriate marker and explanation. 4. Use the keyboard to open the calendar, select a date range, and navigate by day, week, month, and year. 5. Verify that closing the calendar returns focus to the booking period field. 6. Run: `yarn cypress run --component --browser electron --spec 't/cypress/component/BookingCalendar_spec.ts, t/cypress/component/useBookingCalendarMaps_spec.ts'` Sponsored-by: Büchereizentrale Schleswig-Holstein <https://www.bz-sh.de/> Assisted-by: Opus 4.8 (Anthropic) Assisted-by: GPT 5.6 (OpenAI) Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk> Signed-off-by: Jacob O'Mara <jacob.omara@openfifth.co.uk> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #125 from Paul Derscheid <paul.derscheid@lmscloud.de> --- Created attachment 205339 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=205339&action=edit Bug 41129: Allow assigned unbookable items when editing bookings An item or its effective item type can become unbookable after the item has been assigned to a booking. The availability endpoint then rejects the item, and updating the booking can report a false capacity clash. Allow the assigned item only when excluded_booking_id identifies its booking. Include that item in the capacity calculation, but exclude bookings and checkouts for items outside the resulting item pool. Return a 400 response when an update has a genuine booking clash. Test plan: 1. Create a booking assigned to a specific item. 2. Make the item unbookable and edit the booking. 3. Verify that the assigned item remains selected and its availability request succeeds. 4. Change the booking period and select Update. 5. Verify that the existing booking is updated and no duplicate is created. 6. Repeat the test after making the item's effective item type unbookable. 7. Verify that another unbookable item cannot be selected for the booking. 8. Create a conflicting booking for the assigned item and verify that the update is rejected as a booking clash. 9. Run: `prove -v t/db_dependent/Koha/Biblio.t t/db_dependent/Koha/Biblio/Availability/Booking.t t/db_dependent/api/v1/biblios.t t/db_dependent/api/v1/bookings.t` Sponsored-by: Büchereizentrale Schleswig-Holstein <https://www.bz-sh.de/> Assisted-by: GPT 5.6 (OpenAI) Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk> Signed-off-by: Jacob O'Mara <jacob.omara@openfifth.co.uk> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #127 from Paul Derscheid <paul.derscheid@lmscloud.de> --- Created attachment 205341 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=205341&action=edit Bug 41129: (QA follow-up) Restore Escape-to-close on the booking modal Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk> Signed-off-by: Jacob O'Mara <jacob.omara@openfifth.co.uk> Signed-off-by: Paul Derscheid <paul.derscheid@lmscloud.de> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #129 from Paul Derscheid <paul.derscheid@lmscloud.de> --- Created attachment 205343 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=205343&action=edit Bug 41129: (QA follow-up) Add error_code to booking conflict/duplicate responses Surface a stable error_code on booking conflict/duplicate API responses, and assert it in the conflict tests. Co-authored-by: Jacob O'Mara <jacob.omara@openfifth.co.uk> Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk> Signed-off-by: Jacob O'Mara <jacob.omara@openfifth.co.uk> Signed-off-by: Paul Derscheid <paul.derscheid@lmscloud.de> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #131 from Paul Derscheid <paul.derscheid@lmscloud.de> --- Created attachment 205345 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=205345&action=edit Bug 41129: (QA follow-up) Add DST-transition coverage for booking date math Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk> Signed-off-by: Jacob O'Mara <jacob.omara@openfifth.co.uk> Signed-off-by: Paul Derscheid <paul.derscheid@lmscloud.de> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #133 from Paul Derscheid <paul.derscheid@lmscloud.de> --- Created attachment 205347 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=205347&action=edit Bug 41129: (QA follow-up) Harden booking date/range utilities against invalid input Make addDays/addMonths fail loudly on invalid input instead of silently producing nonsense dates, and make rangeHasConflict return false for a null date range instead of throwing. Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk> Signed-off-by: Jacob O'Mara <jacob.omara@openfifth.co.uk> Signed-off-by: Paul Derscheid <paul.derscheid@lmscloud.de> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #135 from Paul Derscheid <paul.derscheid@lmscloud.de> --- Created attachment 205349 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=205349&action=edit Bug 41129: (QA follow-up) Extract a shared PatronSelect component Pull the patron-selection widget out into its own reusable PatronSelect component, use it from the patron step, and stop it depending on booking-only CSS tokens so it can be reused elsewhere. Co-authored-by: Martin Renvoize <martin.renvoize@openfifth.co.uk> Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk> Signed-off-by: Jacob O'Mara <jacob.omara@openfifth.co.uk> Signed-off-by: Paul Derscheid <paul.derscheid@lmscloud.de> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #137 from Paul Derscheid <paul.derscheid@lmscloud.de> --- Created attachment 205351 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=205351&action=edit Bug 41129: (QA follow-up) Replace the floating booking tooltip with an inline day-details panel The floating, positioned tooltip was fragile (z-index/positioning inside a modal) and awkward for assistive tech. Replace it with an inline panel that updates in place as the pointer moves. Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk> Signed-off-by: Jacob O'Mara <jacob.omara@openfifth.co.uk> Signed-off-by: Paul Derscheid <paul.derscheid@lmscloud.de> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #139 from Paul Derscheid <paul.derscheid@lmscloud.de> --- Created attachment 205353 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=205353&action=edit Bug 41129: (QA follow-up) Fix the picker getting stuck open inside the modal Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk> Signed-off-by: Jacob O'Mara <jacob.omara@openfifth.co.uk> Signed-off-by: Paul Derscheid <paul.derscheid@lmscloud.de> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #141 from Paul Derscheid <paul.derscheid@lmscloud.de> --- Created attachment 205355 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=205355&action=edit Bug 41129: (QA follow-up) Trial an always-visible, two-month booking calendar Proposed UX improvement on top of the rest of this follow-up series: replace the click-to-open, single-month popup with an always-visible, two-month inline calendar. Move the legend and hover panels out of flatpickr's own container, stop re-emitting an unchanged viewport on every date pick, keep the picker mounted under v-show, harden date-range selection against redraw churn, overlay the clear button, add a not-ready placeholder, reorder input/legend/calendar, fix the lead/trail hover highlight breaking at a month's last row, and style the hover-feedback bar and day-details panel as standard Bootstrap alerts. This is offered as a proposed improvement, not a required fix - the preceding commits stand on their own with the original single-month popup calendar. Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk> Signed-off-by: Jacob O'Mara <jacob.omara@openfifth.co.uk> Signed-off-by: Paul Derscheid <paul.derscheid@lmscloud.de> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #143 from Paul Derscheid <paul.derscheid@lmscloud.de> --- Created attachment 205357 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=205357&action=edit Bug 41129: (QA follow-up) Show a single month when the calendar has no room for two - BookingCalendar picks showMonths from its wrapper's width: two when two day grids fit, else one, so a phone-width modal no longer overflows; measured once after the first build, then from a ResizeObserver, rebuilding Flatpickr on the next frame when the count changes - Stretch .booking-date-picker so that width is the space available - Write BookingPeriodStep's render width on the next frame too, or the two observers trip the browser's ResizeObserver loop error - Load Flatpickr's stylesheet in the component harness, as calendar.inc does, so the grid has its real width there - Cover the single-month and resize cases in BookingCalendar_spec.ts Test plan: 1. yarn js:build; open the booking modal at ~440px width: one month; widen past ~620px of modal width: two months, selection kept 2. cypress run --component 3. cypress run --spec 't/cypress/integration/Circulation/bookingsModal*_spec.ts' Assisted-by: Fable 5.1 (Anthropic) Signed-off-by: Paul Derscheid <paul.derscheid@lmscloud.de> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #145 from Paul Derscheid <paul.derscheid@lmscloud.de> --- Created attachment 205359 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=205359&action=edit Bug 41129: (QA follow-up) Remove the calendar drop shadow and clip highlight bands to the grid - Replace Flatpickr's popup drop shadow with a 1px border in the modal's neutral colour and its small radius, as the closed-days calendar (tools/holidays.pl) is embedded; report the border-box width so legend and panels line up with the border - Bands (selected, unavailable, lead, trail, clash and hover variants) bridge the ~5px cell gaps with a 5px box-shadow each side; at a row's ends the outward half spilled past the grid or into the next month. Hoist each state's colour into --booking-band, draw the shadow from one rule, and trim the outward half on row-end cells (nth-child 7n+1 and 7n+7) to the 2.5px that reach the grid edge - Assert the Saturday case in bookingsModalDatePicker_spec.ts Test plan: 1. yarn js:build; enable the calendar: no shadow, plain border, legend and panels as wide as the bordered calendar 2. With a booking spanning a weekend: the band runs to the grid edge on Saturday and from it on Sunday 3. cypress run --spec t/cypress/integration/Circulation/bookingsModalDatePicker_spec.ts 4. cypress run --component Assisted-by: Fable 5.1 (Anthropic) Signed-off-by: Paul Derscheid <paul.derscheid@lmscloud.de> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #205334|0 |1 is obsolete| | Attachment #205335|0 |1 is obsolete| | Attachment #205336|0 |1 is obsolete| | Attachment #205337|0 |1 is obsolete| | Attachment #205338|0 |1 is obsolete| | Attachment #205339|0 |1 is obsolete| | Attachment #205340|0 |1 is obsolete| | Attachment #205341|0 |1 is obsolete| | Attachment #205342|0 |1 is obsolete| | Attachment #205343|0 |1 is obsolete| | Attachment #205344|0 |1 is obsolete| | Attachment #205345|0 |1 is obsolete| | Attachment #205346|0 |1 is obsolete| | Attachment #205347|0 |1 is obsolete| | Attachment #205348|0 |1 is obsolete| | Attachment #205349|0 |1 is obsolete| | Attachment #205350|0 |1 is obsolete| | Attachment #205351|0 |1 is obsolete| | Attachment #205352|0 |1 is obsolete| | Attachment #205353|0 |1 is obsolete| | Attachment #205354|0 |1 is obsolete| | Attachment #205355|0 |1 is obsolete| | Attachment #205356|0 |1 is obsolete| | Attachment #205357|0 |1 is obsolete| | Attachment #205358|0 |1 is obsolete| | Attachment #205359|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #147 from Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> --- Created attachment 205499 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=205499&action=edit Bug 41129: Add client-side booking rules and utilities Add the client-side rules used to determine which booking dates and options are available. - evaluate booking, checkout, lead-period, trail-period, holiday, and maximum booking-period restrictions - convert booking availability responses into item and date indexes - filter items, item types, and pickup locations as the booking context changes - convert calendar dates to library-timezone day boundaries for API requests - build calendar markers and accessible descriptions for date restrictions - add booking type declarations and shared ID and debounce utilities - add component tests for rules, constraints, availability maps, and dates Test plan: 1. Run: `yarn cypress run --component --browser electron --spec 't/cypress/component/lib/booking/**/*_spec.ts'` 2. Verify that all booking rule, constraint, availability-map, and timezone tests pass. Sponsored-by: Büchereizentrale Schleswig-Holstein <https://www.bz-sh.de/> Assisted-by: Opus 4.8 (Anthropic) Assisted-by: GPT 5.6 (OpenAI) Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk> Signed-off-by: Jacob O'Mara <jacob.omara@openfifth.co.uk> Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #149 from Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> --- Created attachment 205501 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=205501&action=edit Bug 41129: Add accessible booking calendar and Vue type checking Add a booking date-range picker that presents availability and booking rules in the calendar. - display bookings, checkouts, holidays, lead and trail periods, and booking limits on Flatpickr dates - recalculate item and date availability when the selected range or calendar month changes - retain existing booking ranges while editing - keep restricted dates keyboard-focusable and announce why they cannot be selected - support keyboard selection and day, week, month, and year navigation - exclude Koha's Flatpickr date-shortcut plugins because they bypass booking range validation - add Cypress support and component tests for the calendar and availability maps - enable TypeScript parsing and vue-tsc checking for booking Vue components Test plan: 1. Open the booking modal for a record with a bookable item. 2. Open the booking period calendar and verify that it does not display the Yesterday, Today, and Tomorrow shortcuts. 3. Verify that past dates and dates occupied by a booking or checkout cannot be selected and display the appropriate marker and explanation. 4. Use the keyboard to open the calendar, select a date range, and navigate by day, week, month, and year. 5. Verify that closing the calendar returns focus to the booking period field. 6. Run: `yarn cypress run --component --browser electron --spec 't/cypress/component/BookingCalendar_spec.ts, t/cypress/component/useBookingCalendarMaps_spec.ts'` Sponsored-by: Büchereizentrale Schleswig-Holstein <https://www.bz-sh.de/> Assisted-by: Opus 4.8 (Anthropic) Assisted-by: GPT 5.6 (OpenAI) Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk> Signed-off-by: Jacob O'Mara <jacob.omara@openfifth.co.uk> Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #151 from Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> --- Created attachment 205503 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=205503&action=edit Bug 41129: Allow assigned unbookable items when editing bookings An item or its effective item type can become unbookable after the item has been assigned to a booking. The availability endpoint then rejects the item, and updating the booking can report a false capacity clash. Allow the assigned item only when excluded_booking_id identifies its booking. Include that item in the capacity calculation, but exclude bookings and checkouts for items outside the resulting item pool. Return a 400 response when an update has a genuine booking clash. Test plan: 1. Create a booking assigned to a specific item. 2. Make the item unbookable and edit the booking. 3. Verify that the assigned item remains selected and its availability request succeeds. 4. Change the booking period and select Update. 5. Verify that the existing booking is updated and no duplicate is created. 6. Repeat the test after making the item's effective item type unbookable. 7. Verify that another unbookable item cannot be selected for the booking. 8. Create a conflicting booking for the assigned item and verify that the update is rejected as a booking clash. 9. Run: `prove -v t/db_dependent/Koha/Biblio.t t/db_dependent/Koha/Biblio/Availability/Booking.t t/db_dependent/api/v1/biblios.t t/db_dependent/api/v1/bookings.t` Sponsored-by: Büchereizentrale Schleswig-Holstein <https://www.bz-sh.de/> Assisted-by: GPT 5.6 (OpenAI) Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk> Signed-off-by: Jacob O'Mara <jacob.omara@openfifth.co.uk> Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41129 --- Comment #153 from Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> --- Created attachment 205505 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=205505&action=edit Bug 41129: (QA follow-up) Restore Escape-to-close on the booking modal Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk> Signed-off-by: Jacob O'Mara <jacob.omara@openfifth.co.uk> Signed-off-by: Paul Derscheid <paul.derscheid@lmscloud.de> Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk> -- You are receiving this mail because: You are watching all bug changes.