[Catalyst] Re: Warnings when upgrading Catalyst

[Tomas Doran]

George Nistorica wrote:
> I've checked it out and indeed using the version of Controller::REST you
> provided fixes the warning.
> 
> Not only that, but it seems it doesn't break the functionality for both
> the test application I provided and another more complex one I'm using.

Great, thanks for testing.

I'll see if I can make the code any less gross and/or come up with a 
neater way of doing it, then get it integrated and shipped.

>> I guess I should also go fix the warning in Catalyst, and get the stats
>> to display properly without this patch - but that is at best papering
>> over the cracks :)
> 
> :)
> 
>> From your original mail, you said:
>>> that worked for me, and now the debug output shows the profiling for the
>>> methods I ->forward() to as well:
>>>
>>> the methods from another Controller to which I forward to, where missing
>>> from the profiling output, getting that warning instead.
>> I can't see any methods missing from the stats table with / without this
>> patch when using your supplied testapp however.
> 
> Indeed, the test application provided didn't show the behaviour I was
> talking about. my bad :)

heh, no worries.

>> Can you retry with the latest version of Catalyst so see if you can
>> replicate this behavior once again, before we kill the warning and
>> forget about this?
> 
> this replicates with Catalyst: 5.80005 Catalyst 5.80007
> without the REST patch
> the REST patch fixes the warning with both versions of Catalyst
> 
> Use of uninitialized value in concatenation (.) or string at
> /usr/lib/perl5/site_perl/5.10.0/Catalyst.pm line 1573.

Nono - I was _specifically_ talking about the lines missing from the 
stats table behavior.

I'm still concerned about working out the root cause of that, before I 
do something to 'just shut the warning up', as it were :)

Cheers
t0m