From c491f66952dd440c449c966727f9f08f3247e9d3 Mon Sep 17 00:00:00 2001 From: Lin Jen-Shin <godfat@godfat.org> Date: Mon, 20 Jun 2016 17:58:05 +0800 Subject: [PATCH] Use a separate line for separate actions: Feedback from: https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/3363#note_12420487 --- lib/gitlab/email/handler/base_handler.rb | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/lib/gitlab/email/handler/base_handler.rb b/lib/gitlab/email/handler/base_handler.rb index 4d74e770b91..1691fca6465 100644 --- a/lib/gitlab/email/handler/base_handler.rb +++ b/lib/gitlab/email/handler/base_handler.rb @@ -31,7 +31,8 @@ module Gitlab end def process_message - add_attachments(ReplyParser.new(mail).execute.strip) + message = ReplyParser.new(mail).execute.strip + add_attachments(message) end def add_attachments(reply) -- GitLab