stop using && for the short circuit side effect

2011-06-27 18:16:01 -07:00 · 2011-06-27 18:16:01 -07:00 · 4006e738ed
commit 4006e738ed
parent 4ec47cac10
1 changed files with 1 additions and 1 deletions
--- a/activerecord/lib/active_record/base.rb
+++ b/activerecord/lib/active_record/base.rb
@ -2082,7 +2082,7 @@ def convert_number_column_value(value)

      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