Koha-community/Koha
13
7
Fork 0
Code Issues Releases Wiki Activity

Bug 10883 - AllowHoldDateInFuture can let a patron jump to the top of the holds list

Browse source 
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 <cbrannon@cdalibrary.org>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
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 <tomascohen@gmail.com>
This commit is contained in:
Kyle Hall 2014-08-27 09:05:12 -04:00 committed by Tomas Cohen Arazi
parent eb4df0eff2
commit cd69768f72
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt
View file

@ -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 %]
});