EVEREST-1712-fix-infinite-loop (#889)

* EVEREST-1712-fix-infinite-loop --------- Co-authored-by: solovevayaroslavna <[email protected]>
percona · Dec 2, 2024 · 8e3f5a7 · 8e3f5a7
1 parent a54e5db
commit 8e3f5a7
Showing 1 changed file with 18 additions and 17 deletions.
diff --git a/ui/apps/everest/src/hooks/rbac/rbac.ts b/ui/apps/everest/src/hooks/rbac/rbac.ts
@@ -68,7 +68,8 @@ export const useNamespacePermissionsForResource = (
     create: [],
     delete: [],
   });
-  const { data: namespaces = [], isFetching } = useNamespaces();
+
+  const { data: namespaces, isFetching } = useNamespaces();
 
   const checkPermissions = useCallback(async () => {
     const newPermissions: Record<RBACAction, string[]> = {
@@ -79,24 +80,24 @@ export const useNamespacePermissionsForResource = (
     };
     const permissionsPromisesArr: Promise<void>[] = [];
 
-    for (const namespace of namespaces) {
-      ['read', 'update', 'delete', 'create'].forEach((action) => {
-        permissionsPromisesArr.push(
-          can(
-            action as RBACAction,
-            resource,
-            `${namespace}/${specificResource}`
-          ).then((canDo) => {
-            if (canDo) {
-              newPermissions[action as RBACAction].push(namespace);
-            }
-          })
-        );
-      });
+    if (namespaces) {
+      for (const namespace of namespaces) {
+        ['read', 'update', 'delete', 'create'].forEach((action) => {
+          permissionsPromisesArr.push(
+            can(
+              action as RBACAction,
+              resource,
+              `${namespace}/${specificResource}`
+            ).then((canDo) => {
+              if (canDo) {
+                newPermissions[action as RBACAction].push(namespace);
+              }
+            })
+          );
+        });
+      }
     }
-
     await Promise.all(permissionsPromisesArr);
-
     setPermissions(newPermissions);
   }, [namespaces, resource, specificResource]);