From 8b52957a544a16e512c95c1aff6c068fc05c07f2 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Johnny=20Marie=CC=81thoz?= Date: Tue, 3 Nov 2020 09:21:37 +0100 Subject: [PATCH] permissions: fix system librarian role only MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit A system librarian should have all the librarian rights. * Removes the librarian role to the system librarian of Martigny in the test fixtures to test this specific case. * Closes #1340. Co-Authored-by: Johnny MarieĢthoz --- rero_ils/permissions.py | 3 ++- tests/data/data.json | 3 +-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/rero_ils/permissions.py b/rero_ils/permissions.py index 0646527ce4..20d349344e 100644 --- a/rero_ils/permissions.py +++ b/rero_ils/permissions.py @@ -28,7 +28,8 @@ from .modules.patrons.api import Patron request_item_permission = Permission(RoleNeed('patron')) -librarian_permission = Permission(RoleNeed('librarian')) +librarian_permission = Permission( + RoleNeed('librarian'), RoleNeed('system_librarian')) admin_permission = Permission(RoleNeed('admin')) editor_permission = Permission(RoleNeed('editor'), RoleNeed('admin')) diff --git a/tests/data/data.json b/tests/data/data.json index 0dbd570e3b..de1ab635b5 100644 --- a/tests/data/data.json +++ b/tests/data/data.json @@ -2767,8 +2767,7 @@ "username": "simonetta", "phone": "+41324993585", "roles": [ - "system_librarian", - "librarian" + "system_librarian" ], "patron": { "expiration_date": "2023-10-07",