https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40755 Jonathan Druart <jonathan.druart@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #186167|0 |1 is obsolete| | --- Comment #3 from Jonathan Druart <jonathan.druart@gmail.com> --- Created attachment 186188 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=186188&action=edit Bug 40755: Fix typo in waiting_holds.inc Fix a typo that caused an error 500 on the holds awaiting pickup page when there are holds with cancellation requests. To test: 1. Enable hold cancellation requests from the OPAC i. Go to Administration > Circulation and Fines rules ii. In the "Default waiting hold cancellation policy" section, add the rule: All / All / Yes 2. Place at least two holds for pickup at the logged-in library (for ease of testing, place both holds for the same patron) 3. Check in items for both holds and confirm to set holds to Waiting 4. Log into the OPAC as the patron from step 2 5. Go to the patron's holds tab and request cancellation for both holds 6. In the staff interface, go to Circulation > Holds awaiting pickup ==> Error 500! :( 7. Apply patch and refresh the page ==> The page loads! :) Bonus testing (KTD only): 1. Edit one of the cancellation request dates, so that they are not both the same date: ktd --shell koha-mysql kohadev SELECT * FROM hold_cancellation_requests; UPDATE hold_cancellation_requests SET creation_date=<date> WHERE hold_cancellation_request_id = <id>; 2. Refresh the Holds awaiting pickup page again, and switch to the Holds with cancellation requests tab 3. Sort by hold cancellation request date and confirm the sorting works correctly Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> -- You are receiving this mail because: You are watching all bug changes.