From 512c870ed46b5e441fd0b8daa8bd9cab449f7ac0 Mon Sep 17 00:00:00 2001
From: Gabriel Gizotti <gabriel@gizotti.com>
Date: Mon, 28 Nov 2016 19:06:18 +1000
Subject: [PATCH] Remove unnecessary code from
MergeRequest#issues_mentioned_but_not_closing
---
app/models/merge_request.rb | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/app/models/merge_request.rb b/app/models/merge_request.rb
index 5a5b8bd6010..dba0c463fd6 100644
--- a/app/models/merge_request.rb
+++ b/app/models/merge_request.rb
@@ -573,14 +573,12 @@ class MergeRequest < ActiveRecord::Base
closing_issues = []
if target_branch == project.default_branch
- messages = [description]
-
ext = Gitlab::ReferenceExtractor.new(project, current_user)
- ext.analyze(messages.join("\n"))
+ ext.analyze(description)
issues = ext.issues
closing_issues = Gitlab::ClosingIssueExtractor.new(project, current_user).
- closed_by_message(messages.join("\n"))
+ closed_by_message(description)
end
issues - closing_issues
--
GitLab