Attachment 118368 Details for Bug 27937 – Bug 27937: Set datepicker when date is valid

[patch] Bug 27937: Set datepicker when date is valid

Bug-27937-Set-datepicker-when-date-is-valid.patch (text/plain), 1.71 KB, created by ByWater Sandboxes on 2021-03-17 12:03:55 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: ByWater Sandboxes

Created: 2021-03-17 12:03:55 UTC

Size: 1.71 KB

patch

obsolete

>From 07ac0d426010c91347b8d5122d948c06917d811b Mon Sep 17 00:00:00 2001
>From: Nick Clemens <nick@bywatersolutions.com>
>Date: Tue, 16 Mar 2021 13:13:57 +0000
>Subject: [PATCH] Bug 27937: Set datepicker when date is valid
>
>When validating dates in JavaScript it is acceptable to have single digit days or months e.g,:
>2011-6-16
>
>When parsing in Koha::DateUtils->dt_from_string we die on these
>
>This patch uses JS to set the datepicker which normalizes the date as Koha expects
>
>To test:
> 1 - Add new patron
> 2 - Enter birthdate as 2/11/1986
> 3 - Complete required fields
> 4 - Save
> 5 - Get ISE
>     The given date (2/11/1986) does not match the date format (us) at /kohadevbox/koha/Koha/DateUtils.pm line 175
> 6 - Apply patch
> 7 - Repeat 1-2
> 8 - Note that when you change focus (click on another field) the date is formatted as '02/11/1986'
> 9 - Save patron, success!
>10 - Edit patron, try setting DOB to 2/1/1986
>11 - Change focus and note date is formatted as '02/01/1986', save
>12 - Repeat with other settings of system preference 'dateformat'
>
>Signed-off-by: Donna <donna@bywatersolutions.com>
>---
> koha-tmpl/intranet-tmpl/prog/js/calendar.js | 2 ++
> 1 file changed, 2 insertions(+)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/js/calendar.js b/koha-tmpl/intranet-tmpl/prog/js/calendar.js
>index 1f312e813d..5759fdf7ec 100644
>--- a/koha-tmpl/intranet-tmpl/prog/js/calendar.js
>+++ b/koha-tmpl/intranet-tmpl/prog/js/calendar.js
>@@ -192,6 +192,8 @@ $(document).ready(function () {
>     }).on("change", function () {
>         if (!is_valid_date($(this).val())) {
>             $(this).val("");
>+        } else {
>+          $(this).datepicker("setDate",$(this).val());
>         }
>     });
>     // http://jqueryui.com/demos/datepicker/#date-range
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 27937: 118315 | 118316 | 118317 | 118368 | 119069 | 119070 | 119084