Don't only conditionally use load_class since..
Shawn M Moore [Sat, 28 Mar 2009 22:45:52 +0000 (18:45 -0400)]
we need the metaclass (which load_class gives us)
and load_class already checks is_class_loaded

lib/Moose/Meta/Attribute.pm

index 50079d4..3ee8bcf 100644 (file)
@@ -620,10 +620,7 @@ sub _canonicalize_handles {
         }
     }
     else {
-        Class::MOP::load_class($handles) 
-            unless Class::MOP::is_class_loaded($handles);
-            
-        my $role_meta = Class::MOP::class_of($handles);
+        my $role_meta = Class::MOP::load_class($handles);
 
         (blessed $role_meta && $role_meta->isa('Moose::Meta::Role'))
             || $self->throw_error("Unable to canonicalize the 'handles' option with $handles because its metaclass is not a Moose::Meta::Role", data => $handles);