Re: [Rails-core] Exclude Fields from Eager Loaded Models

2010-01-20 Thread Pratik
I don't believe this should be in core for a few reasons : 1) It's not needed very often 2) Difficult to make it work with eager loading. Your patch only deals with preloading. 3) I'd prefer using a plugin like http://github.com/mcmire/ar_attr_lazy/ On Wed, Jan 20, 2010 at 8:49 PM, Josh Symonds

Re: [Rails-core] Exclude Fields from Eager Loaded Models

2010-01-20 Thread Josh Symonds
I read the Rails 3 Beta call email and I wanted to try to raise this ticket again to maybe get it a bit more visibility before the beta comes out. Anyone feel like commenting on this patch? I've found it helpful, anyway! On Tue, Jan 5, 2010 at 2:38 PM, Josh Symonds wrote: > This isn't *necessari

Re: [Rails-core] Exclude Fields from Eager Loaded Models

2010-01-05 Thread Josh Symonds
This isn't *necessarily* a problem. If you have an association named :except you can still eagerly load it using either the array syntax: User.find(:all, :include => [:comments, :except]) Or just the regular symbol notation: User.find(:all, :include => :except) If you include it in the hash not

Re: [Rails-core] Exclude Fields from Eager Loaded Models

2010-01-05 Thread Jeremy Evans
On Tue, Jan 5, 2010 at 12:06 PM, Josh Symonds wrote: > I made a patch to allow eager loading to exclude specified columns on the > eagerly-loaded models. If you have a model that you want to load for a > specific reason -- like, say, you have lots of users, and you want to grab > all their posts b

[Rails-core] Exclude Fields from Eager Loaded Models

2010-01-05 Thread Josh Symonds
I made a patch to allow eager loading to exclude specified columns on the eagerly-loaded models. If you have a model that you want to load for a specific reason -- like, say, you have lots of users, and you want to grab all their posts but exclude the posts' bodies -- you can now specify that by go