From 7f3d57bfaa7726089a250428a622fe810d7f8f35 Mon Sep 17 00:00:00 2001
From: anovazzi1 <otavio2204@gmail.com>
Date: Fri, 22 Sep 2023 14:46:01 -0300
Subject: [PATCH] fix(nodeToolbarComponent): refactor handleSelectChange
 function to use switch statement for better readability and maintainability
 feat(nodeToolbarComponent): add "SaveAll" option to the select dropdown and
 log a message when selected

---
 .../components/nodeToolbarComponent/index.tsx | 26 ++++++++++---------
 1 file changed, 14 insertions(+), 12 deletions(-)

diff --git a/src/frontend/src/pages/FlowPage/components/nodeToolbarComponent/index.tsx b/src/frontend/src/pages/FlowPage/components/nodeToolbarComponent/index.tsx
index 14984a974..ebc29643d 100644
--- a/src/frontend/src/pages/FlowPage/components/nodeToolbarComponent/index.tsx
+++ b/src/frontend/src/pages/FlowPage/components/nodeToolbarComponent/index.tsx
@@ -11,7 +11,7 @@ import {
 import { TabsContext } from "../../../../contexts/tabsContext";
 import EditNodeModal from "../../../../modals/EditNodeModal";
 import { nodeToolbarPropsType } from "../../../../types/components";
-import { classNames, getRandomKeyByssmm } from "../../../../utils/utils";
+import { classNames } from "../../../../utils/utils";
 
 export default function NodeToolbarComponent({
   data,
@@ -53,14 +53,16 @@ export default function NodeToolbarComponent({
   const [selectedValue, setSelectedValue] = useState("");
 
   const handleSelectChange = (event) => {
-    setSelectedValue(event);
-    if (event.includes("advanced")) {
-      return setShowModalAdvanced(true);
-    }
-    setShowModalAdvanced(false);
-    if (event.includes("show")) {
-      setShowNode((prev) => !prev);
-      updateNodeInternals(data.id);
+    switch (event) {
+      case "advanced":
+        setShowModalAdvanced(true);
+        break;
+      case "show":
+        setShowNode((prev) => !prev);
+        updateNodeInternals(data.id);
+        break;
+      case "SaveAll":
+        console.log("SaveAll");
     }
   };
 
@@ -152,7 +154,7 @@ export default function NodeToolbarComponent({
               </SelectTrigger>
             </ShadTooltip>
             <SelectContent>
-              <SelectItem value={getRandomKeyByssmm() + "advanced"}>
+              <SelectItem value={"advanced"}>
                 <div className="flex">
                   <IconComponent
                     name="Settings2"
@@ -161,7 +163,7 @@ export default function NodeToolbarComponent({
                   Edit{" "}
                 </div>{" "}
               </SelectItem>
-              <SelectItem value={getRandomKeyByssmm() + "advanced"}>
+              <SelectItem value={"SaveAll"}>
                 <div className="flex">
                   <IconComponent
                     name="SaveAll"
@@ -171,7 +173,7 @@ export default function NodeToolbarComponent({
                 </div>{" "}
               </SelectItem>
               {isMinimal && (
-                <SelectItem value={getRandomKeyByssmm() + "show"}>
+                <SelectItem value={"show"}>
                   <div className="flex">
                     <IconComponent
                       name={showNode ? "Minimize2" : "Maximize2"}