diff options
| author | francis <francis> | 2008-09-25 12:23:01 +0000 |
|---|---|---|
| committer | francis <francis> | 2008-09-25 12:23:01 +0000 |
| commit | 0b9158310cd2da280ca50c60bcce821bf9ffe6c0 (patch) | |
| tree | 631077af3a4bcb57e1a4fbd2a2a3b98ba93accc7 | |
| parent | 5c421e042ee1679ec0d29661d6056b4158de91ca (diff) | |
Don't try and send response reminders for holding pen, it won't work if the
messages have been moved out.
| -rw-r--r-- | app/models/request_mailer.rb | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/app/models/request_mailer.rb b/app/models/request_mailer.rb index 2876ddded..4f3e34806 100644 --- a/app/models/request_mailer.rb +++ b/app/models/request_mailer.rb @@ -4,7 +4,7 @@ # Copyright (c) 2007 UK Citizens Online Democracy. All rights reserved. # Email: francis@mysociety.org; WWW: http://www.mysociety.org/ # -# $Id: request_mailer.rb,v 1.58 2008-09-22 04:06:37 francis Exp $ +# $Id: request_mailer.rb,v 1.59 2008-09-25 12:23:01 francis Exp $ class RequestMailer < ApplicationMailer @@ -248,7 +248,7 @@ class RequestMailer < ApplicationMailer end def self.alert_new_response_reminders_internal(days_since, type_code) #STDERR.puts "alert_new_response_reminders_internal days:" + days_since.to_s + " type: " + type_code - info_requests = InfoRequest.find(:all, :conditions => [ "awaiting_description = ? and info_requests.updated_at < ?", true, Time.now() - days_since.days ], :include => [ :user ], :order => "info_requests.id" ) + info_requests = InfoRequest.find(:all, :conditions => [ "awaiting_description = ? and info_requests.updated_at < ? and url_title != 'holding_pen'", true, Time.now() - days_since.days ], :include => [ :user ], :order => "info_requests.id" ) for info_request in info_requests alert_event_id = info_request.get_last_response_event_id last_response_message = info_request.get_last_response |