diff options
Diffstat (limited to 'app/models')
| -rw-r--r-- | app/models/info_request.rb | 9 | ||||
| -rw-r--r-- | app/models/request_mailer.rb | 6 |
2 files changed, 9 insertions, 6 deletions
diff --git a/app/models/info_request.rb b/app/models/info_request.rb index 07dbbccd5..f32a7b3e7 100644 --- a/app/models/info_request.rb +++ b/app/models/info_request.rb @@ -23,7 +23,7 @@ # Copyright (c) 2007 UK Citizens Online Democracy. All rights reserved. # Email: francis@mysociety.org; WWW: http://www.mysociety.org/ # -# $Id: info_request.rb,v 1.134 2008-08-31 15:46:00 francis Exp $ +# $Id: info_request.rb,v 1.135 2008-08-31 23:43:53 francis Exp $ require 'digest/sha1' require File.join(File.dirname(__FILE__),'../../vendor/plugins/acts_as_xapian/lib/acts_as_xapian') @@ -308,7 +308,7 @@ public end # change status, including for last event for later historical purposes - def set_described_state(new_state) + def set_described_state(new_state, details = nil) ActiveRecord::Base.transaction do self.awaiting_description = false last_event = self.get_last_event @@ -321,7 +321,10 @@ public self.calculate_event_states if new_state == 'requires_admin' - RequestMailer.deliver_requires_admin(self) + RequestMailer.deliver_requires_admin(self, details) + else + # XXX this chucks details if we are not moving to requires_admin - + # the user is not meant to have entered any. end end diff --git a/app/models/request_mailer.rb b/app/models/request_mailer.rb index eb4d83349..a4e92252e 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.48 2008-08-31 16:02:26 francis Exp $ +# $Id: request_mailer.rb,v 1.49 2008-08-31 23:43:53 francis Exp $ class RequestMailer < ApplicationMailer @@ -56,13 +56,13 @@ class RequestMailer < ApplicationMailer end # An FOI response is outside the scope of the system, and needs admin attention - def requires_admin(info_request) + def requires_admin(info_request, details) @from = contact_from_name_and_email @recipients = @from @subject = "Unusual FOI response - " + info_request.title url = main_url(request_url(info_request)) admin_url = request_admin_url(info_request) - @body = {:info_request => info_request, :url => url, :admin_url => admin_url } + @body = {:info_request => info_request, :url => url, :admin_url => admin_url, :details => details } end # Tell the requester that a new response has arrived |