refactor in progress, beware (still passing all my tests though :P)

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

diff --git a/lib/Moose/Object.pm b/lib/Moose/Object.pm
index c497104..43cc8b7 100644 (file)
--- a/lib/Moose/Object.pm
+++ b/lib/Moose/Object.pm
@@ -3,34 +3,53 @@ package Moose::Object;
 
 use strict;
 use warnings;
-use metaclass 'Moose::Meta::Class' => (
-       ':attribute_metaclass' => 'Moose::Meta::Attribute'
-);
+
+use Moose::Meta::Class;
+use metaclass 'Moose::Meta::Class';
 
 use Carp 'confess';
 
-our $VERSION = '0.04';
+our $VERSION   = '0.09';
+our $AUTHORITY = 'cpan:STEVAN';
 
 sub new {
-    my $class  = shift;
-    my %params = (scalar @_ == 1) ? %{$_[0]} : @_;
-       my $self = $class->meta->new_object(%params);
-       $self->BUILDALL(\%params);
-       return $self;
+    my $class = shift;
+    my %params;
+    if (scalar @_ == 1) {
+        if (defined $_[0]) {
+            (ref($_[0]) eq 'HASH')
+                || confess "Single parameters to new() must be a HASH ref";
+            %params = %{$_[0]};
+        }
+    }
+    else {
+        %params = @_;
+    }
+    my $self = $class->meta->new_object(%params);
+    $self->BUILDALL(\%params);
+    return $self;
 }
 
 sub BUILDALL {
-       my ($self, $params) = @_;
-       foreach my $method (reverse $self->meta->find_all_methods_by_name('BUILD')) {
-               $method->{code}->($self, $params);
-       }
+    # NOTE: we ask Perl if we even 
+    # need to do this first, to avoid
+    # extra meta level calls
+    return unless $_[0]->can('BUILD');    
+    my ($self, $params) = @_;
+    foreach my $method (reverse $self->meta->find_all_methods_by_name('BUILD')) {
+        $method->{code}->($self, $params);
+    }
 }
 
 sub DEMOLISHALL {
-       my $self = shift;
-       foreach my $method ($self->meta->find_all_methods_by_name('DEMOLISH')) {
-               $method->{code}->($self);
-       }       
+    # NOTE: we ask Perl if we even 
+    # need to do this first, to avoid
+    # extra meta level calls    
+    return unless $_[0]->can('DEMOLISH');    
+    my $self = shift;    
+    foreach my $method ($self->meta->find_all_methods_by_name('DEMOLISH')) {
+        $method->{code}->($self);
+    }    
 }
 
 sub DESTROY { goto &DEMOLISHALL }
@@ -38,10 +57,31 @@ sub DESTROY { goto &DEMOLISHALL }
 # new does() methods will be created 
 # as approiate see Moose::Meta::Role
 sub does {
-    my (undef, $role_name) = @_;
+    my ($self, $role_name) = @_;
     (defined $role_name)
         || confess "You much supply a role name to does()";
-    0;    
+    my $meta = $self->meta;
+    foreach my $class ($meta->class_precedence_list) {
+        return 1 
+            if $meta->initialize($class)->does_role($role_name);            
+    }
+    return 0;   
+}
+
+# RANT:
+# Cmon, how many times have you written 
+# the following code while debugging:
+# 
+#  use Data::Dumper; 
+#  warn Dumper \%thing;
+#
+# It can get seriously annoying, so why 
+# not just do this ...
+sub dump { 
+    my $self = shift;
+    require Data::Dumper;
+    $Data::Dumper::Maxdepth = shift if @_;
+    Data::Dumper::Dumper $self;
 }
 
 1;
@@ -88,6 +128,18 @@ This will call every C<DEMOLISH> method in the inheritance hierarchy.
 
 =item B<does ($role_name)>
 
+This will check if the invocant's class C<does> a given C<$role_name>. 
+This is similar to C<isa> for object, but it checks the roles instead.
+
+=item B<dump ($maxdepth)>
+
+Cmon, 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.
+
 =back
 
 =head1 BUGS
@@ -102,7 +154,7 @@ Stevan Little E<lt>stevan@iinteractive.comE<gt>
 
 =head1 COPYRIGHT AND LICENSE
 
-Copyright 2006 by Infinity Interactive, Inc.
+Copyright 2006, 2007 by Infinity Interactive, Inc.
 
 L<http://www.iinteractive.com>