Add --down-using-run-order option to run down migrations based on run order rather than creation order.

Author: Andrew Kerr

README.md

@@ -18,13 +18,15 @@ Database migration framework for node.js
 Usage: db-migrate [up|down|create] migrationName [options]
 
 Options:
-  --env, -e             The environment to run the migrations under.    [default: "dev"]
-  --migrations-dir, -m  The directory containing your migration files.  [default: "./migrations"]
-  --count, -c           Max number of migrations to run.
-  --dry-run             Prints the SQL but doesn't run it.              [boolean]
-  --verbose, -v         Verbose mode.                                   [default: false]
-  --config              Location of the database.json file.             [default: "./database.json"]
-  --force-exit          Call system.exit() after migration run          [default: false]
+  --env, -e               The environment to run the migrations under.    [default: "dev"]
+  --migrations-dir, -m    The directory containing your migration files.  [default: "./migrations"]
+  --count, -c             Max number of migrations to run.
+  --dry-run               Prints the SQL but doesn't run it.              [boolean]
+  --verbose, -v           Verbose mode.                                   [default: false]
+  --config                Location of the database.json file.             [default: "./database.json"]
+  --force-exit            Call system.exit() after migration run          [default: false]
+  --down-using-run-order  Run down migrations in reverse run order,       [default: false]
+                          rather than reverse creation order.
 ```
 
 ## Creating Migrations

bin/db-migrate

@@ -20,6 +20,7 @@ var argv = optimist
     .default({
       verbose: false,
       "force-exit": false,
+      "down-using-run-order": false,
       config: process.cwd() + '/database.json',
       'migrations-dir': process.cwd() + '/migrations' })
     .usage('Usage: db-migrate [up|down|create] migrationName [options]')
@@ -42,6 +43,9 @@ var argv = optimist
     .describe('force-exit', 'Forcibly exit the migration process on completion.')
     .boolean('force-exit')
 
+    .describe('down-using-run-order', 'Run down migrations in reverse run order, rather than reverse creation order.')
+    .boolean('down-using-run-order')
+
     .describe('verbose', 'Verbose mode.')
     .alias('v', 'verbose')
     .boolean('v')
@@ -71,6 +75,7 @@ if (argv.help || argv._.length == 0) {
 
 global.verbose = argv.verbose;
 global.dryRun = argv['dry-run'];
+global.downUsingRunOrder = argv['down-using-run-order'];
 if(global.dryRun) {
   log.info('dry run');
 }
@@ -165,11 +170,20 @@ function run() {
     case 'down':
       loadConfig();
       if(argv._.length > 0) {
-        argv.destination = argv._.shift().toString();
+        if (global.downUsingRunOrder) {
+          log.info('Ignoring migration name since --down-using-run-order is set.');
+        } else {
+          argv.destination = argv._.shift().toString();
+        }
       }
       if(action == 'up') {
         executeUp();
       } else {
+        if(global.downUsingRunOrder) {
+          log.info('using run order for down migrations');
+        } else {
+          log.info('using creation order for down migrations');
+        }
         executeDown();
       }
       break;

lib/migration.js

@@ -134,7 +134,10 @@ Migration.loadFromFilesystem = function(dir, callback) {
 
 Migration.loadFromDatabase = function(dir, driver, callback) {
   log.verbose('loading migrations from database');
-  driver.all('SELECT * FROM migrations ORDER BY name DESC', function(err, dbResults) {
+  var query = 'SELECT * FROM migrations ORDER BY ';
+  query += global.downUsingRunOrder ? 'run_on DESC, name DESC' : 'name DESC';
+
+  driver.all(query, function(err, dbResults) {
     if (err) { callback(err); return; }
     var migrations = dbResults.map(function(result) {
       return new Migration(path.join(dir, result.name));