[Catalyst-dev] Possible bug in Catalyst-Devel-1.05 [Patch]

Emanuele Zeppieri ema_zep at libero.it
Wed Apr 9 19:26:42 BST 2008


Jonathan Rockway wrote:

> * On Wed, Apr 09 2008, Emanuele Zeppieri wrote:
>> Good morning!
>>
>> Marcus,
>>
>> is the following (line 622) in Helper.pm correct?
>>
>> [% name %]->plugins(qw/-Debug ConfigLoader Static::Simple/);
>>
>> since there is no "plugins" method in Catalyst.pm <= 5.7012 ...
>>
>> If I understand correctly, according to the /new style/, it should
>> instead be:
>>
>> __PACKAGE__->setup(qw/-Debug ConfigLoader Static::Simple/);
>>
>> In case I'm right, the attached patch does just that.
>>
>> Cheers,
>> Emanuele
>> --- Helper.pm	2008-04-07 11:55:12.000000000 +0200
>> +++ Helper.pm.new	2008-04-09 07:17:21.014000000 +0200
>> @@ -619,8 +619,6 @@
>>  
>>  use parent qw/Catalyst/;
>>  
>> -[% name %]->plugins(qw/-Debug ConfigLoader Static::Simple/);
>> -
> 
> Where did this come from anyway?  "use parent" looks like a change for
> the Moose branch, but svn log doesn't reveal anything interesting.

I'm afraid you're asking the wrong person, I'm not a committer ;-)
I can just point you to this previous message from Marcus:
http://lists.scsys.co.uk/pipermail/catalyst-dev/2008-April/001104.html

Cheers,
Emanuele.



More information about the Catalyst-dev mailing list