trjhtr

In my Rails controller with strong parameters, I have the typical my_model_params private function to filter out any "scary internet params". One of the parameters is a list from a Select2 form input. This Select2 form input always passes an empty or null value as the first element, like ["", "23", "32"]. I can easily clean it up in my_model_params by removing it with reject function like this: .reject!

Since there are multiple fields I ended up with the entire function looking like this:

 # Never trust parameters from the scary internet, only allow the white list through.
 def my_model_params
 filtered_params = params.require(:my_model)
 .permit(:name, :description, :status,
 :start_date, :end_date,
 geo_graphics: , geo_media_types: ,
 geo_counties: , geo_states: )

 filtered_params[:geo_states].reject! 
 filtered_params[:geo_counties].reject! 
 filtered_params
 end

This looks bloated and ugly. How can I clean this up to look better or is there a different way to reject the empty strings in the array? It seems like this could look better if it were chained or somehow filter after the permit function

EDIT:

Here is the view code. It use the simple_form select2 gem. I had been using just the select2 js library although I couldn't get the library to load without the blank entry to loading as an item within the select2 input. It would load as a blank box, which make sense as the submitted form value was "".

<%= simple_form_for([@account, @request_for_proposal]) do |f| %>
 <%= f.error_notification %>

 <div class="form-inputs">
 <%= f.input :name %>
 <%= f.input :description %>
 <%= f.input :status, collection: RequestForProposal.statuses.keys %>
 <%= f.input :start_date %>
 <%= f.input :end_date %>

 <%= f.association :campaign, collection: @account.campaigns %>
 <%= f.association :user, as: :hidden %>
 <%= f.association :account, as: :hidden %>


 <%= f.input :geo_counties, as: :select2, collection: , include_blank: false, multiple: true %>
 <%= f.input :geo_states, as: :select2, collection: , include_blank: false, multiple: true %>

 </div>

 <div class="form-actions">
 <%= f.button :submit %>
 </div>
<% end %>


<script>

 $('#request_for_proposal_geo_counties').select2(
 theme: "bootstrap",
 cache: true,
 multiple: true,
 closeOnSelect: false,
 placeholder: 
 id: '-1', // the value of the option
 text: 'Select an option'
 ,
 ajax: 
 url: '/api/v1/counties.json',
 dataType: 'json'
 
 );

 var selected_states = <%= raw @request_for_proposal.geo_path_states.to_json %>;
 var keys = [ "id", "text", "zip_codes", "counties","population_0", "population_5", "population_18", "population_21"];

 $('#request_for_proposal_geo_states').select2(
 theme: "bootstrap",
 placeholder: 
 id: '-1', // the value of the option
 text: 'Select an option'
 ,
 multiple: true,
 ajax: 
 url: '/api/v1/states.json',
 processResults: function (data) 
 // Tranforms the top-level key of the response object from 'items' to 'results'
 $.each(data.rows ,function(pos,obj)
 var counter = 0;
 $.each(obj,function(key,value)
 data.rows [pos][keys[counter]] = value;
 delete data.rows [pos][key];
 counter++;
 )
 )
 return 
 results: data.rows
 ;
 
 

 );

</script>