diff options
| author | Matthew Landauer <matthew@openaustralia.org> | 2013-02-12 12:53:30 +1100 |
|---|---|---|
| committer | Matthew Landauer <matthew@openaustralia.org> | 2013-02-12 12:53:30 +1100 |
| commit | 2eadcc3417b11237f9e40dd5eb5804b2fcc44c82 (patch) | |
| tree | cc05a244b04fcfcb6858cd256a0ff4bd5c016b30 /app/controllers/application_controller.rb | |
| parent | c4ee8fc6aeba6f1bb6eaad161c7bbc766a19e4d1 (diff) | |
Pass parameters to method rather explicitly
Diffstat (limited to 'app/controllers/application_controller.rb')
| -rw-r--r-- | app/controllers/application_controller.rb | 18 |
1 files changed, 9 insertions, 9 deletions
diff --git a/app/controllers/application_controller.rb b/app/controllers/application_controller.rb index e149d2de7..f3deeb64a 100644 --- a/app/controllers/application_controller.rb +++ b/app/controllers/application_controller.rb @@ -456,7 +456,7 @@ class ApplicationController < ActionController::Base end end - def get_request_variety_from_params + def get_request_variety_from_params(params) query = "" sortby = "newest" varieties = [] @@ -478,7 +478,7 @@ class ApplicationController < ActionController::Base return query end - def get_status_from_params + def get_status_from_params(params) query = "" if params[:latest_status] statuses = [] @@ -513,7 +513,7 @@ class ApplicationController < ActionController::Base return query end - def get_date_range_from_params + def get_date_range_from_params(params) query = "" if params.has_key?(:request_date_after) && !params.has_key?(:request_date_before) params[:request_date_before] = Time.now.strftime("%d/%m/%Y") @@ -527,7 +527,7 @@ class ApplicationController < ActionController::Base return query end - def get_tags_from_params + def get_tags_from_params(params) query = "" tags = [] if params.has_key?(:tags) @@ -541,12 +541,12 @@ class ApplicationController < ActionController::Base return query end - def make_query_from_params + def make_query_from_params(params) query = params[:query] || "" if query.nil? - query += get_date_range_from_params - query += get_request_variety_from_params - query += get_status_from_params - query += get_tags_from_params + query += get_date_range_from_params(params) + query += get_request_variety_from_params(params) + query += get_status_from_params(params) + query += get_tags_from_params(params) return query end |