Bug 41345 - Regression: Clicking the 'Ignore' button on hold found modal for already-waiting hold does not dismiss the modal (again)
Summary: Regression: Clicking the 'Ignore' button on hold found modal for already-wait...
Status: Signed Off
Alias: None
Product: Koha
Classification: Unclassified
Component: Circulation (show other bugs)
Version: Main
Hardware: All All
: P5 - low normal
Assignee: Andrew Fuerste-Henry
QA Contact: Testopia
URL:
Keywords:
Depends on: 34596
Blocks:
  Show dependency treegraph
 
Reported: 2025-12-01 23:41 UTC by Andrew Fuerste-Henry
Modified: 2025-12-03 22:52 UTC (History)
5 users (show)

See Also:
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:


Attachments
Bug 41345: Restore ignore button functionality (1.81 KB, patch)
2025-12-01 23:48 UTC, Andrew Fuerste-Henry
Details | Diff | Splinter Review
Bug 41345: Restore ignore button functionality (1.86 KB, patch)
2025-12-03 22:48 UTC, David Nind
Details | Diff | Splinter Review

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