提交 7d7ae494 编写于 作者: G Gabriel Gizotti

add guard clause to MergeRequest#issues_mentioned_but_not_closing

上级 5d478e5c
......@@ -569,17 +569,14 @@ class MergeRequest < ActiveRecord::Base
end
def issues_mentioned_but_not_closing(current_user = self.author)
issues = []
closing_issues = []
return [] unless target_branch == project.default_branch
if target_branch == project.default_branch
ext = Gitlab::ReferenceExtractor.new(project, current_user)
ext.analyze(description)
ext = Gitlab::ReferenceExtractor.new(project, current_user)
ext.analyze(description)
issues = ext.issues
closing_issues = Gitlab::ClosingIssueExtractor.new(project, current_user).
closed_by_message(description)
end
issues = ext.issues
closing_issues = Gitlab::ClosingIssueExtractor.new(project, current_user).
closed_by_message(description)
issues - closing_issues
end
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册