[Rails-core] Re: Looking for comments on a rails patch, lazy association instantiation)

2007-10-28 Thread Rick Olson
On Oct 28, 12:43 pm, "Rick DeNatale" <[EMAIL PROTECTED]> wrote: > Thanks to a great suggestion by Josh Susser. I've spent the past two > days completely re-working the code I submitted in the eariler patch. > > The new patch;http://dev.rubyonrails.org/ticket/10012 > accomplishes the same result

[Rails-core] Re: Looking for comments on a rails patch, lazy association instantiation)

2007-10-28 Thread Rick DeNatale
Thanks to a great suggestion by Josh Susser. I've spent the past two days completely re-working the code I submitted in the eariler patch. The new patch; http://dev.rubyonrails.org/ticket/10012 accomplishes the same results, but with a much better api. The earlier patch added a new option for u

[Rails-core] Re: Looking for comments on a rails patch, lazy association instantiation

2007-10-26 Thread Rick DeNatale
On 10/26/07, Josh Susser <[EMAIL PROTECTED]> wrote: > If you wanted something friendlier than a SQL joins clause string, > like the hash/array syntax used by the :include option, that could be > a good alternate syntax for :joins. Much more Rails-like too. I'd > be interested in seeing somethin

[Rails-core] Re: Looking for comments on a rails patch, lazy association instantiation

2007-10-26 Thread Rick DeNatale
On 10/26/07, Josh Susser <[EMAIL PROTECTED]> wrote: > > > On Oct 24, 2007, at 3:55 PM, Rick DeNatale wrote: > > Prompted by some discussion on rails-talk last week, I worked up a > > patch to add an option to ActiveRecord::Base#find which allows > > associations to be :included for purposes of th

[Rails-core] Re: Looking for comments on a rails patch, lazy association instantiation

2007-10-26 Thread Josh Susser
On Oct 24, 2007, at 3:55 PM, Rick DeNatale wrote: > Prompted by some discussion on rails-talk last week, I worked up a > patch to add an option to ActiveRecord::Base#find which allows > associations to be :included for purposes of the where clause, but > defers instantiating the included associa

[Rails-core] Re: Looking for comments on a rails patch, lazy association instantiation

2007-10-24 Thread Rick DeNatale
Sorry I intended this to go to the rails-talk list but gmail outsmarted me. On 10/24/07, Rick DeNatale <[EMAIL PROTECTED]> wrote: > Prompted by some discussion on rails-talk last week, I worked up a > patch to add an option to ActiveRecord::Base#find which allows > associations to be :included f