X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=lib%2FMoose%2FMeta%2FRole%2FApplication%2FToClass.pm;h=48e4658e1406eeeb055d341c820b1a721bee0a68;hb=0f8380b0cdbda1e13ed7c456edd3f0d1c0315ec9;hp=5d05e60780acf9a3fed3b29fe14a43b75053874c;hpb=154a4aae762133d9f18cd1df5d40743a1878d78e;p=gitmo%2FMoose.git diff --git a/lib/Moose/Meta/Role/Application/ToClass.pm b/lib/Moose/Meta/Role/Application/ToClass.pm index 5d05e60..48e4658 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.77'; +our $VERSION = '0.86'; $VERSION = eval $VERSION; our $AUTHORITY = 'cpan:STEVAN'; @@ -58,13 +58,14 @@ sub check_required_methods { # attribute accessors. However I am thinking # that maybe those are somehow exempt from # the require methods stuff. - foreach my $required_method_name ($role->get_required_method_list) { + foreach my $required_method ($role->get_required_method_list) { + my $required_method_name = $required_method->name; if (!$class->find_method_by_name($required_method_name)) { next if $self->is_aliased_method($required_method_name); - push @missing, $required_method_name; + push @missing, $required_method; } } @@ -72,7 +73,22 @@ sub check_required_methods { my $error = ''; - if (@missing) { + my @conflicts = grep { $_->isa('Moose::Meta::Role::Method::Conflicting') } @missing; + + if (@conflicts) { + my $conflict = $conflicts[0]; + my $roles = Moose::Util::english_list( map { q{'} . $_ . q{'} } @{ $conflict->roles } ); + + $error + .= "Due to a method name conflict in roles " + . $roles + . ", the method '" + . $conflict->name + . "' must be implemented or excluded by '" + . $class->name + . q{'}; + } + elsif (@missing) { my $noun = @missing == 1 ? 'method' : 'methods'; my $list