Skip to content

EE: Add a thin encapsulation around .pluck(:id)

Nick Thomas requested to merge (removed):ee-add-pluck-primary-key into master

What does this MR do?

EE port of https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/26669

This adds a general-purpose pluck_primary_key method to avoid .pluck(:id) in the codebase. To make this generally available, we have to modify all records to inherit from ApplicationRecord instead of ActiveRecord::Base.

What are the relevant issue numbers?

Does this MR meet the acceptance criteria?

Merge request reports