Bug 25606

Summary: Adds "Remove all reserves" button to course details
Product: Koha Reporter: Joe Sikowitz <joe>
Component: Course reservesAssignee: Joe Sikowitz <joe>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: joe, lucas, martin.renvoize, nick
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
20.11.00
Circulation function:
Attachments: Bug 25606: Add remove all reserves button to course details page
Bug 25606: Remove all reserve items button
Bug 25606: Remove all reserve items button
Bug 25606: (QA follow-up) Don't show remove all button if no reserves

Description Joe Sikowitz 2020-05-26 14:40:42 UTC
Adds a "Remove all reserves" button to the course details screen for each course reserve. It allows for quick removal of courses with many reserve items attached.
Comment 1 Joe Sikowitz 2020-05-26 14:43:17 UTC
Created attachment 105360 [details] [review]
Bug 25606: Add remove all reserves button to course details page
Comment 2 Lucas Gass (lukeg) 2020-06-01 22:38:45 UTC
Hey Joe,

This seems like a great enhancement! Would you mind adding a test plan to this patch?
Comment 3 Joe Sikowitz 2020-06-02 13:17:36 UTC
Test plan
1. Install patch
2. Log in as user with permission to delete reserves
3. View a course reserve record with multiple items attached
4. There should now be a button that says "- Batch remove reserves"
5. When you click the button a dialog box should appear with a warning
6. Click "Ok"
7. All course reserves should be removed from the record
8. Add an item to more than course record
9. Redo 3-7
10. Items should have been removed from that course but remain on all other courses
11. Log in as a user without delete reserves permission
12. You should not see the batch remove option
Comment 4 Lucas Gass (lukeg) 2020-06-02 14:34:13 UTC
Created attachment 105494 [details] [review]
Bug 25606: Remove all reserve items button

Adds button to course records to remove all attached reserve items. A warning dialog box will appear before executed.

Test plan
1. Install patch
2. Log in as user with permission to delete reserves
3. View a course reserve record with multiple items attached
4. There should now be a button that says "- Batch remove reserves"
5. When you click the button a dialog box should appear with a warning
6. Click "Ok"
7. All course reserves should be removed from the record
8. Add an item to more than course record
9. Redo 3-7
10. Items should have been removed from that course but remain on all other courses
11. Log in as a user without delete reserves permission
12. You should not see the batch remove option

Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
Comment 5 Nick Clemens (kidclamp) 2020-08-06 18:20:25 UTC
Created attachment 107904 [details] [review]
Bug 25606: Remove all reserve items button

Adds button to course records to remove all attached reserve items. A warning dialog box will appear before executed.

Test plan
1. Install patch
2. Log in as user with permission to delete reserves
3. View a course reserve record with multiple items attached
4. There should now be a button that says "- Batch remove reserves"
5. When you click the button a dialog box should appear with a warning
6. Click "Ok"
7. All course reserves should be removed from the record
8. Add an item to more than course record
9. Redo 3-7
10. Items should have been removed from that course but remain on all other courses
11. Log in as a user without delete reserves permission
12. You should not see the batch remove option

Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Comment 6 Nick Clemens (kidclamp) 2020-08-06 18:20:28 UTC
Created attachment 107905 [details] [review]
Bug 25606: (QA follow-up) Don't show remove all button if no reserves

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Comment 7 Jonathan Druart 2020-08-07 14:55:56 UTC
Pushed to master for 20.11, thanks to everybody involved!
Comment 8 Lucas Gass (lukeg) 2020-08-14 22:30:19 UTC
enhancement will not be backported to 20.05.x series
Comment 9 Martin Renvoize (ashimema) 2020-09-27 06:40:41 UTC
*** Bug 26332 has been marked as a duplicate of this bug. ***