Bug 36341: Add futureinclusive to future holds
authorMartin Renvoize <martin.renvoize@ptfs-europe.com>
Mon, 18 Mar 2024 11:35:13 +0000 (11:35 +0000)
committerFridolin Somers <fridolin.somers@biblibre.com>
Thu, 23 May 2024 09:20:40 +0000 (11:20 +0200)
commite3bfca4f02b72e00909272fbfb5fc24abbab49a7
tree4c2fa3dc2fbc6ad96d810568dfaa76d965ba67c8
parentb0c862533d74505c8f4fa8760d809942973ea57c
Bug 36341: Add futureinclusive to future holds

This patch adds the flatpickr-futureinclusive data to the input field
for "Hold starts on date" in the OPAC.

We also add 'flatpickr-future' data to the input field for expiration
date as it also doesn't make sense to allow this to be a past data
either.

Test plan
1) Attempt to place a hold on an item from the OPAC
2) Expand the 'Show more options' dropdown
3) Note that without the patch 'Hold not needed after' allows setting to
   a date in the past.
4) Enable `AllowHoldDateInFuture` and `OPACAllowHoldDateInFuture` system
   preferences
5) The 'Hold starts on date' option should now appear if you refresh
   your hold placing page in the OPAC.
6) Note that the above input also allows setting a past date
7) Apply patch
8) Neith input should allow past dates now.

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
(cherry picked from commit c16ec21f4b77971b19e5c4b5140521c4133f0608)
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-reserve.tt