fix: adds dependency and handler variable to use in cleanup function

terrestris · Sep 19, 2024 · 325c153 · 325c153
1 parent 296955c
commit 325c153
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/src/Panel/TimeLayerSliderPanel/TimeLayerSliderPanel.tsx b/src/Panel/TimeLayerSliderPanel/TimeLayerSliderPanel.tsx
@@ -127,7 +127,7 @@ export const TimeLayerSliderPanel: React.FC<TimeLayerSliderPanelProps> = memo(
       setCurrentValue(initEndDate);
       setEndDate(initEndDate);
       wmsTimeHandler(initEndDate);
-    }, []);
+    }, [initEndDate]);
 
     const onPlaybackSpeedChange = useCallback(
       (val: number | PlaybackSpeedType) => {
@@ -170,8 +170,10 @@ export const TimeLayerSliderPanel: React.FC<TimeLayerSliderPanelProps> = memo(
     );
 
     useEffect(() => {
+      const handler = debouncedWmsTimeHandlerRef.current;
+
       return () => {
-        debouncedWmsTimeHandlerRef.current.cancel();
+        handler.cancel();
       };
     }, []);