From 145ea9d1606db3e0397fac8f92f27d7645c3c821 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Tue, 14 Jul 2015 16:44:37 +0100 Subject: [PATCH] Bug 14528: Inconsistency in permissions check when listing shelves The svc script needed borrowers when the shelves.pl needed catalogue. There is no sense to have different permissions here. Test plan: 1/ Remove the borrowers permission to a user 2/ Use this user to log in 3/ Go on the shelf list 4/ Without this patch, you should get an endless "Processing..." which is fixed if you apply it. Signed-off-by: Nick Clemens Signed-off-by: Katrin Fischer Signed-off-by: Tomas Cohen Arazi (cherry picked from commit 122d41816bde4e5111d51d4937c240f343ad253b) Signed-off-by: Chris Cormack --- svc/virtualshelves/search | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/svc/virtualshelves/search b/svc/virtualshelves/search index ef1448cec8..5e991fc630 100755 --- a/svc/virtualshelves/search +++ b/svc/virtualshelves/search @@ -17,7 +17,7 @@ my ($template, $user, $cookie) = get_template_and_user({ query => $input, type => "intranet", authnotrequired => 0, - flagsrequired => { borrowers => 1 } + flagsrequired => { catalogue => 1 } }); my $shelfname = $input->param('shelfname'); -- 2.39.5