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