aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorTeddy Wing2018-02-01 15:20:43 +0100
committerTeddy Wing2018-02-01 15:20:43 +0100
commit720d11cb7d2475277e7b79595d857f5ffef5bee1 (patch)
tree48c554a07fab6e98b7a9c7bf99ce9f884297b89a
parentd896ba8753f9877dfd29a97cf1310d54d0b25874 (diff)
downloadchouette-core-720d11cb7d2475277e7b79595d857f5ffef5bee1.tar.bz2
ReferentialVehicleJourneys#index: Fix `nil` error on `stop_area_ids`
This wasn't checking that `params[:q]` was present before trying to access `[:stop_area_ids]`, and thus caused an error on the page when no filters were applied. Refs #5809
-rw-r--r--app/controllers/referential_vehicle_journeys_controller.rb2
1 files changed, 1 insertions, 1 deletions
diff --git a/app/controllers/referential_vehicle_journeys_controller.rb b/app/controllers/referential_vehicle_journeys_controller.rb
index a199157dd..2ce28a5cc 100644
--- a/app/controllers/referential_vehicle_journeys_controller.rb
+++ b/app/controllers/referential_vehicle_journeys_controller.rb
@@ -23,7 +23,7 @@ class ReferentialVehicleJourneysController < ChouetteController
@vehicle_journeys ||= @q.result.order(:published_journey_name).includes(:vehicle_journey_at_stops).paginate page: params[:page], per_page: params[:per_page] || 10
@all_companies = Chouette::Company.where("id IN (#{@referential.vehicle_journeys.select(:company_id).to_sql})").distinct
@all_stop_areas = Chouette::StopArea.where("id IN (#{@referential.vehicle_journeys.joins(:stop_areas).select("stop_areas.id").to_sql})").distinct
- stop_area_ids = params[:q][:stop_area_ids].select(&:present?)
+ stop_area_ids = params[:q].try(:[], :stop_area_ids).try(:select, &:present?)
@filters_stop_areas = Chouette::StopArea.find(stop_area_ids) if stop_area_ids.present? && stop_area_ids.size <= 2
end