Default PR reviewer/forwarder should be used when that certain flag i…

…s empty. Otherwise it will cause conflicts with other commands #283

lib/cmds/pull-request.js

@@ -131,11 +131,9 @@ PullRequest.prototype.run = function() {
         options = instance.options,
         fetchType;
 
-    options.fwd = options.fwd || config.default_pr_forwarder;
     options.number = options.number || instance.getPullRequestNumberFromBranch_();
     options.pullBranch = instance.getBranchNameFromPullNumber_(options.number);
     options.state = options.state || PullRequest.STATE_OPEN;
-    options.submit = options.submit || config.default_pr_reviewer;
 
     if (options.browser) {
         instance.browser(options.user, options.repo, options.number);
@@ -211,6 +209,10 @@ PullRequest.prototype.run = function() {
         });
     }
 
+    if (options.fwd === '') {
+        options.fwd = config.default_pr_forwarder;
+    }
+
     if (options.fwd) {
         hooks.invoke('pull-request.fwd', instance, function(afterHooksCallback) {
             logger.logTemplate(
@@ -283,6 +285,10 @@ PullRequest.prototype.run = function() {
         });
     }
 
+    if (options.submit === '') {
+        options.submit = config.default_pr_reviewer;
+    }
+
     if (options.submit) {
         hooks.invoke('pull-request.submit', instance, function(afterHooksCallback) {
             logger.logTemplate(