gitea
/
Koha-clms
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Bug 42868: (follow-up) Fix stale comment referencing removed UTC boundary logic 

The comment above the day-boundary calculation still described the
previous dayjs.utc() approach, which was replaced by dayjs.tz() in the
parent commit. Update it to describe the current behaviour so it
doesn't mislead future maintainers into re-introducing the timezone
shift bug this patch fixed.

Signed-off-by: Pedro Amorim <pedro.amorim@openfifth.co.uk>
This commit is contained in:
Martin Renvoize 2026-07-01 13:01:34 +01:00 committed by Pedro Amorim
parent 084660a6df
commit 5a7c328475
No known key found for this signature in database
GPG Key ID: 06B75BA7F3894D55
1 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
koha-tmpl/intranet-tmpl/prog/js/modals/place_booking.js
View File

@ -959,9 +959,10 @@ $("#placeBookingModal").on("show.bs.modal", function (e) {
// Range set, update hidden fields and set available items
else if (selectedDates[0] && selectedDates[1]) {
// set form fields from picker
// Extract local date and send as explicit UTC day boundaries
// This preserves the user's selected DATE regardless of browser timezone
// Using dayjs.utc() ensures startOf/endOf operate in UTC, not browser TZ
// Send day boundaries anchored to the library's configured
// timezone ($timezone()), not UTC or the browser's local timezone.
// This preserves the user's selected DATE through the server's
// round-trip conversion back to library time (see Bug 42868).
let startDate = dayjs(selectedDates[0]).format(
"YYYY-MM-DD"
);