message: Use || operator instead of inline if to improve readability
diff: @@ -127,7 +127,7 @@ module.exports = function(bot, config) {
  * @param {function} cb A NodeJS-style callback
  */
  slack_api.callAPI = function(command, data, cb, multipart) {
  - data.token = data.token ? data.token : config.token;
  + data.token = data.token || config.token;
  bot.debug(command, data);
  postForm(slack_api.api_url + command, data, cb, multipart);
  };
  @@ -140,9 +140,9 @@ module.exports = function(bot, config) {
  * @param {function} cb A NodeJS-style callback
  */
  slack_api.callAPIWithoutToken = function(command, data, cb) {
  - data.client_id = data.client_id ? data.client_id : bot.config.clientId;
  - data.client_secret = data.client_secret ? data.client_secret : bot.config.clientSecret;
  - data.redirect_uri = data.redirect_uri ? data.redirect_uri : bot.config.redirectUri;
  + data.client_id = data.client_id || bot.config.clientId;
  + data.client_secret = data.client_secret || bot.config.clientSecret;
  + data.redirect_uri = data.redirect_uri || bot.config.redirectUri;
  // DON'T log options: that could expose the client secret!
  postForm(slack_api.api_url + command, data, cb);
  };
  @@ -160,7 +160,7 @@ module.exports = function(bot, config) {
  var currentGroup = slack_api;
  groups.forEach(function(nextGroupName) {
  - currentGroup[nextGroupName] = currentGroup[nextGroupName] ? currentGroup[nextGroupName] : {};
  + currentGroup[nextGroupName] = currentGroup[nextGroupName] || {};
  currentGroup = currentGroup[nextGroupName];
  });
  