Use a separate line for separate actions:

Feedback from: https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/3363#note_12420487
2016-06-20 17:58:05 +08:00 · 2016-06-20 17:58:05 +08:00 · c491f66952
commit c491f66952
parent 09c38e46e5
1 changed files with 2 additions and 1 deletions
--- 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)