Skip to content

Commit

Permalink
Merge pull request #115 from ayaman/master
Browse files Browse the repository at this point in the history
Fix 'permit' params to allow multiple form inputs for a single scope
  • Loading branch information
jhund committed Mar 8, 2017
2 parents 5564769 + 292ecd9 commit 40ecd19
Showing 1 changed file with 11 additions and 1 deletion.
12 changes: 11 additions & 1 deletion lib/filterrific/param_set.rb
Expand Up @@ -29,7 +29,17 @@ def initialize(a_model_class, filterrific_params = {})
# will be already initialized with the defaults.
filterrific_params = model_class.filterrific_default_filter_params if filterrific_params.blank?
if defined?(ActionController::Parameters) && filterrific_params.is_a?(ActionController::Parameters)
filterrific_params = filterrific_params.permit(model_class.filterrific_available_filters).to_h.stringify_keys
permissible_filter_params = []
model_class.filterrific_available_filters.each do |p|
if filterrific_params[p].is_a?(ActionController::Parameters)
permissible_filter_params << { p => filterrific_params[p].keys }
elsif filterrific_params[p].is_a?(Array)
permissible_filter_params << { p => [] }
else
permissible_filter_params << p
end
end
filterrific_params = filterrific_params.permit(permissible_filter_params).to_h.stringify_keys
else
filterrific_params.stringify_keys!
end
Expand Down

1 comment on commit 40ecd19

@xneceo
Copy link

@xneceo xneceo commented on 40ecd19 Aug 1, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would be great if you could release a new version including this fix

Please sign in to comment.