From 16c8b91092fa0fe0b5822648c22ee545e11cb4bc Mon Sep 17 00:00:00 2001 From: "micael.bergeron" Date: Wed, 6 Dec 2017 08:32:12 -0500 Subject: [PATCH] revert the `#all_pipelines` method to use the pluck the `ci_pipelines.sha` column is not the same type than the `merge_request_diff_commits.sha` column (varchar, bytea) --- app/models/merge_request.rb | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/app/models/merge_request.rb b/app/models/merge_request.rb index 22a79da9879..2dad036639a 100644 --- a/app/models/merge_request.rb +++ b/app/models/merge_request.rb @@ -921,9 +921,8 @@ class MergeRequest < ActiveRecord::Base def all_pipelines return Ci::Pipeline.none unless source_project - commit_shas = all_commits.unscope(:limit).select(:sha) @all_pipelines ||= source_project.pipelines - .where(sha: commit_shas, ref: source_branch) + .where(sha: all_commit_shas, ref: source_branch) .order(id: :desc) end