Merge pull request #259 from chrishiguto/fix/date-picker-duplicated-f…

…unctions fix: date filter duplicated functions
conceptadev · Oct 17, 2024 · 7f619da · 7f619da
2 parents f2f67b3 + 6f9c096
commit 7f619da
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/packages/react-material-ui/src/components/DatePickerField/index.tsx b/packages/react-material-ui/src/components/DatePickerField/index.tsx
@@ -17,7 +17,7 @@ const DatePickerField = ({
   const [search, setSearch] = useState<Date | null>(null);
 
   const handleDebouncedSearch = useMemo(
-    () => debounce(onDebouncedSearchChange, wait),
+    () => onDebouncedSearchChange && debounce(onDebouncedSearchChange, wait),
     [wait, props?.value],
   );
 
@@ -27,7 +27,7 @@ const DatePickerField = ({
     // Keep track of the first render to avoid triggering onDebouncedSearchChange
     // on the initial render. Only trigger when the 'value' changes.
     if (!firstRender.current) {
-      handleDebouncedSearch(props?.value ?? search);
+      handleDebouncedSearch?.(props?.value ?? search);
     } else {
       firstRender.current = false;
     }
@@ -45,7 +45,7 @@ const DatePickerField = ({
         field: {
           clearable: true,
           onClear: () => {
-            handleDebouncedSearch(null);
+            handleDebouncedSearch?.(null);
           },
         },
         textField: {

diff --git a/packages/react-material-ui/src/components/submodules/Filter/index.tsx b/packages/react-material-ui/src/components/submodules/Filter/index.tsx
@@ -226,7 +226,6 @@ const FilterSubmodule = (props: Props) => {
           options,
           value: value as unknown as Date,
           onChange: (val: Date | null) => onFilterChange(id, val, false),
-          onDebouncedSearchChange: (val: Date) => onFilterChange(id, val, true),
         };
 
       default: