Merged r6314 from trunk (#8633).

git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/branches/1.2-stable@7765 e93f8b46-1217-0410-a6f0-8f06a7374b81
This commit is contained in:
Jean-Philippe Lang 2011-11-11 11:31:49 +00:00
parent 35ec8e3570
commit c605477da5
2 changed files with 16 additions and 2 deletions

View File

@ -521,7 +521,7 @@ class Query < ActiveRecord::Base
# Returns the issue count
def issue_count
Issue.count(:include => [:status, :project], :conditions => statement)
Issue.visible.count(:include => [:status, :project], :conditions => statement)
rescue ::ActiveRecord::StatementInvalid => e
raise StatementInvalid.new(e.message)
end

View File

@ -356,7 +356,21 @@ class QueryTest < ActiveSupport::TestCase
q.issues(:conditions => "foo = 1")
end
end
def test_issue_count
q = Query.new(:name => '_')
issue_count = q.issue_count
assert_equal q.issues.size, issue_count
end
def test_issue_count_with_archived_issues
p = Project.generate!( :status => Project::STATUS_ARCHIVED )
i = Issue.generate!( :project => p, :tracker => p.trackers.first )
assert !i.visible?
test_issue_count
end
def test_issue_count_by_association_group
q = Query.new(:name => '_', :group_by => 'assigned_to')
count_by_group = q.issue_count_by_group