On Tue, Oct 19, 2010 at 08:57:02AM -0700, Tim Morgan wrote: > Sorry I forgot to link to the ticket: > https://rails.lighthouseapp.com/projects/8994/tickets/1812-default_scope-cant-take-procs > > And the patch is here: > https://rails.lighthouseapp.com/projects/8994/tickets/1812/a/721344/allow-default_scope-to-accept-a-proc.patch > > Or you can pull from my Rails fork here: http://github.com/seven1m/rails
Two things: 1. Can you change it so that it uses "&&" rather than "and" 2. Change from m.is_a?(Proc) to m.respond_to?(:call) #1 is for style. #2 is because we don't care that this is a Proc object, we only care that it responds to "call". -- Aaron Patterson http://tenderlovemaking.com/
pgpDywb73h7qs.pgp
Description: PGP signature
