filterrific icon indicating copy to clipboard operation
filterrific copied to clipboard

Multi-select and multiple inputs for a single scope don't work together

Open neilvcarvalho opened this issue 1 year ago • 1 comments

Hi, folks,

I have a scope involving multiple fields, and some of those fields are multi-select fields.

I noticed that Filterrific drops some of the params for this multi-input scope. After investigating, I found out that the dropped params are the array-type params.

For example:

<%= fields_for :with_skills do |skills_form| %>
  <%= f.select :skills, skill_options, multiple: true %>
  <%= f.select :expertise, expertise_options %>
<% end %>

In this case, params will receive something like: {filters: { with_skills: {skills: ["knitting", "kungfu"], expertise: "mentor"}}}

Filterrific will accept the expertise filter, but not the skills filter.

I noticed https://github.com/jhund/filterrific/blob/master/lib/filterrific/param_set.rb#L27 handles multi-input scopes and multi-select fields, but not both. I plan to open a PR fixing this.

neilvcarvalho avatar Sep 01 '23 20:09 neilvcarvalho

This issue is fixed by #213

@jhund I'd be so thankful if #213 is merged and you cut a new release with this fix

neilvcarvalho avatar Sep 04 '23 16:09 neilvcarvalho