This project is archived and is in readonly mode.

#3317 ✓stale
Eric Chapweske

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

    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>

Attachments

Referenced by

Pages