[Catalyst] Change in $c->engine->prepare_parameters

Tomas Doran bobtfish at bobtfish.net
Sat Aug 4 13:53:34 GMT 2012


On 3 Aug 2012, at 14:39, Bill Moseley wrote:

> What it doesn't do any more is set $c->req->parameters when called directly, only when called as a builder.
> 
> 
> I don't think prepare_parameters should be both a builder AND a method.   What about this approach:

+1, except I think the clearer should be _clear_parameters - we don't want to add a new public method for this if it can be avoided.

Fancy making a 'proper' patch someone can apply which does these changes and updates the relevant doc?

Cheers
t0m




More information about the Catalyst mailing list