X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=lib%2FMooseX%2FAlwaysCoerce.pm;h=e2360bc226c48055ae3d0b15f684113d59ecd109;hb=refs%2Ftags%2F0.12;hp=b8ad4f39063213fef4b7561b57f3b774410ccacd;hpb=ad1917d79f5490b78c921ce0e7ae708c300d97e4;p=gitmo%2FMooseX-AlwaysCoerce.git

diff --git a/lib/MooseX/AlwaysCoerce.pm b/lib/MooseX/AlwaysCoerce.pm
index b8ad4f3..e2360bc 100644
--- a/lib/MooseX/AlwaysCoerce.pm
+++ b/lib/MooseX/AlwaysCoerce.pm
@@ -5,12 +5,12 @@ use warnings;
 
 use namespace::autoclean;
 use Moose ();
+use MooseX::ClassAttribute ();
 use Moose::Exporter;
+use Moose::Util::MetaRole;
 use Carp;
 
-Moose::Exporter->setup_import_methods (
-    with_caller => [ 'has', 'class_has' ]
-);
+Moose::Exporter->setup_import_methods;
 
 =head1 NAME
 
@@ -18,25 +18,23 @@ MooseX::AlwaysCoerce - Automatically enable coercions for Moose attributes
 
 =head1 VERSION
 
-Version 0.01
+Version 0.12
 
 =cut
 
-our $VERSION = '0.01';
+our $VERSION = '0.12';
 
 =head1 SYNOPSIS
 
     package MyClass;
 
     use Moose;
-    use MooseX::ClassAttribute;
     use MooseX::AlwaysCoerce;
     use MyTypeLib 'SomeType';
 
     has foo => (is => 'rw', isa => SomeType); # coerce => 1 automatically added
 
-    # same, but you must load MooseX::ClassAttribute *BEFORE*
-    # MooseX::AlwaysCoerce
+    # same, MooseX::ClassAttribute is automatically applied
     class_has bar => (is => 'rw', isa => SomeType);
 
 =head1 DESCRIPTION
@@ -45,24 +43,83 @@ Have you ever spent an hour or more trying to figure out "WTF, why did my
 coercion not run?" only to find out that you forgot C<< coerce => 1 >> ?
 
 Just load this module in your L<Moose> class and C<< coerce => 1 >> will be
-enabled for every attribute automatically.
+enabled for every attribute and class attribute automatically.
+
+Use C<< coerce => 0 >> to disable a coercion explicitly.
 
 =cut
 
-sub has {
-    push @_, (coerce => 1);
-    goto &Moose::has;
+{
+    package MooseX::AlwaysCoerce::Role::Meta::Attribute;
+    use namespace::autoclean;
+    use Moose::Role;
+
+    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;
+
+    around add_class_attribute => sub {
+        my $next = shift;
+        my $self = shift;
+        my ($what, %opts) = @_;
+
+        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);
+    };
 }
 
-sub class_has {
-    push @_, (coerce => 1);
-    goto &MooseX::ClassAttribute::class_has;
+my (undef, undef, $init_meta) = Moose::Exporter->build_import_methods(
+
+    install => [ qw(import unimport) ],
+
+    class_metaroles => {
+        attribute   => ['MooseX::AlwaysCoerce::Role::Meta::Attribute'],
+        class       => ['MooseX::AlwaysCoerce::Role::Meta::Class'],
+    },
+
+    role_metaroles => {
+        # applied_attribute should be available soon, for now roles are borked
+        # 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};
+
+    MooseX::ClassAttribute->import({ into => $for_class });
+
+    # call generated method to do the rest of the work.
+    goto $init_meta;
 }
 
 =head1 AUTHOR
 
 Rafael Kitover, C<< <rkitover at cpan.org> >>
 
+=head1 CONTRIBUTORS
+
+Schwern: Michael G. Schwern <mschwern@cpan.org>
+Ether: Karen Etheridge <ether@cpan.org>
+
 =head1 BUGS
 
 Please report any bugs or feature requests to C<bug-moosex-alwayscoerce at rt.cpan.org>, or through
@@ -97,9 +154,11 @@ L<http://search.cpan.org/dist/MooseX-AlwaysCoerce/>
 
 My own stupidity, for inspiring me to write this module.
 
+Dave Rolsky, for telling me how to do it the L<Moose> way.
+
 =head1 COPYRIGHT & LICENSE
 
-Copyright (c) 2009 Rafael Kitover
+Copyright (c) 2009-2010 Rafael Kitover
 
 This program is free software; you can redistribute it and/or modify it
 under the same terms as Perl itself.
@@ -107,3 +166,4 @@ under the same terms as Perl itself.
 =cut
 
 1; # End of MooseX::AlwaysCoerce
+# vim:et sts=4 sw=4 tw=0: