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.

Jon Leighton's latest activity

Tuesday, December 21 2010
5442 Model.has_many_through_association.find(id) returns a read-only record was updated in Ruby on Rails
  • Jon Leighton
    • Assigned user set to Aaron Patterson
    by Jon Leighton at 12:48 PM

    Hi,

    This patch didn't apply cleanly for me initially. I merged in the tests and ran them, and they passed on master and on 3-0-stable. I've attached an updated ...

5137 [PATCH] has_one :through through belongs_to associations does not build throu... was updated in Ruby on Rails
  • Jon Leighton
    Jon Leighton commented at 12:12 PM

    This bug seems to be related to (or possibly a duplicate of) #142.

5131 Preloading associations generates invalid sql for STI and has_many :through w... was updated in Ruby on Rails
  • Jon Leighton
    • State changed from new to resolved
    • Importance changed from to Low
    by Jon Leighton at 11:25 AM

    This is no longer an issue. It's covered by test_find_with_sti_join in inner_join_associations_test.rb. (FYI, doing Foo.includes(:bar).where(:conditions) genera...

Monday, December 20 2010
5970 Strange has_many :through issue with adding objects to a collection was updated in Ruby on Rails
  • Jon Leighton
    • State changed from new to invalid
    • Importance changed from to Low
    by Jon Leighton at 8:14 PM
4887 has_many through belongs_to association bug was updated in Ruby on Rails
  • Jon Leighton
    • State changed from open to resolved
    by Jon Leighton at 8:14 PM
2461 Joined table not quoted with has_many :through was updated in Ruby on Rails
  • Jon Leighton
    • State changed from new to resolved
    by Jon Leighton at 8:11 PM
3684 find creates wrong SQL when you set "has_one :through" association on :join o... was updated in Ruby on Rails
  • Jon Leighton
    • State changed from new to duplicate
    by Jon Leighton at 8:07 PM
3979 [PATCH] Join conditions removed on habtm and has many through with conditions... was updated in Ruby on Rails
  • Jon Leighton
    • State changed from new to resolved
    • Importance changed from to Low
    by Jon Leighton at 7:49 PM
  • Jon Leighton
    • Assigned user set to Aaron Patterson
    by Jon Leighton at 4:00 PM

    This is no longer an issue and so it can be closed.

    I've run both of David's tests and they work. It's already covered by test_eager_with_has_many_through_join_...

Jon Leighton Jon Leighton was invited to Ruby on Rails at 7:09 PM