Remove old TODO and commented out code
This commit is contained in:
parent
5ba6687c7c
commit
6848b72e22
|
@ -493,8 +493,6 @@ function mixinMigration(MySQL, mysql) {
|
||||||
};
|
};
|
||||||
|
|
||||||
MySQL.prototype.buildForeignKeyDefinition = function(model, keyName) {
|
MySQL.prototype.buildForeignKeyDefinition = function(model, keyName) {
|
||||||
//TODO validate that these are in the same DB, etc.
|
|
||||||
//TODO this is probably a better place to check this than in autoupdate or buildColumnDefinitions
|
|
||||||
var definition = this.getModelDefinition(model);
|
var definition = this.getModelDefinition(model);
|
||||||
|
|
||||||
var fk = definition.settings.foreignKeys[keyName];
|
var fk = definition.settings.foreignKeys[keyName];
|
||||||
|
@ -551,13 +549,6 @@ function mixinMigration(MySQL, mysql) {
|
||||||
sql.push(i);
|
sql.push(i);
|
||||||
});
|
});
|
||||||
|
|
||||||
// var foreignKeys = definition.settings.foreignKeys || {};
|
|
||||||
// for (var fk in foreignKeys) {
|
|
||||||
// var constraint = self.buildForeignKeyDefinition(model, fk);
|
|
||||||
// if (constraint) {
|
|
||||||
// sql.push(constraint);
|
|
||||||
// }
|
|
||||||
// }
|
|
||||||
return sql.join(',\n ');
|
return sql.join(',\n ');
|
||||||
};
|
};
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue