From 8d21602c1d5e1896d1329370c4be9934c6af0bda Mon Sep 17 00:00:00 2001 From: Kyle M Hall Date: Thu, 19 Sep 2013 12:18:11 -0400 Subject: [PATCH] [PASSED QA] Bug 10917 - Librarian cannot update hold pickup location without modify_holds_priority A librarian cannot modify a hold's pickup location unless he or she has the permission modify_holds_priority. This appears to be a bug and not by design. The reason the update fails is due to the priority not being passed to ModReserve. The priority is not displayed when a librarian does not have the modify_holds_priority permission. Test Plan: 1) Remove the modify_holds_priority from your logged in user * User cannot be a superlibrarian 2) Attempt to change the pickup location for a hold 3) Note the change fails 4) Apply this patch 5) Repeat step 2 6) Note the change succeeds Signed-off-by: Owen Leonard Signed-off-by: Katrin Fischer Template change only. Passes all tests and QA script. Tested: - Confirmed that changing the pickup location is not possible in master without the modify_holds_priority permission - Confirmed that the patch fixes the problem - pickup location can now be changed with and without the permission - Pickup location can still not be changed, when IndependentBranches is activated --- .../prog/en/modules/reserve/request.tt | 31 +++++++++++++------- 1 file changed, 20 insertions(+), 11 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt index 20853b3..7f7d05f 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt @@ -600,17 +600,26 @@ function checkMultiHold() { -- 1.7.9.5