[Catalyst] html-widget/accessor - by design?

Carl Franks fireartist at gmail.com
Mon Mar 13 12:57:35 CET 2006


On 10/03/06, Matt S Trout <dbix-class at trout.me.uk> wrote:
> Sebastian Riedel wrote:
>
> > We use chained accessors for everything, it's one of the design
> > fundamentals of H::W.
>
> Right, but the chaining should be on set, not on get, so I think this
> behaviour is still erroneous.

I just tried changing
return ( $_[0]->{attributes}->{$name} || $_[0] ) unless @_ > 1;
to this
return ( $_[0]->{attributes}->{$name} ) unless @_ > 1;

and all tests pass.
Is it okay to commit?

Carl



More information about the Catalyst mailing list