This project is archived and is in readonly mode.
Scaffolding Cleanup
Reported by CancelProfileIsBroken | February 1st, 2009 @ 04:49 PM | in 2.x
We should clean up two things in scaffolding that are a source of confusion to new developers (as measured by questions via IRC and elsewhere):
1) The index action in the controller should use .all instead of .find(:all)
2) The index view should use +@entities.each do |entity|+ instead of +for entity in @entities+.
Comments and changes to this ticket
-
Repository February 5th, 2009 @ 07:38 PM
- State changed from “new” to “committed”
(from [44f02de2ad6d22d0d781c0c7303bcd8162aaa5d6]) Update scaffolding [#1840 state:committed]
Signed-off-by: David Heinemeier Hansson david@loudthinking.com http://github.com/rails/rails/co...
-
-
CancelProfileIsBroken February 5th, 2009 @ 10:14 PM
Because using "for" has been one of the perennial confusion points for people using Rails as an intro to Ruby. Have run into too many people who don't grok .each do because scaffolding & AWDWR taught them for first.
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
CancelProfileIsBroken
Pratik
Attachments
Tags
Referenced by
-
1840
Scaffolding Cleanup
(from [44f02de2ad6d22d0d781c0c7303bcd8162aaa5d6]) Update
...