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.

Eloy Duran

Eloy Duran

Developer · Member of 1 project: Ruby on Rails

Eloy Duran's latest activity

Wednesday, March 11 2009
2144 Don't autosave new_record associations was updated in Ruby on Rails
  • Eloy Duran
    Eloy Duran commented at 2:51 PM

    Apparently the "always been the case" autosave exists for has_one relationships too....

    That's correct.

    I would really appreciate the ability to explicitly d...
2213 Saving objects with associated has_one :throughs loaded broken was updated in Ruby on Rails
  • Eloy Duran
    • Assigned user set to Eloy Duran
    • State changed from new to open
    • Milestone cleared.
    by Eloy Duran at 2:26 PM

    Patch applies and fixes mentioned problem. So +1.

    I just wonder if the tests shouldt be added to the "default" tests in autosave_association_test.rb.
Tuesday, March 10 2009
2146 AutosaveAssociation doesn't run removing association callbacks was updated in Ruby on Rails
  • Eloy Duran
    Eloy Duran commented at 6:32 PM

    Arg, and the patch…
  • Eloy Duran
    • Title changed from Nested attributes doesn't run removing association callbacks to AutosaveAssociation doesn't run removing association callbacks
    by Eloy Duran at 6:31 PM
  • Eloy Duran
    • State changed from open to verified
    by Eloy Duran at 6:31 PM

    I applied and test your patch and it works as expected.

    I then moved the tests to the AutosaveAssociation tests, because that's actually where it happens, not i...
  • Eloy Duran
    • State changed from new to open
    by Eloy Duran at 2:02 PM

    Looks great!

    I'll try to find some time tonight to apply and check it out.
  • Eloy Duran
    Eloy Duran commented at 11:41 AM

    Very good, a few comments:

    • There is now a little bit of repetition in association_collection #delete and #destroy. How about refatoring the common code into a ...
1 earlier change…
  • Eloy Duran
    Eloy Duran commented at 8:58 AM

    Ah I understand now. The difference is that you don't get callbacks to the parent that a associated records will be removed. Ok, I'll pick this up again when I ...
2160 nested_attributes: validates_uniqueness_of fails was updated in Ruby on Rails
  • Eloy Duran
    Eloy Duran commented at 8:59 AM

    Any news on this?
2149 Inconsistent has_many validation. was updated in Ruby on Rails
  • Eloy Duran
    Eloy Duran commented at 8:54 AM

    I tried this, as per your thoughts Eloy ( hopefully I understood them correctly, your saying the problem is the parent not being loaded by just doing a #first?...