set version to 0.94
[gitmo/Moose.git] / lib / Moose / Meta / Role.pm
index bea6656..caad021 100644 (file)
@@ -5,106 +5,376 @@ use strict;
 use warnings;
 use metaclass;
 
-use Carp         'confess';
 use Scalar::Util 'blessed';
-use B            'svref_2object';
+use Carp         'confess';
+use Devel::GlobalDestruction 'in_global_destruction';
+
+our $VERSION   = '0.94';
+$VERSION = eval $VERSION;
+our $AUTHORITY = 'cpan:STEVAN';
 
 use Moose::Meta::Class;
+use Moose::Meta::Role::Attribute;
+use Moose::Meta::Role::Method;
+use Moose::Meta::Role::Method::Required;
+use Moose::Meta::Role::Method::Conflicting;
+use Moose::Util qw( ensure_all_roles );
+
+use base 'Class::MOP::Module', 'Class::MOP::Mixin::HasAttributes';
+
+## ------------------------------------------------------------------
+## NOTE:
+## I normally don't do this, but I am doing
+## a whole bunch of meta-programmin in this
+## module, so it just makes sense. For a clearer
+## picture of what is going on in the next
+## several lines of code, look at the really
+## big comment at the end of this file (right
+## before the POD).
+## - SL
+## ------------------------------------------------------------------
+
+my $META = __PACKAGE__->meta;
+
+## ------------------------------------------------------------------
+## attributes ...
 
-our $VERSION = '0.04';
+# NOTE:
+# since roles are lazy, we hold all the attributes
+# of the individual role in 'statis' until which
+# time when it is applied to a class. This means
+# keeping a lot of things in hash maps, so we are
+# using a little of that meta-programmin' magic
+# here an saving lots of extra typin. And since
+# many of these attributes above require similar
+# functionality to support them, so we again use
+# the wonders of meta-programmin' to deliver a
+# very compact solution to this normally verbose
+# problem.
+# - SL
+
+foreach my $action (
+    {
+        name        => 'excluded_roles_map',
+        attr_reader => 'get_excluded_roles_map' ,
+        methods     => {
+            add       => 'add_excluded_roles',
+            get_keys  => 'get_excluded_roles_list',
+            existence => 'excludes_role',
+        }
+    },
+    {
+        name        => 'required_methods',
+        attr_reader => 'get_required_methods_map',
+        methods     => {
+            remove     => 'remove_required_methods',
+            get_values => 'get_required_method_list',
+            existence  => 'requires_method',
+        }
+    },
+) {
+
+    my $attr_reader = $action->{attr_reader};
+    my $methods     = $action->{methods};
+
+    # create the attribute
+    $META->add_attribute($action->{name} => (
+        reader  => $attr_reader,
+        default => sub { {} }
+    ));
+
+    # create some helper methods
+    $META->add_method($methods->{add} => sub {
+        my ($self, @values) = @_;
+        $self->$attr_reader->{$_} = undef foreach @values;
+    }) if exists $methods->{add};
+
+    $META->add_method($methods->{get_keys} => sub {
+        my ($self) = @_;
+        keys %{$self->$attr_reader};
+    }) if exists $methods->{get_keys};
+
+    $META->add_method($methods->{get_values} => sub {
+        my ($self) = @_;
+        values %{$self->$attr_reader};
+    }) if exists $methods->{get_values};
+
+    $META->add_method($methods->{get} => sub {
+        my ($self, $name) = @_;
+        $self->$attr_reader->{$name}
+    }) if exists $methods->{get};
+
+    $META->add_method($methods->{existence} => sub {
+        my ($self, $name) = @_;
+        exists $self->$attr_reader->{$name} ? 1 : 0;
+    }) if exists $methods->{existence};
+
+    $META->add_method($methods->{remove} => sub {
+        my ($self, @values) = @_;
+        delete $self->$attr_reader->{$_} foreach @values;
+    }) if exists $methods->{remove};
+}
+
+$META->add_attribute(
+    'method_metaclass',
+    reader  => 'method_metaclass',
+    default => 'Moose::Meta::Role::Method',
+);
+
+$META->add_attribute(
+    'required_method_metaclass',
+    reader  => 'required_method_metaclass',
+    default => 'Moose::Meta::Role::Method::Required',
+);
+
+$META->add_attribute(
+    'conflicting_method_metaclass',
+    reader  => 'conflicting_method_metaclass',
+    default => 'Moose::Meta::Role::Method::Conflicting',
+);
+
+$META->add_attribute(
+    'application_to_class_class',
+    reader  => 'application_to_class_class',
+    default => 'Moose::Meta::Role::Application::ToClass',
+);
+
+$META->add_attribute(
+    'application_to_role_class',
+    reader  => 'application_to_role_class',
+    default => 'Moose::Meta::Role::Application::ToRole',
+);
+
+$META->add_attribute(
+    'application_to_instance_class',
+    reader  => 'application_to_instance_class',
+    default => 'Moose::Meta::Role::Application::ToInstance',
+);
+
+# More or less copied from Moose::Meta::Class
+sub initialize {
+    my $class = shift;
+    my $pkg   = shift;
+    return Class::MOP::get_metaclass_by_name($pkg)
+        || $class->SUPER::initialize(
+        $pkg,
+        'attribute_metaclass' => 'Moose::Meta::Role::Attribute',
+        @_
+        );
+}
+
+sub reinitialize {
+    my $self = shift;
+    my $pkg  = shift;
+
+    my $meta = blessed $pkg ? $pkg : Class::MOP::class_of($pkg);
+
+    my %existing_classes;
+    if ($meta) {
+        %existing_classes = map { $_ => $meta->$_() } qw(
+            attribute_metaclass
+            method_metaclass
+            wrapped_method_metaclass
+            required_method_metaclass
+            conflicting_method_metaclass
+            application_to_class_class
+            application_to_role_class
+            application_to_instance_class
+        );
+    }
 
-## Attributes
+    return $self->SUPER::reinitialize(
+        $pkg,
+        %existing_classes,
+        @_,
+    );
+}
 
