Backport #6008 (#7431)

Fix for MySQL fk constraint names to incorporate referring table name
and avoid possible name collisions.

Cherry-picked from 85477ddc
See also issue #5826

lib/dialects/mysql/query-generator.js

@@ -103,6 +103,7 @@ var QueryGenerator = {
         key: this.quoteIdentifier(key),
         key: this.quoteIdentifier(key),
         definition: this.attributeToSQL(dataType, {
         definition: this.attributeToSQL(dataType, {
           context: 'addColumn',
           context: 'addColumn',
+          tableName: table,
           foreignKey: key
           foreignKey: key
         })
         })
       });
       });
@@ -129,7 +130,7 @@ var QueryGenerator = {
       var definition = attributes[attributeName];
       var definition = attributes[attributeName];
       if (definition.match(/REFERENCES/)) {
       if (definition.match(/REFERENCES/)) {
         constraintString.push(Utils._.template('<%= fkName %> FOREIGN KEY (<%= attrName %>) <%= definition %>')({
         constraintString.push(Utils._.template('<%= fkName %> FOREIGN KEY (<%= attrName %>) <%= definition %>')({
-          fkName: this.quoteIdentifier(attributeName + '_foreign_idx'),
+          fkName: this.quoteIdentifier(tableName + '_' + attributeName + '_foreign_idx'),
           attrName: this.quoteIdentifier(attributeName),
           attrName: this.quoteIdentifier(attributeName),
           definition: definition.replace(/.+?(?=REFERENCES)/,'')
           definition: definition.replace(/.+?(?=REFERENCES)/,'')
         }));
         }));
@@ -271,7 +272,7 @@ var QueryGenerator = {
         var attrName = options.foreignKey;
         var attrName = options.foreignKey;
 
 
         template += Utils._.template(', ADD CONSTRAINT <%= fkName %> FOREIGN KEY (<%= attrName %>)')({
         template += Utils._.template(', ADD CONSTRAINT <%= fkName %> FOREIGN KEY (<%= attrName %>)')({
-          fkName: this.quoteIdentifier(attrName + '_foreign_idx'),
+          fkName: this.quoteIdentifier(options.tableName + '_' + attrName + '_foreign_idx'),
           attrName: this.quoteIdentifier(attrName)
           attrName: this.quoteIdentifier(attrName)
         });
         });
       }
       }

test/unit/sql/add-column.test.js

@@ -39,7 +39,7 @@ if (current.dialect.name === 'mysql') {
           onUpdate: 'cascade',
           onUpdate: 'cascade',
           onDelete: 'cascade'
           onDelete: 'cascade'
         })), {
         })), {
-            mysql: 'ALTER TABLE `users` ADD `level_id` INTEGER, ADD CONSTRAINT `level_id_foreign_idx` FOREIGN KEY (`level_id`) REFERENCES `level` (`id`) ON DELETE CASCADE ON UPDATE CASCADE;',
+            mysql: 'ALTER TABLE `users` ADD `level_id` INTEGER, ADD CONSTRAINT `users_level_id_foreign_idx` FOREIGN KEY (`level_id`) REFERENCES `level` (`id`) ON DELETE CASCADE ON UPDATE CASCADE;',
         });
         });
       });
       });
 
 

test/unit/sql/change-column.test.js

@@ -64,7 +64,7 @@ if (current.dialect.name !== 'sqlite') {
         }).then(function(sql){
         }).then(function(sql){
           expectsql(sql, {
           expectsql(sql, {
             mssql: 'ALTER TABLE [users] ADD CONSTRAINT [level_id_foreign_idx] FOREIGN KEY ([level_id]) REFERENCES [level] ([id]) ON DELETE CASCADE;',
             mssql: 'ALTER TABLE [users] ADD CONSTRAINT [level_id_foreign_idx] FOREIGN KEY ([level_id]) REFERENCES [level] ([id]) ON DELETE CASCADE;',
-            mysql: 'ALTER TABLE `users` ADD CONSTRAINT `level_id_foreign_idx` FOREIGN KEY (`level_id`) REFERENCES `level` (`id`) ON DELETE CASCADE ON UPDATE CASCADE;',
+            mysql: 'ALTER TABLE `users` ADD CONSTRAINT `users_level_id_foreign_idx` FOREIGN KEY (`level_id`) REFERENCES `level` (`id`) ON DELETE CASCADE ON UPDATE CASCADE;',
             postgres: 'ALTER TABLE "users"  ADD CONSTRAINT "level_id_foreign_idx" FOREIGN KEY ("level_id") REFERENCES "level" ("id") ON DELETE CASCADE ON UPDATE CASCADE;',
             postgres: 'ALTER TABLE "users"  ADD CONSTRAINT "level_id_foreign_idx" FOREIGN KEY ("level_id") REFERENCES "level" ("id") ON DELETE CASCADE ON UPDATE CASCADE;',
           });
           });
         });
         });