find prereqs automatically

[gitmo/MooseX-AlwaysCoerce.git] / lib / MooseX / AlwaysCoerce.pm

diff --git a/lib/MooseX/AlwaysCoerce.pm b/lib/MooseX/AlwaysCoerce.pm
index 96b6213..14d5767 100644 (file)
--- a/lib/MooseX/AlwaysCoerce.pm
+++ b/lib/MooseX/AlwaysCoerce.pm
@@ -3,9 +3,9 @@ package MooseX::AlwaysCoerce;
 use strict;
 use warnings;
 
-use namespace::autoclean;
+use namespace::autoclean 0.12;
 use Moose ();
-use MooseX::ClassAttribute ();
+use MooseX::ClassAttribute 0.24 ();
 use Moose::Exporter;
 use Moose::Util::MetaRole;
 use Carp;
@@ -18,11 +18,11 @@ MooseX::AlwaysCoerce - Automatically enable coercions for Moose attributes
 
 =head1 VERSION
 
-Version 0.06
+Version 0.16
 
 =cut
 
-our $VERSION = '0.06';
+our $VERSION = '0.16';
 
 =head1 SYNOPSIS
 
@@ -54,29 +54,32 @@ Use C<< coerce => 0 >> to disable a coercion explicitly.
     use namespace::autoclean;
     use Moose::Role;
 
-    has coerce => (
-        lazy    => 1,
-        reader  => "should_coerce",
-        default => sub {
-            return 1 if shift->type_constraint->has_coercion;
-            return 0;
-        }
-    );
+    around should_coerce => sub {
+        my $orig = shift;
+        my $self = shift;
+
+        my $current_val = $self->$orig(@_);
+
+        return $current_val if defined $current_val;
 
+        return 1 if $self->type_constraint && $self->type_constraint->has_coercion;
+        return 0;
+    };
 
     package MooseX::AlwaysCoerce::Role::Meta::Class;
     use namespace::autoclean;
     use Moose::Role;
     use Moose::Util::TypeConstraints;
-    use MooseX::ClassAttribute;
 
     around add_class_attribute => sub {
         my $next = shift;
         my $self = shift;
         my ($what, %opts) = @_;
 
-        my $type = Moose::Util::TypeConstraints::find_or_parse_type_constraint($opts{isa});
-        $opts{coerce} = 1 if !exists $opts{coerce} and $type->has_coercion;
+        if (exists $opts{isa}) {
+            my $type = Moose::Util::TypeConstraints::find_or_parse_type_constraint($opts{isa});
+            $opts{coerce} = 1 if not exists $opts{coerce} and $type->has_coercion;
+        }
 
         $self->$next($what, %opts);
     };
@@ -91,19 +94,19 @@ my (undef, undef, $init_meta) = Moose::Exporter->build_import_methods(
         class       => ['MooseX::AlwaysCoerce::Role::Meta::Class'],
     },
 
-    also            => ['MooseX::ClassAttribute'],
+    role_metaroles => {
+        (Moose->VERSION >= 1.9900
+            ? (applied_attribute => ['MooseX::AlwaysCoerce::Role::Meta::Attribute'])
+            : ()),
+        role                => ['MooseX::AlwaysCoerce::Role::Meta::Class'],
+    }
 );
 
 sub init_meta {
     my ($class, %options) = @_;
     my $for_class = $options{for_class};
 
-    # Bring this in only if we are being applied to a
-    # metaclass, but not a metarole.
-    if (Class::MOP::class_of($for_class)->isa('Class::MOP::Class'))
-    {
-        MooseX::ClassAttribute->import({ into => $for_class });
-    }
+    MooseX::ClassAttribute->import({ into => $for_class });
 
     # call generated method to do the rest of the work.
     goto $init_meta;
@@ -164,3 +167,4 @@ under the same terms as Perl itself.
 =cut
 
 1; # End of MooseX::AlwaysCoerce
+# vim:et sts=4 sw=4 tw=0: