Sha256: eb62a6dea6d92cfb8a673a7e3d5b5931b1f267f2a9ea1749d61f7dc913e06607
Contents?: true
Size: 1.91 KB
Versions: 1
Compression:
Stored size: 1.91 KB
Contents
# frozen_string_literal: true module RuboCop module Cop module Rails # Enforces the use of `pluck` over `map`. # # `pluck` can be used instead of `map` to extract a single key from each # element in an enumerable. When called on an Active Record relation, it # results in a more efficient query that only selects the necessary key. # # @example # # bad # Post.published.map { |post| post[:title] } # [{ a: :b, c: :d }].collect { |el| el[:a] } # # # good # Post.published.pluck(:title) # [{ a: :b, c: :d }].pluck(:a) class Pluck < Base extend AutoCorrector extend TargetRailsVersion MSG = 'Prefer `%<replacement>s` over `%<current>s`.' minimum_target_rails_version 5.0 def_node_matcher :pluck_candidate?, <<~PATTERN ({block numblock} (send _ {:map :collect}) $_argument (send (lvar $_element) :[] $_key)) PATTERN def on_block(node) pluck_candidate?(node) do |argument, element, key| match = if node.block_type? argument.children.first.source.to_sym == element else # numblock argument == 1 && element == :_1 end next unless match replacement = "pluck(#{key.source})" message = message(replacement, node) add_offense(offense_range(node), message: message) do |corrector| corrector.replace(offense_range(node), replacement) end end end alias on_numblock on_block private def offense_range(node) node.send_node.loc.selector.join(node.loc.end) end def message(replacement, node) current = offense_range(node).source format(MSG, replacement: replacement, current: current) end end end end end
Version data entries
1 entries across 1 versions & 1 rubygems
Version | Path |
---|---|
rubocop-rails-2.17.0 | lib/rubocop/cop/rails/pluck.rb |