diff options
author | Gareth Rees <gareth@mysociety.org> | 2014-10-01 14:13:25 +0100 |
---|---|---|
committer | Louise Crow <louise.crow@gmail.com> | 2014-12-18 15:16:38 +0000 |
commit | 90a69bd6277152be141dd43577bb516e023c98d6 (patch) | |
tree | ef1e8e98ae77c673bccbd20c598d11de8fe39d42 | |
parent | 01625ed1baef70c23499124afe090642665a6157 (diff) |
Improve InfoRequestEvent#incoming_message_selective_columns SQL
-rw-r--r-- | app/models/info_request_event.rb | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/app/models/info_request_event.rb b/app/models/info_request_event.rb index 9dde3ba80..635ba8f58 100644 --- a/app/models/info_request_event.rb +++ b/app/models/info_request_event.rb @@ -161,11 +161,10 @@ class InfoRequestEvent < ActiveRecord::Base end def incoming_message_selective_columns(fields) - message = IncomingMessage.find(:all, - :select => fields + ", incoming_messages.info_request_id", - :joins => "INNER JOIN info_request_events ON incoming_messages.id = incoming_message_id ", - :conditions => "info_request_events.id = #{self.id}" - ) + message = IncomingMessage.select("#{ fields }, incoming_messages.info_request_id"). + joins('INNER JOIN info_request_events ON incoming_messages.id = incoming_message_id'). + where('info_request_events.id = ?', id) + message = message[0] if !message.nil? message.info_request = InfoRequest.find(message.info_request_id) |