From 47fcd7ea7c9065e2727f1954be8bbffcf17e4114 Mon Sep 17 00:00:00 2001 From: Gareth Rees Date: Wed, 22 Apr 2015 17:40:17 +0100 Subject: Don't load PublicBody when initalizing InfoRequest Doesn't need to try to set the default law_used once it's persisted, so we save a query each time we find an InfoRequest --- app/models/info_request.rb | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'app/models/info_request.rb') diff --git a/app/models/info_request.rb b/app/models/info_request.rb index 8f464e786..0ca3a1279 100644 --- a/app/models/info_request.rb +++ b/app/models/info_request.rb @@ -1394,8 +1394,9 @@ public # this should only happen on Model.exists?() call. It can be safely ignored. # See http://www.tatvartha.com/2011/03/activerecordmissingattributeerror-missing-attribute-a-bug-or-a-features/ end + # FOI or EIR? - if !self.public_body.nil? && self.public_body.eir_only? + if new_record? && public_body && public_body.eir_only? self.law_used = 'eir' end end -- cgit v1.2.3