Merge pull request #201 from wzrdtales/bugfix/foreignkeyBehavior

[Bugfix] Deleting the index together with the foreign key is now optional

It's an unexpected behavior, instead it can be explicitly done via options.dropIndex boolean.

Author: wzrdtales

lib/driver/mysql.js

@@ -250,13 +250,24 @@ var MysqlDriver = Base.extend({
     this.runSql(sql, callback);
   },
 
-  removeForeignKey: function(tableName, keyName, callback) {
+  removeForeignKey: function(tableName, keyName, options, callback) {
     var sql = util.format('ALTER TABLE `%s` DROP FOREIGN KEY `%s`', tableName, keyName);
     this.runSql(sql, function () {
-      sql = util.format('ALTER TABLE `%s` DROP INDEX `%s`', tableName, keyName);
-      this.runSql(sql, function () {
+
+      if( typeof(options) === 'function' ) {
+
+          options();
+      }
+      else if(options.dropIndex === true) {
+
+        sql = util.format('ALTER TABLE `%s` DROP INDEX `%s`', tableName, keyName);
+        this.runSql(sql, function () {
+          callback();
+        });
+      }
+      else
         callback();
-      });
+
     }.bind(this));
   },