@@ -, +, @@ permission - Create a staff user with place_holds permission - Check the result list, you should see 2 links to the holds page: - 'Place holds' on top of the results list - 'Holds (x)' at the bottom of each entry in the results list - Verify both links work - Create a staff user without place_holds_permission - Without the patch you'll see the second link, but it will lead to a permission error - With the patch you'll still see the 'Holds (x)', but it will no longer be a link. - Go to the detail page of a record with an existing hold. - Repeat test with both staff users. - One time the Hold information in the bibliographic information should show as a link, the other time as a simple text. --- koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt | 6 +++++- koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/results.tt | 10 +++++++--- 2 files changed, 12 insertions(+), 4 deletions(-) --- a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt +++ a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt @@ -112,7 +112,11 @@ Holds: - [% holdcount %] + [% IF CAN_user_reserveforothers_place_holds %] + [% holdcount %] + [% ELSE %] + [% holdcount %] + [% END %] [% END %] --- a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/results.tt +++ a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/results.tt @@ -119,7 +119,7 @@ [% END %] - [% IF ( CAN_user_reserveforothers && DisplayMultiPlaceHold ) %] + [% IF ( CAN_user_reserveforothers_place_holds && DisplayMultiPlaceHold ) %] [% IF ( holdfor ) %]
@@ -441,8 +441,12 @@ [% IF ( SEARCH_RESULT.norequests ) %] No holds allowed [% ELSE %] - Holds ([% Biblio.HoldsCount( SEARCH_RESULT.biblionumber ) %]) - [% IF ( holdfor ) %] | Place hold for [% holdfor_firstname %] [% holdfor_surname %] ([% holdfor_cardnumber %])[% END %] + [% IF CAN_user_reserveforothers_place_holds %] + Holds ([% Biblio.HoldsCount( SEARCH_RESULT.biblionumber ) %]) + [% IF ( holdfor ) %] | Place hold for [% holdfor_firstname %] [% holdfor_surname %] ([% holdfor_cardnumber %])[% END %] + [% ELSE %] + Holds ([% Biblio.HoldsCount( SEARCH_RESULT.biblionumber ) %]) + [% END %] [% END %] [% IF Koha.Preference('intranetbookbag') == 1 %] --