add check for arr

n8n-io · Aug 4, 2022 · 40a43f5 · 40a43f5
1 parent 43a91b8
commit 40a43f5
Show file tree

Hide file tree

Showing 3 changed files with 22 additions and 0 deletions.
diff --git a/packages/cli/src/databases/migrations/mysqldb/1658932910559-AddNodeIds.ts b/packages/cli/src/databases/migrations/mysqldb/1658932910559-AddNodeIds.ts
@@ -20,6 +20,9 @@ export class AddNodeIds1658932910559 implements MigrationInterface {
 		await runChunked(queryRunner, workflowsQuery, (workflows) => {
 			workflows.forEach(async (workflow) => {
 				const nodes = workflow.nodes;
+				if (!Array.isArray(nodes)) {
+					return;
+				}
 				// @ts-ignore
 				nodes.forEach((node) => {
 					if (!node.id) {
@@ -55,6 +58,9 @@ export class AddNodeIds1658932910559 implements MigrationInterface {
 		await runChunked(queryRunner, workflowsQuery, (workflows) => {
 			workflows.forEach(async (workflow) => {
 				const nodes = workflow.nodes;
+				if (!Array.isArray(nodes)) {
+					return;
+				}
 				// @ts-ignore
 				nodes.forEach((node) => delete node.id );
 

diff --git a/packages/cli/src/databases/migrations/postgresdb/1658932090381-AddNodeIds.ts b/packages/cli/src/databases/migrations/postgresdb/1658932090381-AddNodeIds.ts
@@ -26,6 +26,10 @@ export class AddNodeIds1658932090381 implements MigrationInterface {
 		await runChunked(queryRunner, workflowsQuery, (workflows) => {
 			workflows.forEach(async (workflow) => {
 				const nodes = workflow.nodes;
+				if (!Array.isArray(nodes)) {
+					return;
+				}
+
 				// @ts-ignore
 				nodes.forEach((node) => {
 					if (!node.id) {
@@ -67,6 +71,10 @@ export class AddNodeIds1658932090381 implements MigrationInterface {
 		await runChunked(queryRunner, workflowsQuery, (workflows) => {
 			workflows.forEach(async (workflow) => {
 				const nodes = workflow.nodes;
+				if (!Array.isArray(nodes)) {
+					return;
+				}
+
 				// @ts-ignore
 				nodes.forEach((node) => delete node.id );
 

diff --git a/packages/cli/src/databases/migrations/sqlite/1658930531669-AddNodeIds.ts b/packages/cli/src/databases/migrations/sqlite/1658930531669-AddNodeIds.ts
@@ -24,6 +24,10 @@ export class AddNodeIds1658930531669 implements MigrationInterface {
 		await runChunked(queryRunner, workflowsQuery, (workflows) => {
 			workflows.forEach(async (workflow) => {
 				const nodes = JSON.parse(workflow.nodes);
+				if (!Array.isArray(nodes)) {
+					return;
+				}
+
 				nodes.forEach((node: INode) => {
 					if (!node.id) {
 						node.id = uuid();
@@ -61,6 +65,10 @@ export class AddNodeIds1658930531669 implements MigrationInterface {
 		await runChunked(queryRunner, workflowsQuery, (workflows) => {
 			workflows.forEach(async (workflow) => {
 				const nodes = JSON.parse(workflow.nodes);
+				if (!Array.isArray(nodes)) {
+					return;
+				}
+
 				// @ts-ignore
 				nodes.forEach((node) => delete node.id );