-## the meta for the role package
+sub add_attribute {
+    my $self = shift;
 
-__PACKAGE__->meta->add_attribute('_role_meta' => (
-    reader   => '_role_meta',
-    init_arg => ':role_meta'
-));
+    if (blessed $_[0] && ! $_[0]->isa('Moose::Meta::Role::Attribute') ) {
+        my $class = ref $_[0];
+        Moose->throw_error( "Cannot add a $class as an attribute to a role" );
+    }
 
-## roles
+    return $self->SUPER::add_attribute(@_);
+}
 
-__PACKAGE__->meta->add_attribute('roles' => (
-    reader  => 'get_roles',
-    default => sub { [] }
-));
+sub _attach_attribute {
+    my ( $self, $attribute ) = @_;
 
-## excluded roles
+    $attribute->attach_to_role($self);
+}
 
-__PACKAGE__->meta->add_attribute('excluded_roles_map' => (
-    reader  => 'get_excluded_roles_map',
-    default => sub { {} }
-));
+sub add_required_methods {
+    my $self = shift;
 
-## attributes
+    for (@_) {
+        my $method = $_;
+        if (!blessed($method)) {
+            $method = $self->required_method_metaclass->new(
+                name => $method,
+            );
+        }
+        $self->get_required_methods_map->{$method->name} = $method;
+    }
+}
 
-__PACKAGE__->meta->add_attribute('attribute_map' => (
-    reader   => 'get_attribute_map',
-    default  => sub { {} }
-));
+sub add_conflicting_method {
+    my $self = shift;
 
-## required methods
+    my $method;
+    if (@_ == 1 && blessed($_[0])) {
+        $method = shift;
+    }
+    else {
+        $method = $self->conflicting_method_metaclass->new(@_);
+    }
 
-__PACKAGE__->meta->add_attribute('required_methods' => (
-    reader  => 'get_required_methods_map',
-    default => sub { {} }
-));
+    $self->add_required_methods($method);
+}
 
+## ------------------------------------------------------------------
 ## method modifiers
 
