Pattern: Missing use of Enumerable#detect
Issue: -
This rule is used to identify usages of select.first, select.last, find_all.first, and find_all.last and change them to use detect instead.
ActiveRecord compatibility:
ActiveRecord does not implement a detect method and find has its
own meaning. Correcting ActiveRecord methods with this rule should be
considered unsafe.
# bad
[].select { |item| true }.first
[].select { |item| true }.last
[].find_all { |item| true }.first
[].find_all { |item| true }.last
# good
[].detect { |item| true }
[].reverse.detect { |item| true }| Attribute | Value |
|---|---|
| SafeMode | true |