Cleaned up code before merging with dev50.

Co-authored-by: Chris LeBrett <[email protected]>
Co-authored-by: Robby Tipton <[email protected]>
Co-authored-by: Joseph Park <[email protected]>
Co-authored-by: David Kim <[email protected]>
Co-authored-by: Kevin HoEun Lee <[email protected]>

Author: 5 people

src/backend/linkFiber.ts

@@ -134,6 +134,7 @@ function updateSnapShotTree(snap: Snapshot, mode: Mode): void {
     // Clears circular component table
     circularComponentTable.clear();
     // creates snapshot that is a tree based on properties in fiberRoot object
+    componentActionsRecord.clear();
     snap.tree = createTree(current);
   }
   // sends the updated tree back

src/backend/masterState.ts

@@ -15,21 +15,19 @@ import {
 // Information on these components include ComponentData as well as state
 // For class components, there will be one "component" for each snapshot
 // For functional components that utilize Hooks, there will be one "component" for each setter/getter every time we have a new snapshot
-const componentActionsRecord: HookStates = [];
+let componentActionsRecord: HookStates = [];
 let index = 0;
 
 export default {
+  clear: () => {
+    componentActionsRecord = [];
+    index = 0;
+  },
   // adds new component to ComponentActionsRecord
   saveNew: (state, component, name): number => {
     componentActionsRecord[index] = { state, component, name };
     index++;
 
-    for (let i = 0; i < componentActionsRecord.length - 2; i++) {
-      if (componentActionsRecord[i].name === name) {
-        componentActionsRecord[i] = { state, component, name };
-      }
-    }
-
     return index - 1;
   },
   getRecordByIndex: (inputIndex: number): HookStateItem =>