X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=lib%2FMoose%2FMeta%2FTypeConstraint%2FRole.pm;h=b79e639614f9407628b83ff44e6de4d1c2dc4918;hb=7e0492d32ce0338c6feba7710d00cb8b72e794a8;hp=f589217a93e9c5905b665f2d2c872a4a6fb8658d;hpb=0779da922a64aa891bc2708b6fc8de4d4fdffa11;p=gitmo%2FMoose.git diff --git a/lib/Moose/Meta/TypeConstraint/Role.pm b/lib/Moose/Meta/TypeConstraint/Role.pm index f589217..b79e639 100644 --- a/lib/Moose/Meta/TypeConstraint/Role.pm +++ b/lib/Moose/Meta/TypeConstraint/Role.pm @@ -7,7 +7,8 @@ use metaclass; use Scalar::Util 'blessed'; use Moose::Util::TypeConstraints (); -our $VERSION = '0.56'; +our $VERSION = '0.93'; +$VERSION = eval $VERSION; our $AUTHORITY = 'cpan:STEVAN'; use base 'Moose::Meta::TypeConstraint'; @@ -19,7 +20,7 @@ __PACKAGE__->meta->add_attribute('role' => ( sub new { my ( $class, %args ) = @_; - $args{parent} = Moose::Util::TypeConstraints::find_type_constraint('Role'); + $args{parent} = Moose::Util::TypeConstraints::find_type_constraint('Object'); my $self = $class->_new(\%args); $self->_create_hand_optimized_type_constraint; @@ -46,10 +47,10 @@ sub parents { # if anybody thinks this problematic please discuss on IRC. # a possible fix is to add by attr indexing to the type registry to find types of a certain property # regardless of their name - Moose::Util::TypeConstraints::find_type_constraint($_) - || + Moose::Util::TypeConstraints::find_type_constraint($_) + || __PACKAGE__->new( role => $_, name => "__ANON__" ) - } @{ $self->role->meta->get_roles }, + } @{ Class::MOP::class_of($self->role)->get_roles }, ); } @@ -58,6 +59,7 @@ sub equals { my $other = Moose::Util::TypeConstraints::find_type_constraint($type_or_name); + return unless defined $other; return unless $other->isa(__PACKAGE__); return $self->role eq $other->role; @@ -76,21 +78,28 @@ sub is_subtype_of { if ( not ref $type_or_name_or_role ) { # it might be a role - return 1 if $self->role->meta->does_role( $type_or_name_or_role ); + return 1 if Class::MOP::class_of($self->role)->does_role( $type_or_name_or_role ); } my $type = Moose::Util::TypeConstraints::find_type_constraint($type_or_name_or_role); + return unless defined $type; + if ( $type->isa(__PACKAGE__) ) { # if $type_or_name_or_role isn't a role, it might be the TC name of another ::Role type # or it could also just be a type object in this branch - return $self->role->meta->does_role( $type->role ); + return Class::MOP::class_of($self->role)->does_role( $type->role ); } else { # the only other thing we are a subtype of is Object $self->SUPER::is_subtype_of($type); } } +sub create_child_type { + my ($self, @args) = @_; + return Moose::Meta::TypeConstraint->new(@args, parent => $self); +} + 1; __END__ @@ -101,35 +110,62 @@ __END__ Moose::Meta::TypeConstraint::Role - Role/TypeConstraint parallel hierarchy +=head1 DESCRIPTION + +This class represents type constraints for a role. + +=head1 INHERITANCE + +C is a subclass of +L. + =head1 METHODS =over 4 -=item B +=item B<< Moose::Meta::TypeConstraint::Role->new(%options) >> + +This creates a new role type constraint based on the given +C<%options>. + +It takes the same options as its parent, with two exceptions. First, +it requires an additional option, C, which is name of the +constraint's role. Second, it automatically sets the parent to the +C type. + +The constructor also overrides the hand optimized type constraint with +one it creates internally. + +=item B<< $constraint->role >> -=item B +Returns the role name associated with the constraint. -=item B +=item B<< $constraint->parents >> -=item B +Returns all the type's parent types, corresponding to the roles that +its role does. -=item B +=item B<< $constraint->is_subtype_of($type_name_or_object) >> -=item B +If the given type is also a role type, then this checks that the +type's role does the other type's role. -=item B +Otherwise it falls back to the implementation in +L. -=item B +=item B<< $constraint->create_child_type(%options) >> -Return all the parent types, corresponding to the parent classes. +This returns a new L object with the type +as its parent. -=item B +Note that it does I return a C +object! =back =head1 BUGS -All complex software has bugs lurking in it, and this module is no +All complex software has bugs lurking in it, and this module is no exception. If you find a bug please either email me, or add the bug to cpan-RT. @@ -139,7 +175,7 @@ Yuval Kogman Enothingmuch@cpan.orgE =head1 COPYRIGHT AND LICENSE -Copyright 2006-2008 by Infinity Interactive, Inc. +Copyright 2006-2010 by Infinity Interactive, Inc. L