class RuboCop::Cop::Rails::Blank
end
!present?
def blank?
# good
end
something
if foo.blank?
# good
end
something
unless foo.present?
# bad
something if foo.blank?
# good
something unless foo.present?
# bad
# Converts usages of ‘unless present?` to `if blank?`
@example UnlessPresent: true (default)
foo.blank?
# good
!foo.present?
# bad
# Converts usages of `!present?` to `blank?`
@example NotPresent: true (default)
foo.blank?
# good
foo == nil || foo.empty?
foo.nil? || foo.empty?
# bad
# Converts usages of `nil? || empty?` to `blank?`
@example NilOrEmpty: true (default)
if the receiver is a non-empty blank string, tab, or newline meta characters.
but `’ ‘.blank?` returns true. Therefore, autocorrection is not compatible
This cop is unsafe autocorrection, because `’ ‘.empty?` returns false,
@safety
to prevent interference between the autocorrection of the two cops.
context of `unless else` if `Style/UnlessElse` is enabled. This is
The configuration of `NotPresent` will not produce an offense in the
Interaction with `Style/UnlessElse`:
using `Object#blank?` defined by Active Support.
Checks for code that can be written with simpler conditionals
def autocorrect(corrector, node)
def autocorrect(corrector, node) method_call, variable1 = unless_present?(node) if method_call corrector.replace(node.loc.keyword, 'if') range = method_call.source_range else variable1, _variable2 = nil_or_empty?(node) || not_present?(node) range = node.source_range end corrector.replace(range, replacement(variable1)) end
def on_if(node)
def on_if(node) return unless cop_config['UnlessPresent'] return unless node.unless? return if node.else? && config.cop_enabled?('Style/UnlessElse') unless_present?(node) do |method_call, receiver| range = unless_condition(node, method_call) message = format(MSG_UNLESS_PRESENT, prefer: replacement(receiver), current: range.source) add_offense(range, message: message) do |corrector| autocorrect(corrector, node) end end end
def on_or(node)
def on_or(node) return unless cop_config['NilOrEmpty'] nil_or_empty?(node) do |var1, var2| return unless var1 == var2 message = format(MSG_NIL_OR_EMPTY, prefer: replacement(var1), current: node.source) add_offense(node, message: message) do |corrector| autocorrect(corrector, node) end end end
def on_send(node)
def on_send(node) return unless cop_config['NotPresent'] not_present?(node) do |receiver| # accepts !present? if its in the body of a `blank?` method next if defining_blank?(node.parent) message = format(MSG_NOT_PRESENT, prefer: replacement(receiver), current: node.source) add_offense(node, message: message) do |corrector| autocorrect(corrector, node) end end end
def replacement(node)
def replacement(node) node.respond_to?(:source) ? "#{node.source}.blank?" : 'blank?' end
def unless_condition(node, method_call)
def unless_condition(node, method_call) if node.modifier_form? node.loc.keyword.join(node.source_range.end) else node.source_range.begin.join(method_call.source_range) end end