From b6cdde78a35d3bb1ea8b4d0d3d8afbe7f9790799 Mon Sep 17 00:00:00 2001 From: CJ Lynce Date: Thu, 3 Oct 2024 18:02:47 +0000 Subject: [PATCH] Bug 38081: maskitoTimeOptionsGenerator does not properly support 12-hour times in calendar.inc This patch corrects an issue where flatpickr time-only input boxes boxes were not having their inputs masked (limited) properly due to maskitoTimeOptionsGenerator not properly supporting 12-hour time inputs for time-only input boxes. To test: 1. Login to the staff intranet. 2. Open Administration->Libraries-> Edit any library. 3. Open your browser's development console (typically via F12) Verify a 'TypeError' message has been thrown for this page. 4. Type in any text into any of the opening hours This should be limiting only to properly formatted HH:MM. 5. Apply patch 6. Repeat steps 2-4 Verify no errors show on your browser's development console. Verify opening hours entry are limited to proper HH:MM format. 7. Open Administration->System Preferences and change TimeFormat to 12-hours, Save. 8. Repeat steps 2-4 Verify opening hours text entry are limited to properly formatted HH:MM AM/PM (or am/pm) --- koha-tmpl/intranet-tmpl/prog/en/includes/calendar.inc | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/calendar.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/calendar.inc index 8280be9ba9..2dbb7d9d1c 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/includes/calendar.inc +++ b/koha-tmpl/intranet-tmpl/prog/en/includes/calendar.inc @@ -99,9 +99,12 @@ dateSeparator: delimiter, overwiteMode: 'replace', }); - } else if ( accepts_time_only ) { - maskitoOptions = maskitoTimeOptionsGenerator({ - }); + } else if ( accepts_time_only ) { + maskitoOptions = { + [% IF Koha.Preference('TimeFormat') == '12hr' -%] mask: [/\d/, /\d/, ':', /\d/, /\d/, ' ', /(a|A|p|P)/, /(m|M)/] + [%- ELSE -%]mask: [/\d/, /\d/, ':', /\d/, /\d/] + [%- END %], + }; } else { maskitoOptions = maskitoDateOptionsGenerator({ mode: altinput_dateformat, -- 2.39.5