Bug 38465 - Cannot schedule a curbside pickup
Summary: Cannot schedule a curbside pickup
Status: Signed Off
Alias: None
Product: Koha
Classification: Unclassified
Component: Staff interface (show other bugs)
Version: unspecified
Hardware: All All
: P5 - low minor
Assignee: Jonathan Druart
QA Contact: Testopia
URL:
Keywords:
Depends on: 36454
Blocks:
  Show dependency treegraph
 
Reported: 2024-11-16 07:17 UTC by Jonathan Druart
Modified: 2024-12-01 17:13 UTC (History)
4 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:


Attachments
Bug 38465: (bug 36454 follow-up) Fix curbside pickup's patron autocomplete (1.54 KB, patch)
2024-11-18 08:40 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38465: (bug 36454 follow-up) Fix curbside pickup's patron autocomplete (1.59 KB, patch)
2024-12-01 17:11 UTC, David Nind
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Jonathan Druart 2024-11-16 07:17:49 UTC
When searching for patron the autocomplete code raised a JS error in the console:

Uncaught TypeError: m.tz is not a function
Comment 1 Jonathan Druart 2024-11-18 08:40:49 UTC
Created attachment 174661 [details] [review]
Bug 38465: (bug 36454 follow-up) Fix curbside pickup's patron autocomplete

js-date-format.inc already include those 2 JS files. If had them in
curbside pickups template because it was the first using dayjs.
Now it's included in js_includes.inc.

Test plan:
Configure curbside pickup for your library and go to circ/curbside_pickups.pl
Last tab, search for a patron
Without this patch you get a JS error:
 Uncaught TypeError: m.tz is not a function
With this patch applied the patron autocomplete is fixed
Comment 2 Pedro Amorim 2024-11-18 12:43:34 UTC
This is fixed by the follow-up in bug 36454:
https://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html&bug=36454&attachment=174671

Can't reproduce the issue after applying the patch from there.
Comment 3 Jonathan Druart 2024-11-18 12:49:52 UTC
(In reply to Pedro Amorim from comment #2)
> This is fixed by the follow-up in bug 36454:
> https://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.
> html&bug=36454&attachment=174671
> 
> Can't reproduce the issue after applying the patch from there.

Yes, but it is still "valid" in the sense that the patch removes 2 includes that are already included.
Comment 4 Jonathan Druart 2024-11-18 12:50:35 UTC
No longer critical however.

New test plan: confirm that the curbide pickup module is still working correctly, ie. you can schedule a new pickup.
Comment 5 David Nind 2024-12-01 17:11:54 UTC
Created attachment 175041 [details] [review]
Bug 38465: (bug 36454 follow-up) Fix curbside pickup's patron autocomplete

js-date-format.inc already include those 2 JS files. If had them in
curbside pickups template because it was the first using dayjs.
Now it's included in js_includes.inc.

Test plan:
Configure curbside pickup for your library and go to circ/curbside_pickups.pl
Last tab, search for a patron
Without this patch you get a JS error:
 Uncaught TypeError: m.tz is not a function
With this patch applied the patron autocomplete is fixed

Signed-off-by: David Nind <david@davidnind.com>
Comment 6 David Nind 2024-12-01 17:13:02 UTC
Testing notes (using KTD):

1. Curbside pickup continues to work as expected.