X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=lib%2FRole%2FTiny.pm;h=b076c8e5398c3a8d8c5d645e70c5d7d280eab062;hb=f656cac92a3274607aa97844ad90522110395e30;hp=66209643aa406daffe2b27684f7fb175333e9bdf;hpb=fbe6dc87c11b77b9e1fe31666b00168d3a9182dc;p=gitmo%2FRole-Tiny.git diff --git a/lib/Role/Tiny.pm b/lib/Role/Tiny.pm index 6620964..b076c8e 100644 --- a/lib/Role/Tiny.pm +++ b/lib/Role/Tiny.pm @@ -6,7 +6,7 @@ sub _getstash { \%{"$_[0]::"} } use strict; use warnings FATAL => 'all'; -our $VERSION = '1.001005'; # 1.1.5 +our $VERSION = '1.002005'; # 1.2.5 $VERSION = eval $VERSION; our %INFO; @@ -45,7 +45,7 @@ sub import { warnings->import(FATAL => 'all'); return if $INFO{$target}; # already exported into this package # get symbol table reference - my $stash = do { no strict 'refs'; \%{"${target}::"} }; + my $stash = _getstash($target); # install before/after/around subs foreach my $type (qw(before after around)) { *{_getglob "${target}::${type}"} = sub { @@ -86,11 +86,6 @@ sub apply_single_role_to_package { $me->_install_modifiers($to, $info->{modifiers}); - # only add does() method to classes and only if they don't have one - if (not $INFO{$to} and not $to->can('does')) { - *{_getglob "${to}::does"} = \&does_role; - } - # copy our role list into the target's @{$APPLIED_TO{$to}||={}}{keys %{$APPLIED_TO{$role}}} = (); } @@ -108,6 +103,7 @@ sub create_class_with_roles { die "No roles supplied!" unless @roles; + _load_module($superclass); { my %seen; $seen{$_}++ for @roles; @@ -156,8 +152,6 @@ sub create_class_with_roles { do { my %h; @h{map @{$_->{requires}||[]}, @info} = (); keys %h } ); - *{_getglob "${new_name}::does"} = \&does_role unless $new_name->can('does'); - @{$APPLIED_TO{$new_name}||={}}{ map keys %{$APPLIED_TO{$_}}, @roles } = (); @@ -196,14 +190,18 @@ sub apply_roles_to_package { delete $INFO{$to}{methods}; # reset since we're about to add methods } - $me->apply_role_to_package($to, $_) for @roles; + foreach my $role (@roles) { + $me->apply_single_role_to_package($to, $role); + } $APPLIED_TO{$to}{join('|',@roles)} = 1; } sub _composite_info_for { my ($me, @roles) = @_; $COMPOSITE_INFO{join('|', sort @roles)} ||= do { - _load_module($_) for @roles; + foreach my $role (@roles) { + _load_module($role); + } my %methods; foreach my $role (@roles) { my $this_methods = $me->_concrete_methods_of($role); @@ -220,7 +218,11 @@ sub _composable_package_for { return $composed_name if $COMPOSED{role}{$composed_name}; $me->_install_methods($composed_name, $role); my $base_name = $composed_name.'::_BASE'; - *{_getglob("${composed_name}::ISA")} = [ $base_name ]; + # Not using _getglob, since setting @ISA via the typeglob breaks + # inheritance on 5.10.0 if the stash has previously been accessed an + # then a method called on the class (in that order!), which + # ->_install_methods (with the help of ->_install_does) ends up doing. + { no strict 'refs'; @{"${composed_name}::ISA"} = ( $base_name ); } my $modifiers = $INFO{$role}{modifiers}||[]; my @mod_base; foreach my $modified ( @@ -256,7 +258,7 @@ sub _concrete_methods_of { my ($me, $role) = @_; my $info = $INFO{$role}; # grab role symbol table - my $stash = do { no strict 'refs'; \%{"${role}::"}}; + my $stash = _getstash($role); # reverse so our keys become the values (captured coderefs) in case # they got copied or re-used since my $not_methods = { reverse %{$info->{not_methods}||{}} }; @@ -265,7 +267,7 @@ sub _concrete_methods_of { map { my $code = *{$stash->{$_}}{CODE}; # rely on the '' key we added in import for "no code here" - exists $not_methods->{$code||''} ? () : ($_ => $code) + ( ! $code or exists $not_methods->{$code||''} ) ? () : ($_ => $code) } grep !ref($stash->{$_}), keys %$stash }; } @@ -284,7 +286,7 @@ sub _install_methods { my $methods = $me->_concrete_methods_of($role); # grab target symbol table - my $stash = do { no strict 'refs'; \%{"${to}::"}}; + my $stash = _getstash($to); # determine already extant methods of target my %has_methods; @@ -297,6 +299,8 @@ sub _install_methods { no warnings 'once'; *{_getglob "${to}::${i}"} = $methods->{$i}; } + + $me->_install_does($to); } sub _install_modifiers { @@ -310,11 +314,41 @@ sub _install_modifiers { } } +my $vcheck_error; + sub _install_single_modifier { my ($me, @args) = @_; + defined($vcheck_error) or $vcheck_error = do { + local $@; + eval { Class::Method::Modifiers->VERSION(1.05); 1 } + ? 0 + : $@ + }; + $vcheck_error and die $vcheck_error; Class::Method::Modifiers::install_modifier(@args); } +my $FALLBACK = sub { 0 }; +sub _install_does { + my ($me, $to) = @_; + + # only add does() method to classes + return if $INFO{$to}; + + # add does() only if they don't have one + *{_getglob "${to}::does"} = \&does_role unless $to->can('does'); + + return if ($to->can('DOES') and $to->can('DOES') != (UNIVERSAL->can('DOES') || 0)); + + my $existing = $to->can('DOES') || $to->can('isa') || $FALLBACK; + my $new_sub = sub { + my ($proto, $role) = @_; + Role::Tiny::does_role($proto, $role) or $proto->$existing($role); + }; + no warnings 'redefine'; + *{_getglob "${to}::DOES"} = $new_sub; +} + sub does_role { my ($proto, $role) = @_; if ($] >= 5.010) { @@ -330,6 +364,8 @@ sub does_role { 1; +=encoding utf-8 + =head1 NAME Role::Tiny - Roles. Like a nouvelle cuisine portion size slice of Moose. @@ -473,7 +509,11 @@ composed into unless that class already has an ->does method, so ... } -will work for classes but to test a role, one must use ::does_role directly +will work for classes but to test a role, one must use ::does_role directly. + +Additionally, Role::Tiny will override the standard Perl C method +for your class. However, if C class in your class' inheritance +heirarchy provides C, then Role::Tiny will not override it. =head1 METHODS @@ -539,6 +579,8 @@ Mithaldu - Christian Walde (cpan:MITHALDU) ilmari - Dagfinn Ilmari Mannsåker (cpan:ILMARI) +tobyink - Toby Inkster (cpan:TOBYINK) + =head1 COPYRIGHT Copyright (c) 2010-2012 the Role::Tiny L and L