Class: RuboCop::Cop::Rails::RedundantActiveRecordAllMethod

Inherits:

Base

• Object
• Base
• RuboCop::Cop::Rails::RedundantActiveRecordAllMethod

Extended by:

AutoCorrector

Includes:

ActiveRecordHelper

Defined in:

lib/rubocop/cop/rails/redundant_active_record_all_method.rb

Overview

Detect redundant ‘all` used as a receiver for Active Record query methods.

Examples:

# bad
User.all.find(id)
User.all.order(:created_at)
users.all.where(id: ids)
user.articles.all.order(:created_at)

# good
User.find(id)
User.order(:created_at)
users.where(id: ids)
user.articles.order(:created_at)

Constant Summary

MSG =

'Redundant `all` detected.'

RESTRICT_ON_SEND =

[:all].freeze

QUERYING_METHODS =

Defined methods in ‘ActiveRecord::Querying::QUERYING_METHODS` on activerecord 7.0.5.

%i[
  and
  annotate
  any?
  average
  calculate
  count
  create_or_find_by
  create_or_find_by!
  create_with
  delete_all
  delete_by
  destroy_all
  destroy_by
  distinct
  eager_load
  except
  excluding
  exists?
  extending
  extract_associated
  fifth
  fifth!
  find
  find_by
  find_by!
  find_each
  find_in_batches
  find_or_create_by
  find_or_create_by!
  find_or_initialize_by
  find_sole_by
  first
  first!
  first_or_create
  first_or_create!
  first_or_initialize
  forty_two
  forty_two!
  fourth
  fourth!
  from
  group
  having
  ids
  in_batches
  in_order_of
  includes
  invert_where
  joins
  last
  last!
  left_joins
  left_outer_joins
  limit
  lock
  many?
  maximum
  merge
  minimum
  none
  none?
  offset
  one?
  only
  optimizer_hints
  or
  order
  pick
  pluck
  preload
  readonly
  references
  reorder
  reselect
  rewhere
  second
  second!
  second_to_last
  second_to_last!
  select
  sole
  strict_loading
  sum
  take
  take!
  third
  third!
  third_to_last
  third_to_last!
  touch_all
  unscope
  update_all
  where
  without
].freeze

Constants included from ActiveRecordHelper

ActiveRecordHelper::WHERE_METHODS

Instance Method Summary

• #on_send(node) ⇒ Object

Methods included from ActiveRecordHelper

#external_dependency_checksum, #foreign_key_of, #in_where?, #inherit_active_record_base?, #polymorphic?, #resolve_relation_into_column, #schema, #table_name

Instance Method Details

#on_send(node) ⇒ Object

# File 'lib/rubocop/cop/rails/redundant_active_record_all_method.rb', line 129

def on_send(node)
  query_node = node.parent

  return unless query_node&.send_type?
  return unless QUERYING_METHODS.include?(query_node.method_name)
  return if node.receiver.nil? && !inherit_active_record_base?(node)

  range_of_all_method = node.loc.selector
  add_offense(range_of_all_method) do |collector|
    collector.remove(range_of_all_method)
    collector.remove(query_node.loc.dot)
  end
end