class RuboCop::Cop::Rails::PluckId
end
ids
def self.user_ids
user.posts.ids
User.ids
# good
end
pluck(primary_key)
def self.user_ids
user.posts.pluck(:id)
User.pluck(:id)
# bad
@example
This cop is unsafe if the receiver object is not an Active Record object.
@safety
Enforces the use of ‘ids` over `pluck(:id)` and `pluck(primary_key)`.
def offense_range(node)
def offense_range(node) range_between(node.loc.selector.begin_pos, node.source_range.end_pos) end
def on_send(node)
def on_send(node) return if !pluck_id_call?(node) || in_where?(node) range = offense_range(node) message = format(MSG, bad_method: range.source) add_offense(range, message: message) do |corrector| corrector.replace(offense_range(node), 'ids') end end