stop using && for the short circuit side effect

2026-04-26 03:00:59 -04:00 · 2011-06-27 18:16:01 -07:00
parent 5d954b4e38
commit 992b3b5e06
1 changed files with 1 additions and 1 deletions
--- a/activerecord/lib/active_record/base.rb
+++ b/activerecord/lib/active_record/base.rb
@@ -2092,7 +2092,7 @@ MSG

      def populate_with_current_scope_attributes
        self.class.scope_attributes.each do |att,value|
-          respond_to?("#{att}=") && send("#{att}=", value)
+          send("#{att}=", value) if respond_to?("#{att}=")
        end
      end