From d4cbb8c173f564ce17c892bddf6acc91b89b0043 Mon Sep 17 00:00:00 2001
From: Kyle M Hall <kyle@bywatersolutions.com>
Date: Tue, 17 Nov 2020 07:00:18 -0500
Subject: [PATCH] Bug 20256: (QA follow-up) Implement for course reserves

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
---
 course_reserves/course-details.pl             |  1 +
 .../modules/course_reserves/course-details.tt | 20 ++++++++++---------
 2 files changed, 12 insertions(+), 9 deletions(-)

diff --git a/course_reserves/course-details.pl b/course_reserves/course-details.pl
index e5c7efe5b2..c5d74f950e 100755
--- a/course_reserves/course-details.pl
+++ b/course_reserves/course-details.pl
@@ -66,6 +66,7 @@ my $course_reserves = GetCourseReserves(
 $template->param(
     course          => $course,
     course_reserves => $course_reserves,
+    user            => Koha::Patrons->find( $loggedinuser ),
 );
 
 output_html_with_http_headers $cgi, $cookie, $template->output;
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 12efce92bb..a0e41144d5 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
@@ -274,16 +274,18 @@
                             </td>
 
                             [% IF CAN_user_coursereserves_add_reserves || CAN_user_coursereserves_delete_reserves %]
-                                <td class="actions">
-                                    [% IF CAN_user_coursereserves_add_reserves %]
-                                        <a class="btn btn-default btn-xs" href="add_items.pl?course_id=[% course.course_id | html %]&amp;itemnumber=[% cr.item.itemnumber | html %]&amp;biblionumber=[% cr.biblio.biblionumber | html %]&amp;action=lookup&amp;return=[% course.course_id | html %]&amp;is_edit=1"><i class="fa fa-pencil"></i> Edit</a>
-                                    [% END %]
+                                [% IF user.can_edit_item( cr.item ) %]
+                                    <td class="actions">
+                                        [% IF CAN_user_coursereserves_add_reserves %]
+                                            <a class="btn btn-default btn-xs" href="add_items.pl?course_id=[% course.course_id | html %]&amp;itemnumber=[% cr.item.itemnumber | html %]&amp;action=lookup&amp;return=[% course.course_id | html %]&amp;is_edit=1"><i class="fa fa-pencil"></i> Edit</a>
+                                        [% END %]
 
-                                    [% IF CAN_user_coursereserves_delete_reserves %]
-                                        <a class="btn btn-default btn-xs delete_item" href="course-details.pl?course_id=[% course.course_id | html %]&amp;action=del_reserve&amp;cr_id=[% cr.cr_id | html %]">
-                                        <i class="fa fa-trash"></i> Remove</a>
-                                    [% END %]
-                                </td>
+                                        [% IF CAN_user_coursereserves_delete_reserves %]
+                                            <a class="btn btn-default btn-xs delete_item" href="course-details.pl?course_id=[% course.course_id | html %]&amp;action=del_reserve&amp;cr_id=[% cr.cr_id | html %]">
+                                            <i class="fa fa-trash"></i> Remove</a>
+                                        [% END %]
+                                    </td>
+                                [% END %]
                             [% END %]
 
                         </tr>
-- 
2.20.1