fix: force update when some data properties are updated on reearth/co…

…re (#483) fix: force update when some data properties are updated
reearth · Feb 22, 2023 · 867238c · 867238c
1 parent 1d94558
commit 867238c
Showing 1 changed file with 16 additions and 0 deletions.
diff --git a/src/core/Map/Layer/hooks.ts b/src/core/Map/Layer/hooks.ts
@@ -1,4 +1,5 @@
 import { useAtom } from "jotai";
+import { isEqual, pick } from "lodash-es";
 import { useCallback, useEffect, useLayoutEffect, useMemo, useRef } from "react";
 
 import {
@@ -8,6 +9,7 @@ import {
   type Atom,
   evalFeature,
   ComputedFeature,
+  type Data,
 } from "../../mantle";
 import type { DataRange, Feature, Layer } from "../../mantle";
 
@@ -81,6 +83,20 @@ export default function useHooks(
     };
   }, [layer, forceUpdateFeatures]);
 
+  const prevForceUpdatableData = useRef<Pick<Data, "csv" | "jsonProperties">>();
+  useLayoutEffect(() => {
+    const data = layer?.type === "simple" ? layer.data : undefined;
+    const forceUpdatableData = pick(data, "csv", "jsonProperties");
+
+    if (isEqual(forceUpdatableData, prevForceUpdatableData.current) || !data?.url) {
+      return;
+    }
+
+    forceUpdateFeatures();
+
+    prevForceUpdatableData.current = forceUpdatableData;
+  }, [layer, forceUpdateFeatures]);
+
   // Clear expression cache if layer is unmounted
   useEffect(
     () => () => {