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)

revert the `#all_pipelines` method to use the pluck
16c8b910 · micael.bergeron · cb6f51ec · 16c8b910
Commit 16c8b910 authored Dec 06, 2017 by micael.bergeron
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 2 deletions

merge_request.rb app/models/merge_request.rb +1 -2

No files found.
--- 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