From eb39ce87ca47e93ead2133cea667b2f9e9072f99 Mon Sep 17 00:00:00 2001
From: Julian Maurice <julian.maurice@biblibre.com>
Date: Fri, 4 Oct 2024 16:03:32 +0200
Subject: [PATCH] Bug 38027: (QA follow-up) Fix flatpickr onChange when input
 is cleared

With the onChange event handler fixed, it's ok to call clear() instead
of setDate('')

Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>
Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
(cherry picked from commit b892af285b8b7ea58b88b91c671631ece1295c01)
Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
---
 koha-tmpl/intranet-tmpl/prog/en/includes/calendar.inc | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/calendar.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/calendar.inc
index 95234ba8e3..0565b0faac 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/includes/calendar.inc
+++ b/koha-tmpl/intranet-tmpl/prog/en/includes/calendar.inc
@@ -74,7 +74,7 @@
                     .addClass("clear_date")
                     .on("click", function(e){
                         e.preventDefault();
-                        instance.setDate('');
+                        instance.clear();
                     })
                     .addClass("fa fa-fw fa-times")
                     .attr("aria-hidden", true)
@@ -110,6 +110,10 @@
             new Maskito( instance.altInput, maskitoOptions );
         },
         onChange: function( selectedDates, dateText, instance) {
+            if (selectedDates.length === 0) {
+                return;
+            }
+
             var thisInput = instance.input;
             let accepts_time = $(thisInput).data('flatpickr-enable-time');
             let accepts_period = $(thisInput).data('flatpickr-period');