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

Thursday, December 17 2009
2578 accepts_nested_attributes_for causes stack level too deep was updated in Ruby on Rails
  • Eloy Duran
    Eloy Duran commented at 3:01 PM

    @Bryan: After some discussion the :inverse_of patches have been backported. It would be great if you could verify that they fix the problem you had. All the pat...
2815 nested models: build should directly assign the parent was updated in Ruby on Rails
3588 :autosave => true + accepts_nested_attributes_for = circular validation & blo... was updated in Ruby on Rails
  • Eloy Duran
    Eloy Duran commented at 2:41 PM

    Actually, instead of only that patch, could you try this Rails branch? http://github.com/Fingertips/rails/tree/2-3-stable

    It contains the aforementioned patch, ...
  • Eloy Duran
    Eloy Duran commented at 10:44 AM

    I’m actually pretty glad you run into this problem, as you could verify the patch on #3533 for me :)

    Please report on that ticket if the patch fixes your proble...
3513 :inverse_of not honored when replacing a has_one association was updated in Ruby on Rails
3520 Add inverse polymorphic association support was updated in Ruby on Rails
  • Eloy Duran
    • State changed from new to verified
    by Eloy Duran at 11:15 AM

    Oh, and it applies and runs :)
  • Eloy Duran
    • Milestone set to 2.3.6
    by Eloy Duran at 11:01 AM
  • Eloy Duran
    • Assigned user set to Michael Koziarski
    by Eloy Duran at 11:01 AM

    I’m wondering, why do you need to fix all these other tests? Eg:

    -    m = Man.find(:first, :include => :interests)
+    m = Man.find(:first, :conditions => {:n...
Friday, December 11 2009
3560 accepts_nested_attributes_for with :touch => ture cause ActiveRecord::StaleOb... was updated in Ruby on Rails
  • Eloy Duran
    Eloy Duran commented at 10:21 AM

    Seems valid, however, you patch is lacking test coverage. Could you please add tests to the patch?

    Also, it seems by glancing over the patch that this would alw...