diff options
author | Lin Jen-Shin <godfat@godfat.org> | 2016-10-14 19:13:11 +0800 |
---|---|---|
committer | Lin Jen-Shin <godfat@godfat.org> | 2016-10-14 19:13:11 +0800 |
commit | b5f9d4c4bc48b252d3175432a3bb6fb1ca394af9 (patch) | |
tree | 45c5305aaf0baf3b36c9692d8e37cba1011935b1 /app/models/ci | |
parent | 5b9971fa9b967eb5591242155d6f5fce2c292b7d (diff) | |
download | gitlab-ce-b5f9d4c4bc48b252d3175432a3bb6fb1ca394af9.tar.gz |
Introduce Pipeline#merge_requests_with_active_first,
Feedback:
https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/6019#note_16956802
Diffstat (limited to 'app/models/ci')
-rw-r--r-- | app/models/ci/pipeline.rb | 12 |
1 files changed, 8 insertions, 4 deletions
diff --git a/app/models/ci/pipeline.rb b/app/models/ci/pipeline.rb index 2cf9892edc5..bf2d861b86f 100644 --- a/app/models/ci/pipeline.rb +++ b/app/models/ci/pipeline.rb @@ -293,10 +293,14 @@ module Ci # the merge request's latest commit. def merge_requests @merge_requests ||= - begin - project.merge_requests.where(source_branch: self.ref). - select { |merge_request| merge_request.pipeline.try(:id) == self.id } - end + project.merge_requests.where(source_branch: ref). + select { |merge_request| merge_request.pipeline.try(:id) == id } + end + + def merge_requests_with_active_first + merge_requests.sort_by do |merge_request| + [merge_request.state_priority, -merge_request.updated_at.to_i] + end end private |