Fix many

[gitmo/Mouse.git] / lib / Mouse / Object.pm

diff --git a/lib/Mouse/Object.pm b/lib/Mouse/Object.pm
index e870e78..7f20548 100644 (file)
--- a/lib/Mouse/Object.pm
+++ b/lib/Mouse/Object.pm
@@ -1,69 +1,8 @@
-#!/usr/bin/env perl
 package Mouse::Object;
-use strict;
-use warnings;
-use MRO::Compat;
-
-use Scalar::Util qw/blessed weaken/;
-use Carp 'confess';
-
-sub new {
-    my $class = shift;
-    my %args  = @_;
-    my $instance = bless {}, $class;
-
-    for my $attribute ($class->meta->attributes) {
-        my $key = $attribute->init_arg;
-        my $default;
-
-        if (!exists($args{$key})) {
-            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->()
-                                  : $default;
-
-                    $attribute->verify_type_constraint($value)
-                        if $attribute->has_type_constraint;
-
-                    $instance->{$key} = $value;
-
-                    weaken($instance->{$key})
-                        if $attribute->weak_ref;
-                }
-            }
-            else {
-                if ($attribute->is_required) {
-                    confess "Attribute '".$attribute->name."' is required";
-                }
-            }
-        }
-
-        if (exists($args{$key})) {
-            $attribute->verify_type_constraint($args{$key})
-                if $attribute->has_type_constraint;
-
-            $instance->{$key} = $args{$key};
-
-            weaken($instance->{$key})
-                if $attribute->weak_ref;
-
-            if ($attribute->has_trigger) {
-                $attribute->trigger->($instance, $args{$key}, $attribute);
-            }
-        }
-    }
-
-    $instance->BUILDALL(\%args);
+use Mouse::Util qw(does dump meta); # enables strict and warnings
 
-    return $instance;
-}
+sub new;
 
-sub DESTROY { shift->DEMOLISHALL }
 
 sub BUILDALL {
     my $self = shift;
@@ -71,13 +10,13 @@ sub BUILDALL {
     # short circuit
     return unless $self->can('BUILD');
 
-    no strict 'refs';
+    for my $class (reverse $self->meta->linearized_isa) {
+        my $build = Mouse::Util::get_code_ref($class, 'BUILD')
+            || next;
 
-    for my $class ($self->meta->linearized_isa) {
-        my $code = *{ $class . '::BUILD' }{CODE}
-            or next;
-        $code->($self, @_);
+        $self->$build(@_);
     }
+    return;
 }
 
 sub DEMOLISHALL {
@@ -86,13 +25,18 @@ sub DEMOLISHALL {
     # short circuit
     return unless $self->can('DEMOLISH');
 
-    no strict 'refs';
+    # 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 ;)
 
-    for my $class ($self->meta->linearized_isa) {
-        my $code = *{ $class . '::DEMOLISH' }{CODE}
-            or next;
-        $code->($self, @_);
+    foreach my $class (@{ Mouse::Util::get_linear_isa(ref $self) }) {
+        my $demolish = Mouse::Util::get_code_ref($class, 'DEMOLISH')
+            || next;
+
+        $self->$demolish();
     }
+    return;
 }
 
 1;
@@ -101,33 +45,65 @@ __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 C<Mouse::Object>. This is obviously intended for subclasses.
+
+=head2 C<< BUILDARGS (Arguments) -> HashRef >>
 
-Instantiates a new Mouse::Object. This is obviously intended for subclasses.
+Lets you override the arguments that C<new> takes. Return a hashref of
+parameters.
 
-=head2 BUILDALL \%args
+=head2 C<< BUILDALL (\%args) >>
 
-Calls L</BUILD> on each class in the class hierarchy. This is called at the
-end of L</new>.
+Calls C<BUILD> on each class in the class hierarchy. This is called at the
+end of C<new>.
 
-=head2 BUILD \%args
+=head2 C<< BUILD (\%args) >>
 
 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 C<< does ($role_name) -> Bool >>
+
+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 C<<dump ($maxdepth) -> Str >>
+
+From the Moose POD:
+
+    C'mon, how many times have you written the following code while debugging:
+
+     use Data::Dumper; 
+     warn Dumper $obj;
+
+    It can get seriously annoying, so why not just use this.
+
+The implementation was lifted directly from Moose::Object.
+
+=head1 SEE ALSO
+
+L<Moose::Object>
+
 =cut