Sure! Here is the updated patch: https://rails.lighthouseapp.com/projects/8994/tickets/1812/a/727956/allow-default_scope-to-accept-a-proc2.patch
Also, the updated commit on GitHub: http://github.com/seven1m/rails/commit/e0b73b64c30e81e5b4279ce55dd2b7d65cfae00a <http://github.com/seven1m/rails/commit/e0b73b64c30e81e5b4279ce55dd2b7d65cfae00a> -Tim On Tue, Oct 19, 2010 at 3:35 PM, Aaron Patterson <[email protected] > wrote: > 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/ > -- You received this message because you are subscribed to the Google Groups "Ruby on Rails: Core" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/rubyonrails-core?hl=en.
