From cd69768f72493d9ec2f2e8015e6f435e9cff18f4 Mon Sep 17 00:00:00 2001 From: Kyle M Hall Date: Wed, 27 Aug 2014 09:05:12 -0400 Subject: [PATCH] Bug 10883 - AllowHoldDateInFuture can let a patron jump to the top of the holds list If a patron is allowed to add a future date for the hold using the "Hold starts on date" field, that patron can put a date in that is in the *past* which will cause them to move to the top of the holds queue ( assuming they make the past date far enough back that it is earlier than all the other hold dates ). Test Plan: 1) Enable future holds 2) Note you can use dates in the past for "hold starts on date" 3) Note you can use dates in the past for "hold expires on date" 4) Apply this patch 5) Refresh the page 6) Note you can no longer use dates in the past for those fields Signed-off-by: Christopher Brannon Signed-off-by: Katrin Fischer Changed subject: OpacAllowHoldDateInFuture to AllowHoldDateInFuture as this fix applies to staff. Works as described, passes tests and QA script. Signed-off-by: Tomas Cohen Arazi --- koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) 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 60551f440e..6c061e4611 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt @@ -22,7 +22,7 @@ var MSG_NO_ITEMS_AVAILABLE = _("A hold cannot be requested on any of these items $(document).ready(function() { [% IF AutoResumeSuspendedHolds %] - $(".suspend_until_datepicker").datepicker("option", "minDate", 1); + $(".suspend_until_datepicker, .datepickerfrom, .datepickerto").datepicker("option", "minDate", 1); [% END %] });