fix union with string types

[gitmo/MooseX-Types.git] / lib / MooseX / Types / TypeDecorator.pm

diff --git a/lib/MooseX/Types/TypeDecorator.pm b/lib/MooseX/Types/TypeDecorator.pm
index 5472646..12c7f48 100644 (file)
--- a/lib/MooseX/Types/TypeDecorator.pm
+++ b/lib/MooseX/Types/TypeDecorator.pm
@@ -3,10 +3,37 @@ package MooseX::Types::TypeDecorator;
 use strict;
 use warnings;
 
+
+use Carp::Clan qw( ^MooseX::Types );
+use Moose::Util::TypeConstraints ();
+use Moose::Meta::TypeConstraint::Union;
+use Scalar::Util qw(blessed);
+
 use overload(
     '""' => sub {
-        shift->type_constraint->name;  
+               my $self = shift @_;
+               if(blessed $self) {
+                       return $self->__type_constraint->name;                  
+               } else {
+                       return "$self";
+               }
     },
+    '|' => sub {
+        
+        ## It's kind of ugly that we need to know about Union Types, but this
+        ## is needed for syntax compatibility.  Maybe someday we'll all just do
+        ## Or[Str,Str,Int]
+
+       my @tc = map {
+           blessed $_ ? $_ :
+             Moose::Util::TypeConstraints::find_or_parse_type_constraint($_)
+       } @_;
+
+        my $union = Moose::Meta::TypeConstraint::Union->new(type_constraints=>\@tc);
+        return Moose::Util::TypeConstraints::register_type_constraint($union);
+    },
+    fallback => 1,
+    
 );
 
 =head1 NAME
@@ -29,24 +56,96 @@ Old school instantiation
 =cut
 
 sub new {
-    my ($class, %args) = @_;
-    return bless \%args, $class;
+    my $class = shift @_;
+    if(my $arg = shift @_) {
+        if(blessed $arg && $arg->isa('Moose::Meta::TypeConstraint')) {
+            return bless {'__type_constraint'=>$arg}, $class;
+        } elsif(
+            blessed $arg &&
+            $arg->isa('MooseX::Types::UndefinedType') 
+          ) {
+            ## stub in case we'll need to handle these types differently
+            return bless {'__type_constraint'=>$arg}, $class;
+        } elsif(blessed $arg) {
+            croak "Argument must be ->isa('Moose::Meta::TypeConstraint') or ->isa('MooseX::Types::UndefinedType'), not ". blessed $arg;
+        } else {
+            croak "Argument cannot be '$arg'";
+        }
+    } else {
+        croak "This method [new] requires a single argument.";        
+    }
+}
+
+=head2 __type_constraint ($type_constraint)
+
+Set/Get the type_constraint.
+
+=cut
+
+sub __type_constraint {
+    my $self = shift @_;    
+    if(blessed $self) {
+        if(defined(my $tc = shift @_)) {
+            $self->{__type_constraint} = $tc;
+        }
+        return $self->{__type_constraint};        
+    } else {
+        croak 'cannot call __type_constraint as a class method';
+    }
+}
+
+=head2 isa
+
+handle $self->isa since AUTOLOAD can't.
+
+=cut
+
+sub isa {
+    my ($self, $target) = @_;  
+    if(defined $target) {
+       if(blessed $self) {
+               return $self->__type_constraint->isa($target);
+       } else {
+               return;
+       }
+    } else {
+        return;
+    }
 }
 
-=head type_constraint ($type_constraint)
+=head2 can
 
-Set/Get the type_constraint
+handle $self->can since AUTOLOAD can't.
 
 =cut
 
-sub type_constraint {
-    my $self = shift @_;
-    if(my $tc = shift @_) {
-        $self->{type_constraint} = $tc;
+sub can {
+    my ($self, $target) = @_;
+    if(defined $target) {
+       if(blessed $self) {
+               return $self->__type_constraint->can($target);
+       } else {
+               return;
+       }
+    } else {
+        return;
     }
-    return $self->{type_constraint};
 }
 
+=head2 meta
+
+have meta examine the underlying type constraints
+
+=cut
+
+sub meta {
+       my $self = shift @_;
+       if(blessed $self) {
+               return $self->__type_constraint->meta;
+       } 
+}
+
+
 =head2 DESTROY
 
 We might need it later
@@ -63,10 +162,24 @@ Delegate to the decorator targe
 
 =cut
 
-sub AUTOLOAD
-{
+sub AUTOLOAD {
+    
+    my ($self, @args) = @_;
     my ($method) = (our $AUTOLOAD =~ /([^:]+)$/);
-    return shift->type_constraint->$method(@_);
+    
+    ## We delegate with this method in an attempt to support a value of
+    ## __type_constraint which is also AUTOLOADing, in particular the class
+    ## MooseX::Types::UndefinedType which AUTOLOADs during autovivication.
+    
+    my $return;
+    
+    eval {
+        $return = $self->__type_constraint->$method(@args);
+    }; if($@) {
+        croak $@;
+    } else {
+        return $return;
+    }
 }
 
 =head1 AUTHOR AND COPYRIGHT