Attachment 30645 Details for Bug 8539 – [SIGNED OFF] Bug 8539: Library transfer limits page has incorrect permission

[patch] [SIGNED OFF] Bug 8539: Library transfer limits page has incorrect permission

SIGNED-OFF-Bug-8539-Library-transfer-limits-page-h.patch (text/plain), 1.37 KB, created by Nick Clemens (kidclamp) on 2014-08-09 05:56:40 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Nick Clemens (kidclamp)

Created: 2014-08-09 05:56:40 UTC

Size: 1.37 KB

patch

obsolete

>From 87d50a210703742e6d82bce5655f011776953d65 Mon Sep 17 00:00:00 2001
>From: Katrin Fischer <Katrin.Fischer.83@web.de>
>Date: Sat, 12 Jul 2014 14:22:53 +0200
>Subject: [PATCH] [SIGNED OFF] Bug 8539: Library transfer limits page has
> incorrect permission
>
>Without this patch the library transfer page required the
>'borrowers' permission, which is not quite right for a page
>in the administration module.
>
>This patch changes the permission to 'parameters_remaining_permissions'
>
>To test:
>Verify the page is accessbile with:
>- superlibrarian
>- administration (full)
>- administration, remaining permissions (without manage_circ_rules)
>Verify the page is not accessible without all of those permissions.
>
>Signed-off-by: Nick Clemens <nick@quecheelibrary.org>
>---
> admin/branch_transfer_limits.pl |    2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/admin/branch_transfer_limits.pl b/admin/branch_transfer_limits.pl
>index 4aa41dd..bd4cb24 100755
>--- a/admin/branch_transfer_limits.pl
>+++ b/admin/branch_transfer_limits.pl
>@@ -35,7 +35,7 @@ my ($template, $loggedinuser, $cookie)
>     = get_template_and_user({template_name => "admin/branch_transfer_limits.tt",
> 			     query => $input,
> 			     type => "intranet",
>-			     flagsrequired => {borrowers => 1},
>+                 flagsrequired => {parameters => 'parameters_remaining_permissions'},
> 			     debug => 1,
> 			     });
> 
>-- 
>1.7.10.4

Actions: View | Diff | Splinter Review

Attachments on bug 8539: 29658 | 30645 | 30859