fix: Update copyNodeEnabled type and ensure proper handling in thunks

Author: priyanshu6238

src/flowTypes.ts

@@ -53,7 +53,7 @@ export interface Endpoints {
   validateMedia: string;
   interactives: string;
   sheets: string;
-  copyNodeEnabled?: string | boolean;
+  copyNodeEnabled?: string;
 }
 
 export interface FlowEditorConfig {

src/store/__snapshots__/thunks.test.ts.snap

@@ -369,6 +369,7 @@ Array [
           "segments": Object {},
         },
         "activityInterval": 60000,
+        "copyNodeEnabled": false,
         "currentRevision": null,
         "debug": null,
         "dragActive": false,
@@ -409,6 +410,7 @@ Array [
           "segments": Object {},
         },
         "activityInterval": 60000,
+        "copyNodeEnabled": false,
         "currentRevision": null,
         "debug": null,
         "dragActive": false,
@@ -1135,6 +1137,7 @@ Array [
           "segments": Object {},
         },
         "activityInterval": 60000,
+        "copyNodeEnabled": false,
         "currentRevision": null,
         "debug": null,
         "dragActive": false,

src/store/editor.ts

@@ -83,7 +83,7 @@ export interface EditorState {
   scrollToNode: string;
   scrollToAction: string;
   toast: ToastMessage | null;
-  copyNodeEnabled?: boolean;
+  copyNodeEnabled: boolean;
 }
 
 export interface ModalMessage {
@@ -126,7 +126,8 @@ export const initialState: EditorState = {
 
   scrollToNode: null,
   scrollToAction: null,
-  toast: null
+  toast: null,
+  copyNodeEnabled: false
 };
 
 // Action Creator

src/store/helpers.ts

@@ -820,20 +820,20 @@ export const remapLocalization = (
   localization: LocalizationMap,
   uuidMap: Record<string, string>
 ): LocalizationMap => {
-  let changed = false;
   const result: LocalizationMap = {};
   Object.keys(localization).forEach(lang => {
     const langMap = localization[lang];
-    const newLangMap: { [uuid: string]: any } = { ...langMap };
+    const newLangMap: { [uuid: string]: any } = {};
     Object.keys(uuidMap).forEach(oldUUID => {
       if (langMap[oldUUID]) {
         newLangMap[uuidMap[oldUUID]] = langMap[oldUUID];
-        changed = true;
       }
     });
-    result[lang] = newLangMap;
+    if (Object.keys(newLangMap).length > 0) {
+      result[lang] = newLangMap;
+    }
   });
-  return changed ? result : localization;
+  return result;
 };
 
 export const extractLocalizationForNode = (

src/store/thunks.ts

@@ -437,13 +437,15 @@ export const fetchFlow = (endpoints: Endpoints, uuid: string, forceSave = false)
 
   if (endpoints.copyNodeEnabled) {
     axios
-      .get(endpoints.copyNodeEnabled as string)
+      .get(endpoints.copyNodeEnabled)
       .then((response: any) => {
         dispatch(mergeEditorState({ copyNodeEnabled: !!response.data?.is_enabled }));
       })
       .catch(() => {
         dispatch(mergeEditorState({ copyNodeEnabled: false }));
       });
+  } else {
+    dispatch(mergeEditorState({ copyNodeEnabled: false }));
   }
 
   getFlowDetails(assetStore.revisions)
@@ -1257,7 +1259,8 @@ export const copyNode = (nodeUUID: string) => (
   const primary = nodes[nodeUUID];
   if (!primary) return;
 
-  let sourceLocalization = extractLocalizationForNode(definition.localization, primary);
+  const localization = definition?.localization || {};
+  let sourceLocalization = extractLocalizationForNode(localization, primary);
 
   const payload: ClipboardPayload = {
     primary,
@@ -1276,7 +1279,7 @@ export const copyNode = (nodeUUID: string) => (
       };
       sourceLocalization = mergeLocalizations(
         sourceLocalization,
-        extractLocalizationForNode(definition.localization, paired)
+        extractLocalizationForNode(localization, paired)
       );
       payload.sourceLocalization = sourceLocalization;
     }
@@ -1322,16 +1325,15 @@ export const pasteNode = (position: FlowPosition) => (
     flowContext: { nodes, assetStore, definition }
   } = getState();
 
+  const localization = definition?.localization || {};
+
   const cloned = cloneNodeWithNewUUIDs(primary);
   cloned.node = resolveResultNames(cloned.node, nodes);
   cloned.ui = { ...cloned.ui, position };
 
   const primaryUUIDMap = buildUUIDMap(primary, cloned);
-  const remappedPrimary = remapLocalization(
-    sourceLocalization || definition.localization,
-    primaryUUIDMap
-  );
-  let updatedLocalization = mergeLocalizations(definition.localization, remappedPrimary);
+  const remappedPrimary = remapLocalization(sourceLocalization || localization, primaryUUIDMap);
+  let updatedLocalization = mergeLocalizations(localization, remappedPrimary);
 
   let updatedNodes = nodes;
 
@@ -1351,7 +1353,7 @@ export const pasteNode = (position: FlowPosition) => (
     clonedPaired.inboundConnections = { [cloned.node.exits[0].uuid]: cloned.node.uuid };
 
     const remappedPaired = remapLocalization(
-      sourceLocalization || definition.localization,
+      sourceLocalization || localization,
       buildUUIDMap(paired, clonedPaired)
     );
     updatedLocalization = mergeLocalizations(updatedLocalization, remappedPaired);
@@ -1371,7 +1373,7 @@ export const pasteNode = (position: FlowPosition) => (
     dispatch(updateAssets(mutators.addFlowResult(assetStore, cloned.node)));
   }
 
-  if (updatedLocalization !== definition.localization) {
+  if (updatedLocalization !== localization) {
     dispatch(updateDefinition({ ...definition, localization: updatedLocalization }));
   }