This project is archived and is in readonly mode.
overwrite_params no longer works with url_for
Reported by Sam Ruby | March 1st, 2010 @ 12:59 AM | in 2.3.6
Overwrite_params is described here:
http://api.rubyonrails.org/classes/ActionController/Base.html
This functionality works in 2.3.5, and worked in 3.0 up to the following commit:
http://github.com/rails/rails/commit/226dfc2681c98deaf14e4ae82e973d...
An example of the breakage can be found at the bottom of the following page:
http://intertwingly.net/projects/AWDwR3/checkdepot/section-21.2.html
The same example under 2.3.5:
http://intertwingly.net/projects/AWDwR3/checkdepot-235/section-21.2...
Either this is an intentional change in behavior (and the rdocs need to be updated), or the function needs to be restored.
Comments and changes to this ticket
-
José Valim March 3rd, 2010 @ 08:50 AM
- Assigned user set to Yehuda Katz (wycats)
- State changed from new to open
-
Sam Ruby March 10th, 2010 @ 03:09 PM
- Milestone set to 2.3.6
Patch provided to deprecate :overwrite_params in 2.3.6
-
Repository March 11th, 2010 @ 12:26 AM
- State changed from open to committed
(from [cbc0201a3e0145da51baa9d228d88633033521ce]) Add deprecation warning for overwrite_params and remove rdoc
[#4073 state:committed]
Signed-off-by: Jeremy Kemper jeremy@bitsweat.net
http://github.com/rails/rails/commit/cbc0201a3e0145da51baa9d228d886... -
Jeff Kreeftmeijer November 7th, 2010 @ 04:52 PM
- Importance changed from to Low
Automatic cleanup of spam.
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
Attachments
Referenced by
- 4073 overwrite_params no longer works with url_for [#4073 state:committed]