aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorGareth Rees <gareth@mysociety.org>2014-10-01 14:13:25 +0100
committerLouise Crow <louise.crow@gmail.com>2014-12-18 15:16:38 +0000
commit90a69bd6277152be141dd43577bb516e023c98d6 (patch)
treeef1e8e98ae77c673bccbd20c598d11de8fe39d42
parent01625ed1baef70c23499124afe090642665a6157 (diff)
Improve InfoRequestEvent#incoming_message_selective_columns SQL
-rw-r--r--app/models/info_request_event.rb9
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)