kolan/Redmine
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Use scopes instead of ARCondition.

Browse Source 
git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@8087 e93f8b46-1217-0410-a6f0-8f06a7374b81
This commit is contained in:
Jean-Philippe Lang 2011-12-04 23:36:47 +00:00
parent b9e02477e9
commit 0a92e382fa
1 changed files with 8 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
app/models/mailer.rb
View File

@ -343,16 +343,15 @@ class Mailer < ActionMailer::Base
tracker = options[:tracker] ? Tracker.find(options[:tracker]) : nil tracker = options[:tracker] ? Tracker.find(options[:tracker]) : nil
user_ids = options[:users] user_ids = options[:users]
s = ARCondition.new ["#{IssueStatus.table_name}.is_closed = ? AND #{Issue.table_name}.due_date <= ?", false, days.day.from_now.to_date] scope = Issue.scoped(:conditions => ["#{Issue.table_name}.assigned_to_id IS NOT NULL" +
s << "#{Issue.table_name}.assigned_to_id IS NOT NULL" " AND #{Project.table_name}.status = #{Project::STATUS_ACTIVE}" +
s << ["#{Issue.table_name}.assigned_to_id IN (?)", user_ids] if user_ids.present? " AND #{Issue.table_name}.due_date <= ?", days.day.from_now.to_date]
s << "#{Project.table_name}.status = #{Project::STATUS_ACTIVE}" )
s << "#{Issue.table_name}.project_id = #{project.id}" if project scope = scope.scoped(:conditions => {:assigned_to_id => user_ids}) if user_ids.present?
s << "#{Issue.table_name}.tracker_id = #{tracker.id}" if tracker scope = scope.scoped(:conditions => {:project_id => project.id}) if project
scope = scope.scoped(:conditions => {:tracker_id => tracker.id}) if tracker
issues_by_assignee = Issue.find(:all, :include => [:status, :assigned_to, :project, :tracker], issues_by_assignee = scope.all(:include => [:status, :assigned_to, :project, :tracker]).group_by(&:assigned_to)
:conditions => s.conditions
).group_by(&:assigned_to)
issues_by_assignee.each do |assignee, issues| issues_by_assignee.each do |assignee, issues|
deliver_reminder(assignee, issues, days) if assignee && assignee.active? deliver_reminder(assignee, issues, days) if assignee && assignee.active?
end end