Add support for undef init_arg
Shawn M Moore [Fri, 13 Jun 2008 01:53:45 +0000 (01:53 +0000)]
Changes
lib/Mouse/Object.pm

diff --git a/Changes b/Changes
index 4debfb4..be60339 100644 (file)
--- a/Changes
+++ b/Changes
@@ -8,6 +8,7 @@ Revision history for Mouse
        - Add support for ->new({...})
        - Use compute_all_applicable_attributes in the constructor to get the
          attributes of superclasses
+       - Add better support for undef init_arg
 
      * Mouse::Meta::Class:
        - More methods: compute_all_applicable_attributes, has_attribute
index 2138adf..5b7d17e 100644 (file)
@@ -28,7 +28,20 @@ sub new {
         my $key  = $attribute->name;
         my $default;
 
-        if (!exists($args{$from})) {
+        if (defined($from) && exists($args{$from})) {
+            $attribute->verify_type_constraint($args{$from})
+                if $attribute->has_type_constraint;
+
+            $instance->{$key} = $args{$from};
+
+            weaken($instance->{$key})
+                if ref($instance->{$key}) && $attribute->is_weak_ref;
+
+            if ($attribute->has_trigger) {
+                $attribute->trigger->($instance, $args{$from}, $attribute);
+            }
+        }
+        else {
             if ($attribute->has_default || $attribute->has_builder) {
                 unless ($attribute->is_lazy) {
                     my $default = $attribute->default;
@@ -54,20 +67,6 @@ sub new {
                 }
             }
         }
-
-        if (exists($args{$from})) {
-            $attribute->verify_type_constraint($args{$from})
-                if $attribute->has_type_constraint;
-
-            $instance->{$key} = $args{$from};
-
-            weaken($instance->{$key})
-                if ref($instance->{$key}) && $attribute->is_weak_ref;
-
-            if ($attribute->has_trigger) {
-                $attribute->trigger->($instance, $args{$from}, $attribute);
-            }
-        }
     }
 
     $instance->BUILDALL(\%args);