@@ -136,6 +136,10 @@ module ActiveRecord end end + def sum(*args) + super || 0 + end + def empty? @found ? @found.empty? : count.zero? end lib/named_scope/named_scope.rb @@ -29,6 +29,13 @@ class NamedScopeTest < NamedScope::TestCase @david.reload.posts.last end end + + should 'return 0 when calculating on an empty set returned from a scope' do + new_author = Factory :author + assert_equal 0, @david.posts.containing_the_letter_a.sum('comments_count') # non-empty + assert_equal 0, new_author.posts.containing_the_letter_a.count # empty + assert_equal 0, new_author.posts.containing_the_letter_a.sum('comments_count') + end end test/named_scope_test.rb 6f12f9f302fa83cc8cf63ba5d313df544dd91c39 Brennan Dunn me@brennandunn.com http://github.com/metaskills/named_scope/commit/c3cb61f34bbbf6450b7b6cc00f382ff8008e536f c3cb61f34bbbf6450b7b6cc00f382ff8008e536f 2008-10-17T11:10:43-07:00 2008-10-17T11:10:43-07:00 Made #sum default to 0 when nil 71076133b5d098496eee0813b5ff69fcb853a7f7 Brennan Dunn me@brennandunn.com