fix(spreadsheet): reset equipments on update failure (#3312)

* fix(spreadsheet): reset equipments on update failure

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

* fix(spreadsheet): filter count raises error if empty

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

---------

Signed-off-by: Joris Mancini <[email protected]>
Co-authored-by: Achour berrahma <[email protected]>

Author: TheMaskedTurtle

src/components/spreadsheet-view/hooks/use-update-equipments-on-notification.ts

@@ -58,11 +58,17 @@ export function useUpdateEquipmentsOnNotification(nodeAliases: NodeAlias[] | und
             }
 
             if (impactedSubstationsIds.length > 0 && studyUuid && currentRootNetworkUuid) {
-                fetchAllEquipments(studyUuid, nodeUuid, currentRootNetworkUuid, impactedSubstationsIds).then(
-                    (values) => {
+                fetchAllEquipments(studyUuid, nodeUuid, currentRootNetworkUuid, impactedSubstationsIds)
+                    .then((values) => {
                         dispatch(updateEquipments(values, nodeUuid));
-                    }
-                );
+                    })
+                    .catch((error: unknown) => {
+                        console.warn(
+                            `Failed to update spreadsheet equipments on notification, it will be reset`,
+                            error
+                        );
+                        dispatch(resetEquipments());
+                    });
             }
 
             if (deletedEquipments.length > 0) {
@@ -105,17 +111,22 @@ export function useUpdateEquipmentsOnNotification(nodeAliases: NodeAlias[] | und
                     currentRootNetworkUuid === eventRootNetworkUuid &&
                     builtNodesIds.has(eventNodeUuid)
                 ) {
-                    const networkImpacts = JSON.parse(eventData.payload) as NetworkImpactsInfos;
-                    updateEquipmentsLocal(
-                        eventNodeUuid,
-                        networkImpacts.impactedSubstationsIds,
-                        networkImpacts.deletedEquipments,
-                        networkImpacts.impactedElementTypes ?? []
-                    );
+                    try {
+                        const networkImpacts = JSON.parse(eventData.payload) as NetworkImpactsInfos;
+                        updateEquipmentsLocal(
+                            eventNodeUuid,
+                            networkImpacts.impactedSubstationsIds,
+                            networkImpacts.deletedEquipments,
+                            networkImpacts.impactedElementTypes ?? []
+                        );
+                    } catch (error: unknown) {
+                        console.warn(`Something failed during spreadsheet update, it will be reset`, error);
+                        dispatch(resetEquipments());
+                    }
                 }
             }
         },
-        [builtNodesIds, currentRootNetworkUuid, studyUuid, updateEquipmentsLocal]
+        [builtNodesIds, currentRootNetworkUuid, dispatch, studyUuid, updateEquipmentsLocal]
     );
 
     useNotificationsListener(NotificationsUrlKeys.STUDY, {

src/components/spreadsheet-view/spreadsheet/spreadsheet-toolbar/row-counter/use-filtered-row-counter.tsx

@@ -60,7 +60,7 @@ export function useFilteredRowCounterInfo({
                 }
                 const api = gridRef.current.api;
                 setDisplayedRows(api.getDisplayedRowCount());
-                setTotalRows(Object.values(equipments.equipmentsByNodeId[currentNode.id]).length ?? 0);
+                setTotalRows(Object.values(equipments.equipmentsByNodeId[currentNode.id] ?? {}).length ?? 0);
                 setIsLoading(false);
             }, 600),
         [gridRef, currentNode, disabled, equipments.equipmentsByNodeId]