🐛 Fixing pushing nodes downstream when inserting new node

n8n-io · Nov 7, 2022 · c761186 · c761186
1 parent d66f9d2
commit c761186
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/packages/editor-ui/src/stores/workflows.ts b/packages/editor-ui/src/stores/workflows.ts
@@ -642,7 +642,7 @@ export const useWorkflowsStore = defineStore(STORES.WORKFLOWS, {
 				return node.name === updateInformation.name;
 			});
 
-			if (node === undefined || node === null) {
+			if (node === undefined || node === null || !updateInformation.key) {
 				throw new Error(`Node with the name "${updateInformation.name}" could not be found to set parameter.`);
 			}
 

diff --git a/packages/editor-ui/src/views/NodeView.vue b/packages/editor-ui/src/views/NodeView.vue
@@ -1161,14 +1161,15 @@ export default mixins(
 				const childNodes = workflow.getChildNodes(sourceNodeName);
 				for (const nodeName of childNodes) {
 					const node = this.workflowsStore.nodesByName[nodeName] as INodeUi;
+
 					if (node.position[0] < sourceNode.position[0]) {
 						continue;
 					}
 
 					const updateInformation: INodeUpdatePropertiesInformation = {
 						name: nodeName,
 						properties: {
-							position: { position: [node.position[0] + margin, node.position[1]] },
+							position: [node.position[0] + margin, node.position[1]],
 						},
 					};