diff options
author | Carlos Antonio da Silva <carlosantoniodasilva@gmail.com> | 2012-05-24 20:50:44 -0300 |
---|---|---|
committer | Carlos Antonio da Silva <carlosantoniodasilva@gmail.com> | 2012-05-28 23:16:34 -0300 |
commit | f9cb645dfcb5cc89f59d2f8b58a019486c828c73 (patch) | |
tree | fe959ec9be9888c650975a75e6e5e185fd261290 /activerecord/lib | |
parent | 9cc2bf69ce296b7351dc612a8366193390a305f3 (diff) | |
download | rails-f9cb645dfcb5cc89f59d2f8b58a019486c828c73.tar.gz rails-f9cb645dfcb5cc89f59d2f8b58a019486c828c73.tar.bz2 rails-f9cb645dfcb5cc89f59d2f8b58a019486c828c73.zip |
Yield only one argument instead of splatting.
Add Changelog entry. Closes #4003
Diffstat (limited to 'activerecord/lib')
-rw-r--r-- | activerecord/lib/active_record/relation/calculations.rb | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/activerecord/lib/active_record/relation/calculations.rb b/activerecord/lib/active_record/relation/calculations.rb index a0d24f28d0..04b4fcf379 100644 --- a/activerecord/lib/active_record/relation/calculations.rb +++ b/activerecord/lib/active_record/relation/calculations.rb @@ -21,7 +21,7 @@ module ActiveRecord # # => queries people where "age > 26" then count the loaded results filtering by gender def count(column_name = nil, options = {}) if block_given? - self.to_a.count { |*block_args| yield(*block_args) } + self.to_a.count { |item| yield item } else column_name, options = nil, column_name if column_name.is_a?(Hash) calculate(:count, column_name, options) @@ -65,7 +65,7 @@ module ActiveRecord # # queries people where "age > 100" then perform a sum calculation with the block returns def sum(*args) if block_given? - self.to_a.sum(*args) { |*block_args| yield(*block_args) } + self.to_a.sum(*args) { |item| yield item } else calculate(:sum, *args) end |