James Le Cuirot

Pratik, please read the rest of the ticket. This isn't just about default_scope. The title is actually misleading and I will now change it. Note in particular t...

I think you guys need to see ticket #2348.

I was going to say I forgot to tackle construct_finder_sql_for_association_limiting but it turns out I didn't need to. It already works with the above patch.

Here's a patch with a test for the non-preloading case. This one is less pretty because with_scope is awkward to work with. I suppose I could have split it out ...

This may be even more serious than I thought. I haven't tried it but it appears that my with_scope example above would have worked in Rails 2.2. The change in b...

I gather that adding a &block parameter to your method signature can impact performance. In the case of with_scope, the parameter actually isn't used so it shou...

Sorry. Stupid Markdown...

I hit a very similar issue, which I reported in ticket #2329. As well as having .* in :select, I also had a calculated column, hence t...

I hit a very similar issue, which I reported in ticket #2329. As well as having . in :select, I also had a calculated column, hence there was a comma in there. ...

I've thought about it some more and an easy workaround is to duplicate the default_scope in the association definition like so.

class Tag < ActiveRecord::Bas...

Actually the problem goes deeper. This doesn't work either.

Tagging.send(:with_scope, :find => { :conditions => { :enabled => true } }) do
  Tag.first(:incl...