Bug 13838

Summary: Redirect to 'expired holds' tab after cancelling a hold
Product: Koha Reporter: Katrin Fischer <katrin.fischer>
Component: CirculationAssignee: Kyle M Hall (khall) <kyle>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: minor    
Priority: P5 - low CC: cbrannon, f.demians, gmcharlt, jonathan.druart, julian.maurice, kyle.m.hall, kyle, sally.healey, tmisilo
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:
Attachments: Bug 13838 - Redirect to 'expired holds' tab after cancelling a hold
Bug 13838 - Redirect to 'expired holds' tab after cancelling a hold
Bug 13838 - Redirect to 'expired holds' tab after cancelling a hold
Bug 13838 - Redirect to 'expired holds' tab after cancelling a hold

Description Katrin Fischer 2015-03-16 10:47:12 UTC
When working with the 'holds awaiting pickup' report, after cancelling a hold that has not been picked up, you are redirected to the first tab. It would be nice if the redirect led you to the second tab to save a click when working on cancelling the holds.

To test:
- Go to the holds awaiting pickup report from the circulation start page
- you will need some holds that appear on the 'hold(s) over' tab
- cancel a hold from this tab
- verify you are redirected to the report page with the first tab 'Hold(s) waiting' selected
Comment 1 Katrin Fischer 2015-04-20 19:31:46 UTC
*** Bug 14028 has been marked as a duplicate of this bug. ***
Comment 2 Kyle M Hall (khall) 2015-04-20 19:36:14 UTC Comment hidden (obsolete)
Comment 3 Jonathan Druart 2015-04-28 15:42:27 UTC
The test plan works, but I am redirected to the Hold(s) over tab even if I deleted a waiting hold (first tab).
Comment 4 Kyle M Hall (khall) 2015-12-09 15:01:55 UTC Comment hidden (obsolete)
Comment 5 Tom Misilo 2015-12-09 19:29:07 UTC Comment hidden (obsolete)
Comment 6 Tom Misilo 2015-12-09 19:32:08 UTC
Jonathan,

I tested your case as well, and it does not appear to do that with the latest patch.
Comment 7 Jonathan Druart 2015-12-11 10:30:51 UTC
Created attachment 45596 [details] [review]
Bug 13838 - Redirect to 'expired holds' tab after cancelling a hold

When canceling a hold from the holds over tab, you are redirected back to the holds waiting tab. Instead, we should the librarian should continue to see the list of holds past the waiting length.

Test Plan:
1) Apply this patch
2) Cancel a hold from the "Holds over" tab on circ/waitingreserves.pl
3) Note you are redirected back to the holds over tab

Signed-off-by: Tom Misilo <misilot@fit.edu>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 8 Kyle M Hall (khall) 2015-12-11 16:44:30 UTC
Pushed to master!
Comment 9 Julian Maurice 2015-12-14 07:40:43 UTC
Patch pushed to 3.22.x, will be in 3.22.1
Comment 10 Frédéric Demians 2015-12-14 17:59:10 UTC
This patch has been pushed to 3.20.x, will be in 3.20.7.