Contents

# frozen_string_literal: true

module RailsBestPractices
  module Reviews
    # Review a view file to make sure there is no finder, finder should be moved to controller.
    #
    # See the best practice details here https://rails-bestpractices.com/posts/2010/07/24/move-code-into-controller/
    #
    # Implementation:
    #
    # Review process:
    #   only check all view files to see if there are finders, then the finders should be moved to controller.
    class MoveCodeIntoControllerReview < Review
      interesting_nodes :method_add_arg, :assign
      interesting_files VIEW_FILES
      url 'https://rails-bestpractices.com/posts/2010/07/24/move-code-into-controller/'

      FINDERS = %w[find all first last].freeze

      # check method_add_arg nodes.
      #
      # if the receiver of the method_add_arg node is a constant,
      # and the message of the method_add_arg node is one of the find, all, first and last,
      # then it is a finder and should be moved to controller.
      add_callback :start_method_add_arg do |node|
        add_error 'move code into controller' if finder?(node)
      end

      # check assign nodes.
      #
      # if the receiver of the right value node is a constant,
      # and the message of the right value node is one of the find, all, first and last,
      # then it is a finder and should be moved to controller.
      add_callback :start_assign do |node|
        add_error 'move code into controller', node.file, node.right_value.line_number if finder?(node.right_value)
      end

      private

        # check if the node is a finder call node.
        def finder?(node)
          node.receiver.const? && FINDERS.include?(node.message.to_s)
        end
    end
  end
end

Version data entries

2 entries across 2 versions & 1 rubygems

Version	Path
rails_best_practices-1.19.3	lib/rails_best_practices/reviews/move_code_into_controller_review.rb
rails_best_practices-1.19.2	lib/rails_best_practices/reviews/move_code_into_controller_review.rb