With flatPickr you can manually enter a date. If that date date allows for the input of hours and minutes and you do not manually specify those the default will be 00:00. When you use the flatPickr calendar to pick a date time and no do specify the hour/minute it will default to 23:59. This should be consistent.
To recreate: 1. Find some places in the staff interface where data-flatpickr-enable-time is set to true. Examples: renew.tt or circulation.tt 2. Directly input a date in whatever date format you have specified in DateFormat but omit the hour/minute. Example: 12/12/2023 3. See that the hour/minute default to 00:00 if your TimeFormat is 24hr and 12:00 AM if your time format is 12hr
Created attachment 156309 [details] [review] Bug 34939: Set hour:minute to 23:59 when inputing dates unless explicitly set 1. Find some places in the staff interface where data-flatpickr-enable-time is set to true. Examples: renew.tt or circulation.tt 2. Directly input a date in whatever date format you have specified in DateFormat but omit the hour/minute. Example: 12/12/2023 3. See that the hour/minute default to 00:00 if your TimeFormat is 24hr and 12:00 AM if your time format is 12hr 4. Apply patch, clear browser cache. 5. Try step again, setting a date like '12/12/2023'. The hour:minute should be added as 23:59. 6. With TimeFormat set to 24hr try entering a date with the hour:minute like '12/12/2023 11:22'. The hour:minute should be set to 11:22. 7. With TimeFormat set to 12hr try entering a date with the hour:minute like '12/12/2023 11:22 AM'. The hour:minute should be set to 11:22 AM. 8. Play with this in as many TimeFormat and DateFormat combonatinons as you can.
Created attachment 156311 [details] [review] Bug 34939: Set hour:minute to 23:59 when inputing dates unless explicitly set 1. Find some places in the staff interface where data-flatpickr-enable-time is set to true. Examples: renew.tt or circulation.tt 2. Directly input a date in whatever date format you have specified in DateFormat but omit the hour/minute. Example: 12/12/2023 3. See that the hour/minute default to 00:00 if your TimeFormat is 24hr and 12:00 AM if your time format is 12hr 4. Apply patch, clear browser cache. 5. Try step again, setting a date like '12/12/2023'. The hour:minute should be added as 23:59. 6. With TimeFormat set to 24hr try entering a date with the hour:minute like '12/12/2023 11:22'. The hour:minute should be set to 11:22. 7. With TimeFormat set to 12hr try entering a date with the hour:minute like '12/12/2023 11:22 AM'. The hour:minute should be set to 11:22 AM. 8. Play with this in as many TimeFormat and DateFormat combonatinons as you can.
Created attachment 156405 [details] [review] Bug 34939: Set hour:minute to 23:59 when inputing dates unless explicitly set 1. Find some places in the staff interface where data-flatpickr-enable-time is set to true. Examples: renew.tt or circulation.tt 2. Directly input a date in whatever date format you have specified in DateFormat but omit the hour/minute. Example: 12/12/2023 3. See that the hour/minute default to 00:00 if your TimeFormat is 24hr and 12:00 AM if your time format is 12hr 4. Apply patch, clear browser cache. 5. Try step again, setting a date like '12/12/2023'. The hour:minute should be added as 23:59. 6. With TimeFormat set to 24hr try entering a date with the hour:minute like '12/12/2023 11:22'. The hour:minute should be set to 11:22. 7. With TimeFormat set to 12hr try entering a date with the hour:minute like '12/12/2023 11:22 AM'. The hour:minute should be set to 11:22 AM. 8. Play with this in as many TimeFormat and DateFormat combonatinons as you can. Signed-off-by: Kristi Krueger <kkrueger@cuyahogalibrary.org>
Hi Kristi. I think you've signed this off, but the status needs changing. Could you change the status to signed off? (This also means that you will get credit on the dashboard under the sign-offs section https://dashboard.koha-community.org/ ) David Nind New Zealand
Created attachment 156461 [details] [review] Bug 34939: Set hour:minute to 23:59 when inputing dates unless explicitly set 1. Find some places in the staff interface where data-flatpickr-enable-time is set to true. Examples: renew.tt or circulation.tt 2. Directly input a date in whatever date format you have specified in DateFormat but omit the hour/minute. Example: 12/12/2023 3. See that the hour/minute default to 00:00 if your TimeFormat is 24hr and 12:00 AM if your time format is 12hr 4. Apply patch, clear browser cache. 5. Try step again, setting a date like '12/12/2023'. The hour:minute should be added as 23:59. 6. With TimeFormat set to 24hr try entering a date with the hour:minute like '12/12/2023 11:22'. The hour:minute should be set to 11:22. 7. With TimeFormat set to 12hr try entering a date with the hour:minute like '12/12/2023 11:22 AM'. The hour:minute should be set to 11:22 AM. 8. Play with this in as many TimeFormat and DateFormat combonatinons as you can. Signed-off-by: Kristi Krueger <kkrueger@cuyahogalibrary.org> Signed-off-by: Andrew Fuerste-Henry <andrewfh@dubcolib.org>
Created attachment 156495 [details] [review] Bug 34939: Set hour:minute to 23:59 when inputing dates unless explicitly set 1. Find some places in the staff interface where data-flatpickr-enable-time is set to true. Examples: renew.tt or circulation.tt 2. Directly input a date in whatever date format you have specified in DateFormat but omit the hour/minute. Example: 12/12/2023 3. See that the hour/minute default to 00:00 if your TimeFormat is 24hr and 12:00 AM if your time format is 12hr 4. Apply patch, clear browser cache. 5. Try step again, setting a date like '12/12/2023'. The hour:minute should be added as 23:59. 6. With TimeFormat set to 24hr try entering a date with the hour:minute like '12/12/2023 11:22'. The hour:minute should be set to 11:22. 7. With TimeFormat set to 12hr try entering a date with the hour:minute like '12/12/2023 11:22 AM'. The hour:minute should be set to 11:22 AM. 8. Play with this in as many TimeFormat and DateFormat combonatinons as you can. Signed-off-by: Kristi Krueger <kkrueger@cuyahogalibrary.org> Signed-off-by: Andrew Fuerste-Henry <andrewfh@dubcolib.org> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Key here is to type the date vs. picking it from the widget.
Pushed to master for 23.11. Nice work everyone, thanks!
Pushed to 23.05.x for 23.05.05
I think this should only fire when enable-time is present on the input.. not for all flatpickrs Opening a follow-up bug to fix that quickly.
Discussion on IRC - breaks bookings feature (under development- Bug 29002): https://irc.koha-community.org/koha/2023-10-09#i_2513012
Nice work everyone! Pushed to oldstable for 22.11.x
*** Bug 34680 has been marked as a duplicate of this bug. ***