fix: improve getAllConnections — rename param, add optional chaining, fallback to empty array

- Rename connectionData → updatedConnection for clarity
- Add optional chaining (updatedConnection?.id, c?.id) for edge cases
- Ensure connectionList always falls back to [] if API returns undefined/null
- Applied to both NewEnv.jsx and NewProject.jsx

Author: wicky-zipstack

frontend/src/base/components/environment/NewEnv.jsx

@@ -250,23 +250,24 @@ const NewEnv = ({
   );
 
   const getAllConnections = useCallback(
-    async (connectionData) => {
-      if (connectionData) {
+    async (updatedConnection) => {
+      if (updatedConnection?.id) {
         setConnectionList((prev) => {
-          const exists = prev.some((c) => c.id === connectionData.id);
+          const list = prev || [];
+          const exists = list.some((c) => c?.id === updatedConnection.id);
           if (exists) {
-            return prev.map((c) =>
-              c.id === connectionData.id ? connectionData : c
+            return list.map((c) =>
+              c?.id === updatedConnection.id ? updatedConnection : c
             );
           }
-          return [...prev, connectionData];
+          return [...list, updatedConnection];
         });
         return;
       }
       setLoading(true);
       try {
         const data = await fetchAllConnections(axiosRef, selectedOrgId);
-        setConnectionList(data?.filter((el) => !el?.is_sample_project));
+        setConnectionList(data?.filter((el) => !el?.is_sample_project) || []);
       } catch (error) {
         console.error(error);
         notify({ error });

frontend/src/base/new-project/NewProject.jsx

@@ -67,22 +67,23 @@ function NewProject({ open, setOpen, getAllProject, id }) {
   }, [form, setOpen]);
 
   const getAllConnections = useCallback(
-    async (connectionData) => {
-      if (connectionData) {
+    async (updatedConnection) => {
+      if (updatedConnection?.id) {
         setConnectionList((prev) => {
-          const exists = prev.some((c) => c.id === connectionData.id);
+          const list = prev || [];
+          const exists = list.some((c) => c?.id === updatedConnection.id);
           if (exists) {
-            return prev.map((c) =>
-              c.id === connectionData.id ? connectionData : c
+            return list.map((c) =>
+              c?.id === updatedConnection.id ? updatedConnection : c
             );
           }
-          return [...prev, connectionData];
+          return [...list, updatedConnection];
         });
         return;
       }
       try {
         const data = await getAllConnectionsApi(axiosPrivate, selectedOrgId);
-        setConnectionList(data);
+        setConnectionList(data || []);
       } catch (error) {
         console.error(error);
         notify({ error });