refactor: improve zone and asset handler structure and update context usage

app/src/functions/outlineHelpers/useZoneAssetHandlers.ts

@@ -1,111 +1,95 @@
 import { useCallback } from "react";
 import { useParams } from "react-router-dom";
 import { getUserData } from "../getUserData";
-import { useVersionContext } from "../../modules/builder/version/versionContext";
 import { getZoneData } from "../../services/visulization/zone/getZones";
 import { zoneCameraUpdate } from "../../services/visulization/zone/zoneCameraUpdation";
 import { setAssetsApi } from "../../services/factoryBuilder/asset/floorAsset/setAssetsApi";
 import { useSelectedZoneStore } from "../../store/visualization/useZoneStore";
-import { useSubModuleStore } from "../../store/useModuleStore";
+import { useSubModuleStore } from "../../store/ui/useModuleStore";
 import { useSceneContext } from "../../modules/scene/sceneContext";
 import { useZoneAssetId } from "../../store/builder/store";
 
 export const useZoneAssetHandlers = () => {
-  const { projectId } = useParams();
-  const { organization } = getUserData();
-  const { selectedVersionStore } = useVersionContext();
-  const { selectedVersion } = selectedVersionStore();
+    const { projectId } = useParams();
+    const { organization } = getUserData();
 
-  const { selectedZone, setSelectedZone } = useSelectedZoneStore();
-  const { setSubModule } = useSubModuleStore();
-  const { zoneAssetId, setZoneAssetId } = useZoneAssetId();
-  const { assetStore, zoneStore } = useSceneContext();
-  const { setName } = assetStore();
-  const { zones, setZoneName } = zoneStore();
+    const { selectedZone, setSelectedZone } = useSelectedZoneStore();
+    const { setSubModule } = useSubModuleStore();
+    const { zoneAssetId, setZoneAssetId } = useZoneAssetId();
+    const { assetStore, zoneStore, versionStore } = useSceneContext();
+    const { selectedVersion } = versionStore();
+    const { setName } = assetStore();
+    const { zones, setZoneName } = zoneStore();
 
-  // 🔹 Zone selection
-  const handleSelectZone = useCallback(
-    async (id: string) => {
-      if (selectedZone?.zoneUuid === id || id === "unassigned-zone") return;
-      setSubModule("zoneProperties");
+    // 🔹 Zone selection
+    const handleSelectZone = useCallback(
+        async (id: string) => {
+            if (selectedZone?.zoneUuid === id || id === "unassigned-zone") return;
+            setSubModule("zoneProperties");
 
-      const response = await getZoneData(
-        id,
-        organization,
-        projectId,
-        selectedVersion?.versionId || ""
-      );
-      if (!response) return;
+            const response = await getZoneData(id, organization, projectId, selectedVersion?.versionId || "");
+            if (!response) return;
 
-      setSelectedZone({
-        zoneName: response.zoneName,
-        activeSides: response.activeSides ?? [],
-        panelOrder: response.panelOrder ?? [],
-        lockedPanels: response.lockedPanels ?? [],
-        widgets: response.widgets ?? [],
-        zoneUuid: response.zoneUuid,
-        zoneViewPortTarget: response.viewPortTarget ?? [],
-        zoneViewPortPosition: response.viewPortPosition ?? [],
-      });
-    },
-    [organization, projectId, selectedVersion, selectedZone?.zoneUuid, setSelectedZone, setSubModule]
-  );
+            setSelectedZone({
+                zoneName: response.zoneName,
+                activeSides: response.activeSides ?? [],
+                panelOrder: response.panelOrder ?? [],
+                lockedPanels: response.lockedPanels ?? [],
+                widgets: response.widgets ?? [],
+                zoneUuid: response.zoneUuid,
+                zoneViewPortTarget: response.viewPortTarget ?? [],
+                zoneViewPortPosition: response.viewPortPosition ?? [],
+            });
+        },
+        [organization, projectId, selectedVersion, selectedZone?.zoneUuid, setSelectedZone, setSubModule]
+    );
 
-  // 🔹 Zone rename
-  const handleZoneNameChange = useCallback(
-    async (newName: string) => {
-      const isDuplicate = zones.some(
-        (zone: any) =>
-          zone.zoneName?.trim().toLowerCase() === newName.trim().toLowerCase() &&
-          zone.zoneUuid !== selectedZone.zoneUuid
-      );
-      if (isDuplicate) {
-        alert("Zone name already exists. Please choose a different name.");
-        return;
-      }
+    // 🔹 Zone rename
+    const handleZoneNameChange = useCallback(
+        async (newName: string) => {
+            const isDuplicate = zones.some((zone: any) => zone.zoneName?.trim().toLowerCase() === newName.trim().toLowerCase() && zone.zoneUuid !== selectedZone.zoneUuid);
+            if (isDuplicate) {
+                alert("Zone name already exists. Please choose a different name.");
+                return;
+            }
 
-      const response = await zoneCameraUpdate(
-        { zoneUuid: selectedZone.zoneUuid, zoneName: newName },
-        organization,
-        projectId,
-        selectedVersion?.versionId || ""
-      );
+            const response = await zoneCameraUpdate({ zoneUuid: selectedZone.zoneUuid, zoneName: newName }, organization, projectId, selectedVersion?.versionId || "");
 
-      if (response.message === "zone updated") {
-        setSelectedZone((prev) => ({ ...prev, zoneName: newName }));
-        setZoneName(selectedZone.zoneUuid, newName);
-      }
-    },
-    [organization, projectId, selectedVersion, selectedZone, setSelectedZone, setZoneName, zones]
-  );
+            if (response.message === "zone updated") {
+                setSelectedZone((prev) => ({ ...prev, zoneName: newName }));
+                setZoneName(selectedZone.zoneUuid, newName);
+            }
+        },
+        [organization, projectId, selectedVersion, selectedZone, setSelectedZone, setZoneName, zones]
+    );
 
-  // 🔹 Asset selection
-  const handleAssetClick = useCallback(
-    (asset: ListAsset) => {
-      setZoneAssetId(asset);
-    },
-    [setZoneAssetId]
-  );
+    // 🔹 Asset selection
+    const handleAssetClick = useCallback(
+        (asset: ListAsset) => {
+            setZoneAssetId(asset);
+        },
+        [setZoneAssetId]
+    );
 
-  // 🔹 Asset rename
-  const handleZoneAssetName = useCallback(
-    async (newName: string) => {
-      if (!zoneAssetId?.id) return;
-      const response = await setAssetsApi({
-        modelUuid: zoneAssetId.id,
-        modelName: newName,
-        projectId,
-        versionId: selectedVersion?.versionId || "",
-      });
-      setName(zoneAssetId.id, response.modelName);
-    },
-    [projectId, selectedVersion, setName, zoneAssetId]
-  );
+    // 🔹 Asset rename
+    const handleZoneAssetName = useCallback(
+        async (newName: string) => {
+            if (!zoneAssetId?.id) return;
+            const response = await setAssetsApi({
+                modelUuid: zoneAssetId.id,
+                modelName: newName,
+                projectId,
+                versionId: selectedVersion?.versionId || "",
+            });
+            setName(zoneAssetId.id, response.modelName);
+        },
+        [projectId, selectedVersion, setName, zoneAssetId]
+    );
 
-  return {
-    handleSelectZone,
-    handleZoneNameChange,
-    handleAssetClick,
-    handleZoneAssetName,
-  };
+    return {
+        handleSelectZone,
+        handleZoneNameChange,
+        handleAssetClick,
+        handleZoneAssetName,
+    };
 };