[Catalyst] Re: Warnings when upgrading Catalyst

[George Nistorica]

Tomas Doran wrote:

[...]

>>> 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 :)

Running $ perl script/myapp_test.pl /rest/test
in several scenarios:

* Catalyst 5.80007 from CPAN, Catalyst::Controller::REST 0.73 from CPAN

.------------------------------------------------------------+-----------.
| Action                                                     | Time      |
+------------------------------------------------------------+-----------+
| /rest/begin                                                | 0.000156s |
| /rest/test                                                 | 0.000118s |
| -> /test/private_forwarded_action                          | 0.000239s |
| /rest/end                                                  | 0.002551s |
'------------------------------------------------------------+-----------'

* Catalyst 5.80007, with warning supressed, Catalyst::Controller::REST
0.73 from CPAN

.------------------------------------------------------------+-----------.
| Action                                                     | Time      |
+------------------------------------------------------------+-----------+
| /rest/begin                                                | 0.000176s |
| /rest/test                                                 | 0.000131s |
| -> /test/private_forwarded_action                          | 0.000236s |
| /rest/end                                                  | 0.002756s |
'------------------------------------------------------------+-----------'

* Catalyst 5.80007 from CPAN, Catalyst::Controller::REST 0.73 you provided

.------------------------------------------------------------+-----------.
| Action                                                     | Time      |
+------------------------------------------------------------+-----------+
| /rest/begin                                                | 0.000165s |
| /rest/test                                                 | 0.000140s |
| /rest/test_GET                                             | 0.001048s |
|  -> /test/private_forwarded_action                         | 0.000259s |
| /rest/end                                                  | 0.004020s |
'------------------------------------------------------------+-----------'

* Catalyst 5.80007, with warning supressed, Catalyst::Controller::REST
0.73 you provided

.------------------------------------------------------------+-----------.
| Action                                                     | Time      |
+------------------------------------------------------------+-----------+
| /rest/begin                                                | 0.000202s |
| /rest/test                                                 | 0.000128s |
| /rest/test_GET                                             | 0.000937s |
|  -> /test/private_forwarded_action                         | 0.000260s |
| /rest/end                                                  | 0.003944s |
'------------------------------------------------------------+-----------'

It looks like I was mistaken when talking about missing actions in the
stats output, in the sense that supressing the Catalyst warning didn't
automagically make more actions to be displayed in the stats.

However I see that using the REST controller you provided, test_GET pops up.



> 
> 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/


-- 
#!/usr/bin/perl -w ###################
use strict;###########################

( $_ = qq{0912 3c1217 709073043p2it//,
'"70kc1H 270P 70htonA t3uJ" tni7p'8;})
=~    tr/42179038/(larves)/;eval#uate;

######################################