Bug 41149

Summary: Spinner/loader does not disappear when a renewal fails with AllowRenewalOnHoldOverride set to dont allow
Product: Koha Reporter: Lucas Gass (lukeg) <lucas>
Component: CirculationAssignee: Lucas Gass (lukeg) <lucas>
Status: Signed Off --- QA Contact: Testopia <testopia>
Severity: normal    
Priority: P5 - low CC: gmcharlt, kyle
Version: Main   
Hardware: All   
OS: All   
GIT URL: Initiative type: ---
Sponsorship status: --- Crowdfunding goal: 0
Patch complexity: Trivial patch Documentation contact:
Documentation submission: Text to go in the release notes:
Version(s) released in:
Circulation function:
Attachments: Bug 41149: Replace replaceWith() with empty() and append()
Bug 41149: Replace replaceWith() with empty() and append()

Description Lucas Gass (lukeg) 2025-10-30 20:14:53 UTC
To recreate:

1. Set AllowRenewalOnHoldOverride to 'Dont allow'
2. Set  OverduesBlockRenewing to "block renewing"
3. Checkout an item and mark it as overdue.
4. Place a hold on that item to someone else 
5. Now go look at the issues table for that patron/item
6. Check "Override renewal restrictions:"
7. Now a checkbox should appear in the "Renew" column, check it.
8. Click "Renew selected items"
9. You should correctly see "Renew failed: on hold" with a spinning icon that never goes away. 

The fact that the spinning icon doesn't go away is confusing to staff. It makes it seem like something is still happening or something is wrong, neither are the case.
Comment 1 Lucas Gass (lukeg) 2025-10-30 20:26:58 UTC Comment hidden (obsolete)
Comment 2 Owen Leonard 2025-10-31 12:56:30 UTC
Created attachment 188824 [details] [review]
Bug 41149: Replace replaceWith() with empty() and append()

To test:

1. Set AllowRenewalOnHoldOverride to 'Dont allow'
2. Set  OverduesBlockRenewing to "block renewing"
3. Checkout an item and mark it as overdue.
4. Place a hold on that item to someone else
5. Now go look at the issues table for that patron/item
6. Check "Override renewal restrictions:"
7. Now a checkbox should appear in the "Renew" column, check it.
8. Click "Renew selected items"
9. After the "Renew failed: on hold" appears the spinning icon should
   disappear.

Signed-off-by: Owen Leonard <oleonard@myacpl.org>