X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=lib%2FMoose%2FMeta%2FRole%2FApplication%2FToRole.pm;h=c1f91c7a2ba322dca14e36e0f2a69010cdff8b02;hb=baf46b9edc7dc3665c7eaf9d1684b157efb09e1a;hp=399bc72cf9b4fc944051699697ab7a105a0a28ef;hpb=1c9db35c77752fc918e23bee1613dc6567087dc5;p=gitmo%2FMoose.git diff --git a/lib/Moose/Meta/Role/Application/ToRole.pm b/lib/Moose/Meta/Role/Application/ToRole.pm index 399bc72..c1f91c7 100644 --- a/lib/Moose/Meta/Role/Application/ToRole.pm +++ b/lib/Moose/Meta/Role/Application/ToRole.pm @@ -4,29 +4,31 @@ use strict; use warnings; use metaclass; -use Carp 'confess'; use Scalar::Util 'blessed'; -use Data::Dumper; - -our $VERSION = '0.01'; +our $VERSION = '0.72_01'; +$VERSION = eval $VERSION; our $AUTHORITY = 'cpan:STEVAN'; use base 'Moose::Meta::Role::Application'; sub apply { - my ($self, $role1, $role2) = @_; - $self->SUPER::apply($role1, $role2); - $role2->add_role($role1); + my ($self, $role1, $role2) = @_; + $self->SUPER::apply($role1, $role2); + $role2->add_role($role1); } sub check_role_exclusions { my ($self, $role1, $role2) = @_; - confess "Conflict detected: " . $role2->name . " excludes role '" . $role1->name . "'" - if $role2->excludes_role($role1->name); + if ( $role2->excludes_role($role1->name) ) { + require Moose; + Moose->throw_error("Conflict detected: " . $role2->name . " excludes role '" . $role1->name . "'"); + } foreach my $excluded_role_name ($role1->get_excluded_roles_list) { - confess "The class " . $role2->name . " does the excluded role '$excluded_role_name'" - if $role2->does_role($excluded_role_name); + if ( $role2->does_role($excluded_role_name) ) { + require Moose; + Moose->throw_error("The class " . $role2->name . " does the excluded role '$excluded_role_name'"); + } $role2->add_excluded_roles($excluded_role_name); } } @@ -34,11 +36,18 @@ sub check_role_exclusions { sub check_required_methods { my ($self, $role1, $role2) = @_; foreach my $required_method_name ($role1->get_required_method_list) { + + next if $self->is_aliased_method($required_method_name); + $role2->add_required_methods($required_method_name) unless $role2->find_method_by_name($required_method_name); } } +sub check_required_attributes { + +} + sub apply_attributes { my ($self, $role1, $role2) = @_; foreach my $attribute_name ($role1->get_attribute_list) { @@ -46,8 +55,10 @@ sub apply_attributes { if ($role2->has_attribute($attribute_name) && # make sure we haven't seen this one already too $role2->get_attribute($attribute_name) != $role1->get_attribute($attribute_name)) { - confess "Role '" . $role1->name . "' has encountered an attribute conflict " . - "during composition. This is fatal error and cannot be disambiguated."; + + require Moose; + Moose->throw_error("Role '" . $role1->name . "' has encountered an attribute conflict " . + "during composition. This is fatal error and cannot be disambiguated."); } else { $role2->add_attribute( @@ -61,6 +72,33 @@ sub apply_attributes { sub apply_methods { my ($self, $role1, $role2) = @_; foreach my $method_name ($role1->get_method_list) { + + if ($self->is_method_aliased($method_name)) { + my $aliased_method_name = $self->get_method_aliases->{$method_name}; + # it if it has one already + if ($role2->has_method($aliased_method_name) && + # and if they are not the same thing ... + $role2->get_method($aliased_method_name)->body != $role1->get_method($method_name)->body) { + + require Moose; + Moose->throw_error("Cannot create a method alias if a local method of the same name exists"); + } + + $role2->add_method( + $aliased_method_name, + $role1->get_method($method_name) + ); + + if (!$role2->has_method($method_name)) { + $role2->add_required_methods($method_name) + unless $self->is_method_excluded($method_name); + } + + next; + } + + next if $self->is_method_excluded($method_name); + # it if it has one already if ($role2->has_method($method_name) && # and if they are not the same thing ... @@ -70,12 +108,14 @@ sub apply_methods { $role2->add_required_methods($method_name); } else { - # add it, although it could be overriden - $role2->alias_method( + # add it, although it could be overridden + $role2->add_method( $method_name, $role1->get_method($method_name) ); + } + } } @@ -86,11 +126,12 @@ sub apply_override_method_modifiers { if ($role2->has_method($method_name)) { # if it is being composed into another role # we have a conflict here, because you cannot - # combine an overriden method with a locally + # combine an overridden method with a locally # defined one - confess "Role '" . $role1->name . "' has encountered an 'override' method conflict " . + require Moose; + Moose->throw_error("Role '" . $role1->name . "' has encountered an 'override' method conflict " . "during composition (A local method of the same name as been found). This " . - "is fatal error."; + "is fatal error."); } else { # if we are a role, we need to make sure @@ -98,9 +139,11 @@ sub apply_override_method_modifiers { # we are composing into if ($role2->has_override_method_modifier($method_name) && $role2->get_override_method_modifier($method_name) != $role2->get_override_method_modifier($method_name)) { - confess "Role '" . $role1->name . "' has encountered an 'override' method conflict " . + + require Moose; + Moose->throw_error("Role '" . $role1->name . "' has encountered an 'override' method conflict " . "during composition (Two 'override' methods of the same name encountered). " . - "This is fatal error."; + "This is fatal error."); } else { # if there is no conflict, @@ -126,10 +169,6 @@ sub apply_method_modifiers { } } -sub apply_before_method_modifiers { (shift)->apply_method_modifiers('before' => @_) } -sub apply_around_method_modifiers { (shift)->apply_method_modifiers('around' => @_) } -sub apply_after_method_modifiers { (shift)->apply_method_modifiers('after' => @_) } - 1; @@ -139,7 +178,7 @@ __END__ =head1 NAME -Moose::Meta::Role::Application::ToRole +Moose::Meta::Role::Application::ToRole - Compose a role into another role =head1 DESCRIPTION @@ -153,9 +192,11 @@ Moose::Meta::Role::Application::ToRole =item B +=item B + =item B -=item B +=item B =item B @@ -163,12 +204,6 @@ Moose::Meta::Role::Application::ToRole =item B -=item B - -=item B - -=item B - =item B =back @@ -185,7 +220,7 @@ Stevan Little Estevan@iinteractive.comE =head1 COPYRIGHT AND LICENSE -Copyright 2006, 2007 by Infinity Interactive, Inc. +Copyright 2006-2009 by Infinity Interactive, Inc. L