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

Bill Moseley moseley at hank.org
Tue Aug 7 19:52:35 GMT 2012


Did this help?

On Sat, Aug 4, 2012 at 6:19 PM, Bill Moseley <moseley at hank.org> wrote:

> See attached.
>
> Let me know if you want something else.
>
> I'm mixed on of the clear should be private.  I made it that way, but I
> wonder if there could be a use case to reset the parameters so next time
> $c->req->parameters is called they get built again.   I guess in that case
> why wouldn't they just call $c->prepare_parameters again?
>
> On Sat, Aug 4, 2012 at 6:53 AM, Tomas Doran <bobtfish at 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 at 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
> moseley at hank.org
>



-- =

Bill Moseley
moseley at hank.org
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.scsys.co.uk/pipermail/catalyst/attachments/20120807/7ac05=
6c4/attachment.htm


More information about the Catalyst mailing list