Bug 40899 - When placing multiple holds at once the individual "Pickup location:" dropdowns do not update when changing the top level "Pickup at:" dropdown"
Summary: When placing multiple holds at once the individual "Pickup location:" dropdow...
Status: Signed Off
Alias: None
Product: Koha
Classification: Unclassified
Component: Circulation (show other bugs)
Version: Main
Hardware: All All
: P5 - low normal
Assignee: Lucas Gass (lukeg)
QA Contact: Testopia
URL:
Keywords: regression
Depends on:
Blocks:
 
Reported: 2025-09-29 16:48 UTC by Lucas Gass (lukeg)
Modified: 2025-10-06 09:42 UTC (History)
13 users (show)

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


Attachments
Bug 40899: Fix rendering of JSON in data-attribute with html filter (1.63 KB, patch)
2025-09-30 13:21 UTC, Lucas Gass (lukeg)
Details | Diff | Splinter Review
Bug 40899: Fix rendering of JSON in data-attribute with html filter (1.70 KB, patch)
2025-09-30 21:01 UTC, WPPL Sandboxes
Details | Diff | Splinter Review
Screenshot of Place holds modal (30.79 KB, image/jpeg)
2025-09-30 21:08 UTC, Chip Halvorsen
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Lucas Gass (lukeg) 2025-09-29 16:48:55 UTC
To recreate:

1. Place a hold on many items 
2. Change the "Pickup at:" dropdown" to something else 
2. Notice the "Pickup location:" dropdowns do not change with it 

This is a regression.
Comment 1 Lucas Gass (lukeg) 2025-09-30 13:07:05 UTC
The rendered data-pickup-locations attribute does not look right.
Comment 2 Lucas Gass (lukeg) 2025-09-30 13:21:01 UTC
Created attachment 187102 [details] [review]
Bug 40899: Fix rendering of JSON in data-attribute with html filter

To test:
1. Place a hold on many items
2. Change the "Pickup at:" dropdown" to something else
3. Notice the "Pickup location:" dropdowns do not change with it
4. APPLY PATCH
5. Try steps 1 & 2 again. This time the dropdpwns should adjust accordingly
Comment 3 WPPL Sandboxes 2025-09-30 21:01:47 UTC
Created attachment 187140 [details] [review]
Bug 40899: Fix rendering of JSON in data-attribute with html filter

To test:
1. Place a hold on many items
2. Change the "Pickup at:" dropdown" to something else
3. Notice the "Pickup location:" dropdowns do not change with it
4. APPLY PATCH
5. Try steps 1 & 2 again. This time the dropdpwns should adjust accordingly

Signed-off-by: Chip Halvorsen <chip.halvorsen@westlakelibrary.org>
Comment 4 Chip Halvorsen 2025-09-30 21:06:06 UTC
The patch is functioning as intended. However, the Pickup locations that are filled for each hold are not readable. The column is too narrow so only the first letter or two displays.
Comment 5 Chip Halvorsen 2025-09-30 21:08:35 UTC
Created attachment 187141 [details]
Screenshot of Place holds modal
Comment 6 CJ Lynce 2025-10-01 16:32:30 UTC
(In reply to Chip Halvorsen from comment #4)
> The patch is functioning as intended. However, the Pickup locations that are
> filled for each hold are not readable. The column is too narrow so only the
> first letter or two displays.

I submitted bug 40931 for this issue since it is a css template issue.
Comment 7 Jonathan Druart 2025-10-06 09:42:29 UTC
What caused the regression?