fix: Always clear popupWindowState before showing popup from form trigger (n8n-io#13363)

michael-radency · web-flow · commit b7f12650f1f4 · 2025-02-19T15:03:26.000+02:00
diff --git a/packages/editor-ui/src/utils/executionUtils.ts b/packages/editor-ui/src/utils/executionUtils.ts
@@ -130,7 +130,10 @@ export function displayForm({
 		if (node.name === destinationNode || !node.disabled) {
 			let testUrl = '';
 			if (node.type === FORM_TRIGGER_NODE_TYPE) testUrl = getTestUrl(node);
-			if (testUrl && source !== 'RunData.ManualChatMessage') openFormPopupWindow(testUrl);
+			if (testUrl && source !== 'RunData.ManualChatMessage') {
+				clearPopupWindowState();
+				openFormPopupWindow(testUrl);
+			}
 		}
 	}
 }

Original file line number	Diff line number	Diff line change
`@@ -130,7 +130,10 @@ export function displayForm({`
`130`	`130`	`if (node.name === destinationNode \|\| !node.disabled) {`
`131`	`131`	`let testUrl = '';`
`132`	`132`	`if (node.type === FORM_TRIGGER_NODE_TYPE) testUrl = getTestUrl(node);`
`133`		`- if (testUrl && source !== 'RunData.ManualChatMessage') openFormPopupWindow(testUrl);`
	`133`	`+ if (testUrl && source !== 'RunData.ManualChatMessage') {`
	`134`	`+ clearPopupWindowState();`
	`135`	`+ openFormPopupWindow(testUrl);`
	`136`	`+ }`
`134`	`137`	`}`
`135`	`138`	`}`
`136`	`139`	`}`