admin/n8n-enterprise-unlocked
1
0
Fork 0
mirror of https://github.com/Abdulazizzn/n8n-enterprise-unlocked.git synced 2025-12-16 09:36:44 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(core): Ensure getNodeOutputs always returns an array (#19069)

Browse Source
This commit is contained in:
Csaba Tuncsik
2025-09-02 15:16:42 +02:00
committed by GitHub
parent 9569965a0b
commit 0f22f3be92
2 changed files with 9 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
packages/workflow/src/node-helpers.ts
View File

@@ -1047,12 +1047,15 @@ export function getNodeOutputs(
} else {
// Calculate the outputs dynamically
try {
outputs = (workflow.expression.getSimpleParameterValue(
const result = workflow.expression.getSimpleParameterValue(
node,
nodeTypeData.outputs,
'internal',
{},
) || []) as NodeConnectionType[];
);
outputs = Array.isArray(result)
? (result as Array<NodeConnectionType | INodeOutputConfiguration>)
: [];
} catch (e) {
console.warn('Could not calculate outputs dynamically for node: ', node.name);
}

13
packages/workflow/src/workflow.ts
View File

@@ -697,15 +697,10 @@ export class Workflow {
const outputs = NodeHelpers.getNodeOutputs(this, node, nodeType.description);
const nonMainConnectionTypes: NodeConnectionType[] = [];
// Defensive check: NodeHelpers.getNodeOutputs should always return an array,
// but in some edge cases (particularly during testing with incomplete node setup),
// it may return undefined or null
if (Array.isArray(outputs)) {
for (const output of outputs) {
const type = typeof output === 'string' ? output : output.type;
if (type !== NodeConnectionTypes.Main) {
nonMainConnectionTypes.push(type);
}
for (const output of outputs) {
const type = typeof output === 'string' ? output : output.type;
if (type !== NodeConnectionTypes.Main) {
nonMainConnectionTypes.push(type);
}
}