diff options
| author | Louise Crow <louise.crow@gmail.com> | 2013-10-24 16:19:22 +0100 |
|---|---|---|
| committer | Louise Crow <louise.crow@gmail.com> | 2013-12-04 09:32:42 +0000 |
| commit | 4b2e3ef0b6f5b881597242a4afe1404415256f3b (patch) | |
| tree | b2317647312a8960a988e61cc8e1b704d3a92bc2 /app/models/info_request.rb | |
| parent | cf11b959899d54cf3c7f4e018f11c2c89c83d4af (diff) | |
Make method names for finding existing objects clearer
They're not finding by the existing object, they're finding an existing
object.
Diffstat (limited to 'app/models/info_request.rb')
| -rw-r--r-- | app/models/info_request.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/app/models/info_request.rb b/app/models/info_request.rb index b990f4b41..c95b7427e 100644 --- a/app/models/info_request.rb +++ b/app/models/info_request.rb @@ -387,7 +387,7 @@ public # repeated requests, say once a quarter for time information, then might need to do that. # XXX this *should* also check outgoing message joined to is an initial # request (rather than follow up) - def InfoRequest.find_by_existing_request(title, public_body_id, body) + def InfoRequest.find_existing(title, public_body_id, body) return InfoRequest.find(:first, :conditions => [ "title = ? and public_body_id = ? and outgoing_messages.body = ?", title, public_body_id, body ], :include => [ :outgoing_messages ] ) end |