From 1cd99829e8a161c5720add8eb7efd0d471622580 Mon Sep 17 00:00:00 2001 From: "Stanislav N. aka pztrn" Date: Tue, 19 Sep 2017 01:08:24 +0500 Subject: [PATCH] Parse attachments only if theyre exist and message.Text is empty. --- parsers/gitlab/gitlabparser.go | 17 +++++++++++++---- 1 file changed, 13 insertions(+), 4 deletions(-) diff --git a/parsers/gitlab/gitlabparser.go b/parsers/gitlab/gitlabparser.go index e71b023..8437770 100644 --- a/parsers/gitlab/gitlabparser.go +++ b/parsers/gitlab/gitlabparser.go @@ -300,15 +300,24 @@ func (gp GitlabParser) parseTagPush(message slackmessage.SlackMessage) map[strin return data } +func (gp GitlabParser) parseAttachments(message slackmessage.SlackMessage) map[string]string { + c.Log.Debugln("Parsing attachments...") + + var data map[string]string + if strings.Contains(message.Attachments[0].Text, "Pipeline") && strings.Contains(message.Attachments[0].Text, "of branch") { + data = gp.parsePipelineMessage(message) + } + + return data +} + func (gp GitlabParser) ParseMessage(message slackmessage.SlackMessage) map[string]string { c.Log.Debugln("Parsing Gitlab message...") var data map[string]string - if len(message.Attachments) > 0 { - if strings.Contains(message.Attachments[0].Text, "Pipeline") && strings.Contains(message.Attachments[0].Text, "of branch") { - data = gp.parsePipelineMessage(message) - } + if len(message.Attachments) > 0 && message.Text == "" { + return gp.parseAttachments(message) } if strings.Contains(message.Text, "pushed to") {