Lighthouse has a new layout. Prefer the old one? Return to the old layout, and switch back any time from the link at the top of each page.

There was a problem

You must be a member of this account.

Eloy Duran's latest activity

Monday, January 10 2011
1912 accepts_nested_attributes_for :reject_if option not honored for has_one assoc... was updated in Ruby on Rails
  • Eloy Duran
    Eloy Duran commented at 12:43 PM

    @Dave, adding a failing test case against the ActiveRecord test suite will make it much easier for others to confirm and/or fix.

Tuesday, September 21 2010
2933 2.3.2 update_attributes save many-to-many associations was updated in Ruby on Rails
  • Eloy Duran
    Eloy Duran commented at 4:40 PM

    @José Gotcha :)

    @lcars Ah, now I see the problem. #attributes= does basically the following for each attribute/value pair in the hash: send("#{attribute}=", val...

  • Eloy Duran
    Eloy Duran commented at 3:43 PM

    Hmm, I don’t really get what the test is trying to explain here… Afaik assigning to #assoc_ids= will always perform an implicit save directly. However, it seems...

  • Eloy Duran
    Eloy Duran commented at 3:08 PM

    #attributes= should never save any records by itself, that’s what #update_attributes is for. IMO it might be better if you’d start on a failing test case for th...

Wednesday, July 21 2010
5053 child/grandchild associations reload from db after nested attributes assignment was updated in Ruby on Rails
Thursday, July 15 2010
5053 child/grandchild associations reload from db after nested attributes assignment was updated in Ruby on Rails
  • Eloy Duran
    Eloy Duran commented at 11:17 AM

    Sure thing, added comments to each commit.

Thursday, July 08 2010
5053 child/grandchild associations reload from db after nested attributes assignment was updated in Ruby on Rails
  • Eloy Duran
    Eloy Duran commented at 6:32 PM

    I hope I got the class etc right, I haven't looked at AR internals for a while :)

  • Eloy Duran
    Eloy Duran commented at 6:31 PM

    @Neeraj Well, I can think of two options. Either create/merge a patch like the one I mentioned earlier: #1751.

    Or add something like this to your ~/.irbrc, beca...