Correct usage of simple_form collection_check_boxes

Question

I have set up 2 models as following :

class Sector < ActiveRecord::Base
  attr_accessible :summary, :title, :sector_ids

  belongs_to :platform
end

and

class Platform < ActiveRecord::Base
attr_accessible :name, :url
    has_many :sectors
end

and a form which tries to use example from here as follows :

<%= simple_form_for @platform, :html => { :class => 'form-vertical'} do |f| %>
  <%= f.error_notification %>

  <div class="form-inputs">
    <div class="row-fluid">
        <div class="span12">
            <div class="span6">
              <%=  field_set_tag "General" do %>
    <%= f.input :name %>
    <%= f.input :url %>
<%= f.collection_check_boxes :sector_ids, Sector.all, :title, :title %>

    <% end %>
</div>
</div>
</div>
  </div>

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

nevertheless, after I try to submit the form I get the following error :

Can't mass-assign protected attributes: sector_ids

What am I missing here? I successfully migrated the database after adding appropriate associations, but it seems like Rails doesnt really know what to do now wit the sector ids that are selected.

Solution :

class Sector < ActiveRecord::Base
  attr_accessible :summary, :title

  belongs_to :platforms
end

and

class Platform < ActiveRecord::Base
attr_accessible :name, :url, :platform_attributes, :sector_ids
    has_many :sectors
end

and in the view :

<%= f.association :sectors, :as => :check_boxes %>

Ofcourse, do not forget to run "rake db"migrate", in case you havent done it yet. I was also required to restart the server in order for the changes to apply.

I hope this helps someone.

score 4 · Answer 1 · answered Apr 15 '13 at 17:16

4

Try adding { :sector_ids => [] } to your Platform permits list.

https://github.com/rails/strong_parameters#nested-parameters

answered Apr 15 '13 at 17:16

Eric R

41
3

pablomarti · Accepted Answer · 2012-12-11T02:44:52.523

2

First I think you should remove :sectors_ids from the attr_accessible of Sector and try.

If the above doesn't works (which probably may not work), see the documentation of simple_form, specially read https://github.com/plataformatec/simple_form#associations and https://github.com/plataformatec/simple_form#collection-check-boxes; the last one makes what you are trying to do, but, in the associations they use has_and_belongs_to_many associations.

Finally, check this out https://github.com/plataformatec/simple_form/issues/341

Update The solution is to make a has_and_belongs_to_many relationship, there is no way out, because, you want to associate some sectors to one platform, but, you want to associate the same sectors to other platforms, otherwise you won't need checkbox, only nested forms to keep adding new records. You have to make this change in your database design and structure.

edited Dec 11 '12 at 02:44

answered Dec 09 '12 at 19:45

pablomarti

2,087
2
22
35

I am using one of the latest simple_form releases, and the issue you are referencing was solved more then a year ago. Removing :sectors_ids from the model indeed didnt solve the issue. Followed your reading material, but I still cannot figure it out, maybe some ideas for debugging? – Cninroh Dec 10 '12 at 00:41
@Cninroh checking again, you have to make a has_and_belongs_to_many relationship, there is no way out, because, you want to associate some sectors to one platform, but, you want to associate the same sectors to other platforms, otherwise you won't need checkbox, only nested forms to keep adding new records. You have to make this change in your database design and structure. – pablomarti Dec 11 '12 at 02:43

Correct usage of simple_form collection_check_boxes

2 Answers2