X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=lib%2FMoose%2FMeta%2FRole%2FApplication%2FToClass.pm;h=f6a86efd52f35554af7791b97c760c1fd038635e;hb=db236a63c00371129c2254251aaa1408f044cd89;hp=e1b07f60af2d42a4351120f4b2c9ec2b6ad5ba7b;hpb=d4048ef33f6cad8a3453766505ee0c67690796f6;p=gitmo%2FMoose.git diff --git a/lib/Moose/Meta/Role/Application/ToClass.pm b/lib/Moose/Meta/Role/Application/ToClass.pm index e1b07f6..f6a86ef 100644 --- a/lib/Moose/Meta/Role/Application/ToClass.pm +++ b/lib/Moose/Meta/Role/Application/ToClass.pm @@ -7,7 +7,7 @@ use metaclass; use Moose::Util 'english_list'; use Scalar::Util 'weaken', 'blessed'; -our $VERSION = '0.93'; +our $VERSION = '1.10'; $VERSION = eval $VERSION; our $AUTHORITY = 'cpan:STEVAN'; @@ -147,40 +147,43 @@ sub apply_attributes { } sub apply_methods { - my ($self, $role, $class) = @_; - foreach my $method_name ($role->get_method_list) { + my ( $self, $role, $class ) = @_; + + foreach my $method ( $role->_get_local_methods ) { + my $method_name = $method->name; + next if $method_name eq 'meta'; - unless ($self->is_method_excluded($method_name)) { - # it if it has one already - if ($class->has_method($method_name) && - # and if they are not the same thing ... - $class->get_method($method_name)->body != $role->get_method($method_name)->body) { - next; - } - else { - # add it, although it could be overridden - $class->add_method( - $method_name, - $role->get_method($method_name) - ); - } - } + unless ( $self->is_method_excluded($method_name) ) { + + my $class_method = $class->get_method($method_name); + + next if $class_method && $class_method->body != $method->body; - if ($self->is_method_aliased($method_name)) { - my $aliased_method_name = $self->get_method_aliases->{$method_name}; - # it if it has one already - if ($class->has_method($aliased_method_name) && - # and if they are not the same thing ... - $class->get_method($aliased_method_name)->body != $role->get_method($method_name)->body) { - $class->throw_error("Cannot create a method alias if a local method of the same name exists"); - } $class->add_method( - $aliased_method_name, - $role->get_method($method_name) + $method_name, + $method, + ); + } + + next unless $self->is_method_aliased($method_name); + + my $aliased_method_name = $self->get_method_aliases->{$method_name}; + + my $class_method = $class->get_method($aliased_method_name); + + if ( $class_method && $class_method->body != $method->body ) { + $class->throw_error( + "Cannot create a method alias if a local method of the same name exists" ); } + + $class->add_method( + $aliased_method_name, + $method, + ); } + # we must reset the cache here since # we are just aliasing methods, otherwise # the modifiers go wonky.