Merge pull request #97 from StackStorm/hotfix/duplicate-help

Hotfix: avoid duplicating help entries
StackStorm · Mar 7, 2016 · b9e7abc · b9e7abc
2 parents a0e1583 + 13916fd
commit b9e7abc
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/lib/command_factory.js b/lib/command_factory.js
@@ -73,16 +73,16 @@ CommandFactory.prototype.addCommand = function(command, name, format, action_ali
   compiled_template = _.template('hubot ${command}');
   command_string = compiled_template(context);
 
-  if (!flag || flag === utils.DISPLAY) {
-    this.robot.commands.push(command_string);
-  }
   if (!flag || flag === utils.REPRESENTATION) {
     this.st2_hubot_commands.push(command_string);
     this.st2_commands_name_map[name] = action_alias;
     this.st2_commands_format_map[format] = action_alias;
     regex = this.getRegexForFormatString(format);
     this.st2_commands_regex_map[format] = regex;
   }
+  if ((!flag || flag === utils.DISPLAY) && this.robot.commands.indexOf(command_string) === -1) {
+    this.robot.commands.push(command_string);
+  }
 
   this.robot.logger.debug('Added command: ' + command);
 };