fix: fix parameters undo logic

Author: pyphilia

src/js/app.js

@@ -39,6 +39,7 @@ import {
   MESSAGE_COPY_ERROR
 } from "./constants/messages";
 import { saveWorkflow } from "./workflows/saveWorkflow";
+import {setInputValueInElement, setSelectValueInElement} from "./layout/inputs"
 
 export let app;
 
@@ -92,6 +93,11 @@ export let app;
           return { boxId, size };
         });
       },
+      defaultParamsElements() {
+        return this.elements.map(({ boxId, defaultParams }) => {
+          return { boxId, defaultParams };
+        });
+      },
       dataElements() {
         return this.dataTest;
       },
@@ -229,6 +235,27 @@ export let app;
           }
         }
       },
+      
+      /**
+       * watch parameters of elements
+       * on direct changes, nothing changes (mechanic operation)
+       * only apply changes on undo-redo
+       */
+      defaultParamsElements: {
+        deep: true,
+        handler(newValue, oldValue) {
+          const difference = findDifferenceBy(
+            newValue,
+            oldValue,
+            "defaultParams"
+          );
+          for (const box of difference) {
+            setSelectValueInElement(box);
+            setInputValueInElement(box);
+          }
+        }
+      },
+
       /**
        * watch moved elements
        * on direct move operation, nothing changes (mechanic operation)

src/js/layout/inputs.js

@@ -45,21 +45,21 @@ export const closeSelects = () => {
  * @param {string} boxId
  * @param {object} defaultParams
  */
-// export const setSelectValueInElement = ({ boxId, defaultParams }) => {
-//   for (const [key, { value }] of Object.entries(
-//     defaultParams[Types.SELECT.type]
-//   )) {
-//     // we need to precise paper root because of the minimap duplicate elements
-//     const el = $(
-//       `${INTERFACE_ROOT} foreignObject[boxId="${boxId}"] .select[name="${key}"] ${Types.SELECT.tag}`
-//     );
-//     if (el.find(":selected").val() != value) {
-//       $(el)
-//         .val(value)
-//         .trigger("change.select2");
-//     }
-//   }
-// };
+export const setSelectValueInElement = ({ boxId, defaultParams }) => {
+  for (const [key, { value }] of Object.entries(
+    defaultParams[Types.SELECT.type]
+  )) {
+    // we need to precise paper root because of the minimap duplicate elements
+    const el = $(
+      `${INTERFACE_ROOT} foreignObject[boxId="${boxId}"] .select[name="${key}"] ${Types.SELECT.tag}`
+    );
+    if (el.find(":selected").val() != value) {
+      $(el)
+        .val(value)
+        .trigger("change.select2");
+    }
+  }
+};
 
 /**
  * set input value in DOM
@@ -68,22 +68,22 @@ export const closeSelects = () => {
  * @param {string} type
  * @param {object} defaultParams
  */
-// export const setInputValueInElement = ({
-//   boxId,
-//   type: boxName,
-//   defaultParams
-// }) => {
-//   for (const [key, { value }] of Object.entries(
-//     defaultParams[Types.NUMBER.type]
-//   )) {
-//     // we need to precise paper root because of the minimap duplicate elements
-//     const el = $(
-//       `${INTERFACE_ROOT} foreignObject[boxId="${boxId}"] ${Types.NUMBER.tag}[name="${key}"]`
-//     );
-//     el.val(value);
-//     checkInputValue(el, { boxId, boxName });
-//   }
-// };
+export const setInputValueInElement = ({
+  boxId,
+  type: boxName,
+  defaultParams
+}) => {
+  for (const [key, { value }] of Object.entries(
+    defaultParams[Types.NUMBER.type]
+  )) {
+    // we need to precise paper root because of the minimap duplicate elements
+    const el = $(
+      `${INTERFACE_ROOT} foreignObject[boxId="${boxId}"] ${Types.NUMBER.tag}[name="${key}"]`
+    );
+    el.val(value);
+    checkInputValue(el, { boxId, boxName });
+  }
+};
 
 /**
  * reset value of input or select element