TryGhost/Ghost
1
0
Fork 0
mirror of https://github.com/TryGhost/Ghost.git synced 2024-12-25 03:44:29 +03:00
Code Issues Projects Releases Wiki Activity

Updated logging messages in migration commands

Browse Source 
no issue

- by switching around the the columns and tables, it becomes a
  lot easier to read the log line in a natural order
This commit is contained in:
Daniel Lockyer 2021-03-04 13:53:06 +00:00
parent ae090b0a13
commit 4e18606942
No known key found for this signature in database
GPG Key ID: FFBC6FA2A6F6ABC1
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
core/server/data/schema/commands.js
View File

@ -199,7 +199,7 @@ async function addForeign({fromTable, fromColumn, toTable, toColumn, cascadeDele
const hasForeignKey = await hasForeign({fromTable, fromColumn, toTable, toColumn, transaction}); const hasForeignKey = await hasForeign({fromTable, fromColumn, toTable, toColumn, transaction});
if (!hasForeignKey) { if (!hasForeignKey) {
logging.info(`Adding foreign key for: ${fromColumn} in ${fromTable} to ${toColumn} in ${toTable}`); logging.info(`Adding foreign key from ${fromTable}.${fromColumn} to ${toTable}.${toColumn}`);
//disable and re-enable foreign key checks on sqlite because of https://github.com/knex/knex/issues/4155 //disable and re-enable foreign key checks on sqlite because of https://github.com/knex/knex/issues/4155
let foreignKeysEnabled; let foreignKeysEnabled;
@ -224,7 +224,7 @@ async function addForeign({fromTable, fromColumn, toTable, toColumn, cascadeDele
} }
} }
} else { } else {
logging.warn(`Skipped adding foreign key for ${fromColumn} in ${fromTable} to ${toColumn} in ${toTable} - foreign key already exists`); logging.warn(`Skipped adding foreign key from ${fromTable}.${fromColumn} to ${toTable}.${toColumn} - foreign key already exists`);
} }
} }
@ -243,7 +243,7 @@ async function dropForeign({fromTable, fromColumn, toTable, toColumn, transactio
const hasForeignKey = await hasForeign({fromTable, fromColumn, toTable, toColumn, transaction}); const hasForeignKey = await hasForeign({fromTable, fromColumn, toTable, toColumn, transaction});
if (hasForeignKey) { if (hasForeignKey) {
logging.info(`Dropping foreign key for: ${fromColumn} in ${fromTable} to ${toColumn} in ${toTable}`); logging.info(`Dropping foreign key from ${fromTable}.${fromColumn} to ${toTable}.${toColumn}`);
//disable and re-enable foreign key checks on sqlite because of https://github.com/knex/knex/issues/4155 //disable and re-enable foreign key checks on sqlite because of https://github.com/knex/knex/issues/4155
let foreignKeysEnabled; let foreignKeysEnabled;
@ -264,7 +264,7 @@ async function dropForeign({fromTable, fromColumn, toTable, toColumn, transactio
} }
} }
} else { } else {
logging.warn(`Skipped dropping foreign key for ${fromColumn} in ${fromTable} to ${toColumn} in ${toTable} - foreign key does not exist`); logging.warn(`Skipped dropping foreign key from ${fromTable}.${fromColumn} to ${toTable}.${toColumn} - foreign key does not exist`);
} }
} }