From 5cf05ff53d7284500a2653f7b0988d6a46a7ea13 Mon Sep 17 00:00:00 2001
From: Joseph Sikowitz <joe@flo.org>
Date: Sat, 23 May 2020 17:45:51 -0400
Subject: [PATCH] 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>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
---
 course_reserves/course-details.pl                      | 10 +++++++++-
 .../prog/en/modules/course_reserves/course-details.tt  |  7 +++++++
 2 files changed, 16 insertions(+), 1 deletion(-)

diff --git a/course_reserves/course-details.pl b/course_reserves/course-details.pl
index af1eb62337..7604e2377c 100755
--- a/course_reserves/course-details.pl
+++ b/course_reserves/course-details.pl
@@ -34,7 +34,7 @@ my $action = $cgi->param('action') || '';
 my $course_id = $cgi->param('course_id');
 
 my $flagsrequired;
-$flagsrequired->{coursereserves} = 'delete_reserves' if ( $action eq 'del_reserve' );
+$flagsrequired->{coursereserves} = 'delete_reserves' if ( $action eq 'del_reserve' or $action eq 'rm_all' );
 
 my $tmpl = ($course_id) ? "course-details.tt" : "invalid-course.tt";
 my ( $template, $loggedinuser, $cookie ) = get_template_and_user(
@@ -49,6 +49,14 @@ my ( $template, $loggedinuser, $cookie ) = get_template_and_user(
 if ( $action eq 'del_reserve' ) {
     DelCourseReserve( cr_id => scalar $cgi->param('cr_id') );
 }
+elsif ( $action eq 'rm_all' ) {
+    my $course_res = GetCourseReserves(course_id => $course_id);
+    foreach my $cr (@$course_res) {
+        if(exists $cr->{'cr_id'}) {
+            DelCourseReserve(cr_id => $cr->{cr_id});
+        }
+    }
+}
 
 my $course          = GetCourse($course_id);
 my $course_reserves = GetCourseReserves(
diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/course_reserves/course-details.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/course_reserves/course-details.tt
index 1ad1fa7f04..9173d9bc00 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/course_reserves/course-details.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/course_reserves/course-details.tt
@@ -36,6 +36,9 @@
                     <a class="btn btn-default" id="add_items" href="/cgi-bin/koha/course_reserves/add_items.pl?course_id=[% course.course_id | html %]"><i class="fa fa-plus"></i> Add reserves</a>
                     <a class="btn btn-default" id="add_items" href="/cgi-bin/koha/course_reserves/batch_add_items.pl?course_id=[% course.course_id | html %]"><i class="fa fa-plus"></i> Batch add reserves</a>
                 [% END %]
+                [% IF CAN_user_coursereserves_delete_reserves %]
+                    <a class="btn btn-default" id="rm_items" href="/cgi-bin/koha/course_reserves/course-details.pl?course_id=[% course.course_id | html %]&amp;action=rm_all"><i class="fa fa-minus"></i> Remove all reserves</a>
+                [% END %]
                 [% IF ( CAN_user_coursereserves_manage_courses ) %]
                     <a class="btn btn-default" id="edit_course" href="/cgi-bin/koha/course_reserves/course.pl?course_id=[% course.course_id | html %]"><i class="fa fa-pencil"></i> Edit course</a>
                 [% END %]
@@ -284,6 +287,10 @@
                 return confirmDelete(_("Are you sure you want to remove this item from the course?"));
             });
 
+            $("#rm_items").click(function(){
+                return confirmDelete(_("Are you sure you want to remove all items from the course?"));
+            });
+
             $("#delete_course").click(function(){
                 [% SET count = course_reserves.size || 0 %]
                 [% IF count == 1 %]
-- 
2.39.5