]> source.dussan.org Git - redmine.git/commitdiff
RecordNotFound exception still raised with Rails3.
authorJean-Philippe Lang <jp_lang@yahoo.fr>
Sat, 28 Apr 2012 13:00:16 +0000 (13:00 +0000)
committerJean-Philippe Lang <jp_lang@yahoo.fr>
Sat, 28 Apr 2012 13:00:16 +0000 (13:00 +0000)
git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@9569 e93f8b46-1217-0410-a6f0-8f06a7374b81

app/models/query.rb
test/unit/query_test.rb

index e2e1326e231e6f0da07d73c0c598249626adcf14..0640dd68483368e297d370abc06862c813a8fb4a 100644 (file)
@@ -564,7 +564,7 @@ class Query < ActiveRecord::Base
     r = nil
     if grouped?
       begin
-        # Rails will raise an (unexpected) RecordNotFound if there's only a nil group value
+        # Rails3 will raise an (unexpected) RecordNotFound if there's only a nil group value
         r = Issue.visible.count(:group => group_by_statement, :include => [:status, :project], :conditions => statement)
       rescue ActiveRecord::RecordNotFound
         r = {nil => issue_count}
index 8a39d09a13e768cb4370113cd222390a69161a87..c89c2c8c5a1b2439e9687475de79dd303d8d7bf3 100644 (file)
@@ -705,6 +705,16 @@ class QueryTest < ActiveSupport::TestCase
     assert_equal %w(Fixnum), count_by_group.values.collect {|k| k.class.name}.uniq
   end
 
+  def test_issue_count_with_nil_group_only
+    Issue.update_all("assigned_to_id = NULL")
+
+    q = Query.new(:name => '_', :group_by => 'assigned_to')
+    count_by_group = q.issue_count_by_group
+    assert_kind_of Hash, count_by_group
+    assert_equal 1, count_by_group.keys.size
+    assert_nil count_by_group.keys.first
+  end
+
   def test_issue_ids
     q = Query.new(:name => '_')
     order = "issues.subject, issues.id"