BEGIN { *INFO = \%Role::Tiny::INFO }
our %INFO;
+our %APPLY_DEFAULTS;
sub _install_tracked {
my ($target, $name, $code) = @_;
# get symbol table reference
my $stash = do { no strict 'refs'; \%{"${target}::"} };
_install_tracked $target => has => sub {
- my ($name_proto, %spec) = @_;
- my $name_isref = ref $name_proto eq 'ARRAY';
- foreach my $name ($name_isref ? @$name_proto : $name_proto) {
- my $spec_ref = $name_isref ? +{%spec} : \%spec;
+ my $name_proto = shift;
+ my @name_proto = ref $name_proto eq 'ARRAY' ? @$name_proto : $name_proto;
+ if (@_ % 2 != 0) {
+ require Carp;
+ Carp::croak("Invalid options for " . join(', ', map "'$_'", @name_proto)
+ . " attribute(s): even number of arguments expected, got " . scalar @_)
+ }
+ my %spec = @_;
+ foreach my $name (@name_proto) {
+ my $spec_ref = @name_proto > 1 ? +{%spec} : \%spec;
($INFO{$target}{accessor_maker} ||= do {
require Method::Generate::Accessor;
Method::Generate::Accessor->new
}
require Class::Method::Modifiers if @$mods;
$INFO{$role}{inhaled_from_moose} = 1;
+ $INFO{$role}{is_role} = 1;
}
}
sub _maybe_make_accessors {
- my ($self, $role, $target) = @_;
+ my ($self, $target, $role) = @_;
my $m;
- if ($INFO{$role}{inhaled_from_moose}
+ if ($INFO{$role} && $INFO{$role}{inhaled_from_moose}
or $INC{"Moo.pm"}
and $m = Moo->_accessor_maker_for($target)
and ref($m) ne 'Method::Generate::Accessor') {
- $self->_make_accessors($role, $target);
+ $self->_make_accessors($target, $role);
}
}
sub _make_accessors_if_moose {
- my ($self, $role, $target) = @_;
- if ($INFO{$role}{inhaled_from_moose}) {
- $self->_make_accessors($role, $target);
+ my ($self, $target, $role) = @_;
+ if ($INFO{$role} && $INFO{$role}{inhaled_from_moose}) {
+ $self->_make_accessors($target, $role);
}
}
sub _make_accessors {
- my ($self, $role, $target) = @_;
+ my ($self, $target, $role) = @_;
my $acc_gen = ($Moo::MAKERS{$target}{accessor} ||= do {
require Method::Generate::Accessor;
Method::Generate::Accessor->new
}
}
+sub role_application_steps {
+ qw(_handle_constructor _maybe_make_accessors),
+ $_[0]->SUPER::role_application_steps;
+}
+
sub apply_roles_to_package {
my ($me, $to, @roles) = @_;
foreach my $role (@roles) {
- $me->_inhale_if_moose($role);
+ $me->_inhale_if_moose($role);
+ die "${role} is not a Moo::Role" unless $INFO{$role};
}
$me->SUPER::apply_roles_to_package($to, @roles);
}
sub apply_single_role_to_package {
my ($me, $to, $role) = @_;
- die "${role} is not a Moo::Role" unless my $info = $INFO{$role};
$me->_inhale_if_moose($role);
- $me->_handle_constructor($to, $INFO{$role}{attributes});
- $me->_maybe_make_accessors($role, $to);
+ die "${role} is not a Moo::Role" unless $INFO{$role};
$me->SUPER::apply_single_role_to_package($to, $role);
}
$me->SUPER::create_class_with_roles($superclass, @roles);
foreach my $role (@roles) {
- die "${role} is not a Role::Tiny" unless my $info = $INFO{$role};
+ die "${role} is not a Role::Tiny" unless $INFO{$role};
}
- $Moo::MAKERS{$new_name} = {};
+ $Moo::MAKERS{$new_name} = {is_class => 1};
- $me->_handle_constructor(
- $new_name, [ map @{$INFO{$_}{attributes}||[]}, @roles ]
- );
+ $me->_handle_constructor($new_name, $_) for @roles;
return $new_name;
}
sub apply_roles_to_object {
my ($me, $object, @roles) = @_;
my $new = $me->SUPER::apply_roles_to_object($object, @roles);
- if ($INC{'Moo.pm'}
- and my $m = Moo->_accessor_maker_for(ref $new)
- and my $con_gen = Moo->_constructor_maker_for(ref $new)) {
- require Sub::Quote;
- my $specs = $con_gen->all_attribute_specs;
+ my $apply_defaults = $APPLY_DEFAULTS{ref $new} ||= do {
my %attrs = map { @{$INFO{$_}{attributes}||[]} } @roles;
- my $assign = '';
- my %captures;
- foreach my $name ( keys %attrs ) {
- my $spec = $specs->{$name};
- if ($m->has_eager_default($name, $spec)) {
- my ($has, $has_cap)
- = $m->generate_simple_has('$_[0]', $name, $spec);
- my ($code, $pop_cap)
- = $m->generate_use_default('$_[0]', $name, $spec, $has);
-
- $assign .= $code;
- @captures{keys %$has_cap, keys %$pop_cap}
- = (values %$has_cap, values %$pop_cap);
+ if ($INC{'Moo.pm'}
+ and keys %attrs
+ and my $con_gen = Moo->_constructor_maker_for(ref $new)
+ and my $m = Moo->_accessor_maker_for(ref $new)) {
+ require Sub::Quote;
+
+ my $specs = $con_gen->all_attribute_specs;
+
+ my $assign = '';
+ my %captures;
+ foreach my $name ( keys %attrs ) {
+ my $spec = $specs->{$name};
+ if ($m->has_eager_default($name, $spec)) {
+ my ($has, $has_cap)
+ = $m->generate_simple_has('$_[0]', $name, $spec);
+ my ($code, $pop_cap)
+ = $m->generate_use_default('$_[0]', $name, $spec, $has);
+
+ $assign .= $code;
+ @captures{keys %$has_cap, keys %$pop_cap}
+ = (values %$has_cap, values %$pop_cap);
+ }
}
+ Sub::Quote::quote_sub($assign, \%captures);
}
- Sub::Quote::quote_sub($assign, \%captures)->($new);
- }
+ else {
+ sub {};
+ }
+ };
+ $new->$apply_defaults;
return $new;
}
my ($self, $role) = @_;
my $composed_name = 'Role::Tiny::_COMPOSABLE::'.$role;
return $composed_name if $Role::Tiny::COMPOSED{role}{$composed_name};
- $self->_make_accessors_if_moose($role, $composed_name);
+ $self->_make_accessors_if_moose($composed_name, $role);
$self->SUPER::_composable_package_for($role);
}
}
sub _handle_constructor {
- my ($me, $to, $attr_info) = @_;
+ my ($me, $to, $role) = @_;
+ my $attr_info = $INFO{$role} && $INFO{$role}{attributes};
return unless $attr_info && @$attr_info;
if ($INFO{$to}) {
push @{$INFO{$to}{attributes}||=[]}, @$attr_info;