From 516aeb87006f4bd4713baca2775919de1302c045 Mon Sep 17 00:00:00 2001 From: Kyle M Hall Date: Thu, 27 Feb 2014 08:55:22 -0500 Subject: [PATCH] Bug 11866 - Staff side course reserves too restrictive A librarian with no course reserves permissions cannot view course reserves from the staff intranet, yet can from the OPAC. This doesn't make much sense. Librarians should not require any course reserves permissions to view courses and reserves from the staff intranet. Test Plan: 1) Log into staff intranet as a librarian with no course reserves permissions 2) Note you cannot view course reserves 3) Apply this patch 4) Note you can now view course reserves 5) Verify you cannot modify courses or course reserves --- course_reserves/course-reserves.pl | 1 - .../intranet-tmpl/prog/en/includes/header.inc | 2 -- .../en/modules/course_reserves/course-details.tt | 8 ++++---- 3 files changed, 4 insertions(+), 7 deletions(-) diff --git a/course_reserves/course-reserves.pl b/course_reserves/course-reserves.pl index b81bc94..4aa982a 100755 --- a/course_reserves/course-reserves.pl +++ b/course_reserves/course-reserves.pl @@ -35,7 +35,6 @@ my ( $template, $loggedinuser, $cookie ) = get_template_and_user( query => $cgi, type => "intranet", authnotrequired => 0, - flagsrequired => { coursereserves => '*' }, } ); diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/header.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/header.inc index b6bc660..a3489c3 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/includes/header.inc +++ b/koha-tmpl/intranet-tmpl/prog/en/includes/header.inc @@ -22,9 +22,7 @@ [% IF ( CAN_user_serials ) %]
  • Serials
  • [% END %] - [% IF ( CAN_user_coursereserves ) %]
  • Course reserves
  • - [% END %] [% IF ( CAN_user_reports ) %]
  • Reports
  • [% END %] 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 5e69702..ea58136 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 @@ -17,7 +17,7 @@ "aLengthMenu": [[10, 20, 50, 100, -1], [10, 20, 50, 100, "All"]], "iDisplayLength": 20, "aoColumnDefs": [ - { 'bSortable': false, 'aTargets': [ 9, 10, 11 ] } + { 'bSortable': false, 'aTargets': [ 'NoSort' ] } ] })); @@ -82,9 +82,9 @@ Library Staff note Public note - [% IF CAN_user_coursereserves_add_reserves %] [% END %] - [% IF CAN_user_coursereserves_delete_reserves %] [% END %] - Other course reserves + [% IF CAN_user_coursereserves_add_reserves %] [% END %] + [% IF CAN_user_coursereserves_delete_reserves %] [% END %] + Other course reserves -- 1.7.2.5