From e5066c495b8ea9a5195a792ae6f42820d77611d2 Mon Sep 17 00:00:00 2001
From: monilprajapati <monillprajapatii@gmail.com>
Date: Fri, 28 Jun 2024 17:40:08 +0530
Subject: [PATCH] Fixed that dynamic var name changes in the parameters

---
 frontend/src/components/ParsedAction.jsx | 15 ++++++++-------
 frontend/src/views/AngularWorkflow.jsx   |  3 ---
 2 files changed, 8 insertions(+), 10 deletions(-)

diff --git a/frontend/src/components/ParsedAction.jsx b/frontend/src/components/ParsedAction.jsx
index 81cab6f1..e3cf400c 100755
--- a/frontend/src/components/ParsedAction.jsx
+++ b/frontend/src/components/ParsedAction.jsx
@@ -179,7 +179,7 @@ const ParsedAction = (props) => {
   const [activateHidingBodyButton, setActivateHidingBodyButton] = React.useState(false)
   const [appActionName, setAppActionName] = React.useState(selectedAction.label);
   const [delay, setDelay] = React.useState(selectedAction?.execution_delay || 0);
-  
+  const [prevActionName, setPrevActionName] = React.useState(selectedAction.label);
   const [fieldCount, setFieldCount] = React.useState(0);
   const [hiddenDescription, setHiddenDescription] = React.useState(true);
   const [autoCompleting, setAutocompleting] = React.useState(false);
@@ -406,6 +406,10 @@ const ParsedAction = (props) => {
 			if (selectedAction.label !== appActionName) {
 				setAppActionName(selectedAction.label);
 			}
+
+			if(selectedAction.label !== prevActionName){
+				setPrevActionName(selectedAction.label)
+			}
 		
 			// Only set delay if it has changed
 			const newDelay = selectedAction?.execution_delay || 0;
@@ -1292,7 +1296,6 @@ const ParsedAction = (props) => {
 	
 
   const selectedAppIcon = selectedAction.large_image
-  var baselabel = selectedAction.label
   return (
     <div style={appApiViewStyle} id="parsed_action_view">
 
@@ -1611,8 +1614,8 @@ const ParsedAction = (props) => {
 								}
 								onBlur={(e) => {
 									// Copy the name value
-									const name = appActionName
-									const parsedBaseLabel = "$"+baselabel.toLowerCase().replaceAll(" ", "_")
+									const name = e.target.value
+									const parsedBaseLabel = "$"+prevActionName.toLowerCase().replaceAll(" ", "_")
 									const newname = "$"+name.toLowerCase().replaceAll(" ", "_")
 
 									// Check if it's the same as the current name in use
@@ -1741,7 +1744,6 @@ const ParsedAction = (props) => {
 										}
 
 										const params = workflow.actions[key].parameters
-										console.log(params)
 										if (params === null || params === undefined) {
 											continue
 										}
@@ -1765,7 +1767,6 @@ const ParsedAction = (props) => {
 													// Need to make sure e.g. changing the first here doesn't change the 2nd
 													// $change_me
 													// $change_me_2
-													
 													const foundindex = param.value.toLowerCase().indexOf(parsedBaseLabel, previous)
 													if (foundindex === previous && foundindex !== 0) {
 														break
@@ -1808,7 +1809,7 @@ const ParsedAction = (props) => {
 
 									setWorkflow(workflow);
                   					setUpdate(Math.random());
-									baselabel = name
+									setPrevActionName(name)
 								}}
 							/>
 						</div>
diff --git a/frontend/src/views/AngularWorkflow.jsx b/frontend/src/views/AngularWorkflow.jsx
index 889a407b..292a494e 100755
--- a/frontend/src/views/AngularWorkflow.jsx
+++ b/frontend/src/views/AngularWorkflow.jsx
@@ -21607,9 +21607,6 @@ const releaseToConnectLabel = "Release to Connect"
 			</Paper>
 		  )
 	  }
-    //! Logs
-    console.log("Workflow state", workflow)
-    console.log("All revision", allRevisions)
 	  const drawerData = originalWorkflow !== undefined && originalWorkflow !== null ?
       <div style={{ height: "100%"}}>
       <Typography variant="h5" style={{ paddingLeft: 25, paddingTop:25, backgroundColor: theme.palette.surfaceColor,  height: "8%" }}>