Fix many
[gitmo/Mouse.git] / lib / Mouse / Object.pm
index bac4fa0..7f20548 100644 (file)
@@ -1,83 +1,8 @@
 package Mouse::Object;
-use strict;
-use warnings;
-
-use Scalar::Util 'weaken';
-use Carp 'confess';
-
-sub new {
-    my $class = shift;
-
-    my $args = $class->BUILDARGS(@_);
-
-    my $instance = bless {}, $class;
-
-    for my $attribute ($class->meta->compute_all_applicable_attributes) {
-        my $from = $attribute->init_arg;
-        my $key  = $attribute->name;
-
-        if (defined($from) && exists($args->{$from})) {
-            $args->{$from} = $attribute->coerce_constraint($args->{$from})
-                if $attribute->should_coerce;
-            $attribute->verify_against_type_constraint($args->{$from});
-
-            $instance->{$key} = $args->{$from};
-
-            weaken($instance->{$key})
-                if ref($instance->{$key}) && $attribute->is_weak_ref;
-
-            if ($attribute->has_trigger) {
-                $attribute->trigger->($instance, $args->{$from});
-            }
-        }
-        else {
-            if ($attribute->has_default || $attribute->has_builder) {
-                unless ($attribute->is_lazy) {
-                    my $default = $attribute->default;
-                    my $builder = $attribute->builder;
-                    my $value = $attribute->has_builder
-                              ? $instance->$builder
-                              : ref($default) eq 'CODE'
-                                  ? $default->($instance)
-                                  : $default;
-
-                    $value = $attribute->coerce_constraint($value)
-                        if $attribute->should_coerce;
-                    $attribute->verify_against_type_constraint($value);
-
-                    $instance->{$key} = $value;
-
-                    weaken($instance->{$key})
-                        if ref($instance->{$key}) && $attribute->is_weak_ref;
-                }
-            }
-            else {
-                if ($attribute->is_required) {
-                    confess "Attribute (".$attribute->name.") is required";
-                }
-            }
-        }
-    }
-
-    $instance->BUILDALL($args);
-
-    return $instance;
-}
+use Mouse::Util qw(does dump meta); # enables strict and warnings
 
-sub BUILDARGS {
-    my $class = shift;
-
-    if (scalar @_ == 1) {
-        (ref($_[0]) eq 'HASH')
-            || confess "Single parameters to new() must be a HASH ref";
-        return {%{$_[0]}};
-    }
-    else {
-        return {@_};
-    }
-}
+sub new;
 
-sub DESTROY { shift->DEMOLISHALL }
 
 sub BUILDALL {
     my $self = shift;
@@ -86,12 +11,12 @@ sub BUILDALL {
     return unless $self->can('BUILD');
 
     for my $class (reverse $self->meta->linearized_isa) {
-        no strict 'refs';
-        no warnings 'once';
-        my $code = *{ $class . '::BUILD' }{CODE}
-            or next;
-        $code->($self, @_);
+        my $build = Mouse::Util::get_code_ref($class, 'BUILD')
+            || next;
+
+        $self->$build(@_);
     }
+    return;
 }
 
 sub DEMOLISHALL {
@@ -100,35 +25,19 @@ sub DEMOLISHALL {
     # short circuit
     return unless $self->can('DEMOLISH');
 
-    no strict 'refs';
-
-    for my $class ($self->meta->linearized_isa) {
-        my $code = *{ $class . '::DEMOLISH' }{CODE}
-            or next;
-        $code->($self, @_);
-    }
-}
-
-sub dump { 
-    my $self = shift;
-    require Data::Dumper;
-    local $Data::Dumper::Maxdepth = shift if @_;
-    Data::Dumper::Dumper $self;
-}
+    # We cannot count on being able to retrieve a previously made
+    # metaclass, _or_ being able to make a new one during global
+    # destruction. However, we should still be able to use mro at
+    # that time (at least tests suggest so ;)
 
+    foreach my $class (@{ Mouse::Util::get_linear_isa(ref $self) }) {
+        my $demolish = Mouse::Util::get_code_ref($class, 'DEMOLISH')
+            || next;
 
-sub does {
-    my ($self, $role_name) = @_;
-    (defined $role_name)
-        || confess "You must supply a role name to does()";
-    my $meta = $self->meta;
-    foreach my $class ($meta->linearized_isa) {
-        my $m = $meta->initialize($class);
-        return 1 
-            if $m->can('does_role') && $m->does_role($role_name);            
+        $self->$demolish();
     }
-    return 0;   
-};
+    return;
+}
 
 1;
 
@@ -136,47 +45,50 @@ __END__
 
 =head1 NAME
 
-Mouse::Object - we don't need to steenkin' constructor
+Mouse::Object - The base object for Mouse classes
+
+=head1 VERSION
+
+This document describes Mouse version 0.40_07
 
 =head1 METHODS
 
-=head2 new arguments -> object
+=head2 C<< new (Arguments) -> Object >>
 
-Instantiates a new Mouse::Object. This is obviously intended for subclasses.
+Instantiates a new C<Mouse::Object>. This is obviously intended for subclasses.
 
-=head2 BUILDALL \%args
+=head2 C<< BUILDARGS (Arguments) -> HashRef >>
 
-Calls L</BUILD> on each class in the class hierarchy. This is called at the
-end of L</new>.
+Lets you override the arguments that C<new> takes. Return a hashref of
+parameters.
 
-=head2 BUILD \%args
+=head2 C<< BUILDALL (\%args) >>
 
-You may put any business logic initialization in BUILD methods. You don't
-need to redispatch or return any specific value.
+Calls C<BUILD> on each class in the class hierarchy. This is called at the
+end of C<new>.
 
-=head2 BUILDARGS
+=head2 C<< BUILD (\%args) >>
 
-Lets you override the arguments that C<new> takes. Return a hashref of
-parameters.
+You may put any business logic initialization in BUILD methods. You don't
+need to redispatch or return any specific value.
 
-=head2 DEMOLISHALL
+=head2 C<< DEMOLISHALL >>
 
-Calls L</DEMOLISH> on each class in the class hierarchy. This is called at
-L</DESTROY> time.
+Calls C<DEMOLISH> on each class in the class hierarchy. This is called at
+C<DESTROY> time.
 
-=head2 DEMOLISH
+=head2 C<< DEMOLISH >>
 
 You may put any business logic deinitialization in DEMOLISH methods. You don't
 need to redispatch or return any specific value.
 
 
-=head2 does $role_name
+=head2 C<< does ($role_name) -> Bool >>
 
-This will check if the invocant's class "does" a given C<$role_name>.
+This will check if the invocant's class B<does> a given C<$role_name>.
 This is similar to "isa" for object, but it checks the roles instead.
 
-
-=head2 B<dump ($maxdepth)>
+=head2 C<<dump ($maxdepth) -> Str >>
 
 From the Moose POD:
 
@@ -189,6 +101,9 @@ From the Moose POD:
 
 The implementation was lifted directly from Moose::Object.
 
-=cut
+=head1 SEE ALSO
+
+L<Moose::Object>
 
+=cut