diff options
author | James McKinney <james@slashpoundbang.com> | 2015-05-20 15:05:35 +0200 |
---|---|---|
committer | James McKinney <james@slashpoundbang.com> | 2015-05-20 15:05:35 +0200 |
commit | 363e121bdfbdf7f4a5bc156ffa9d7868a3e7ed75 (patch) | |
tree | bca1b0237dc7c0493f64ba54ab077402d90a99ae /app/models/info_request.rb | |
parent | da706b6c05ee7d0be15b59cf0fdec7f6d3ed58ec (diff) |
Extract a method to add conditions from extra_params in InfoRequest
Diffstat (limited to 'app/models/info_request.rb')
-rw-r--r-- | app/models/info_request.rb | 26 |
1 files changed, 11 insertions, 15 deletions
diff --git a/app/models/info_request.rb b/app/models/info_request.rb index 65099b1c4..2770715a2 100644 --- a/app/models/info_request.rb +++ b/app/models/info_request.rb @@ -1015,21 +1015,13 @@ public def InfoRequest.count_old_unclassified(extra_params={}) params = old_unclassified_params(extra_params) - if extra_params[:conditions] - condition_string = extra_params[:conditions].shift - params[:conditions][0] += " AND #{condition_string}" - params[:conditions] += extra_params[:conditions] - end + add_conditions_from_extra_params(params, extra_params) count(:all, params) end def InfoRequest.get_random_old_unclassified(limit, extra_params) params = old_unclassified_params({}) - if extra_params[:conditions] - condition_string = extra_params[:conditions].shift - params[:conditions][0] += " AND #{condition_string}" - params[:conditions] += extra_params[:conditions] - end + add_conditions_from_extra_params(params, extra_params) params[:limit] = limit params[:order] = "random()" find(:all, params) @@ -1044,11 +1036,7 @@ public params[:order] = extra_params[:order] params.delete(:select) end - if extra_params[:conditions] - condition_string = extra_params[:conditions].shift - params[:conditions][0] += " AND #{condition_string}" - params[:conditions] += extra_params[:conditions] - end + add_conditions_from_extra_params(params, extra_params) find(:all, params) end @@ -1436,5 +1424,13 @@ public errors.add(:title, _('Please describe more what the request is about in the subject. There is no need to say it is an FOI request, we add that on anyway.')) end end + + def add_conditions_from_extra_params(params, extra_params) + if extra_params[:conditions] + condition_string = extra_params[:conditions].shift + params[:conditions][0] += " AND #{condition_string}" + params[:conditions] += extra_params[:conditions] + end + end end |