diff options
author | ojab <ojab@ojab.ru> | 2015-08-30 10:02:24 +0000 |
---|---|---|
committer | ojab <ojab@ojab.ru> | 2015-08-30 20:44:09 +0000 |
commit | 7089768b700f79b2ae5171dc434ef45d16d18a63 (patch) | |
tree | b1bbfd6960f56198446b00adad1ca56245aceeda /activerecord/lib/active_record/connection_handling.rb | |
parent | 79f44eb6c84fe1d7a7ba51f3f511d2e744788ef8 (diff) | |
download | rails-7089768b700f79b2ae5171dc434ef45d16d18a63.tar.gz rails-7089768b700f79b2ae5171dc434ef45d16d18a63.tar.bz2 rails-7089768b700f79b2ae5171dc434ef45d16d18a63.zip |
Handle nested fields_for by adding indexes to record_name
In case of the form with nested fields_for, i. e.
<%= form_for :foos, url: root_path do |f| %>
<% @foos.each do |foo| %>
<%= f.fields_for 'foo[]', foo do |f2| %>
<%= f2.text_field :id %>
<% foo.bars.each do |bar| %>
<%= f2.fields_for 'bar[]', bar do |b| %>
<%= b.text_field :id %>
<% end %>
<% end %>
<% end %>
<% end %>
<%= f.submit %>
<% end %>
rails doesn't add index for 'foo' in the inner fields_for block, so field names
in the outer fields_for looks like "foos[foo][#{foo_index}][id]" and in the
inner "foos[foo[]][bar][#{bar_index}][id]". Submitting of such form leads to an
error like:
>ActionController::BadRequest (Invalid request parameters: expected Array
>(got Rack::QueryParser::Params) for param `foo'):
This commit adds indexes for the foos in the inner blocks, so field names
become "foos[foo][#{foo_index}][bar][#{bar_index}][id]" and submitting of such
form works fine as expected.
Fixes #15332
Diffstat (limited to 'activerecord/lib/active_record/connection_handling.rb')
0 files changed, 0 insertions, 0 deletions