This project is archived and is in readonly mode.
Multi-parameter assignment refactoring
Reported by Eric Chapweske | October 4th, 2009 @ 12:46 AM
This cleans up the internal implementation, with most of the code being moved into Multiparameter:Attribute and Multiparameter::Attributes classes. I would like to expand on these changes and look into:
1) Moving typecasting responsibility to the attribute writer
methods, e.g., allowing for @record.published_on = [ 2009, 6, 3
].
2) Move the multi-parameter parsing logic from ActiveRecord to
action pack. This would cleanup the attributes= implementation,
among other things.
Any feedback on the patched implementation, or thoughts on the other proposals is much appreciated! :)
Comments and changes to this ticket
-
Rohit Arondekar October 6th, 2010 @ 06:41 AM
- State changed from “new” to “stale”
- Importance changed from “” to “”
Marking ticket as stale. If this is still an issue please leave a comment with suggested changes, creating a patch with tests, rebasing an existing patch or just confirming the issue on a latest release or master/branches.
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
Eric Chapweske
Jeremy Kemper
Rohit Arondekar
Attachments
Referenced by
-
3178
Mass Assignment Security Refactoring
Also rewrote multi-parameter assignment, the ticket is at...