|
Description
Pete Edwards
2015-07-13 16:21:11 UTC
Created attachment 40995 [details] [review] 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. Created attachment 40996 [details] [review] 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. Created attachment 41090 [details] [review] 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 <nick@quecheelibrary.org> Created attachment 43083 [details] [review] [PASSED QA] 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 <nick@quecheelibrary.org> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Patch pushed to master. Thanks onathan! Pushed to 3.20.x will be in 3.20.5 |