X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?p=gitmo%2FMouse.git;a=blobdiff_plain;f=lib%2FMouse%2FMeta%2FAttribute.pm;h=71a2bbdf178659ab4c40eb3d155210f13efa9a6d;hp=a9a84310e039add3661b2720ea3e2f81a703b845;hb=1372fcc0864d73696fe1b926d489b455be26229e;hpb=612d3e1a65aebf8042df5972079695883a4eec6e diff --git a/lib/Mouse/Meta/Attribute.pm b/lib/Mouse/Meta/Attribute.pm index a9a8431..71a2bbd 100644 --- a/lib/Mouse/Meta/Attribute.pm +++ b/lib/Mouse/Meta/Attribute.pm @@ -1,17 +1,21 @@ package Mouse::Meta::Attribute; -use strict; -use warnings; +use Mouse::Util qw(:meta); # enables strict and warnings use Carp (); -use Mouse::Util qw(:meta); - use Mouse::Meta::TypeConstraint; use Mouse::Meta::Method::Accessor; + sub _process_options{ my($class, $name, $args) = @_; + + # XXX: for backward compatibility (with method modifiers) + if($class->can('canonicalize_args') != \&canonicalize_args){ + %{$args} = $class->canonicalize_args($name, %{$args}); + } + # taken from Class::MOP::Attribute::new defined($name) @@ -25,8 +29,12 @@ sub _process_options{ my $can_be_required = defined( $args->{init_arg} ); if(exists $args->{builder}){ + # XXX: + # Moose refuses a CODE ref builder, but Mouse doesn't for backward compatibility + # This feature will be changed in a future. (gfx) $class->throw_error('builder must be a defined scalar value which is a method name') - if ref $args->{builder} || !(defined $args->{builder}); + #if ref $args->{builder} || !defined $args->{builder}; + if !defined $args->{builder}; $can_be_required++; } @@ -72,8 +80,7 @@ sub _process_options{ $args->{type_constraint} = Mouse::Util::TypeConstraints::find_or_create_isa_type_constraint($args->{isa}); } elsif(exists $args->{does}){ - # TODO - # $args->{type_constraint} = Mouse::Util::TypeConstraints::find_or_create_does_type_constraint($args->{does}); + $args->{type_constraint} = Mouse::Util::TypeConstraints::find_or_create_does_type_constraint($args->{does}); } $tc = $args->{type_constraint}; @@ -119,10 +126,6 @@ sub _process_options{ || $class->throw_error("You cannot have lazy attribute ($name) without specifying a default value for it"); } - # XXX: for backward compatibility (with method modifiers) - if($class->can('canonicalize_args') != \&canonicalize_args){ - %{$args} = $class->canonicalize_args($name, %{$args}); - } return; } @@ -136,11 +139,11 @@ sub new { $args{name} = $name; - my $instance = bless \%args, $class; + my $self = bless \%args, $class; # extra attributes if($class ne __PACKAGE__){ - $class->meta->_initialize_instance($instance,\%args); + $class->meta->_initialize_object($self, \%args); } # XXX: there is no fast way to check attribute validity @@ -150,7 +153,7 @@ sub new { # Carp::cluck("Found unknown argument(s) passed to '$name' attribute constructor in '$class': @bad"); # } - return $instance + return $self; } # readers @@ -179,9 +182,6 @@ sub builder { $_[0]->{builder} } sub should_auto_deref { $_[0]->{auto_deref} } sub should_coerce { $_[0]->{coerce} } -sub get_read_method { $_[0]->{reader} || $_[0]->{accessor} } -sub get_write_method { $_[0]->{writer} || $_[0]->{accessor} } - # predicates sub has_accessor { exists $_[0]->{accessor} } @@ -199,13 +199,13 @@ sub has_builder { exists $_[0]->{builder} } sub has_read_method { exists $_[0]->{reader} || exists $_[0]->{accessor} } sub has_write_method { exists $_[0]->{writer} || exists $_[0]->{accessor} } -sub _create_args { +sub _create_args { # DEPRECATED $_[0]->{_create_args} = $_[1] if @_ > 1; $_[0]->{_create_args} } sub interpolate_class{ - my($class, $name, $args) = @_; + my($class, $args) = @_; if(my $metaclass = delete $args->{metaclass}){ $class = Mouse::Util::resolve_metaclass_alias( Attribute => $metaclass ); @@ -214,17 +214,21 @@ sub interpolate_class{ my @traits; if(my $traits_ref = delete $args->{traits}){ - for (my $i = 0; $i < @{$traits_ref}; $i++) { - my $trait = Mouse::Util::resolve_metaclass_alias(Attribute => $traits_ref->[$i], trait => 1); + local $@; + eval{ + for (my $i = 0; $i < @{$traits_ref}; $i++) { + my $trait = Mouse::Util::resolve_metaclass_alias(Attribute => $traits_ref->[$i], trait => 1); - next if $class->does($trait); + next if $class->does($trait); - push @traits, $trait; + push @traits, $trait; - # are there options? - push @traits, $traits_ref->[++$i] - if ref($traits_ref->[$i+1]); - } + # are there options? + push @traits, $traits_ref->[++$i] + if ref($traits_ref->[$i+1]); + } + }; + warn "Fatal error in traits: $@" if _MOUSE_VERBOSE; if (@traits) { $class = Mouse::Meta::Class->create_anon_class( @@ -238,83 +242,90 @@ sub interpolate_class{ return( $class, @traits ); } -sub canonicalize_args{ +sub canonicalize_args{ # DEPRECATED my ($self, $name, %args) = @_; Carp::cluck("$self->canonicalize_args has been deprecated." - . "Use \$self->_process_options instead."); + . "Use \$self->_process_options instead.") + if _MOUSE_VERBOSE; return %args; } -sub create { +sub create { # DEPRECATED my ($self, $class, $name, %args) = @_; Carp::cluck("$self->create has been deprecated." - . "Use \$meta->add_attribute and \$attr->install_accessors instead."); + . "Use \$meta->add_attribute and \$attr->install_accessors instead.") + if _MOUSE_VERBOSE; # noop return $self; } +sub _coerce_and_verify { + my($self, $value, $instance) = @_; + + my $type_constraint = $self->{type_constraint}; + return $value if !defined $type_constraint; + + if ($self->should_coerce && $type_constraint->has_coercion) { + $value = $type_constraint->coerce($value); + } + + $self->verify_against_type_constraint($value); + + return $value; +} + sub verify_against_type_constraint { my ($self, $value) = @_; - my $tc = $self->type_constraint; - return 1 unless $tc; - local $_ = $value; - return 1 if $tc->check($value); + my $type_constraint = $self->{type_constraint}; + return 1 if !$type_constraint; + return 1 if $type_constraint->check($value); - $self->verify_type_constraint_error($self->name, $value, $tc); + $self->verify_type_constraint_error($self->name, $value, $type_constraint); } sub verify_type_constraint_error { my($self, $name, $value, $type) = @_; - $self->throw_error("Attribute ($name) does not pass the type constraint because: " . $type->get_message($value)); + $self->throw_error("Attribute ($name) does not pass the type constraint because: " + . $type->get_message($value)); } -sub coerce_constraint { ## my($self, $value) = @_; +sub coerce_constraint { # DEPRECATED my $type = $_[0]->{type_constraint} or return $_[1]; - return Mouse::Util::TypeConstraints->typecast_constraints($_[0]->associated_class->name, $type, $_[1]); -} -sub _canonicalize_handles { - my $self = shift; - my $handles = shift; + Carp::cluck("coerce_constraint() has been deprecated, which was an internal utility anyway"); - if (ref($handles) eq 'HASH') { - return %$handles; - } - elsif (ref($handles) eq 'ARRAY') { - return map { $_ => $_ } @$handles; - } - else { - $self->throw_error("Unable to canonicalize the 'handles' option with $handles"); - } + return Mouse::Util::TypeConstraints->typecast_constraints($_[0]->associated_class->name, $type, $_[1]); } sub clone_and_inherit_options{ - my $self = shift; - my $name = shift; + my($self, %args) = @_; - return ref($self)->new($name, %{$self}, (@_ == 1) ? %{$_[0]} : @_); + my($attribute_class, @traits) = ref($self)->interpolate_class(\%args); + + $args{traits} = \@traits if @traits; + return $attribute_class->new($self->name, %{$self}, %args); } -sub clone_parent { +sub clone_parent { # DEPRECATED my $self = shift; my $class = shift; my $name = shift; my %args = ($self->get_parent_args($class, $name), @_); Carp::cluck("$self->clone_parent has been deprecated." - . "Use \$meta->add_attribute and \$attr->install_accessors instead."); - + . "Use \$meta->add_attribute and \$attr->install_accessors instead.") + if _MOUSE_VERBOSE; $self->clone_and_inherited_args($class, $name, %args); } -sub get_parent_args { +sub get_parent_args { # DEPRECATED my $self = shift; my $class = shift; my $name = shift; @@ -328,21 +339,110 @@ sub get_parent_args { $self->throw_error("Could not find an attribute by the name of '$name' to inherit from"); } + +sub get_read_method { + $_[0]->{reader} || $_[0]->{accessor} +} +sub get_write_method { + $_[0]->{writer} || $_[0]->{accessor} +} + +sub get_read_method_ref{ + my($self) = @_; + + $self->{_read_method_ref} ||= do{ + my $metaclass = $self->associated_class + or $self->throw_error('No asocciated class for ' . $self->name); + + my $reader = $self->{reader} || $self->{accessor}; + if($reader){ + $metaclass->name->can($reader); + } + else{ + $self->accessor_metaclass->_generate_reader($self, $metaclass); + } + }; +} + +sub get_write_method_ref{ + my($self) = @_; + + $self->{_write_method_ref} ||= do{ + my $metaclass = $self->associated_class + or $self->throw_error('No asocciated class for ' . $self->name); + + my $reader = $self->{writer} || $self->{accessor}; + if($reader){ + $metaclass->name->can($reader); + } + else{ + $self->accessor_metaclass->_generate_writer($self, $metaclass); + } + }; +} + +sub _canonicalize_handles { + my($self, $handles) = @_; + + if (ref($handles) eq 'HASH') { + return %$handles; + } + elsif (ref($handles) eq 'ARRAY') { + return map { $_ => $_ } @$handles; + } + elsif (ref($handles) eq 'Regexp') { + my $class_or_role = ($self->{isa} || $self->{does}) + || $self->throw_error("Cannot delegate methods based on a Regexp without a type constraint (isa)"); + + my $meta = Mouse::Meta::Class->initialize("$class_or_role"); # "" for stringify + return map { $_ => $_ } + grep { $_ ne 'meta' && !Mouse::Object->can($_) && $_ =~ $handles } + $meta->isa('Mouse::Meta::Class') ? $meta->get_all_method_names : $meta->get_method_list; + } + else { + $self->throw_error("Unable to canonicalize the 'handles' option with $handles"); + } +} + + +sub associate_method{ + my ($attribute, $method) = @_; + $attribute->{associated_methods}++; + return; +} + +sub accessor_metaclass(){ 'Mouse::Meta::Method::Accessor' } + sub install_accessors{ my($attribute) = @_; - my $metaclass = $attribute->{associated_class}; + my $metaclass = $attribute->{associated_class}; + my $accessor_class = $attribute->accessor_metaclass; - foreach my $type(qw(accessor reader writer predicate clearer handles)){ + foreach my $type(qw(accessor reader writer predicate clearer)){ if(exists $attribute->{$type}){ - my $installer = '_install_' . $type; + my $generator = '_generate_' . $type; + my $code = $accessor_class->$generator($attribute, $metaclass); + $metaclass->add_method($attribute->{$type} => $code); + $attribute->associate_method($code); + } + } - Mouse::Meta::Method::Accessor->$installer($attribute, $attribute->{$type}, $metaclass); + # install delegation + if(exists $attribute->{handles}){ + my %handles = $attribute->_canonicalize_handles($attribute->{handles}); + my $reader = $attribute->get_read_method_ref; - $attribute->{associated_methods}++; + while(my($handle_name, $method_to_call) = each %handles){ + my $code = $accessor_class->_generate_delegation($attribute, $metaclass, + $reader, $handle_name, $method_to_call); + + $metaclass->add_method($handle_name => $code); + $attribute->associate_method($code); } } + if($attribute->can('create') != \&create){ # backword compatibility $attribute->create($metaclass, $attribute->name, %{$attribute}); @@ -364,7 +464,11 @@ __END__ =head1 NAME -Mouse::Meta::Attribute - attribute metaclass +Mouse::Meta::Attribute - The Mouse attribute metaclass + +=head1 VERSION + +This document describes Mouse version 0.39 =head1 METHODS @@ -472,6 +576,13 @@ is equivalent to this: =back +=head2 C<< associate_method(Method) >> + +Associates a method with the attribute. Typically, this is called internally +when an attribute generates its accessors. + +Currently the argument I is ignored in Mouse. + =head2 C<< verify_against_type_constraint(Item) -> TRUE | ERROR >> Checks that the given value passes this attribute's type constraint. Returns C @@ -482,9 +593,20 @@ on success, otherwise Ces. Creates a new attribute in the owner class, inheriting options from parent classes. Accessors and helper methods are installed. Some error checking is done. +=head2 C<< get_read_method_ref >> + +=head2 C<< get_write_method_ref >> + +Returns the subroutine reference of a method suitable for reading or +writing the attribute's value in the associated class. These methods +always return a subroutine reference, regardless of whether or not the +attribute is read- or write-only. + =head1 SEE ALSO L +L + =cut