[Rails-core] Re: Nested :through associations

2010-11-28 Thread lardawge
Jon, I would make sure this currently merges into master and send a pull request on github with the lighthouse ticket in the comments... Need get more eyes on it. On Nov 17, 3:40 am, Jon Leighton wrote: > Hi all, > > In October I worked on a patch for a long-requested feature: > associations that

[Rails-core] Re: Nested :through associations

2010-11-29 Thread Jon Leighton
The patch has now been "verified" on Lighthouse. I.e. 3 people have tested and +1ed it. I've done a pull request here: https://github.com/rails/rails/pull/121 I merged latest master over the weekend and will continue to merge as necessary to keep it up to date. Jon On Nov 29, 1:42 am, lardawge

[Rails-core] Re: Nested :through associations

2010-12-02 Thread Jack Kinsella
+1 for the usefulness of Jon's patch. During a previous project I was very disappointed to find that Rails didn't support nested :through associations. Besides my own experiences, a quick search on Google for "nested has many through associations" shows a litany of blog posts and discussions on hac

Re: [Rails-core] Re: Nested :through associations

2010-12-01 Thread Jon Leighton
Hi all, Jose has responded on the pull request: === Hey mate, thanks for the pull request. But could you please send this request to Rails mailing list and get feedback from more people? At first, I am -1 on this feature. Rails has_many :through, without b

Re: [Rails-core] Re: Nested :through associations

2010-12-01 Thread Jason King
I'm very much in favor of hmt working to an infinite depth. From my own experience as well as responding to others, I know it's something that people just expect to work. I'm also concerned about bugs, but I like your suggestion Jon. On Dec 1, 2010 12:18 PM, "Jon Leighton" wrote: > Hi all, > > Jo

Re: [Rails-core] Re: Nested :through associations

2010-12-01 Thread Lunks
+1 for sorting bugs out and commiting this. HMT just helps doing some associations where it would be hard to do. Even if it is not fool proof, allowing us to associate models further where needed is welcomed. I have been in a couple of projects where coding would be easier with this patch. On Wed,