diff options
| author | Teddy Wing | 2018-02-01 15:20:43 +0100 | 
|---|---|---|
| committer | Teddy Wing | 2018-02-01 15:20:43 +0100 | 
| commit | 720d11cb7d2475277e7b79595d857f5ffef5bee1 (patch) | |
| tree | 48c554a07fab6e98b7a9c7bf99ce9f884297b89a | |
| parent | d896ba8753f9877dfd29a97cf1310d54d0b25874 (diff) | |
| download | chouette-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.rb | 2 | 
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 | 
