fix : timeselector, zero value를 failure 처리 하던 현상

version 2.0.5

Author: flamecommit

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@shinyongjun/react-datepicker",
-  "version": "2.0.4",
+  "version": "2.0.5",
   "main": "./dist/cjs/index.js",
   "module": "./dist/esm/index.js",
   "source": "./src/index.tsx",

package/src/components/input/InputUnit.tsx

@@ -96,14 +96,14 @@ function InputUnit({
     month,
     date,
   }: {
-    year?: string | number | null;
-    month?: string | number | null;
-    date?: string | number | null;
+    year: number | null;
+    month: number | null;
+    date: number | null;
   }) => {
     setDateValue({
-      year: year ? year : dateValue.year,
-      month: month ? Number(month) - 1 : dateValue.month,
-      date: date ? date : dateValue.date,
+      year: year !== null ? year : dateValue.year,
+      month: month !== null ? month : dateValue.month,
+      date: date !== null ? date : dateValue.date,
     });
   };
 
@@ -112,14 +112,14 @@ function InputUnit({
     minute,
     second,
   }: {
-    hour?: string | number | null;
-    minute?: string | number | null;
-    second?: string | number | null;
+    hour: number | null;
+    minute: number | null;
+    second: number | null;
   }) => {
     setTimeValue({
-      hour: hour ? Number(hour) : timeValue.hour,
-      minute: minute ? Number(minute) : timeValue.minute,
-      second: second ? Number(second) : timeValue.second,
+      hour: hour !== null ? hour : timeValue.hour,
+      minute: minute !== null ? minute : timeValue.minute,
+      second: second !== null ? second : timeValue.second,
     });
   };
 
@@ -153,28 +153,28 @@ function InputUnit({
 
     switch (type) {
       case 'YYYY':
-        utilSetDateValue({ year: text });
+        utilSetDateValue({ ...dateValue, year: Number(text) });
         setViewDate(
           `${text}-${viewDate.split('-')[1]}-${viewDate.split('-')[2]}`
         );
         return;
       case 'MM':
-        utilSetDateValue({ month: text });
+        utilSetDateValue({ ...dateValue, month: Number(text) - 1 });
         setViewDate(
           `${viewDate.split('-')[0]}-${text}-${viewDate.split('-')[2]}`
         );
         return;
       case 'DD':
-        utilSetDateValue({ date: text });
+        utilSetDateValue({ ...dateValue, date: Number(text) });
         return;
       case 'hh':
-        utilSetTimeValue({ hour: text });
+        utilSetTimeValue({ ...timeValue, hour: Number(text) });
         return;
       case 'mm':
-        utilSetTimeValue({ minute: text });
+        utilSetTimeValue({ ...timeValue, minute: Number(text) });
         return;
       case 'ss':
-        utilSetTimeValue({ second: text });
+        utilSetTimeValue({ ...timeValue, second: Number(text) });
         return;
       default:
         return;

package/src/components/timeselector/SelectorList.tsx

@@ -44,7 +44,7 @@ function TimeselectorSelectorList({
             onClick={() =>
               setTimeValue({
                 ...timeValue,
-                ...{ [timeKey]: item },
+                [timeKey]: item,
               })
             }
           >

package/src/types/props.d.ts

@@ -11,9 +11,9 @@ export interface ITimeValue {
 }
 
 export interface IDateValue {
-  year: number | string | null;
-  month: number | string | null;
-  date: number | string | null;
+  year: number | null;
+  month: number | null;
+  date: number | null;
 }
 
 export type TIsVisible = boolean | 'start' | 'end';

package/src/utils/datetime.ts

@@ -71,15 +71,13 @@ export const getDateValueUnit = (value: IDateValue, unit: string): string => {
 };
 
 export const getTimeValueUnit = (value: ITimeValue, unit: string): string => {
-  if (!value) return unit;
-
   switch (unit) {
     case 'hh':
-      return value.hour ? addLeadingZero(value.hour) : 'hh';
+      return addLeadingZero(value.hour);
     case 'mm':
-      return value.minute ? addLeadingZero(value.minute) : 'mm';
+      return addLeadingZero(value.minute);
     case 'ss':
-      return value.second ? addLeadingZero(value.second) : 'ss';
+      return addLeadingZero(value.second);
     default:
       return '';
   }