Rails multiple search parameters and defaults - ruby-on-rails

Question

I have a search form to search incidents model with 3 fields: incidenttype, dayofweek and created_at. In my incidents.index method I check to see if the search param is present. Then filter the results based on the created_at range etc. How would I set this up so that if the dayofweek and incidenttype params were empty or blank it would just not use them in the query filter? Essentially if none were selected do not use those params, only the created_at.
Index method of incidents controller:
def index
if params[:search] && params[:search][:created_at].present?
start_date, end_date = params[:search][:created_at].split(' - ')
#incidents = Incident.where(created_at: start_date..end_date).where(dayofweek: params[:search][:dayofweek]).where(incidenttype: params[:search][:incidenttype])
#dayofweek = params[:search][:dayofweek]
#incidenttypes = Incident.order("incidenttype").distinct.pluck(:incidenttype)
else
#incidents = Incident.all
#incidenttypes = Incident.order("incidenttype").distinct.pluck(:incidenttype)
end
end
Form:
<%= form_for(:search, url: incidents_path, method: :get) do |f| %>
<div class="ui-bordered px-4 pt-4 mb-4">
<div class="form-row">
<div class="col-md mb-4">
<label class="form-label">Type</label>
<%= f.select :dayofweek, Incident.dayofweeks.keys, {:include_blank=> 'Any days', :selected => #dayofweek}, class: 'custom-select' %>
</div>
<div class="col-md mb-4">
<label class="form-label">Incident Type</label>
<%= f.select :incidenttype, options_for_select(#incidenttypes, :selected => params[:search][:incidenttype]), {include_blank: true}, class: 'form-control' %>
</div>
<div class="col-md mb-4">
<label class="form-label">Created date</label>
<%= f.text_field :created_at, class: 'form-control', id: 'tickets-list-created', :autocomplete => :off, value: created_at_from_parameters %>
</div>
<div class="col-md col-xl-2 mb-4">
<label class="form-label d-none d-md-block"> </label>
<button type="submit" class="btn btn-secondary btn-block">Show</button>
</div>
</div>
</div>
<% end %>
Any thoughts on how to set this up?

Answer 1

Replace your index method with:
def index
#incidenttypes = Incident.order(:incidenttype).distinct.pluck(:incidenttype)
#incidents = if params[:search] && params[:search][:created_at].present?
#dayofweek = params[:search][:dayofweek]
start_date, end_date = params[:search][:created_at].split(' - ')
filters = { created_at: start_date..end_date }
[:dayofweek, :incidenttype].each do |filter|
next if params[:search][filter].blank?
filters.merge!({ filter => params[:search][filter] } )
end
Incident.where(filters)
else
Incident.all
end
end