Bug 41345

Summary: Regression: Clicking the 'Ignore' button on hold found modal for already-waiting hold does not dismiss the modal (again)
Product: Koha Reporter: Andrew Fuerste-Henry <andrew>
Component: CirculationAssignee: Andrew Fuerste-Henry <andrew>
Status: Signed Off --- QA Contact: Testopia <testopia>
Severity: normal    
Priority: P5 - low CC: david, gmcharlt, kebliss, kyle, nick
Version: Main   
Hardware: All   
OS: All   
See Also: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39108
GIT URL: Initiative type: ---
Sponsorship status: --- Comma delimited list of Sponsors:
Crowdfunding goal: 0 Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
This fixes a regression when checking in an item. Clicking the "Ignore" option in the dialog box, when an item already has a waiting status, just reloaded the dialog box. Clicking the "Ignore" option now closes the dialog box and works as expected.
Version(s) released in:
Circulation function:
Bug Depends on: 34596    
Bug Blocks:    
Attachments: Bug 41345: Restore ignore button functionality
Bug 41345: Restore ignore button functionality

Description Andrew Fuerste-Henry 2025-12-01 23:41:04 UTC
To recreate:
1 - place a hold
2 - check in an item that can fill the hold
3 - in the hold found modal, click Ignore
4 - the modal dismisses and nothing changes on the hold (this is the desired behavior)
5 - check in the item again, click Confirm Hold, confirm your hold is now Waiting
6 - check in the item again, confirm the modal says "Item is already waiting"
7 - click Ignore, confirm modal reloads
8 - click Ignore, confirm modal reloads
9 - click Ignore, confirm modal reloads
Comment 1 Andrew Fuerste-Henry 2025-12-01 23:48:49 UTC
Created attachment 190066 [details] [review]
Bug 41345: Restore ignore button functionality

To test:
1 - place a hold
2 - check an item in to fill your hold, confirm so item goes to Waiting status
3 - check item in again, get Hold Already Waiting modal
4 - clck Ignore, confirm modal reloads immediately
5 - apply patch, restart_all
6 - reload checkin page
7 - check item in again, get Hold Already Waiting modal
8 - click Ignore, confirm modal goes away and doesn't return
Comment 2 David Nind 2025-12-03 22:48:04 UTC
Created attachment 190138 [details] [review]
Bug 41345: Restore ignore button functionality

To test:
1 - place a hold
2 - check an item in to fill your hold, confirm so item goes to Waiting status
3 - check item in again, get Hold Already Waiting modal
4 - clck Ignore, confirm modal reloads immediately
5 - apply patch, restart_all
6 - reload checkin page
7 - check item in again, get Hold Already Waiting modal
8 - click Ignore, confirm modal goes away and doesn't return

Signed-off-by: David Nind <david@davidnind.com>