use warnings;
use metaclass;
-use Scalar::Util 'blessed';
+use Scalar::Util 'blessed';
use Moose::Meta::Role::Composite;
-our $VERSION = '0.73_01';
+our $VERSION = '0.89';
$VERSION = eval $VERSION;
our $AUTHORITY = 'cpan:STEVAN';
sub get_exclusions_for_role {
my ($self, $role) = @_;
$role = $role->name if blessed $role;
- if ($self->role_params->{$role} && defined $self->role_params->{$role}->{excludes}) {
- if (ref $self->role_params->{$role}->{excludes} eq 'ARRAY') {
- return $self->role_params->{$role}->{excludes};
+ my $excludes_key = exists $self->role_params->{$role}->{'-excludes'} ?
+ '-excludes' : 'excludes';
+ if ($self->role_params->{$role} && defined $self->role_params->{$role}->{$excludes_key}) {
+ if (ref $self->role_params->{$role}->{$excludes_key} eq 'ARRAY') {
+ return $self->role_params->{$role}->{$excludes_key};
}
- return [ $self->role_params->{$role}->{excludes} ];
+ return [ $self->role_params->{$role}->{$excludes_key} ];
}
return [];
}
sub get_method_aliases_for_role {
my ($self, $role) = @_;
$role = $role->name if blessed $role;
- if ($self->role_params->{$role} && defined $self->role_params->{$role}->{alias}) {
- return $self->role_params->{$role}->{alias};
+ my $alias_key = exists $self->role_params->{$role}->{'-alias'} ?
+ '-alias' : 'alias';
+ if ($self->role_params->{$role} && defined $self->role_params->{$role}->{$alias_key}) {
+ return $self->role_params->{$role}->{$alias_key};
}
- return {};
+ return {};
}
sub is_method_excluded {
sub is_aliased_method {
my ($self, $role, $method_name) = @_;
- my %aliased_names = reverse %{$self->get_method_aliases_for_role($role->name)};
+ my %aliased_names = reverse %{$self->get_method_aliases_for_role($role->name)};
exists $aliased_names{$method_name} ? 1 : 0;
}
-# stolen from List::MoreUtils ...
-my $uniq = sub { my %h; map { $h{$_}++ == 0 ? $_ : () } @_ };
-
sub check_role_exclusions {
my ($self, $c) = @_;
sub check_required_methods {
my ($self, $c) = @_;
- my %all_required_methods = map { $_ => undef } $uniq->(map {
- $_->get_required_method_list
- } @{$c->get_roles});
+ my %all_required_methods =
+ map { $_->name => $_ }
+ map { $_->get_required_method_list }
+ @{$c->get_roles};
foreach my $role (@{$c->get_roles}) {
foreach my $required (keys %all_required_methods) {
-
+
delete $all_required_methods{$required}
if $role->has_method($required)
|| $self->is_aliased_method($role, $required);
}
}
- $c->add_required_methods(keys %all_required_methods);
+ $c->add_required_methods(values %all_required_methods);
}
sub check_required_attributes {
-
+
}
sub apply_attributes {
my ($self, $c) = @_;
-
+
my @all_attributes = map {
my $role = $_;
- map {
- +{
+ map {
+ +{
name => $_,
attr => $role->get_attribute($_),
}
} $role->get_attribute_list
} @{$c->get_roles};
-
+
my %seen;
foreach my $attr (@all_attributes) {
if (exists $seen{$attr->{name}}) {
if ( $seen{$attr->{name}} != $attr->{attr} ) {
require Moose;
- Moose->throw_error("We have encountered an attribute conflict with '" . $attr->{name} . "' "
+ Moose->throw_error("We have encountered an attribute conflict with '" . $attr->{name} . "' "
. "during composition. This is fatal error and cannot be disambiguated.")
}
}
$seen{$attr->{name}} = $attr->{attr};
}
- foreach my $attr (@all_attributes) {
+ foreach my $attr (@all_attributes) {
$c->add_attribute($attr->{name}, $attr->{attr});
}
}
sub apply_methods {
my ($self, $c) = @_;
-
+
my @all_methods = map {
my $role = $_;
my $aliases = $self->get_method_aliases_for_role($role);
my %excludes = map { $_ => undef } @{ $self->get_exclusions_for_role($role) };
(
- (map {
+ (map {
exists $excludes{$_} ? () :
- +{
+ +{
role => $role,
name => $_,
method => $role->get_method($_),
}
} $role->get_method_list),
- (map {
- +{
+ (map {
+ +{
role => $role,
name => $aliases->{$_},
method => $role->get_method($_),
- }
+ }
} keys %$aliases)
);
} @{$c->get_roles};
-
+
my (%seen, %method_map);
foreach my $method (@all_methods) {
- if (exists $seen{$method->{name}}) {
- if ($seen{$method->{name}}->body != $method->{method}->body) {
- $c->add_required_methods($method->{name});
+ my $seen = $seen{$method->{name}};
+
+ if ($seen) {
+ if ($seen->{method}->body != $method->{method}->body) {
+ $c->add_conflicting_method(
+ name => $method->{name},
+ roles => [$method->{role}->name, $seen->{role}->name],
+ );
+
delete $method_map{$method->{name}};
next;
- }
- }
-
- $seen{$method->{name}} = $method->{method};
+ }
+ }
+
+ $seen{$method->{name}} = $method;
$method_map{$method->{name}} = $method->{method};
}
sub apply_override_method_modifiers {
my ($self, $c) = @_;
-
+
my @all_overrides = map {
my $role = $_;
- map {
- +{
+ map {
+ +{
name => $_,
method => $role->get_override_method_modifier($_),
}
} $role->get_method_modifier_list('override');
} @{$c->get_roles};
-
+
my %seen;
foreach my $override (@all_overrides) {
if ( $c->has_method($override->{name}) ){
}
$seen{$override->{name}} = $override->{method};
}
-
+
$c->add_override_method_modifier(
$_->{name}, $_->{method}
) for @all_overrides;
-
+
}
sub apply_method_modifiers {
=head1 DESCRIPTION
-Summation composes two traits, forming the union of non-conflicting
+Summation composes two traits, forming the union of non-conflicting
bindings and 'disabling' the conflicting bindings
=head2 METHODS