X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=lib%2FMoose%2FRole.pm;h=67347a33be2e69a564f03fcaa1d2be5708654a37;hb=759e4e8ff4171bfabf1cb3d3843f760425676fe0;hp=19c66b0e4bbe608347b31cd6222f02d329285c71;hpb=d44714be2bf834a2df5e42da05fb7a760145adf8;p=gitmo%2FMoose.git diff --git a/lib/Moose/Role.pm b/lib/Moose/Role.pm index 19c66b0..67347a3 100644 --- a/lib/Moose/Role.pm +++ b/lib/Moose/Role.pm @@ -8,12 +8,14 @@ use Scalar::Util 'blessed'; use Carp 'confess'; use Sub::Name 'subname'; +use Data::OptList; use Sub::Exporter; -our $VERSION = '0.07'; +our $VERSION = '0.08'; our $AUTHORITY = 'cpan:STEVAN'; -use Moose (); +use Moose (); +use Moose::Util (); use Moose::Meta::Role; use Moose::Util::TypeConstraints; @@ -25,98 +27,89 @@ use Moose::Util::TypeConstraints; my $role = $CALLER; return $METAS{$role} if exists $METAS{$role}; - + # make a subtype for each Moose class subtype $role => as 'Role' - => where { $_->does($role) } - => optimize_as { blessed($_[0]) && $_[0]->can('does') && $_[0]->does($role) } - unless find_type_constraint($role); - - my $meta; - if ($role->can('meta')) { - $meta = $role->meta(); - (blessed($meta) && $meta->isa('Moose::Meta::Role')) + => where { Moose::Util::does_role($_, $role) } + => optimize_as { blessed($_[0]) && Moose::Util::does_role($_[0], $role) } + unless find_type_constraint($role); + + my $meta; + if ($role->can('meta')) { + $meta = $role->meta(); + (blessed($meta) && $meta->isa('Moose::Meta::Role')) || confess "You already have a &meta function, but it does not return a Moose::Meta::Role"; - } - else { - $meta = Moose::Meta::Role->initialize($role); - $meta->Moose::Meta::Class::add_method('meta' => sub { $meta }) - } + } + else { + $meta = Moose::Meta::Role->initialize($role); + $meta->alias_method('meta' => sub { $meta }); + } return $METAS{$role} = $meta; } - - - my %exports = ( + + + my %exports = ( extends => sub { my $meta = _find_meta(); - return subname 'Moose::Role::extends' => sub { + return subname 'Moose::Role::extends' => sub { confess "Moose::Role does not currently support 'extends'" - }; - }, - with => sub { - my $meta = _find_meta(); - return subname 'Moose::Role::with' => sub (@) { - my (@roles) = @_; - confess "Must specify at least one role" unless @roles; - Class::MOP::load_class($_) for @roles; - ($_->can('meta') && $_->meta->isa('Moose::Meta::Role')) - || confess "You can only consume roles, $_ is not a Moose role" - foreach @roles; - if (scalar @roles == 1) { - $roles[0]->meta->apply($meta); - } - else { - Moose::Meta::Role->combine( - map { $_->meta } @roles - )->apply($meta); - } }; - }, + }, + with => sub { + my $meta = _find_meta(); + return subname 'Moose::Role::with' => sub (@) { + Moose::Util::apply_all_roles($meta, @_) + }; + }, requires => sub { my $meta = _find_meta(); - return subname 'Moose::Role::requires' => sub (@) { + return subname 'Moose::Role::requires' => sub (@) { confess "Must specify at least one method" unless @_; $meta->add_required_methods(@_); - }; - }, + }; + }, excludes => sub { my $meta = _find_meta(); - return subname 'Moose::Role::excludes' => sub (@) { + return subname 'Moose::Role::excludes' => sub (@) { confess "Must specify at least one role" unless @_; $meta->add_excluded_roles(@_); - }; - }, + }; + }, has => sub { my $meta = _find_meta(); - return subname 'Moose::Role::has' => sub ($;%) { - my ($name, %options) = @_; - $meta->add_attribute($name, %options) - }; - }, + return subname 'Moose::Role::has' => sub ($;%) { + my ($name, %options) = @_; + $meta->add_attribute($name, %options) + }; + }, before => sub { my $meta = _find_meta(); - return subname 'Moose::Role::before' => sub (@&) { + return subname 'Moose::Role::before' => sub (@&) { my $code = pop @_; $meta->add_before_method_modifier($_, $code) for @_; - }; - }, + }; + }, after => sub { my $meta = _find_meta(); - return subname 'Moose::Role::after' => sub (@&) { - my $code = pop @_; - $meta->add_after_method_modifier($_, $code) for @_; - }; - }, + return subname 'Moose::Role::after' => sub (@&) { + my $code = pop @_; + $meta->add_after_method_modifier($_, $code) for @_; + }; + }, around => sub { my $meta = _find_meta(); - return subname 'Moose::Role::around' => sub (@&) { - my $code = pop @_; - $meta->add_around_method_modifier($_, $code) for @_; - }; - }, - super => sub { + return subname 'Moose::Role::around' => sub (@&) { + my $code = pop @_; + $meta->add_around_method_modifier($_, $code) for @_; + }; + }, + super => sub { + { + no strict 'refs'; + $Moose::SUPER_SLOT{$CALLER} = \*{"${CALLER}::super"}; + } my $meta = _find_meta(); return subname 'Moose::Role::super' => sub {}; }, @@ -125,40 +118,49 @@ use Moose::Util::TypeConstraints; return subname 'Moose::Role::override' => sub ($&) { my ($name, $code) = @_; $meta->add_override_method_modifier($name, $code); - }; - }, + }; + }, inner => sub { my $meta = _find_meta(); return subname 'Moose::Role::inner' => sub { confess "Moose::Role cannot support 'inner'"; - }; - }, + }; + }, augment => sub { my $meta = _find_meta(); return subname 'Moose::Role::augment' => sub { confess "Moose::Role cannot support 'augment'"; - }; - }, + }; + }, confess => sub { return \&Carp::confess; }, blessed => sub { return \&Scalar::Util::blessed; - } - ); + } + ); - my $exporter = Sub::Exporter::build_exporter({ + my $exporter = Sub::Exporter::build_exporter({ exports => \%exports, groups => { default => [':all'] } }); - + sub import { - $CALLER = caller(); - + $CALLER = + ref $_[1] && defined $_[1]->{into} ? $_[1]->{into} + : ref $_[1] + && defined $_[1]->{into_level} ? caller( $_[1]->{into_level} ) + : caller(); + + # this works because both pragmas set $^H (see perldoc perlvar) + # which affects the current compilation - i.e. the file who use'd + # us - which is why we don't need to do anything special to make + # it affect that file rather than this one (which is already compiled) + strict->import; - warnings->import; + warnings->import; # we should never export to main return if $CALLER eq 'main'; @@ -181,26 +183,22 @@ Moose::Role - The Moose Role =head1 SYNOPSIS package Eq; - use strict; - use warnings; - use Moose::Role; - + use Moose::Role; # automatically turns on strict and warnings + requires 'equal'; - - sub no_equal { + + sub no_equal { my ($self, $other) = @_; !$self->equal($other); } - + # ... then in your classes - + package Currency; - use strict; - use warnings; - use Moose; - + use Moose; # automatically turns on strict and warnings + with 'Eq'; - + sub equal { my ($self, $other) = @_; $self->as_float == $other->as_float; @@ -208,78 +206,78 @@ Moose::Role - The Moose Role =head1 DESCRIPTION -Role support in Moose is pretty solid at this point. However, the best -documentation is still the the test suite. It is fairly safe to assume -Perl 6 style behavior and then either refer to the test suite, or ask -questions on #moose if something doesn't quite do what you expect. +Role support in Moose is pretty solid at this point. However, the best +documentation is still the the test suite. It is fairly safe to assume Perl 6 +style behavior and then either refer to the test suite, or ask questions on +#moose if something doesn't quite do what you expect. -We are planning writing some more documentation in the near future, but -nothing is ready yet, sorry. +We are planning writing some more documentation in the near future, but nothing +is ready yet, sorry. =head1 EXPORTED FUNCTIONS -Currently Moose::Role supports all of the functions that L exports, -but differs slightly in how some items are handled (see L below -for details). +Moose::Role currently supports all of the functions that L exports, but +differs slightly in how some items are handled (see L below for +details). -Moose::Role also offers two role specific keyword exports: +Moose::Role also offers two role-specific keyword exports: =over 4 =item B -Roles can require that certain methods are implemented by any class which -C the role. +Roles can require that certain methods are implemented by any class which +C the role. =item B Roles can C other roles, in effect saying "I can never be combined -with these C<@role_names>". This is a feature which should not be used -lightly. +with these C<@role_names>". This is a feature which should not be used +lightly. =back =head1 CAVEATS -The role support now has only a few caveats. They are as follows: +Role support has only a few caveats: =over 4 =item * -Roles cannot use the C keyword, it will throw an exception for now. -The same is true of the C and C keywords (not sure those -really make sense for roles). All other Moose keywords will be I -so that they can be applied to the consuming class. +Roles cannot use the C keyword; it will throw an exception for now. +The same is true of the C and C keywords (not sure those +really make sense for roles). All other Moose keywords will be I +so that they can be applied to the consuming class. -=item * +=item * -Role composition does it's best to B be order sensitive when it comes -to conflict resolution and requirements detection. However, it is order -sensitive when it comes to method modifiers. All before/around/after modifiers -are included whenever a role is composed into a class, and then are applied -in the order the roles are used. This too means that there is no conflict for -before/around/after modifiers as well. +Role composition does its best to B be order-sensitive when it comes to +conflict resolution and requirements detection. However, it is order-sensitive +when it comes to method modifiers. All before/around/after modifiers are +included whenever a role is composed into a class, and then applied in the order +in which the roles are used. This also means that there is no conflict for +before/around/after modifiers. -In most cases, this will be a non issue, however it is something to keep in -mind when using method modifiers in a role. You should never assume any +In most cases, this will be a non-issue; however, it is something to keep in +mind when using method modifiers in a role. You should never assume any ordering. =item * -The C keyword currently only works with actual methods. A method -modifier (before/around/after and override) will not count as a fufillment +The C keyword currently only works with actual methods. A method +modifier (before/around/after and override) will not count as a fufillment of the requirement, and neither will an autogenerated accessor for an attribute. -It is likely that the attribute accessors will eventually be allowed to fufill -those requirements, either that or we will introduce a C keyword -of some kind instead. This descision has not yet been finalized. +It is likely that attribute accessors will eventually be allowed to fufill those +requirements, or we will introduce a C keyword of some kind +instead. This decision has not yet been finalized. =back =head1 BUGS -All complex software has bugs lurking in it, and this module is no +All complex software has bugs lurking in it, and this module is no exception. If you find a bug please either email me, or add the bug to cpan-RT. @@ -291,11 +289,11 @@ Christian Hansen Echansen@cpan.orgE =head1 COPYRIGHT AND LICENSE -Copyright 2006, 2007 by Infinity Interactive, Inc. +Copyright 2006-2008 by Infinity Interactive, Inc. L This library is free software; you can redistribute it and/or modify -it under the same terms as Perl itself. +it under the same terms as Perl itself. =cut