Bug 41149 - Spinner/loader does not disappear when a renewal fails with AllowRenewalOnHoldOverride set to dont allow
Summary: Spinner/loader does not disappear when a renewal fails with AllowRenewalOnHol...
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:
Depends on:
Blocks:
 
Reported: 2025-10-30 20:14 UTC by Lucas Gass (lukeg)
Modified: 2025-10-31 12:56 UTC (History)
2 users (show)

See Also:
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() (987 bytes, patch)
2025-10-30 20:26 UTC, Lucas Gass (lukeg)
Details | Diff | Splinter Review
Bug 41149: Replace replaceWith() with empty() and append() (1.48 KB, patch)
2025-10-31 12:56 UTC, Owen Leonard
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
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>