This project is archived and is in readonly mode.
Fix FormHelper#check_box documentation in 2.3
Reported by jrdioko | January 18th, 2011 @ 08:07 PM
In 2.3, ActionView::Helpers::FormHelper#check_box was changed to put the generated hidden field before the checkbox tag, not after it, because the HTTP parameter extraction order changed. The documentation was fixed in edge at that point, but was never updated for 2.3. See the following discussion in rubyonrails-core:
http://groups.google.com/group/rubyonrails-core/browse_thread/threa...
In that discussion Xavier Noria requested that I create a patch and LH ticket, so I am doing so. I will produce the patch and attach it to this ticket.
Comments and changes to this ticket
-
jrdioko January 18th, 2011 @ 08:23 PM
Attaching the patch. This is my first time doing this, so let me know if I did something wrong. I directly copied over edits to the documentation from commit f97832b1e4406a76d268.
-
Xavier Noria January 19th, 2011 @ 07:31 AM
- State changed from new to open
- Assigned user set to Xavier Noria
- Importance changed from to Low
-
Repository January 19th, 2011 @ 07:49 AM
- State changed from open to resolved
(from [4f0c8ef9f1e68be88b22cce6a857e1a9b6434182]) Fix doc for #check_box [#6311 state:resolved]
Signed-off-by: Xavier Noria fxn@hashref.com
https://github.com/rails/rails/commit/4f0c8ef9f1e68be88b22cce6a857e...
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
- 6311 Fix FormHelper#check_box documentation in 2.3 (from [4f0c8ef9f1e68be88b22cce6a857e1a9b6434182]) Fix doc...