X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=lib%2FMoose%2FMeta%2FMethod%2FConstructor.pm;h=f1cd249b70590e128ec12a9c154c7fd6a8e6da4a;hb=c53c3ec7350a5fc4699ca6053221bc02e60d02d5;hp=8cad19a9035f8fed8e62fb34809772b94d1e8b93;hpb=32a274011e52f50883fa2be1145bce48426ff554;p=gitmo%2FMoose.git diff --git a/lib/Moose/Meta/Method/Constructor.pm b/lib/Moose/Meta/Method/Constructor.pm index 8cad19a..f1cd249 100644 --- a/lib/Moose/Meta/Method/Constructor.pm +++ b/lib/Moose/Meta/Method/Constructor.pm @@ -6,7 +6,7 @@ use warnings; use Scalar::Util 'blessed', 'weaken', 'looks_like_number', 'refaddr'; -our $VERSION = '0.75'; +our $VERSION = '1.05'; our $AUTHORITY = 'cpan:STEVAN'; use base 'Moose::Meta::Method', @@ -25,7 +25,7 @@ sub new { || $class->throw_error("You must supply the package_name and name parameters $Class::MOP::Method::UPGRADE_ERROR_TEXT"); my $self = bless { - 'body' => undef, + 'body' => undef, 'package_name' => $options{package_name}, 'name' => $options{name}, 'options' => $options{options}, @@ -43,11 +43,6 @@ sub new { return $self; } -# This is here so can_be_inlined can be inherited by MooseX modules. -sub _expected_constructor_class { - return 'Moose::Object'; -} - ## method sub _initialize_body { @@ -60,10 +55,12 @@ sub _initialize_body { # requires some adaption on the part of # the author, after all, nothing is free) my $source = 'sub {'; - $source .= "\n" . 'my $class = shift;'; + $source .= "\n" . 'my $_instance = shift;'; + + $source .= "\n" . 'my $class = Scalar::Util::blessed($_instance) || $_instance;'; $source .= "\n" . 'return $class->Moose::Object::new(@_)'; - $source .= "\n if \$class ne '" . $self->associated_metaclass->name + $source .= "\n if \$class ne '" . $self->associated_metaclass->name . "';\n"; $source .= $self->_generate_params('$params', '$class'); @@ -96,7 +93,7 @@ sub _initialize_body { defined $_ ? $_->_compiled_type_constraint : undef; } @type_constraints; - my $code = $self->_compile_code( + my ( $code, $e ) = $self->_compile_code( code => $source, environment => { '$meta' => \$self, @@ -104,8 +101,13 @@ sub _initialize_body { '@type_constraints' => \@type_constraints, '@type_constraint_bodies' => \@type_constraint_bodies, }, - ) or $self->throw_error("Could not eval the constructor :\n\n$source\n\nbecause :\n\n$@", error => $@, data => $source ); - + ); + + $self->throw_error( + "Could not eval the constructor :\n\n$source\n\nbecause :\n\n$e", + error => $e, data => $source ) + if $e; + $self->{'body'} = $code; } @@ -197,7 +199,7 @@ sub _generate_slot_initializer { if ($is_moose && defined($attr->init_arg) && $attr->is_required && !$attr->has_default && !$attr->has_builder) { push @source => ('(exists $params->{\'' . $attr->init_arg . '\'}) ' . - '|| ' . $self->_inline_throw_error('"Attribute (' . $attr->name . ') is required"') .';'); + '|| ' . $self->_inline_throw_error('"Attribute (' . quotemeta($attr->name) . ') is required"') .';'); } if (($attr->has_default || $attr->has_builder) && !($is_moose && $attr->is_lazy)) { @@ -213,18 +215,18 @@ sub _generate_slot_initializer { my $default; if ( $attr->has_default ) { $default = $self->_generate_default_value($attr, $index); - } + } else { my $builder = $attr->builder; $default = '$instance->' . $builder; } - + push @source => '{'; # wrap this to avoid my $val overwrite warnings push @source => ('my $val = ' . $default . ';'); push @source => $self->_generate_type_constraint_and_coercion($attr, $index) - if $is_moose; + if $is_moose; push @source => $self->_generate_slot_assignment($attr, '$val', $index); - push @source => '}'; # close - wrap this to avoid my $val overrite warnings + push @source => '}'; # close - wrap this to avoid my $val overrite warnings push @source => "}" if defined $attr->init_arg; } @@ -235,16 +237,16 @@ sub _generate_slot_initializer { if ($is_moose && $attr->has_type_constraint) { if ($attr->should_coerce && $attr->type_constraint->has_coercion) { push @source => $self->_generate_type_coercion( - $attr, - '$type_constraints[' . $index . ']', - '$val', + $attr, + '$type_constraints[' . $index . ']', + '$val', '$val' ); } push @source => $self->_generate_type_constraint_check( - $attr, - '$type_constraint_bodies[' . $index . ']', - '$type_constraints[' . $index . ']', + $attr, + '$type_constraint_bodies[' . $index . ']', + '$type_constraints[' . $index . ']', '$val' ); } @@ -260,11 +262,11 @@ sub _generate_slot_assignment { my ($self, $attr, $value, $index) = @_; my $source; - + if ($attr->has_initializer) { $source = ( '$attrs->[' . $index . ']->set_initial_value($instance, ' . $value . ');' - ); + ); } else { $source = ( @@ -273,10 +275,10 @@ sub _generate_slot_assignment { $attr->name, $value ) . ';' - ); + ); } - - my $is_moose = $attr->isa('Moose::Meta::Attribute'); # XXX FIXME + + my $is_moose = $attr->isa('Moose::Meta::Attribute'); # XXX FIXME if ($is_moose && $attr->is_weak_ref) { $source .= ( @@ -294,9 +296,9 @@ sub _generate_slot_assignment { sub _generate_type_constraint_and_coercion { my ($self, $attr, $index) = @_; - + return unless $attr->has_type_constraint; - + my @source; if ($attr->should_coerce && $attr->type_constraint->has_coercion) { push @source => $self->_generate_type_coercion( @@ -309,7 +311,7 @@ sub _generate_type_constraint_and_coercion { push @source => $self->_generate_type_constraint_check( $attr, ('$type_constraint_bodies[' . $index . ']'), - ('$type_constraints[' . $index . ']'), + ('$type_constraints[' . $index . ']'), '$val' ); return @source; @@ -324,8 +326,8 @@ sub _generate_type_constraint_check { my ($self, $attr, $type_constraint_cv, $type_constraint_obj, $value_name) = @_; return ( $self->_inline_throw_error('"Attribute (' # FIXME add 'dad' - . $attr->name - . ') does not pass the type constraint because: " . ' + . quotemeta( $attr->name ) + . ') does not pass the type constraint because: " . ' . $type_constraint_obj . '->get_message(' . $value_name . ')') . "\n\t unless " . $type_constraint_cv . '->(' . $value_name . ');' ); @@ -359,39 +361,20 @@ Moose::Meta::Method::Constructor - Method Meta Object for constructors =head1 DESCRIPTION -This class is a subclass of L that +This class is a subclass of L that provides additional Moose-specific functionality To understand this class, you should read the the -L documentation as well. +L documentation as well. =head1 INHERITANCE C is a subclass of L I L. -=head1 METHODS - -=over 4 - -=item B<< $metamethod->can_be_inlined >> - -This returns true if the method can inlined. - -First, it looks at all of the parents of the associated class. If any -of them have an inlined constructor, then the constructor can be -inlined. - -If none of them have been inlined, it checks to make sure that the -pre-inlining constructor for the class matches the constructor from -the expected class. - -By default, it expects this constructor come from L, -but subclasses can change this expectation. - -If the constructor cannot be inlined it warns that this is the case. +=head1 BUGS -=back +See L for details on reporting bugs. =head1 AUTHORS @@ -399,7 +382,7 @@ Stevan Little Estevan@iinteractive.comE =head1 COPYRIGHT AND LICENSE -Copyright 2006-2009 by Infinity Interactive, Inc. +Copyright 2006-2010 by Infinity Interactive, Inc. L