This project is archived and is in readonly mode.

#6727 new
Justin French

ActionView::Helpers::FormHelper#fields_for API changed in master

Reported by Justin French | April 20th, 2011 @ 01:23 PM

ActionView::Helpers::FormHelper#fields_for now has four params (record, record_object, options, block) in master instead of three (record_or_name_or_array, args, &block) in Rails 3.0.6.

I'm mostly surprised that a public method has changed between releases without deprecation warnings, documentation, etc. It also bugs me that the params are different to ActionView::Helpers::FormBuilder#fields_for. The change seems to have been introduced in 58d3b9042302d41f90fc129f12787d83f7bdade9 ("Speed up form_for by specializing extract_options!") by Yehuda in August (can be found in the diff between v3.0.6 and master).

Ideally this change would be reverted, or perhaps a backwards compatible way to keep the optimisation can be found.

No comments found

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile »

<h2 style="font-size: 14px">Tickets have moved to Github</h2>

The new ticket tracker is available at <a href="https://github.com/rails/rails/issues">https://github.com/rails/rails/issues</a>

People watching this ticket

Pages