-__PACKAGE__->meta->add_attribute('before_method_modifiers' => (
-    reader  => 'get_before_method_modifiers_map',
-    default => sub { {} } # (<name> => [ (CODE) ])
-));
+# NOTE:
+# the before/around/after method modifiers are
+# stored by name, but there can be many methods
+# then associated with that name. So again we have
+# lots of similar functionality, so we can do some
+# meta-programmin' and save some time.
+# - SL
+
+foreach my $modifier_type (qw[ before around after ]) {
+
+    my $attr_reader = "get_${modifier_type}_method_modifiers_map";
+
+    # create the attribute ...
+    $META->add_attribute("${modifier_type}_method_modifiers" => (
+        reader  => $attr_reader,
+        default => sub { {} }
+    ));
+
+    # and some helper methods ...
+    $META->add_method("get_${modifier_type}_method_modifiers" => sub {
+        my ($self, $method_name) = @_;
+        #return () unless exists $self->$attr_reader->{$method_name};
+        my $mm = $self->$attr_reader->{$method_name};
+        $mm ? @$mm : ();
+    });
+
+    $META->add_method("has_${modifier_type}_method_modifiers" => sub {
+        my ($self, $method_name) = @_;
+        # NOTE:
+        # for now we assume that if it exists,..
+        # it has at least one modifier in it
+        (exists $self->$attr_reader->{$method_name}) ? 1 : 0;
+    });
 
-__PACKAGE__->meta->add_attribute('after_method_modifiers' => (
-    reader  => 'get_after_method_modifiers_map',
-    default => sub { {} } # (<name> => [ (CODE) ])
-));
+    $META->add_method("add_${modifier_type}_method_modifier" => sub {
+        my ($self, $method_name, $method) = @_;
 
-__PACKAGE__->meta->add_attribute('around_method_modifiers' => (
-    reader  => 'get_around_method_modifiers_map',
-    default => sub { {} } # (<name> => [ (CODE) ])
-));
+        $self->$attr_reader->{$method_name} = []
+            unless exists $self->$attr_reader->{$method_name};
+
+        my $modifiers = $self->$attr_reader->{$method_name};
 
-__PACKAGE__->meta->add_attribute('override_method_modifiers' => (
+        # NOTE:
+        # check to see that we aren't adding the
+        # same code twice. We err in favor of the
+        # first on here, this may not be as expected
+        foreach my $modifier (@{$modifiers}) {
+            return if $modifier == $method;
+        }
+
+        push @{$modifiers} => $method;
+    });
+
+}
+
+## ------------------------------------------------------------------
+## override method mofidiers
+
+$META->add_attribute('override_method_modifiers' => (
     reader  => 'get_override_method_modifiers_map',
-    default => sub { {} } # (<name> => CODE) 
+    default => sub { {} }
 ));
 
-## Methods 
-
-sub new {
-    my $class   = shift;
-    my %options = @_;
-    $options{':role_meta'} = Moose::Meta::Class->initialize(
-        $options{role_name},
-        ':method_metaclass' => 'Moose::Meta::Role::Method'
-    ) unless defined $options{':role_meta'} && 
-             $options{':role_meta'}->isa('Moose::Meta::Class');
-    my $self = $class->meta->new_object(%options);
-    return $self;
+# NOTE:
+# these are a little different because there
+# can only be one per name, whereas the other
+# method modifiers can have multiples.
+# - SL
+
+sub add_override_method_modifier {
+    my ($self, $method_name, $method) = @_;
+    (!$self->has_method($method_name))
+        || Moose->throw_error("Cannot add an override of method '$method_name' " .
+                   "because there is a local version of '$method_name'");
+    $self->get_override_method_modifiers_map->{$method_name} = $method;
+}
+
+sub has_override_method_modifier {
+    my ($self, $method_name) = @_;
+    # NOTE:
+    # for now we assume that if it exists,..
+    # it has at least one modifier in it
+    (exists $self->get_override_method_modifiers_map->{$method_name}) ? 1 : 0;
+}
+
+sub get_override_method_modifier {
+    my ($self, $method_name) = @_;
+    $self->get_override_method_modifiers_map->{$method_name};
 }
 
+## general list accessor ...
+
+sub get_method_modifier_list {
+    my ($self, $modifier_type) = @_;
+    my $accessor = "get_${modifier_type}_method_modifiers_map";
+    keys %{$self->$accessor};
+}
+
+sub reset_package_cache_flag  { (shift)->{'_package_cache_flag'} = undef }
+sub update_package_cache_flag {
+    my $self = shift;
+    $self->{'_package_cache_flag'} = Class::MOP::check_package_cache_flag($self->name);
+}
+
+
+
+## ------------------------------------------------------------------
 ## subroles
 
+$META->add_attribute('roles' => (
+    reader  => 'get_roles',
+    default => sub { [] }
+));
+
 sub add_role {
     my ($self, $role) = @_;
     (blessed($role) && $role->isa('Moose::Meta::Role'))
-        || confess "Roles must be instances of Moose::Meta::Role";
+        || Moose->throw_error("Roles must be instances of Moose::Meta::Role");
     push @{$self->get_roles} => $role;
+    $self->reset_package_cache_flag;
 }
 
 sub calculate_all_roles {
     my $self = shift;
     my %seen;
-    grep { !$seen{$_->name}++ } $self, map { $_->calculate_all_roles } @{ $self->get_roles };
+    grep {
+        !$seen{$_->name}++
+    } ($self, map {
+                  $_->calculate_all_roles
+              } @{ $self->get_roles });
 }
 
 sub does_role {
-    my ($self, $role_name) = @_;
-    (defined $role_name)
-        || confess "You must supply a role name to look for";
+    my ($self, $role) = @_;
+    (defined $role)
+        || Moose->throw_error("You must supply a role name to look for");
+    my $role_name = blessed $role ? $role->name : $role;
     # if we are it,.. then return true
     return 1 if $role_name eq $self->name;
     # otherwise.. check our children
@@ -114,429 +384,292 @@ sub does_role {
     return 0;
 }
 
-## excluded roles
+sub find_method_by_name { (shift)->get_method(@_) }
 
-sub add_excluded_roles {
-    my ($self, @excluded_role_names) = @_;
-    $self->get_excluded_roles_map->{$_} = undef foreach @excluded_role_names;
-}
-
-sub get_excluded_roles_list {
-    my ($self) = @_;
-    keys %{$self->get_excluded_roles_map};
-}
-
-sub excludes_role {
-    my ($self, $role_name) = @_;
-    exists $self->get_excluded_roles_map->{$role_name} ? 1 : 0;
-}
-
-## required methods
-
-sub add_required_methods {
-    my ($self, @methods) = @_;
-    $self->get_required_methods_map->{$_} = undef foreach @methods;
-}
-
-sub remove_required_methods {
-    my ($self, @methods) = @_;
-    delete $self->get_required_methods_map->{$_} foreach @methods;
-}
-
-sub get_required_method_list {
-    my ($self) = @_;
-    keys %{$self->get_required_methods_map};
-}
+sub alias_method {
+    Carp::cluck("The alias_method method is deprecated. Use add_method instead.\n");
 
-sub requires_method {
-    my ($self, $method_name) = @_;
-    exists $self->get_required_methods_map->{$method_name} ? 1 : 0;
-}
-
-sub _clean_up_required_methods {
     my $self = shift;
-    foreach my $method ($self->get_required_method_list) {
-        $self->remove_required_methods($method)
-            if $self->has_method($method);
-    } 
-}
 
-## methods
-
-# NOTE:
-# we delegate to some role_meta methods for convience here
-# the Moose::Meta::Role is meant to be a read-only interface
-# to the underlying role package, if you want to manipulate 
-# that, just use ->role_meta
-
-sub name    { (shift)->_role_meta->name    }
-sub version { (shift)->_role_meta->version }
-
-sub get_method      { (shift)->_role_meta->get_method(@_)   }
-sub has_method      { (shift)->_role_meta->has_method(@_)   }
-sub alias_method    { (shift)->_role_meta->alias_method(@_) }
-sub get_method_list { 
-    my ($self) = @_;
-    grep { 
-        # NOTE:
-        # this is a kludge for now,... these functions 
-        # should not be showing up in the list at all, 
-        # but they do, so we need to switch Moose::Role
-        # and Moose to use Sub::Exporter to prevent this
-        !/^(meta|has|extends|blessed|confess|augment|inner|override|super|before|after|around|with|requires)$/ 
-    } $self->_role_meta->get_method_list;
+    $self->add_method(@_);
 }
 
-# ... however the items in statis (attributes & method modifiers)
-# can be removed and added to through this API
+## ------------------------------------------------------------------
+## role construction
+## ------------------------------------------------------------------
 
-# attributes
+sub apply {
+    my ($self, $other, @args) = @_;
 
-sub add_attribute {
-    my $self = shift;
-    my $name = shift;
-    my $attr_desc;
-    if (scalar @_ == 1 && ref($_[0]) eq 'HASH') {
-        $attr_desc = $_[0];
+    (blessed($other))
+        || Moose->throw_error("You must pass in an blessed instance");
+
+    my $application_class;
+    if ($other->isa('Moose::Meta::Role')) {
+        $application_class = $self->application_to_role_class;
+    }
+    elsif ($other->isa('Moose::Meta::Class')) {
+        $application_class = $self->application_to_class_class;
     }
     else {
-        $attr_desc = { @_ };
+        $application_class = $self->application_to_instance_class;
     }
-    $self->get_attribute_map->{$name} = $attr_desc;
-}
 
-sub has_attribute {
-    my ($self, $name) = @_;
-    exists $self->get_attribute_map->{$name} ? 1 : 0;
+    Class::MOP::load_class($application_class);
+    return $application_class->new(@args)->apply($self, $other);
 }
 
-sub get_attribute {
-    my ($self, $name) = @_;
-    $self->get_attribute_map->{$name}
-}
+sub composition_class_roles { }
 
-sub remove_attribute {
-    my ($self, $name) = @_;
-    delete $self->get_attribute_map->{$name}
-}
+sub combine {
+    my ($class, @role_specs) = @_;
 
-sub get_attribute_list {
-    my ($self) = @_;
-    keys %{$self->get_attribute_map};
-}
+    require Moose::Meta::Role::Composite;
 
-# method modifiers
+    my (@roles, %role_params);
+    while (@role_specs) {
+        my ($role, $params) = @{ splice @role_specs, 0, 1 };
+        my $requested_role
+            = blessed $role
+            ? $role
+            : Class::MOP::class_of($role);
 
-# mimic the metaclass API
-sub add_before_method_modifier { (shift)->_add_method_modifier('before', @_) }
-sub add_around_method_modifier { (shift)->_add_method_modifier('around', @_) }
-sub add_after_method_modifier  { (shift)->_add_method_modifier('after',  @_) }
+        my $actual_role = $requested_role->_role_for_combination($params);
+        push @roles => $actual_role;
 
-sub _add_method_modifier {
-    my ($self, $modifier_type, $method_name, $method) = @_;
-    my $accessor = "get_${modifier_type}_method_modifiers_map";
-    $self->$accessor->{$method_name} = [] 
-        unless exists $self->$accessor->{$method_name};
-    my $modifiers = $self->$accessor->{$method_name};
-    # NOTE:
-    # check to see that we aren't adding the 
-    # same code twice. We err in favor of the 
-    # first on here, this may not be as expected
-    foreach my $modifier (@{$modifiers}) {
-        return if $modifier == $method;
+        next unless defined $params;
+        $role_params{$actual_role->name} = $params;
     }
-    push @{$modifiers} => $method;
+
+    my $c = Moose::Meta::Role::Composite->new(roles => \@roles);
+    return $c->apply_params(\%role_params);
 }
 
-sub add_override_method_modifier {
-    my ($self, $method_name, $method) = @_;
-    (!$self->has_method($method_name))
-        || confess "Cannot add an override of method '$method_name' " . 
-                   "because there is a local version of '$method_name'";
-    $self->get_override_method_modifiers_map->{$method_name} = $method;    
+sub _role_for_combination {
+    my ($self, $params) = @_;
+    return $self;
 }
 
-sub has_before_method_modifiers { (shift)->_has_method_modifiers('before', @_) }
-sub has_around_method_modifiers { (shift)->_has_method_modifiers('around', @_) }
-sub has_after_method_modifiers  { (shift)->_has_method_modifiers('after',  @_) }
+sub create {
+    my ( $role, $package_name, %options ) = @_;
 
-# override just checks for one,.. 
-# but we can still re-use stuff
-sub has_override_method_modifier { (shift)->_has_method_modifiers('override',  @_) }
+    $options{package} = $package_name;
 
-sub _has_method_modifiers {
-    my ($self, $modifier_type, $method_name) = @_;
-    my $accessor = "get_${modifier_type}_method_modifiers_map";   
-    # NOTE:
-    # for now we assume that if it exists,.. 
-    # it has at least one modifier in it
-    (exists $self->$accessor->{$method_name}) ? 1 : 0;
-}
+    (ref $options{attributes} eq 'HASH')
+        || confess "You must pass a HASH ref of attributes"
+            if exists $options{attributes};
 
-sub get_before_method_modifiers { (shift)->_get_method_modifiers('before', @_) }
-sub get_around_method_modifiers { (shift)->_get_method_modifiers('around', @_) }
-sub get_after_method_modifiers  { (shift)->_get_method_modifiers('after',  @_) }
+    (ref $options{methods} eq 'HASH')
+        || confess "You must pass a HASH ref of methods"
+            if exists $options{methods};
 
-sub _get_method_modifiers {
-    my ($self, $modifier_type, $method_name) = @_;
-    my $accessor = "get_${modifier_type}_method_modifiers_map";
-    @{$self->$accessor->{$method_name}};
-}
+    my (%initialize_options) = %options;
+    delete @initialize_options{qw(
+        package
+        attributes
+        methods
+        version
+        authority
+    )};
 
-sub get_override_method_modifier {
-    my ($self, $method_name) = @_;
-    $self->get_override_method_modifiers_map->{$method_name};    
-}
+    my $meta = $role->initialize( $package_name => %initialize_options );
 
-sub get_method_modifier_list {
-    my ($self, $modifier_type) = @_;
-    my $accessor = "get_${modifier_type}_method_modifiers_map";    
-    keys %{$self->$accessor};
-}
+    $meta->_instantiate_module( $options{version}, $options{authority} );
 
-## applying a role to a class ...
+    # FIXME totally lame
+    $meta->add_method('meta' => sub {
+        $role->initialize(ref($_[0]) || $_[0]);
+    });
 
-sub _check_excluded_roles {
-    my ($self, $other) = @_;
-    if ($other->excludes_role($self->name)) {
-        confess "Conflict detected: " . $other->name . " excludes role '" . $self->name . "'";
-    }
-    foreach my $excluded_role_name ($self->get_excluded_roles_list) {
-        if ($other->does_role($excluded_role_name)) { 
-            confess "The class " . $other->name . " does the excluded role '$excluded_role_name'";
+    if (exists $options{attributes}) {
+        foreach my $attribute_name (keys %{$options{attributes}}) {
+            my $attr = $options{attributes}->{$attribute_name};
+            $meta->add_attribute(
+                $attribute_name => blessed $attr ? $attr : %{$attr} );
         }
-        else {
-            if ($other->isa('Moose::Meta::Role')) {
-                $other->add_excluded_roles($excluded_role_name);
-            }
-            # else -> ignore it :) 
-        }
-    }    
-}
-
-sub _check_required_methods {
-    my ($self, $other) = @_;
-    # NOTE:
-    # we might need to move this down below the 
-    # the attributes so that we can require any 
-    # attribute accessors. However I am thinking 
-    # that maybe those are somehow exempt from 
-    # the require methods stuff.  
-    foreach my $required_method_name ($self->get_required_method_list) {
-        unless ($other->has_method($required_method_name)) {
-            if ($other->isa('Moose::Meta::Role')) {
-                $other->add_required_methods($required_method_name);
-            }
-            else {
-                confess "'" . $self->name . "' requires the method '$required_method_name' " . 
-                        "to be implemented by '" . $other->name . "'";
-            }
-        }
-        else {
-            # NOTE:
-            # we need to make sure that the method is 
-            # not a method modifier, because those do 
-            # not satisfy the requirements ...
-            my $method = $other->get_method($required_method_name);
-            # check if it is an override or a generated accessor ..
-            (!$method->isa('Moose::Meta::Method::Overriden') &&
-             !$method->isa('Class::MOP::Attribute::Accessor'))
-                || confess "'" . $self->name . "' requires the method '$required_method_name' " . 
-                           "to be implemented by '" . $other->name . "', the method is only a method modifier";
-            # before/after/around methods are a little trickier
-            # since we wrap the original local method (if applicable)
-            # so we need to check if the original wrapped method is 
-            # from the same package, and not a wrap of the super method 
-            if ($method->isa('Class::MOP::Method::Wrapped')) {
-                ($method->get_original_method->package_name eq $other->name)
-                    || confess "'" . $self->name . "' requires the method '$required_method_name' " . 
-                               "to be implemented by '" . $other->name . "', the method is only a method modifier";            
-            }
-        }
-    }    
-}
+    }
 
-sub _apply_attributes {
-    my ($self, $other) = @_;    
-    foreach my $attribute_name ($self->get_attribute_list) {
-        # it if it has one already
-        if ($other->has_attribute($attribute_name) &&
-            # make sure we haven't seen this one already too
-            $other->get_attribute($attribute_name) != $self->get_attribute($attribute_name)) {
-            # see if we are being composed  
-            # into a role or not
-            if ($other->isa('Moose::Meta::Role')) {                
-                # all attribute conflicts between roles 
-                # result in an immediate fatal error 
-                confess "Role '" . $self->name . "' has encountered an attribute conflict " . 
-                        "during composition. This is fatal error and cannot be disambiguated.";
-            }
-            else {
-                # but if this is a class, we 
-                # can safely skip adding the 
-                # attribute to the class
-                next;
-            }
-        }
-        else {
-            $other->add_attribute(
-                $attribute_name,
-                $self->get_attribute($attribute_name)
-            );
+    if (exists $options{methods}) {
+        foreach my $method_name (keys %{$options{methods}}) {
+            $meta->add_method($method_name, $options{methods}->{$method_name});
         }
-    }    
-}
+    }
 
-sub _apply_methods {
-    my ($self, $other) = @_;   
-    foreach my $method_name ($self->get_method_list) {
-        # it if it has one already
-        if ($other->has_method($method_name) &&
-            # and if they are not the same thing ...
-            $other->get_method($method_name) != $self->get_method($method_name)) {
-            # see if we are composing into a role
-            if ($other->isa('Moose::Meta::Role')) { 
-                # method conflicts between roles result 
-                # in the method becoming a requirement
-                $other->add_required_methods($method_name);
-                # NOTE:
-                # we have to remove the method from our 
-                # role, if this is being called from combine()
-                # which means the meta is an anon class
-                # this *may* cause problems later, but it 
-                # is probably fairly safe to assume that 
-                # anon classes will only be used internally
-                # or by people who know what they are doing
-                $other->_role_meta->remove_method($method_name)
-                    if $other->_role_meta->name =~ /__ANON__/;
-            }
-            else {
-                next;
-            }
-        }
-        else {
-            # add it, although it could be overriden 
-            $other->alias_method(
-                $method_name,
-                $self->get_method($method_name)
-            );
-        }
-    }     
-}
+    Class::MOP::weaken_metaclass($meta->name)
+        if $meta->is_anon_role;
 
-sub _apply_override_method_modifiers {
-    my ($self, $other) = @_;    
-    foreach my $method_name ($self->get_method_modifier_list('override')) {
-        # it if it has one already then ...
-        if ($other->has_method($method_name)) {
-            # if it is being composed into another role
-            # we have a conflict here, because you cannot 
-            # combine an overriden method with a locally
-            # defined one 
-            if ($other->isa('Moose::Meta::Role')) { 
-                confess "Role '" . $self->name . "' has encountered an 'override' method conflict " . 
-                        "during composition (A local method of the same name as been found). This " . 
-                        "is fatal error.";
-            }
-            else {
-                # if it is a class, then we 
-                # just ignore this here ...
-                next;
-            }
-        }
-        else {
-            # if no local method is found, then we 
-            # must check if we are a role or class
-            if ($other->isa('Moose::Meta::Role')) { 
-                # if we are a role, we need to make sure 
-                # we dont have a conflict with the role 
-                # we are composing into
-                if ($other->has_override_method_modifier($method_name) &&
-                    $other->get_override_method_modifier($method_name) != $self->get_override_method_modifier($method_name)) {
-                    confess "Role '" . $self->name . "' has encountered an 'override' method conflict " . 
-                            "during composition (Two 'override' methods of the same name encountered). " . 
-                            "This is fatal error.";
-                }
-                else {   
-                    # if there is no conflict,
-                    # just add it to the role  
-                    $other->add_override_method_modifier(
-                        $method_name, 
-                        $self->get_override_method_modifier($method_name)
-                    );                    
-                }
-            }
-            else {
-                # if this is not a role, then we need to 
-                # find the original package of the method
-                # so that we can tell the class were to 
-                # find the right super() method
-                my $method = $self->get_override_method_modifier($method_name);
-                my $package = svref_2object($method)->GV->STASH->NAME;
-                # if it is a class, we just add it
-                $other->add_override_method_modifier($method_name, $method, $package);
-            }
-        }
-    }    
+    return $meta;
 }
 
-sub _apply_method_modifiers {
-    my ($self, $modifier_type, $other) = @_;    
-    my $add = "add_${modifier_type}_method_modifier";
-    my $get = "get_${modifier_type}_method_modifiers";    
-    foreach my $method_name ($self->get_method_modifier_list($modifier_type)) {
-        $other->$add(
-            $method_name,
-            $_
-        ) foreach $self->$get($method_name);
-    }    
-}
+# anonymous roles. most of it is copied straight out of Class::MOP::Class.
+# an intrepid hacker might find great riches if he unifies this code with that
+# code in Class::MOP::Module or Class::MOP::Package
+{
+    # NOTE:
+    # this should be sufficient, if you have a
+    # use case where it is not, write a test and
+    # I will change it.
+    my $ANON_ROLE_SERIAL = 0;
 
-sub _apply_before_method_modifiers { (shift)->_apply_method_modifiers('before' => @_) }
-sub _apply_around_method_modifiers { (shift)->_apply_method_modifiers('around' => @_) }
-sub _apply_after_method_modifiers  { (shift)->_apply_method_modifiers('after'  => @_) }
+    # NOTE:
+    # we need a sufficiently annoying prefix
+    # this should suffice for now, this is
+    # used in a couple of places below, so
+    # need to put it up here for now.
+    my $ANON_ROLE_PREFIX = 'Moose::Meta::Role::__ANON__::SERIAL::';
+
+    sub is_anon_role {
+        my $self = shift;
+        no warnings 'uninitialized';
+        $self->name =~ /^$ANON_ROLE_PREFIX/;
+    }
 
-sub apply {
-    my ($self, $other) = @_;
-    
-    $self->_check_excluded_roles($other);
-    $self->_check_required_methods($other);  
-
-    $self->_apply_attributes($other);         
-    $self->_apply_methods($other);         
-         
-    $self->_apply_override_method_modifiers($other);                  
-    $self->_apply_before_method_modifiers($other);                  
-    $self->_apply_around_method_modifiers($other);                  
-    $self->_apply_after_method_modifiers($other);                              
-    
-    $other->add_role($self);
-}
+    sub create_anon_role {
+        my ($role, %options) = @_;
+        my $package_name = $ANON_ROLE_PREFIX . ++$ANON_ROLE_SERIAL;
+        return $role->create($package_name, %options);
+    }
 
-sub combine {
-    my ($class, @roles) = @_;
-    
-    my $combined = $class->new(
-        ':role_meta' => Moose::Meta::Class->create_anon_class()
-    );
-    
-    foreach my $role (@roles) {
-        $role->apply($combined);
+    # NOTE:
+    # this will only get called for
+    # anon-roles, all other calls
+    # are assumed to occur during
+    # global destruction and so don't
+    # really need to be handled explicitly
+    sub DESTROY {
+        my $self = shift;
+
+        return if in_global_destruction(); # it'll happen soon anyway and this just makes things more complicated
+
+        no warnings 'uninitialized';
+        return unless $self->name =~ /^$ANON_ROLE_PREFIX/;
+
+        # XXX: is this necessary for us? I don't understand what it's doing
+        # -sartak
+
+        # Moose does a weird thing where it replaces the metaclass for
+        # class when fixing metaclass incompatibility. In that case,
+        # we don't want to clean out the namespace now. We can detect
+        # that because Moose will explicitly update the singleton
+        # cache in Class::MOP.
+        #my $current_meta = Class::MOP::get_metaclass_by_name($self->name);
+        #return if $current_meta ne $self;
+
+        my ($serial_id) = ($self->name =~ /^$ANON_ROLE_PREFIX(\d+)/);
+        no strict 'refs';
+        foreach my $key (keys %{$ANON_ROLE_PREFIX . $serial_id}) {
+            delete ${$ANON_ROLE_PREFIX . $serial_id}{$key};
+        }
+        delete ${'main::' . $ANON_ROLE_PREFIX}{$serial_id . '::'};
     }
-    
-    $combined->_clean_up_required_methods;   
-    
-    return $combined;
 }
 
-package Moose::Meta::Role::Method;
+#####################################################################
+## NOTE:
+## This is Moose::Meta::Role as defined by Moose (plus the use of
+## MooseX::AttributeHelpers module). It is here as a reference to
+## make it easier to see what is happening above with all the meta
+## programming. - SL
+#####################################################################
+#
+# has 'roles' => (
+#     metaclass => 'Array',
+#     reader    => 'get_roles',
+#     isa       => 'ArrayRef[Moose::Meta::Role]',
+#     default   => sub { [] },
+#     provides  => {
+#         'push' => 'add_role',
+#     }
+# );
+#
+# has 'excluded_roles_map' => (
+#     metaclass => 'Hash',
+#     reader    => 'get_excluded_roles_map',
+#     isa       => 'HashRef[Str]',
+#     provides  => {
+#         # Not exactly set, cause it sets multiple
+#         'set'    => 'add_excluded_roles',
+#         'keys'   => 'get_excluded_roles_list',
+#         'exists' => 'excludes_role',
+#     }
+# );
+#
+# has 'required_methods' => (
+#     metaclass => 'Hash',
+#     reader    => 'get_required_methods_map',
+#     isa       => 'HashRef[Moose::Meta::Role::Method::Required]',
+#     provides  => {
+#         # not exactly set, or delete since it works for multiple
+#         'set'    => 'add_required_methods',
+#         'delete' => 'remove_required_methods',
+#         'keys'   => 'get_required_method_list',
+#         'exists' => 'requires_method',
+#     }
+# );
+#
+# # the before, around and after modifiers are
+# # HASH keyed by method-name, with ARRAY of
+# # CODE refs to apply in that order
+#
+# has 'before_method_modifiers' => (
+#     metaclass => 'Hash',
+#     reader    => 'get_before_method_modifiers_map',
+#     isa       => 'HashRef[ArrayRef[CodeRef]]',
+#     provides  => {
+#         'keys'   => 'get_before_method_modifiers',
+#         'exists' => 'has_before_method_modifiers',
+#         # This actually makes sure there is an
+#         # ARRAY at the given key, and pushed onto
+#         # it. It also checks for duplicates as well
+#         # 'add'  => 'add_before_method_modifier'
+#     }
+# );
+#
+# has 'after_method_modifiers' => (
+#     metaclass => 'Hash',
+#     reader    =>'get_after_method_modifiers_map',
+#     isa       => 'HashRef[ArrayRef[CodeRef]]',
+#     provides  => {
+#         'keys'   => 'get_after_method_modifiers',
+#         'exists' => 'has_after_method_modifiers',
+#         # This actually makes sure there is an
+#         # ARRAY at the given key, and pushed onto
+#         # it. It also checks for duplicates as well
+#         # 'add'  => 'add_after_method_modifier'
+#     }
+# );
+#
+# has 'around_method_modifiers' => (
+#     metaclass => 'Hash',
+#     reader    =>'get_around_method_modifiers_map',
+#     isa       => 'HashRef[ArrayRef[CodeRef]]',
+#     provides  => {
+#         'keys'   => 'get_around_method_modifiers',
+#         'exists' => 'has_around_method_modifiers',
+#         # This actually makes sure there is an
+#         # ARRAY at the given key, and pushed onto
+#         # it. It also checks for duplicates as well
+#         # 'add'  => 'add_around_method_modifier'
+#     }
+# );
+#
+# # override is similar to the other modifiers
+# # except that it is not an ARRAY of code refs
+# # but instead just a single name->code mapping
+#
+# has 'override_method_modifiers' => (
+#     metaclass => 'Hash',
+#     reader    =>'get_override_method_modifiers_map',
+#     isa       => 'HashRef[CodeRef]',
+#     provides  => {
+#         'keys'   => 'get_override_method_modifier',
+#         'exists' => 'has_override_method_modifier',
+#         'add'    => 'add_override_method_modifier', # checks for local method ..
+#     }
+# );
+#
+#####################################################################
 
-use strict;
-use warnings;
-
-our $VERSION = '0.01';
-
-use base 'Class::MOP::Method';
 
 1;
 
@@ -550,154 +683,260 @@ Moose::Meta::Role - The Moose Role metaclass
 
 =head1 DESCRIPTION
 
-Moose's Roles are being actively developed, please see L<Moose::Role> 
-for more information. For the most part, this has no user-serviceable 
-parts inside. It's API is still subject to some change (although 
-probably not that much really).
+This class is a subclass of L<Class::MOP::Module> that provides
+additional Moose-specific functionality.
+
+It's API looks a lot like L<Moose::Meta::Class>, but internally it
+implements many things differently. This may change in the future.
+
+=head1 INHERITANCE
+
+C<Moose::Meta::Role> is a subclass of L<Class::MOP::Module>.
 
 =head1 METHODS
 
+=head2 Construction
+
 =over 4
 
-=item B<meta>
+=item B<< Moose::Meta::Role->initialize($role_name) >>
 
-=item B<new>
+This method creates a new role object with the provided name.
 
-=item B<apply>
+=item B<< Moose::Meta::Role->combine( [ $role => { ... } ], [ $role ], ... ) >>
 
-=item B<combine>
+This method accepts a list of array references. Each array reference
+should contain a role name or L<Moose::Meta::Role> object as its first element. The second element is
+an optional hash reference. The hash reference can contain C<-excludes>
+and C<-alias> keys to control how methods are composed from the role.
 
-=back
+The return value is a new L<Moose::Meta::Role::Composite> that
+represents the combined roles.
 
-=over 4
+=item B<< $metarole->composition_class_roles >>
+
+When combining multiple roles using C<combine>, this method is used to obtain a
+list of role names to be applied to the L<Moose::Meta::Role::Composite>
+instance returned by C<combine>. The default implementation returns an empty
+list. Extensions that need to hook into role combination may wrap this method
+to return additional role names.
+
+=item B<< Moose::Meta::Role->create($name, %options) >>
+
+This method is identical to the L<Moose::Meta::Class> C<create>
+method.
+
+=item B<< Moose::Meta::Role->create_anon_role >>
 
-=item B<name>
+This method is identical to the L<Moose::Meta::Class>
+C<create_anon_class> method.
 
-=item B<version>
+=item B<< $metarole->is_anon_role >>
 
-=item B<role_meta>
+Returns true if the role is an anonymous role.
 
 =back
 
+=head2 Role application
+
 =over 4
 
-=item B<get_roles>
+=item B<< $metarole->apply( $thing, @options ) >>
+
+This method applies a role to the given C<$thing>. That can be another
+L<Moose::Meta::Role>, object, a L<Moose::Meta::Class> object, or a
+(non-meta) object instance.
 
-=item B<add_role>
+The options are passed directly to the constructor for the appropriate
+L<Moose::Meta::Role::Application> subclass.
 
-=item B<does_role>
+Note that this will apply the role even if the C<$thing> in question already
+C<does> this role.  L<Moose::Util/does_role> is a convenient wrapper for
+finding out if role application is necessary.
 
 =back
 
+=head2 Roles and other roles
+
 =over 4
 
-=item B<add_excluded_roles>
+=item B<< $metarole->get_roles >>
 
-=item B<excludes_role>
+This returns an array reference of roles which this role does. This
+list may include duplicates.
 
-=item B<get_excluded_roles_list>
+=item B<< $metarole->calculate_all_roles >>
 
-=item B<get_excluded_roles_map>
+This returns a I<unique> list of all roles that this role does, and
+all the roles that its roles do.
 
-=back
+=item B<< $metarole->does_role($role) >>
 
-=over 4
+Given a role I<name> or L<Moose::Meta::Role> object, returns true if this role
+does the given role.
+
+=item B<< $metarole->add_role($role) >>
+
+Given a L<Moose::Meta::Role> object, this adds the role to the list of
+roles that the role does.
 
-=item B<get_method>
+=item B<< $metarole->get_excluded_roles_list >>
 
-=item B<has_method>
+Returns a list of role names which this role excludes.
 
-=item B<alias_method>
+=item B<< $metarole->excludes_role($role_name) >>
 
-=item B<get_method_list>
+Given a role I<name>, returns true if this role excludes the named
+role.
+
+=item B<< $metarole->add_excluded_roles(@role_names) >>
+
+Given one or more role names, adds those roles to the list of excluded
+roles.
 
 =back
 
+=head2 Methods
+
+The methods for dealing with a role's methods are all identical in API
+and behavior to the same methods in L<Class::MOP::Class>.
+
 =over 4
 
-=item B<add_attribute>
+=item B<< $metarole->method_metaclass >>
+
+Returns the method metaclass name for the role. This defaults to
+L<Moose::Meta::Role::Method>.
+
+=item B<< $metarole->get_method($name) >>
 
-=item B<has_attribute>
+=item B<< $metarole->has_method($name) >>
 
-=item B<get_attribute>
+=item B<< $metarole->add_method( $name, $body ) >>
 
-=item B<get_attribute_list>
+=item B<< $metarole->get_method_list >>
 
-=item B<get_attribute_map>
+=item B<< $metarole->find_method_by_name($name) >>
 
-=item B<remove_attribute>
+These methods are all identical to the methods of the same name in
+L<Class::MOP::Package>
 
 =back
 
+=head2 Attributes
+
+As with methods, the methods for dealing with a role's attribute are
+all identical in API and behavior to the same methods in
+L<Class::MOP::Class>.
+
+However, attributes stored in this class are I<not> stored as
+objects. Rather, the attribute definition is stored as a hash
+reference. When a role is composed into a class, this hash reference
+is passed directly to the metaclass's C<add_attribute> method.
+
+This is quite likely to change in the future.
+
 =over 4
 
-=item B<add_required_methods>
+=item B<< $metarole->get_attribute($attribute_name) >>
 
-=item B<remove_required_methods>
+=item B<< $metarole->has_attribute($attribute_name) >>
 
-=item B<get_required_method_list>
+=item B<< $metarole->get_attribute_list >>
 
-=item B<get_required_methods_map>
+=item B<< $metarole->add_attribute($name, %options) >>
 
-=item B<requires_method>
+=item B<< $metarole->remove_attribute($attribute_name) >>
 
 =back
 
+=head2 Required methods
+
 =over 4
 
-=item B<add_after_method_modifier>
+=item B<< $metarole->get_required_method_list >>
 
-=item B<add_around_method_modifier>
+Returns the list of methods required by the role.
 
-=item B<add_before_method_modifier>
+=item B<< $metarole->requires_method($name) >>
 
-=item B<add_override_method_modifier>
+Returns true if the role requires the named method.
 
-=over 4
+=item B<< $metarole->add_required_methods(@names) >>
 
-=back
+Adds the named methods to the role's list of required methods.
 
-=item B<has_after_method_modifiers>
+=item B<< $metarole->remove_required_methods(@names) >>
 
-=item B<has_around_method_modifiers>
+Removes the named methods from the role's list of required methods.
 
-=item B<has_before_method_modifiers>
+=item B<< $metarole->add_conflicting_method(%params) >>
 
-=item B<has_override_method_modifier>
+Instantiate the parameters as a L<Moose::Meta::Role::Method::Conflicting>
+object, then add it to the required method list.
+
+=back
+
+=head2 Method modifiers
+
+These methods act like their counterparts in L<Class::Mop::Class> and
+L<Moose::Meta::Class>.
+
+However, method modifiers are simply stored internally, and are not
+applied until the role itself is applied to a class.
 
 =over 4
 
-=back
+=item B<< $metarole->add_after_method_modifier($method_name, $method) >>
 
-=item B<get_after_method_modifiers>
+=item B<< $metarole->add_around_method_modifier($method_name, $method) >>
 
-=item B<get_around_method_modifiers>
+=item B<< $metarole->add_before_method_modifier($method_name, $method) >>
 
-=item B<get_before_method_modifiers>
+=item B<< $metarole->add_override_method_modifier($method_name, $method) >>
 
-=item B<get_method_modifier_list>
+These methods all add an appropriate modifier to the internal list of
+modifiers.
 
-=over 4
+=item B<< $metarole->has_after_method_modifiers >>
 
-=back
+=item B<< $metarole->has_around_method_modifiers >>
+
+=item B<< $metarole->has_before_method_modifiers >>
+
+=item B<< $metarole->has_override_method_modifier >>
+
+Return true if the role has any modifiers of the given type.
+
+=item B<< $metarole->get_after_method_modifiers($method_name) >>
+
+=item B<< $metarole->get_around_method_modifiers($method_name) >>
 
-=item B<get_override_method_modifier>
+=item B<< $metarole->get_before_method_modifiers($method_name) >>
 
-=item B<get_after_method_modifiers_map>
+Given a method name, returns a list of the appropriate modifiers for
+that method.
 
-=item B<get_around_method_modifiers_map>
+=item B<< $metarole->get_override_method_modifier($method_name) >>
+
+Given a method name, returns the override method modifier for that
+method, if it has one.
+
+=back
+
+=head2 Introspection
+
+=over 4
 
-=item B<get_before_method_modifiers_map>
+=item B<< Moose::Meta::Role->meta >>
 
-=item B<get_override_method_modifiers_map>
+This will return a L<Class::MOP::Class> instance for this class.
 
 =back
 
 =head1 BUGS
 
-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.
+See L<Moose/BUGS> for details on reporting bugs.
 
 =head1 AUTHOR
 
@@ -705,11 +944,11 @@ Stevan Little E<lt>stevan@iinteractive.comE<gt>
 
 =head1 COPYRIGHT AND LICENSE
 
-Copyright 2006 by Infinity Interactive, Inc.
+Copyright 2006-2010 by Infinity Interactive, Inc.
 
 L<http://www.iinteractive.com>
 
 This library is free software; you can redistribute it and/or modify
-it under the same terms as Perl itself. 
+it under the same terms as Perl itself.
 
 =cut