Merged PR 173: Implements API methods through right click + more (read desc)

- Implements API methods through right click
- Refactor events
- Refactor usage of setHistory and setHistoryCurrentStep into a single function
- Update ContainerOperations documentations
- Added AddContainers in order to add multiple containers + refactor AddContainer to use it
- Fix regression
- Fix AddContainer at index

src/Components/Editor/Actions/SymbolOperations.ts

@@ -13,10 +13,8 @@ export function AddSymbol(
   name: string,
   configuration: IConfiguration,
   fullHistory: IHistoryState[],
-  historyCurrentStep: number,
-  setHistory: Dispatch<SetStateAction<IHistoryState[]>>,
-  setHistoryCurrentStep: Dispatch<SetStateAction<number>>
-): void {
+  historyCurrentStep: number
+): IHistoryState[] {
   const history = GetCurrentHistory(fullHistory, historyCurrentStep);
   const current = history[history.length - 1];
 
@@ -44,17 +42,14 @@ export function AddSymbol(
     symbols: newSymbols,
     selectedSymbolId: newSymbol.id
   });
-  setHistory(history);
-  setHistoryCurrentStep(history.length - 1);
+  return history;
 }
 
 export function SelectSymbol(
   symbolId: string,
   fullHistory: IHistoryState[],
-  historyCurrentStep: number,
-  setHistory: Dispatch<SetStateAction<IHistoryState[]>>,
-  setHistoryCurrentStep: Dispatch<SetStateAction<number>>
-): void {
+  historyCurrentStep: number
+): IHistoryState[] {
   const history = GetCurrentHistory(fullHistory, historyCurrentStep);
   const current = history[history.length - 1];
 
@@ -66,17 +61,14 @@ export function SelectSymbol(
     symbols: structuredClone(current.symbols),
     selectedSymbolId: symbolId
   });
-  setHistory(history);
-  setHistoryCurrentStep(history.length - 1);
+  return history;
 }
 
 export function DeleteSymbol(
   symbolId: string,
   fullHistory: IHistoryState[],
-  historyCurrentStep: number,
-  setHistory: Dispatch<SetStateAction<IHistoryState[]>>,
-  setHistoryCurrentStep: Dispatch<SetStateAction<number>>
-): void {
+  historyCurrentStep: number
+): IHistoryState[] {
   const history = GetCurrentHistory(fullHistory, historyCurrentStep);
   const current = history[history.length - 1];
 
@@ -89,7 +81,7 @@ export function DeleteSymbol(
 
   const newMainContainer = structuredClone(current.mainContainer);
 
-  UnlinkContainers(symbol, newMainContainer);
+  UnlinkSymbolFromContainers(symbol, newMainContainer);
 
   newSymbols.delete(symbolId);
 
@@ -101,13 +93,17 @@ export function DeleteSymbol(
     symbols: newSymbols,
     selectedSymbolId: symbolId
   });
-  setHistory(history);
-  setHistoryCurrentStep(history.length - 1);
+  return history;
 }
 
-function UnlinkContainers(symbol: ISymbolModel, newMainContainer: IContainerModel): void {
+/**
+ * Unlink a symbol to a container and its children
+ * @param symbol Symbol to remove
+ * @param root Container and its children to remove a symbol from
+ */
+function UnlinkSymbolFromContainers(symbol: ISymbolModel, root: IContainerModel): void {
   symbol.linkedContainers.forEach((containerId) => {
-    const container = FindContainerById(newMainContainer, containerId);
+    const container = FindContainerById(root, containerId);
 
     if (container === undefined) {
       return;
@@ -127,10 +123,8 @@ export function OnPropertyChange(
   key: string,
   value: string | number | boolean,
   fullHistory: IHistoryState[],
-  historyCurrentStep: number,
-  setHistory: Dispatch<SetStateAction<IHistoryState[]>>,
-  setHistoryCurrentStep: Dispatch<SetStateAction<number>>
-): void {
+  historyCurrentStep: number
+): IHistoryState[] {
   const history = GetCurrentHistory(fullHistory, historyCurrentStep);
   const current = history[history.length - 1];
 
@@ -166,6 +160,5 @@ export function OnPropertyChange(
     symbols: newSymbols,
     selectedSymbolId: symbol.id
   });
-  setHistory(history);
-  setHistoryCurrentStep(history.length - 1);
+  return history;
 }