From 29c22efed0bc4ba4b3820cb7d2f8cd14b3e4de68 Mon Sep 17 00:00:00 2001
From: lindhork <45235524+lindhork@users.noreply.github.com>
Date: Thu, 11 Feb 2021 03:55:16 +0100
Subject: [PATCH] Error message corrected

---
 client/components/cards/cardDate.js | 20 ++++++++++++++++----
 client/lib/datepicker.js            | 20 ++++++++++++++++----
 2 files changed, 32 insertions(+), 8 deletions(-)

diff --git a/client/components/cards/cardDate.js b/client/components/cards/cardDate.js
index 0cab46540..d247fe6d8 100644
--- a/client/components/cards/cardDate.js
+++ b/client/components/cards/cardDate.js
@@ -76,13 +76,25 @@ BlazeComponent.extendComponent({
           const time =
             evt.target.time.value ||
             moment(new Date().setHours(12, 0, 0)).format('LT');
+          const newTime = moment(time, adjustedTimeFormat(), true);
+          const newDate = moment(evt.target.date.value, 'L', true);
           const dateString = `${evt.target.date.value} ${time}`;
-          const newDate = moment(dateString, 'L ' + adjustedTimeFormat(), true);
-          if (newDate.isValid()) {
-            this._storeDate(newDate.toDate());
+          const newCompleteDate = moment(dateString, 'L ' + adjustedTimeFormat(), true);
+          if (!newTime.isValid()) {
+            this.error.set('invalid-time');
+            evt.target.time.focus();
+          }
+          if (!newDate.isValid()) {
+            this.error.set('invalid-date');
+            evt.target.date.focus();
+          }
+          if (newCompleteDate.isValid()) {
+            this._storeDate(newCompleteDate.toDate());
             Popup.close();
           } else {
-            this.error.set('invalid');
+            if (!this.error){
+              this.error.set('invalid');
+            }
           }
         },
         'click .js-delete-date'(evt) {
diff --git a/client/lib/datepicker.js b/client/lib/datepicker.js
index d415fde39..de3986157 100644
--- a/client/lib/datepicker.js
+++ b/client/lib/datepicker.js
@@ -94,13 +94,25 @@ DatePicker = BlazeComponent.extendComponent({
           const time =
             evt.target.time.value ||
             moment(new Date().setHours(12, 0, 0)).format('LT');
+          const newTime = moment(time, adjustedTimeFormat(), true);
+          const newDate = moment(evt.target.date.value, 'L', true);
           const dateString = `${evt.target.date.value} ${time}`;
-          const newDate = moment(dateString, 'L ' + adjustedTimeFormat(), true);
-          if (newDate.isValid()) {
-            this._storeDate(newDate.toDate());
+          const newCompleteDate = moment(dateString, 'L ' + adjustedTimeFormat(), true);
+          if (!newTime.isValid()) {
+            this.error.set('invalid-time');
+            evt.target.time.focus();
+          }
+          if (!newDate.isValid()) {
+            this.error.set('invalid-date');
+            evt.target.date.focus();
+          }
+          if (newCompleteDate.isValid()) {
+            this._storeDate(newCompleteDate.toDate());
             Popup.close();
           } else {
-            this.error.set('invalid');
+            if (!this.error){
+              this.error.set('invalid');
+            }
           }
         },
         'click .js-delete-date'(evt) {