commit 791afd616e17a1bcbfb8e8f4f838383a165ea96a
Author: Wim <wim@42.be>
Date: Thu Nov 08 22:01:29 2018 +0000
Make gocritic linter happier
diff --git a/bridge/mattermost/mattermost.go b/bridge/mattermost/mattermost.go
index c9a7c66..2e93446 100644
--- a/bridge/mattermost/mattermost.go
+++ b/bridge/mattermost/mattermost.go
@@ -3424 +3425 @@ func (b *Bmattermost) Command(cmd string) string {
func (b *Bmattermost) Connect() error {
if b.GetString("WebhookBindAddress") != "" {
- if b.GetString("WebhookURL") != "" {
+ switch {
+ case b.GetString("WebhookURL") != "":
b.Log.Info("Connecting using webhookurl (sending) and webhookbindaddress (receiving)")
b.mh = matterhook.New(b.GetString("WebhookURL"),
matterhook.Config{InsecureSkipVerify: b.GetBool("SkipTLSVerify"),
BindAddress: b.GetString("WebhookBindAddress")})
- } else if b.GetString("Token") != "" {
+ case b.GetString("Token") != "":
b.Log.Info("Connecting using token (sending)")
err := b.apiLogin()
if err != nil {
return err
}
- } else if b.GetString("Login") != "" {
+ case b.GetString("Login") != "":
b.Log.Info("Connecting using login/password (sending)")
err := b.apiLogin()
if err != nil {
return err
}
- } else {
+ default:
b.Log.Info("Connecting using webhookbindaddress (receiving)")
b.mh = matterhook.New(b.GetString("WebhookURL"),
matterhook.Config{InsecureSkipVerify: b.GetBool("SkipTLSVerify"),
@@ -607 +618 @@ func (b *Bmattermost) Connect() error {
go b.handleMatter()
return nil
}
- if b.GetString("WebhookURL") != "" {
+ switch {
+ case b.GetString("WebhookURL") != "":
b.Log.Info("Connecting using webhookurl (sending)")
b.mh = matterhook.New(b.GetString("WebhookURL"),
matterhook.Config{InsecureSkipVerify: b.GetBool("SkipTLSVerify"),
@@ -8114 +8314 @@ func (b *Bmattermost) Connect() error {
go b.handleMatter()
}
return nil
- } else if b.GetString("Token") != "" {
+ case b.GetString("Token") != "":
b.Log.Info("Connecting using token (sending and receiving)")
err := b.apiLogin()
if err != nil {
return err
}
go b.handleMatter()
- } else if b.GetString("Login") != "" {
+ case b.GetString("Login") != "":
b.Log.Info("Connecting using login/password (sending and receiving)")
err := b.apiLogin()
if err != nil {
diff --git a/bridge/slack/helpers.go b/bridge/slack/helpers.go
index 19ac5d0..fdcef2e 100644
--- a/bridge/slack/helpers.go
+++ b/bridge/slack/helpers.go
@@ -20511 +20512 @@ func (b *Bslack) populateMessageWithUserInfo(ev *slack.MessageEvent, rmsg *confi
// Second, deal with "real" users if we have the necessary information.
var userID string
- if ev.User != "" {
+ switch {
+ case ev.User != "":
userID = ev.User
- } else if ev.SubMessage != nil && ev.SubMessage.User != "" {
+ case ev.SubMessage != nil && ev.SubMessage.User != "":
userID = ev.SubMessage.User
- } else {
+ default:
return nil
}
diff --git a/bridge/slack/slack.go b/bridge/slack/slack.go
index 701f476..814aeea 100644
--- a/bridge/slack/slack.go
+++ b/bridge/slack/slack.go
@@ -9813 +9814 @@ func (b *Bslack) Connect() error {
b.RLock()
defer b.RUnlock()
if b.GetString(incomingWebhookConfig) != "" {
- if b.GetString(outgoingWebhookConfig) != "" {
+ switch {
+ case b.GetString(outgoingWebhookConfig) != "":
b.Log.Info("Connecting using webhookurl (sending) and webhookbindaddress (receiving)")
b.mh = matterhook.New(b.GetString(outgoingWebhookConfig), matterhook.Config{
InsecureSkipVerify: b.GetBool(skipTLSConfig),
BindAddress: b.GetString(incomingWebhookConfig),
})
- } else if b.GetString(tokenConfig) != "" {
+ case b.GetString(tokenConfig) != "":
b.Log.Info("Connecting using token (sending)")
b.sc = slack.New(b.GetString(tokenConfig))
b.rtm = b.sc.NewRTM()
@@ -1147 +1157 @@ func (b *Bslack) Connect() error {
InsecureSkipVerify: b.GetBool(skipTLSConfig),
BindAddress: b.GetString(incomingWebhookConfig),
})
- } else {
+ default:
b.Log.Info("Connecting using webhookbindaddress (receiving)")
b.mh = matterhook.New(b.GetString(outgoingWebhookConfig), matterhook.Config{
InsecureSkipVerify: b.GetBool(skipTLSConfig),
diff --git a/bridge/steam/steam.go b/bridge/steam/steam.go
index 27bf284..a35a9d5 100644
--- a/bridge/steam/steam.go
+++ b/bridge/steam/steam.go
@@ -1748 +1746 @@ func (b *Bsteam) handleEvents() {
b.c.Connect()
case steam.FatalErrorEvent:
b.Log.Error(e)
- case error:
- b.Log.Error(e)
default:
b.Log.Debugf("unknown event %#v", e)
}
