MatthewRudy

I think this is because validates_acceptance_of has a default of :allow_nil => true,

      def validates_acceptance_of(*attr_names)
        configuration = {...

hey there nikz,

your diff did upload,

but looking at it,

seems it reverts a whole load of new changes.

like "merge conditions" becoming public.

I imagine ...

Am I missing something?

Doesn't this work already?

object.reload(options)

actually does

def reload(options = nil)
        clear_aggregation_cache
       ...

hmm...

still need to do this,

Tests that are missing are;

"doesnt work with Proc"

"doesnt work through named_scopes with Proc"

I wonder if these shortcomi...

proper testing, bit of documentation.

will do one more bit of testing later.

btw...

apologies if I seem harsh...

I just love "named_scope"

was playing with a similar idea a year ago,

made my day seeing it in edge.

forgot to point out as well...

Proc.new doesn't enforce the number of arguments,

ends up the same as a lambda, but lambda will raise if it doesnt get its req...

I agree that the patch allows the (2nd argument == :symbol)

but my feeling is that it isn't intuitive.

(also, shouldn't it accept symbols and strings)

if al...

can't think what the right name for the key to pass

:scope_name doesnt feel right,

as the "scope name" is the name we just gave it...

so maybe :scope_option...

i think my current test example doesn't quite explain it well.

the point is,

why create a scope, just so you can combine it with another one.

we want 1st co...