Fixes silent DatetimePicker error on midnight value

gingi · gingi · commit d104b7241bfb · 2022-04-21T15:30:55.000-04:00
diff --git a/src/@batch-flask/ui/datetime-picker/datetime-picker.component.spec.ts b/src/@batch-flask/ui/datetime-picker/datetime-picker.component.spec.ts
@@ -72,6 +72,11 @@ describe("DatetimePickerComponent", () => {
             expect(testComponent.control.value).toEqual(new Date(2017, 11, 14, 17, 32));
         });
 
+        it("correctly handles midnight", () => {
+            testComponent.control.setValue(new Date(2022, 3, 15, 0, 0));
+            expect(timeInputEl.nativeElement.value).toEqual("00:00");
+        });
+
         it("updates the inputs when setting the date time as a string", () => {
             testComponent.control.setValue(new Date(2017, 11, 14, 17, 32).toISOString());
             expect(dateInputEl.nativeElement.value).toEqual("12/14/2017");
diff --git a/src/@batch-flask/ui/datetime-picker/datetime-picker.component.ts b/src/@batch-flask/ui/datetime-picker/datetime-picker.component.ts
@@ -106,7 +106,7 @@ export class DatetimePickerComponent implements ControlValueAccessor, OnDestroy
         const datetime = this._getDate(value).setZone(this.currentTimeZone.name);
         this.datetime.patchValue({
             date: datetime.toJSDate(),
-            time: `${datetime.hour}:${datetime.minute}`,
+            time: datetime.toFormat("HH:mm")
         });
         this.changeDetector.markForCheck();
     }

Original file line number	Diff line number	Diff line change
`@@ -106,7 +106,7 @@ export class DatetimePickerComponent implements ControlValueAccessor, OnDestroy`
`106`	`106`	`const datetime = this._getDate(value).setZone(this.currentTimeZone.name);`
`107`	`107`	`this.datetime.patchValue({`
`108`	`108`	`date: datetime.toJSDate(),`
`109`		- time: `${datetime.hour}:${datetime.minute}`,
	`109`	`+ time: datetime.toFormat("HH:mm")`
`110`	`110`	`});`
`111`	`111`	`this.changeDetector.markForCheck();`
`112`	`112`	`}`