Attachment 37096 Details for Bug 13890 – Bug 13890 - Switch from full serials permission to 'routing' permission for reordering routing lists

[patch] Bug 13890 - Switch from full serials permission to 'routing' permission for reordering routing lists

Bug-13890---Switch-from-full-serials-permission-to.patch (text/plain), 1.66 KB, created by Katrin Fischer on 2015-03-22 21:10:57 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Katrin Fischer

Created: 2015-03-22 21:10:57 UTC

Size: 1.66 KB

patch

obsolete

>From 99b19921ab16c75c82589871ea3b5bf2d85fb2ec Mon Sep 17 00:00:00 2001
>From: Katrin Fischer <Katrin.Fischer.83@web.de>
>Date: Sun, 22 Mar 2015 22:07:32 +0100
>Subject: [PATCH] Bug 13890 - Switch from full serials permission to 'routing'
> permission for reordering routing lists
>
>For changing the sequence of patrons on a routing list by changing
>one of the numbers via the pull downs, full serial permission is needed:
>
>To test:
>- Give your patron full serial permissions
>- Create a subscription and a routing list for it
>- Change the sequence on the routing list using the pull downs
>- Verify this works
>- Remove any of the serial sub permissions (claim serials or similar)
>- Verify changing the sequence now results in a permission error
>- Apply patch
>- Verify that the permission error is gone now
>- Remove the routing permission
>- Verify the permission error is back (but this seems to make sense)
>
>In general more fixes would be required to make all routing related
>features depend on the routing permission, but I think this fixes
>the bug making it depend on the correct permission.
>---
> serials/reorder_members.pl | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/serials/reorder_members.pl b/serials/reorder_members.pl
>index ba19b8c..9ff633c 100755
>--- a/serials/reorder_members.pl
>+++ b/serials/reorder_members.pl
>@@ -29,7 +29,7 @@ my $subscriptionid = $query->param('subscriptionid');
> my $routingid      = $query->param('routingid');
> my $rank           = $query->param('rank');
> 
>-checkauth( $query, 0, { serials => 1 }, 'intranet' );
>+checkauth( $query, 0, { serials => 'routing' }, 'intranet' );
> 
> reorder_members( $subscriptionid, $routingid, $rank );
> 
>-- 
>1.9.1

Actions: View | Diff | Splinter Review

Attachments on bug 13890: 37096 | 37099 | 37309