I'll work on a patch and docs, yes.

On Sat, Aug 4, 2012 at 6:53 AM, Tomas Doran <bobtf...@bobtfish.net> wrote:

>
> 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
>
>
> _______________________________________________
> List: Catalyst@lists.scsys.co.uk
> Listinfo: http://lists.scsys.co.uk/cgi-bin/mailman/listinfo/catalyst
> Searchable archive:
> http://www.mail-archive.com/catalyst@lists.scsys.co.uk/
> Dev site: http://dev.catalyst.perl.org/
>



-- 
Bill Moseley
mose...@hank.org
_______________________________________________
List: Catalyst@lists.scsys.co.uk
Listinfo: http://lists.scsys.co.uk/cgi-bin/mailman/listinfo/catalyst
Searchable archive: http://www.mail-archive.com/catalyst@lists.scsys.co.uk/
Dev site: http://dev.catalyst.perl.org/

Reply via email to