[Catalyst] change in Engine prepare_body_parameters
bobtfish at bobtfish.net
Mon Apr 23 08:55:47 GMT 2012
On 21 Apr 2012, at 13:30, Bill Moseley wrote:
> The code above looks like it was trying to fix some flow problems (populating the attribute directly as a hash, for example), so the real fix is probably something else. Perhaps a builder on body_parameters so that it isn't populated every call.
> This look like a bug?
Yes, it does.
Your proposed solution sounds entirely reasonable also.
> Or should I not be adding method modifiers to prepare_body_parameters?
> BTW -- the reason I used "after prepare_body_parameters" is so that $c->req->parameters ends up with any changes I make to the body_parameters.
Yeah, I think it's perfectly reasonable usage, and we should fix it to only be called once - even if only for efficiencies sake.
Could we start with some failing tests for your use-case?
Thanks in advance!
More information about the Catalyst