Sha256: b63210b8adb6e9f65eaa16581828feae574da95ea309bedb2e251de39918836b
Contents?: true
Size: 1.68 KB
Versions: 7
Compression:
Stored size: 1.68 KB
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
7 entries across 7 versions & 1 rubygems