mudhorn/n8n
1
0
Fork 0
mirror of https://github.com/n8n-io/n8n.git synced 2025-03-05 20:50:17 -08:00
Code Issues Actions Packages Projects Releases Wiki Activity

🐛🐘 Fix escaping workflow name dedup

Browse source
This commit is contained in:
Iván Ovejero 2021-05-31 19:14:09 +02:00
parent 7a5e69d4e5
commit a992a2da75
3 changed files with 30 additions and 31 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

20
packages/cli/src/databases/mysqldb/migrations/1620826335440-UniqueWorkflowNames.ts
View file

@ -13,27 +13,27 @@ export class UniqueWorkflowNames1620826335440 implements MigrationInterface {
`); `);
for (const { name } of workflowNames) { for (const { name } of workflowNames) {
const [duplicatesQuery, parameters] = queryRunner.connection.driver.escapeQueryWithParameters(`
const duplicates = await queryRunner.query(`
SELECT id, name SELECT id, name
FROM ${tablePrefix}workflow_entity FROM ${tablePrefix}workflow_entity
WHERE name = '${name}' WHERE name = :name
ORDER BY createdAt ASC ORDER BY createdAt ASC
`); `, { name }, {});
const duplicates = await queryRunner.query(duplicatesQuery, parameters);
if (duplicates.length > 1) { if (duplicates.length > 1) {
await Promise.all(duplicates.map(({ id, name }: { id: number; name: string; }, index: number) => { await Promise.all(duplicates.map(({ id, name }: { id: number; name: string; }, index: number) => {
if (index === 0) return Promise.resolve(); if (index === 0) return Promise.resolve();
return queryRunner.query(` const [updateQuery, updateParams] = queryRunner.connection.driver.escapeQueryWithParameters(`
UPDATE ${tablePrefix}workflow_entity UPDATE ${tablePrefix}workflow_entity
SET name = '${name} ${index + 1}' SET name = :name
WHERE id = '${id}' WHERE id = '${id}'
`); `, { name: `${name} ${index + 1}`}, {});
return queryRunner.query(updateQuery, updateParams);
})); }));
} }
} }
await queryRunner.query('ALTER TABLE `' + tablePrefix + 'workflow_entity` ADD UNIQUE INDEX `IDX_' + tablePrefix + '943d8f922be094eb507cb9a7f9` (`name`)'); await queryRunner.query('ALTER TABLE `' + tablePrefix + 'workflow_entity` ADD UNIQUE INDEX `IDX_' + tablePrefix + '943d8f922be094eb507cb9a7f9` (`name`)');

21
packages/cli/src/databases/postgresdb/migrations/1620824779533-UniqueWorkflowNames.ts
View file

@ -12,34 +12,33 @@ export class UniqueWorkflowNames1620824779533 implements MigrationInterface {
tablePrefix = schema + '.' + tablePrefix; tablePrefix = schema + '.' + tablePrefix;
} }
const workflowNames = await queryRunner.query(` const workflowNames = await queryRunner.query(`
SELECT name SELECT name
FROM ${tablePrefix}workflow_entity FROM ${tablePrefix}workflow_entity
`); `);
for (const { name } of workflowNames) { for (const { name } of workflowNames) {
const [duplicatesQuery, parameters] = queryRunner.connection.driver.escapeQueryWithParameters(`
const duplicates = await queryRunner.query(`
SELECT id, name SELECT id, name
FROM ${tablePrefix}workflow_entity FROM ${tablePrefix}workflow_entity
WHERE name = '${name}' WHERE name = :name
ORDER BY "createdAt" ASC ORDER BY "createdAt" ASC
`); `, { name }, {});
const duplicates = await queryRunner.query(duplicatesQuery, parameters);
if (duplicates.length > 1) { if (duplicates.length > 1) {
await Promise.all(duplicates.map(({ id, name }: { id: number; name: string; }, index: number) => { await Promise.all(duplicates.map(({ id, name }: { id: number; name: string; }, index: number) => {
if (index === 0) return Promise.resolve(); if (index === 0) return Promise.resolve();
return queryRunner.query(` const [updateQuery, updateParams] = queryRunner.connection.driver.escapeQueryWithParameters(`
UPDATE ${tablePrefix}workflow_entity UPDATE ${tablePrefix}workflow_entity
SET name = '${name} ${index + 1}' SET name = :name
WHERE id = '${id}' WHERE id = '${id}'
`); `, { name: `${name} ${index + 1}`}, {});
return queryRunner.query(updateQuery, updateParams);
})); }));
} }
} }
await queryRunner.query(`CREATE UNIQUE INDEX "IDX_${tablePrefixPure}a252c527c4c89237221fe2c0ab" ON ${tablePrefix}workflow_entity ("name") `); await queryRunner.query(`CREATE UNIQUE INDEX "IDX_${tablePrefixPure}a252c527c4c89237221fe2c0ab" ON ${tablePrefix}workflow_entity ("name") `);

20
packages/cli/src/databases/sqlite/migrations/1620821879465-UniqueWorkflowNames.ts
View file

@ -13,27 +13,27 @@ export class UniqueWorkflowNames1620821879465 implements MigrationInterface {
`); `);
for (const { name } of workflowNames) { for (const { name } of workflowNames) {
const [duplicatesQuery, parameters] = queryRunner.connection.driver.escapeQueryWithParameters(`
const duplicates = await queryRunner.query(`
SELECT id, name SELECT id, name
FROM "${tablePrefix}workflow_entity" FROM "${tablePrefix}workflow_entity"
WHERE name = "${name}" WHERE name = :name
ORDER BY createdAt ASC ORDER BY createdAt ASC
`); `, { name }, {});
const duplicates = await queryRunner.query(duplicatesQuery, parameters);
if (duplicates.length > 1) { if (duplicates.length > 1) {
await Promise.all(duplicates.map(({ id, name }: { id: number; name: string; }, index: number) => { await Promise.all(duplicates.map(({ id, name }: { id: number; name: string; }, index: number) => {
if (index === 0) return Promise.resolve(); if (index === 0) return Promise.resolve();
return queryRunner.query(` const [updateQuery, updateParams] = queryRunner.connection.driver.escapeQueryWithParameters(`
UPDATE "${tablePrefix}workflow_entity" UPDATE "${tablePrefix}workflow_entity"
SET name = "${name} ${index + 1}" SET name = :name
WHERE id = '${id}' WHERE id = '${id}'
`); `, { name: `${name} ${index + 1}`}, {});
return queryRunner.query(updateQuery, updateParams);
})); }));
} }
} }
await queryRunner.query(`CREATE UNIQUE INDEX "IDX_${tablePrefix}943d8f922be094eb507cb9a7f9" ON "${tablePrefix}workflow_entity" ("name") `); await queryRunner.query(`CREATE UNIQUE INDEX "IDX_${tablePrefix}943d8f922be094eb507cb9a7f9" ON "${tablePrefix}workflow_entity" ("name") `);