Thanks all. I will patch and let you know the link for the +1's! Adam
On Apr 23, 3:38 am, Jan De Poorter <[EMAIL PROTECTED]> wrote: > Sounds like it should be! Go for the patch! > > regards, > Jan > > On 22 Apr 2008, at 08:18, Adam Meehan wrote: > > > > > Hi all, > > > I just wanted some feedback on a AR hack I have been playing with > > which implements the block syntax for create and update method. > > > For example, instead of: > > > @person = Person.new(params[:person]) > > @person.has_rails_patch = false > > @person.set_status :uncool > > @person.save > > > You could do: > > > @person = Person.create(params[:person]) do |p| > > p.has_rails_patch = false > > p.set_status(:uncool) > > end > > > And the update version > > > @person = Person.update(params[:id], params[:person]) do |p| > > p.set_status(:cool) > > end > > > It makes it more Rubyish with the blocks, don't you think? The bang! > > versions could added easily enough, for create anyway. > > > Let me know what you think before I put a patch in. > > > Thanks, > > Adam. --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Ruby on Rails: Core" group. To post to this group, send email to [email protected] To unsubscribe from this group, send email to [EMAIL PROTECTED] For more options, visit this group at http://groups.google.com/group/rubyonrails-core?hl=en -~----------~----~----~----~------~----~------~--~---
