Re: [Catalyst] [Announce] Catalyst-Runtime 5.8000_04 shipped to CPAN

2008-12-27 Thread Tomas Doran


On 5 Dec 2008, at 23:00, Guillermo Roditi wrote:
I think the trunk version fixes all your problems, but nobody has  
been able to accurately describe their problems or produce a simple  
failing test case.



I've found some plugins which call mk_accessors more than once for  
the same attribute / package, which fails.


This is because, on the second application, the attribute is added  
with no options, which results in the accessor methods for it being  
removed. Please find test attached.


Cheers
t0m



double_apply.t
Description: Binary data
___
List: Catalyst@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/


Re: [Catalyst] [Announce] Catalyst-Runtime 5.8000_04 shipped to CPAN

2008-12-27 Thread Tomas Doran


On 5 Dec 2008, at 23:00, Guillermo Roditi wrote:

All of you who have experienced errors rooted in MooseX::Emulate/ 
Adopt::Class::Accessor::Fast please please please send me error  
messages and show me the code that breaks.


Catalyst::Plugin::HashedCookies calls Catalyst::Request- 
>make_accessor, which in some ways is quite broken (as you're re- 
opening the package from outside), but it isn't actually very hard to  
fix/support..


Tests and patch attached.

Cheers
t0m



reopen_packages.diff
Description: Binary data
___
List: Catalyst@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/


Re: [Catalyst] [Announce] Catalyst-Runtime 5.8000_04 shipped to CPAN

2008-12-27 Thread Tomas Doran

On 5 Dec 2008, at 23:00, Guillermo Roditi wrote:
I think the trunk version fixes all your problems, but nobody has  
been able to accurately describe their problems or produce a simple  
failing test case.


HTML::Widget uses Class::Accessor::Chained::Fast, which overrides  
make_accessor. As mk_accessors no longer uses make_accessor to  
generate the coderef, this breaks horribly...


Test case attached.

Cheers
t0m



class_accessor_chained_fast.t
Description: Binary data





___
List: Catalyst@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/