Attachment 35267 Details for Bug 13584 – Bug 13584 - Able to view menu for Purchase Suggestions even when user does not have permission

[patch] Bug 13584 - Able to view menu for Purchase Suggestions even when user does not have permission

Bug-13584---Able-to-view-menu-for-Purchase-Suggest.patch (text/plain), 1.97 KB, created by Thomas Wright on 2015-01-14 23:29:54 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Thomas Wright

Created: 2015-01-14 23:29:54 UTC

Size: 1.97 KB

patch

obsolete

>From ae8ad4917756434b6dda4aa78e382894c4988f26 Mon Sep 17 00:00:00 2001
>From: Thomas <tomsStudy@gmail.com>
>Date: Wed, 14 Jan 2015 23:26:36 +0000
>Subject: [PATCH] Bug 13584 - Able to view menu for Purchase Suggestions even
> when user does not have permission
>
>Added check for borrowers that stops the Purchase Suggestions link being shown. This could be done through the css but that function currently appears to be broken
>
>Testing Plan:
>
>-Search for a patron with an account that does not have the 'borrowers' permission
>*Purchase suggestions item should not be seen in the menu.
>
>-Change the accounts permissions so they have the 'borrowers' permission
>*Purchase suggestions item should be present in the menu.
>---
> koha-tmpl/intranet-tmpl/prog/en/includes/circ-menu.inc |    5 ++++-
> 1 file changed, 4 insertions(+), 1 deletion(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/circ-menu.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/circ-menu.inc
>index 2fe0a68..eac4a8f 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/includes/circ-menu.inc
>+++ b/koha-tmpl/intranet-tmpl/prog/en/includes/circ-menu.inc
>@@ -82,8 +82,11 @@
>     [% IF (  statisticsview ) %]<li class="active">[% ELSE %]<li>[% END %]<a href="/cgi-bin/koha/members/statistics.pl?borrowernumber=[% borrowernumber %]">Statistics</a></li>
>     [% IF EnableBorrowerFiles %]
>         [% IF ( borrower_files ) %]<li class="active">[% ELSE %]<li>[% END %]<a href="/cgi-bin/koha/members/files.pl?borrowernumber=[% borrowernumber %]">Files</a></li>
>+    [% END %] 
>+
>+    [% IF ( CAN_user_borrowers ) %]
>+        [% IF ( suggestionsview ) %]<li class="active">[% ELSE %]<li>[% END %]<a href="/cgi-bin/koha/members/purchase-suggestions.pl?borrowernumber=[% borrowernumber %]">Purchase suggestions</a></li>
>     [% END %]
>-    [% IF (  suggestionsview ) %]<li class="active">[% ELSE %]<li>[% END %]<a href="/cgi-bin/koha/members/purchase-suggestions.pl?borrowernumber=[% borrowernumber %]">Purchase suggestions</a></li>
> </ul></div>
> [% END %]
> 
>-- 
>1.7.10.4

Actions: View | Diff | Splinter Review

Attachments on bug 13584: 35267 | 35285 | 35298 | 35299