Sha256: 9282f14e91ec4461be40a7ff10db12806cce9309fb7c2daf95001d318d6dfc6d
Contents?: true
Size: 1.27 KB
Versions: 10
Compression:
Stored size: 1.27 KB
Contents
# frozen_string_literal: true
module RuboCop
module Cop
module Rails
# This cop checks to make sure safe navigation isn't used with `blank?` in
# a conditional.
#
# While the safe navigation operator is generally a good idea, when
# checking `foo&.blank?` in a conditional, `foo` being `nil` will actually
# do the opposite of what the author intends.
#
# @example
# # bad
# do_something if foo&.blank?
# do_something unless foo&.blank?
#
# # good
# do_something if foo.blank?
# do_something unless foo.blank?
#
class SafeNavigationWithBlank < Cop
MSG =
'Avoid calling `blank?` with the safe navigation operator ' \
'in conditionals.'
def_node_matcher :safe_navigation_blank_in_conditional?, <<~PATTERN
(if $(csend ... :blank?) ...)
PATTERN
def on_if(node)
return unless safe_navigation_blank_in_conditional?(node)
add_offense(node)
end
def autocorrect(node)
lambda do |corrector|
corrector.replace(
safe_navigation_blank_in_conditional?(node).location.dot,
'.'
)
end
end
end
end
end
end
Version data entries
10 entries across 10 versions & 1 rubygems