fix(spreadsheet): reset spreadsheet data on root-network change (#3323)

Signed-off-by: Joris Mancini <[email protected]>

Author: TheMaskedTurtle

src/components/spreadsheet-view/hooks/use-built-nodes-ids.ts

@@ -17,8 +17,12 @@ import type { NodeAlias } from '../types/node-alias.type';
 export function useBuiltNodesIds(nodeAliases: NodeAlias[] | undefined) {
     const currentNode = useSelector((state: AppState) => state.currentTreeNode);
     const treeNodes = useSelector((state: AppState) => state.networkModificationTreeModel?.treeNodes);
+    const isTreeModelUpToDate = useSelector((state: AppState) => state.isNetworkModificationTreeModelUpToDate);
 
     return useStableComputedSet(() => {
+        if (!isTreeModelUpToDate) {
+            return new Set<UUID>();
+        }
         const aliasedNodesIds = nodeAliases
             ?.filter((nodeAlias) => validAlias(nodeAlias))
             .map((nodeAlias) => nodeAlias.id);

src/components/spreadsheet-view/spreadsheet/spreadsheet-content/hooks/use-spreadsheet-equipments.ts

@@ -5,9 +5,9 @@
  * file, You can obtain one at http://mozilla.org/MPL/2.0/.
  */
 
-import { useEffect, useState } from 'react';
+import { useEffect, useRef, useState } from 'react';
 import { useDispatch, useSelector } from 'react-redux';
-import { cleanEquipments, removeNodeData } from 'redux/actions';
+import { cleanEquipments, removeNodeData, resetEquipments } from 'redux/actions';
 import { type AppState } from 'redux/reducer';
 import type { NodeAlias } from '../../../types/node-alias.type';
 import { useOptionalLoadingParametersForEquipments } from './use-optional-loading-parameters-for-equipments';
@@ -31,6 +31,8 @@ export const useSpreadsheetEquipments = (
     const [isFetching, setIsFetching] = useState<boolean>(false);
     const { fetchNodesEquipmentData } = useFetchEquipment(type);
 
+    const prevCurrentRootNetworkUuidRef = useRef(currentRootNetworkUuid);
+
     const {
         shouldLoadOptionalLoadingParameters,
         equipmentsWithLoadingOptionsLoaded,
@@ -74,6 +76,14 @@ export const useSpreadsheetEquipments = (
         }
     }, [active, dispatch, nodesIdsToRemove]);
 
+    // Reset equipment data on root network change
+    useEffect(() => {
+        if (prevCurrentRootNetworkUuidRef.current !== currentRootNetworkUuid) {
+            dispatch(resetEquipments());
+            prevCurrentRootNetworkUuidRef.current = currentRootNetworkUuid;
+        }
+    }, [dispatch, currentRootNetworkUuid]);
+
     // Note: take care about the dependencies because any execution here implies equipment loading (large fetches).
     // For example, we have 3 currentNode properties in deps rather than currentNode object itself.
     useEffect(() => {