fix(module:time-picker): add null input judgement when using datefns (N…

…G-ZORRO#4283) close NG-ZORRO#3854
hsuanxyz · Oct 14, 2019 · ffc9d2f · ffc9d2f
1 parent 9de0582
commit ffc9d2f
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/components/i18n/date-helper.service.ts b/components/i18n/date-helper.service.ts
@@ -75,8 +75,8 @@ export class DateHelperByDateFns extends DateHelperService {
    * @param date Date
    * @param formatStr format string
    */
-  format(date: Date, formatStr: string): string {
-    return fnsFormat(date, formatStr, { locale: this.i18n.getDateLocale() });
+  format(date: Date | null, formatStr: string): string {
+    return date ? fnsFormat(date, formatStr, { locale: this.i18n.getDateLocale() }) : '';
   }
 }
 
@@ -103,7 +103,7 @@ export class DateHelperByDatePipe extends DateHelperService {
     return this.config.firstDayOfWeek;
   }
 
-  format(date: Date, formatStr: string): string {
+  format(date: Date | null, formatStr: string): string {
     return date ? formatDate(date, formatStr, this.i18n.getLocaleId())! : '';
   }