diff options
| author | Gareth Rees <gareth@mysociety.org> | 2015-05-13 15:02:02 +0100 |
|---|---|---|
| committer | Gareth Rees <gareth@mysociety.org> | 2015-06-04 13:51:02 +0100 |
| commit | 752048a5c3f3fc3fa63e78c665c1fa6a4e65bba8 (patch) | |
| tree | 12495115472ba7ca4de122be5a9ed65dd66f38fc /lib/xapian_queries.rb | |
| parent | 197dd8676b43dced1486feefde09dbd2d1d5d616 (diff) | |
Use Hash#fetch for nil fallback
Easier to understand
Removes explicit return
Diffstat (limited to 'lib/xapian_queries.rb')
| -rw-r--r-- | lib/xapian_queries.rb | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/lib/xapian_queries.rb b/lib/xapian_queries.rb index 5c02ba788..dbb17f592 100644 --- a/lib/xapian_queries.rb +++ b/lib/xapian_queries.rb @@ -77,10 +77,10 @@ module XapianQueries end def make_query_from_params(params) - query = params[:query] || "" if query.nil? + query = params.fetch(:query) { '' } query += get_date_range_from_params(params) query += get_request_variety_from_params(params) query += get_status_from_params(params) - return query + query end end |