Bug 37618

Summary: Select patron's, then item's home library by default for pickup location when creating bookings
Product: Koha Reporter: Paul Derscheid <paul.derscheid>
Component: CirculationAssignee: Paul Derscheid <me>
Status: Needs Signoff --- QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: gmcharlt, kyle.m.hall, martin.renvoize
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function: bookings
Bug Depends on: 36120    
Bug Blocks:    
Attachments: Bug 37618: Select patron's, then item's home library by default for pickup location when creating bookings
Bug 37618: Select patron's, then item's home library by default for pickup location when creating bookings
Bug 37618: (follow-up) Inline single use function containsAny and remove definition
Bug 37618: (follow-up) Remove redundant api call

Description Paul Derscheid 2024-08-12 10:07:54 UTC
When working with many bookings per day, always selecting the pickup location can become cumbersome.

I'd suggest using the patron's home library as a preselection within the pickup library field in the bookings creation modal and to fall back onto the item's home library if the patron's home library is not available.

Would you also consider this a sane default?
Comment 1 Paul Derscheid 2024-08-12 16:26:33 UTC
Created attachment 170235 [details] [review]
Bug 37618: Select patron's, then item's home library by default for pickup location when creating bookings
Comment 2 Martin Renvoize 2024-08-15 15:01:32 UTC
I like the direction here.. will swing back around to it once I've wrapped my head around a few other bits in the bookings tree that are on my list for today.
Comment 3 Martin Renvoize 2024-09-12 11:44:16 UTC
Created attachment 171384 [details] [review]
Bug 37618: Select patron's, then item's home library by default for pickup location when creating bookings

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 4 Martin Renvoize 2024-09-12 11:44:56 UTC
Just a rebase so far.. still testing :)
Comment 5 Paul Derscheid 2024-09-12 13:23:14 UTC
Created attachment 171394 [details] [review]
Bug 37618: (follow-up) Inline single use function containsAny and remove definition
Comment 6 Paul Derscheid 2024-09-12 13:23:17 UTC
Created attachment 171395 [details] [review]
Bug 37618: (follow-up) Remove redundant api call