Re: Overriding additional attribute options with has +$foo

[email protected] (Stevan Little)
Newsgroups perl.moose
Message-ID <[email protected]>
Tom,

Patch looks good, I will apply it this weekend likely.

As for the traits support, it should be as simple as adding 'traits'  
to the list same as you did with 'metaclass'. If you want to add that  
in as well, that would be great. Also feel free to stop by IRC and  
ask mst for a commit bit, then you can bypass the whole "sending  
patch to the list" thing.

- Stevan

On Apr 16, 2008, at 3:21 PM, Tomas Doran wrote:
> Hiya
>
> First of all, just to say I'm pretty new to Moose (started hacking  
> on Saturday), so excuse me if I'm either missing totally obvious  
> things, or messing up the lingo..
>
> One of the things that I've found is that I want to be able to  
> override additional options to attributes when composing roles  
> (making Class::Workflow and MooseX::Storage play together nicely),  
> and I found that you can't change either the 'metaclass' or the  
> 'traits' with has +$foo..
>
> After a quick discussion on #moose the other day Stevan agreed that  
> both of these were sane and nothingmuch volunteered to write the  
> patch if I submitted some tests.
>
> Please find a diff attached with tests, docs and the metaclass  
> option fixed - I haven't touched the attribute traits yet (other  
> than the tests), as I'm yet to wrap my head around them properly...
>
> Cheers
> Tom
>
> <moose-+has-extra-options.diff>
>
>
>
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.