Bug 35341: Improve processing hard due date and auto renewal date
authorMarcel de Rooy <m.de.rooy@rijksmuseum.nl>
Fri, 17 Nov 2023 10:18:01 +0000 (10:18 +0000)
committerKatrin Fischer <katrin.fischer@bsz-bw.de>
Tue, 16 Jan 2024 11:06:15 +0000 (12:06 +0100)
commit963a3a47155cf313d25af6ba10781f4e7922f4dc
treed893ae6c83237df694cc1f233e166dc09f610442
parent9e168686cc7de610275c51c161d02b3bc307a9bd
Bug 35341: Improve processing hard due date and auto renewal date

After quite a struggle, came up with this approach using
flatpickr's parseDate and setDate. Seems to be the best.
The dateformat variable is set in calendar.inc.

Test plan:
Test edit, save, clear rules, focus on both dates.
Toggle all I18N preference date formats.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Bug 35341: Do not reinstantiate flatpickr

We do not need to reinstantiate flatpickr, we need to retrieve the
existing instance.

Test plan:
See previous patch.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
12-01-2024 Squashed both patches. Removed the unused hidden inputs for
hardduedatecompare and added a data attribute for the duedate.
The check with is_valid_date seems no longer needed here. Normally
date is empty or valid. A js warning from parseDate should be
exceptional. Made code for both dates a bit more consistent.

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
koha-tmpl/intranet-tmpl/prog/en/modules/admin/smart-rules.tt