Merge pull request #77 from orthlieb/master

Fix issue #56 where options parsing strips off arguments that it shouldn't
tj · Aug 3, 2012 · 4f1ddc1 · 4f1ddc1
2 parents 7234489 + d65dabd
commit 4f1ddc1
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/lib/commander.js b/lib/commander.js
@@ -213,6 +213,12 @@ Command.prototype.action = function(fn){
       self.unknownOption(parsed.unknown[0]);
     }
 
+    // Leftover arguments need to be pushed back onto the args stack
+    // Fixes issue #56
+    if (parsed.args.length > 0) {
+        args = parsed.args.concat(args);
+    }
+
     self.args.forEach(function(arg, i){
       if (arg.required && null == args[i]) {
         self.missingArgument(arg.name);