refactor in progress, beware (still passing all my tests though :P)
[gitmo/Moose.git] / lib / Moose / Meta / Role.pm
index 3b938cd..60a9727 100644 (file)
@@ -5,89 +5,249 @@ use strict;
 use warnings;
 use metaclass;
 
+use Sub::Name    'subname';
 use Carp         'confess';
-use Scalar::Util 'blessed';
-use B            'svref_2object';
+use Scalar::Util 'blessed', 'reftype';
+
+our $VERSION   = '0.12';
+our $AUTHORITY = 'cpan:STEVAN';
 
 use Moose::Meta::Class;
+use Moose::Meta::Role::Method;
+use Moose::Meta::Role::Method::Required;
 
-our $VERSION = '0.03';
+use base 'Class::MOP::Module';
 
-## Attributes
+## ------------------------------------------------------------------
+## 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
+## ------------------------------------------------------------------
 
-## the meta for the role package
+my $META = __PACKAGE__->meta;
 
-__PACKAGE__->meta->add_attribute('_role_meta' => (
-    reader   => '_role_meta',
-    init_arg => ':role_meta'
-));
+## ------------------------------------------------------------------
+## attributes ...
 
-## roles
+# 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_list  => 'get_excluded_roles_list',
+            existence => 'excludes_role',
+        }
+    },
+    {
+        name        => 'required_methods',
+        attr_reader => 'get_required_methods_map',
+        methods     => {
+            add       => 'add_required_methods',
+            remove    => 'remove_required_methods',
+            get_list  => 'get_required_method_list',
+            existence => 'requires_method',
+        }
+    },
+    {
+        name        => 'attribute_map',
+        attr_reader => 'get_attribute_map',
+        methods     => {
+            get       => 'get_attribute',
+            get_list  => 'get_attribute_list',
+            existence => 'has_attribute',
+            remove    => 'remove_attribute',
+        }
+    }
+) {
+
+    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_list} => sub {
+        my ($self) = @_;
+        keys %{$self->$attr_reader};
+    }) if exists $methods->{get_list};
+
+    $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};
+}
 
-__PACKAGE__->meta->add_attribute('roles' => (
-    reader  => 'get_roles',
-    default => sub { [] }
-));
+## some things don't always fit, so they go here ...
 
-## excluded roles
+sub add_attribute {
+    my $self = shift;
+    my $name = shift;
+    my $attr_desc;
+    if (scalar @_ == 1 && ref($_[0]) eq 'HASH') {
+        $attr_desc = $_[0];
+    }
+    else {
+        $attr_desc = { @_ };
+    }
+    $self->get_attribute_map->{$name} = $attr_desc;
+}
 
-__PACKAGE__->meta->add_attribute('excluded_roles_map' => (
-    reader  => 'get_excluded_roles_map',
-    default => sub { {} }
-));
+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);
+    }
+}
 
-## attributes
+## ------------------------------------------------------------------
+## method modifiers
 
-__PACKAGE__->meta->add_attribute('attribute_map' => (
-    reader   => 'get_attribute_map',
-    default  => sub { {} }
-));
+# 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
 
-## required methods
+foreach my $modifier_type (qw[ before around after ]) {
 
-__PACKAGE__->meta->add_attribute('required_methods' => (
-    reader  => 'get_required_methods_map',
-    default => sub { {} }
-));
+    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};
+        @{$self->$attr_reader->{$method_name}};
+    });
+
+    $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;
+    });
 
-## method modifiers
+    $META->add_method("add_${modifier_type}_method_modifier" => sub {
+        my ($self, $method_name, $method) = @_;
 
-__PACKAGE__->meta->add_attribute('before_method_modifiers' => (
-    reader  => 'get_before_method_modifiers_map',
-    default => sub { {} } # (<name> => [ (CODE) ])
-));
+        $self->$attr_reader->{$method_name} = []
+            unless exists $self->$attr_reader->{$method_name};
 
-__PACKAGE__->meta->add_attribute('after_method_modifiers' => (
-    reader  => 'get_after_method_modifiers_map',
-    default => sub { {} } # (<name> => [ (CODE) ])
-));
+        my $modifiers = $self->$attr_reader->{$method_name};
 
-__PACKAGE__->meta->add_attribute('around_method_modifiers' => (
-    reader  => 'get_around_method_modifiers_map',
-    default => sub { {} } # (<name> => [ (CODE) ])
-));
+        # 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;
+        }
 
-__PACKAGE__->meta->add_attribute('override_method_modifiers' => (
+        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))
+        || 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 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};
+}
+
+## ------------------------------------------------------------------
 ## subroles
 
+__PACKAGE__->meta->add_attribute('roles' => (
+    reader  => 'get_roles',
+    default => sub { [] }
+));
+
 sub add_role {
     my ($self, $role) = @_;
     (blessed($role) && $role->isa('Moose::Meta::Role'))
@@ -95,6 +255,16 @@ sub add_role {
     push @{$self->get_roles} => $role;
 }
 
+sub calculate_all_roles {
+    my $self = shift;
+    my %seen;
+    grep {
+        !$seen{$_->name}++
+    } ($self, map {
+                  $_->calculate_all_roles
+              } @{ $self->get_roles });
+}
+
 sub does_role {
     my ($self, $role_name) = @_;
     (defined $role_name)
@@ -108,410 +278,478 @@ sub does_role {
     return 0;
 }
 
-## excluded roles
+## ------------------------------------------------------------------
+## methods
 
-sub add_excluded_roles {
-    my ($self, @excluded_role_names) = @_;
-    $self->get_excluded_roles_map->{$_} = undef foreach @excluded_role_names;
-}
+sub method_metaclass { 'Moose::Meta::Role::Method' }
 
-sub get_excluded_roles_list {
-    my ($self) = @_;
-    keys %{$self->get_excluded_roles_map};
-}
+sub get_method_map {
+    my $self = shift;
+    my $map  = {};
 
-sub excludes_role {
-    my ($self, $role_name) = @_;
-    exists $self->get_excluded_roles_map->{$role_name} ? 1 : 0;
-}
+    my $role_name        = $self->name;
+    my $method_metaclass = $self->method_metaclass;
 
-## required methods
+    foreach my $symbol ($self->list_all_package_symbols('CODE')) {
 
-sub add_required_methods {
-    my ($self, @methods) = @_;
-    $self->get_required_methods_map->{$_} = undef foreach @methods;
-}
+        my $code = $self->get_package_symbol('&' . $symbol);
+
+        my ($pkg, $name) = Class::MOP::get_code_info($code);
 
-sub remove_required_methods {
-    my ($self, @methods) = @_;
-    delete $self->get_required_methods_map->{$_} foreach @methods;
+        if ($pkg->can('meta')
+            # NOTE:
+            # we don't know what ->meta we are calling
+            # here, so we need to be careful cause it
+            # just might blow up at us, or just complain
+            # loudly (in the case of Curses.pm) so we
+            # just be a little overly cautious here.
+            # - SL
+            && eval { no warnings; blessed($pkg->meta) }
+            && $pkg->meta->isa('Moose::Meta::Role')) {
+            my $role = $pkg->meta->name;
+            next unless $self->does_role($role);
+        }
+        else {
+            next if ($pkg  || '') ne $role_name &&
+                    ($name || '') ne '__ANON__';
+        }
+
+        $map->{$symbol} = $method_metaclass->wrap($code);
+    }
+
+    return $map;    
 }
 
-sub get_required_method_list {
-    my ($self) = @_;
-    keys %{$self->get_required_methods_map};
+sub get_method { 
+    my ($self, $name) = @_;
+    $self->get_method_map->{$name}
 }
 
-sub requires_method {
-    my ($self, $method_name) = @_;
-    exists $self->get_required_methods_map->{$method_name} ? 1 : 0;
+sub has_method {
+    my ($self, $name) = @_;
+    exists $self->get_method_map->{$name} ? 1 : 0
 }
 
-sub _clean_up_required_methods {
+sub find_method_by_name { (shift)->get_method(@_) }
+
+sub get_method_list {
     my $self = shift;
-    foreach my $method ($self->get_required_method_list) {
-        $self->remove_required_methods($method)
-            if $self->has_method($method);
-    } 
+    grep { !/^meta$/ } keys %{$self->get_method_map};
 }
 
-## methods
+sub alias_method {
+    my ($self, $method_name, $method) = @_;
+    (defined $method_name && $method_name)
+        || confess "You must define a method name";
 
-# 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;
+    my $body = (blessed($method) ? $method->body : $method);
+    ('CODE' eq (reftype($body) || ''))
+        || confess "Your code block must be a CODE reference";
+
+    $self->add_package_symbol("&${method_name}" => $body);
 }
 
-# ... however the items in statis (attributes & method modifiers)
-# can be removed and added to through this API
+sub reset_package_cache_flag  { () }
+sub update_package_cache_flag { () }
 
-# attributes
+## ------------------------------------------------------------------
+## role construction
+## ------------------------------------------------------------------
 
-sub add_attribute {
-    my ($self, $name, %attr_desc) = @_;
-    $self->get_attribute_map->{$name} = \%attr_desc;
-}
+my $anon_counter = 0;
 
-sub has_attribute {
-    my ($self, $name) = @_;
-    exists $self->get_attribute_map->{$name} ? 1 : 0;
-}
+sub apply {
+    my ($self, $other) = @_;
 
-sub get_attribute {
-    my ($self, $name) = @_;
-    $self->get_attribute_map->{$name}
-}
+    unless ($other->isa('Moose::Meta::Class') || $other->isa('Moose::Meta::Role')) {
 
-sub remove_attribute {
-    my ($self, $name) = @_;
-    delete $self->get_attribute_map->{$name}
-}
+        # Runtime Role mixins
 
-sub get_attribute_list {
-    my ($self) = @_;
-    keys %{$self->get_attribute_map};
-}
+        # FIXME:
+        # We really should do this better, and
+        # cache the results of our efforts so
+        # that we don't need to repeat them.
 
-# method modifiers
+        my $pkg_name = __PACKAGE__ . "::__RUNTIME_ROLE_ANON_CLASS__::" . $anon_counter++;
+        eval "package " . $pkg_name . "; our \$VERSION = '0.00';";
+        die $@ if $@;
 
-# 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 $object = $other;
 
-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;
-    }
-    push @{$modifiers} => $method;
-}
+        $other = Moose::Meta::Class->initialize($pkg_name);
+        $other->superclasses(blessed($object));
 
-sub add_override_method_modifier {
-    my ($self, $method_name, $method) = @_;
-    $self->get_override_method_modifiers_map->{$method_name} = $method;    
-}
+        bless $object => $pkg_name;
+    }
 
-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',  @_) }
+    $self->_check_excluded_roles($other);
+    $self->_check_required_methods($other);
 
-# override just checks for one,.. 
-# but we can still re-use stuff
-sub has_override_method_modifier { (shift)->_has_method_modifiers('override',  @_) }
+    $self->_apply_attributes($other);
+    $self->_apply_methods($other);
 
-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;
-}
+    # we need a clear cache flag too ...
+    $other->reset_package_cache_flag;
 
-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',  @_) }
+    $self->_apply_override_method_modifiers($other);
+    
+    $self->_apply_before_method_modifiers($other);
+    $self->_apply_around_method_modifiers($other);
+    $self->_apply_after_method_modifiers($other);
 
-sub _get_method_modifiers {
-    my ($self, $modifier_type, $method_name) = @_;
-    my $accessor = "get_${modifier_type}_method_modifiers_map";
-    @{$self->$accessor->{$method_name}};
+    $other->add_role($self);
 }
 
-sub get_override_method_modifier {
-    my ($self, $method_name) = @_;
-    $self->get_override_method_modifiers_map->{$method_name};    
+sub combine {
+    my ($class, @roles) = @_;
+    
+    require Moose::Meta::Role::Application::RoleSummation;
+    require Moose::Meta::Role::Composite;    
+    
+    my $c = Moose::Meta::Role::Composite->new(roles => \@roles);
+    Moose::Meta::Role::Application::RoleSummation->new->apply($c);
+    return $c;
 }
 
-sub get_method_modifier_list {
-    my ($self, $modifier_type) = @_;
-    my $accessor = "get_${modifier_type}_method_modifiers_map";    
-    keys %{$self->$accessor};
-}
+## ------------------------------------------------------------------
 
 ## applying a role to a class ...
 
-sub apply {
+sub _check_excluded_roles {
     my ($self, $other) = @_;
-    
     if ($other->excludes_role($self->name)) {
         confess "Conflict detected: " . $other->name . " excludes role '" . $self->name . "'";
     }
-    
-#    warn "... Checking " . $self->name . " for excluded methods";
     foreach my $excluded_role_name ($self->get_excluded_roles_list) {
-#        warn "... Checking if '$excluded_role_name' is done by " . $other->name . " for " . $self->name;
-        if ($other->does_role($excluded_role_name)) { 
+        if ($other->does_role($excluded_role_name)) {
             confess "The class " . $other->name . " does the excluded role '$excluded_role_name'";
         }
         else {
             if ($other->isa('Moose::Meta::Role')) {
-#                warn ">>> The role " . $other->name . " does not do the excluded role '$excluded_role_name', so we are adding it in";
                 $other->add_excluded_roles($excluded_role_name);
             }
-            else {
-#                warn ">>> The class " . $other->name . " does not do the excluded role '$excluded_role_name', so we can just go about our business";                
-            }
+            # 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.  
+    # 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)) {
+
+        unless ($other->find_method_by_name($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' " . 
+                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 
+            # 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";            
-            }
+            my $method = $other->find_method_by_name($required_method_name);
+
+            # check if it is a generated accessor ...
+            (!$method->isa('Class::MOP::Method::Accessor'))
+                || confess "'" . $self->name . "' requires the method '$required_method_name' " .
+                           "to be implemented by '" . $other->name . "', the method is only an attribute accessor";
+
+            # NOTE:
+            # All other tests here have been removed, they were tests
+            # for overriden methods and before/after/around modifiers.
+            # But we realized that for classes any overriden or modified
+            # methods would be backed by a real method of that name
+            # (and therefore meet the requirement). And for roles, the
+            # overriden and modified methods are "in statis" and so would
+            # not show up in this test anyway (and as a side-effect they
+            # would not fufill the requirement, which is exactly what we
+            # want them to do anyway).
+            # - SL
         }
-    }       
-    
+    }
+}
+
+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)) {
-            # see if we are being composed  
+        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')) {
-                
-                # FIXME:
-                # it is possible for these attributes
-                # to actually both be from the same 
-                # origin (some common ancestor role)
-                # so we need to find a way to check this
-                
-                # all attribute conflicts between roles 
-                # result in an immediate fatal error 
-                confess "Role '" . $self->name . "' has encountered an attribute conflict " . 
+                # 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 
+                # but if this is a class, we
+                # can safely skip adding the
                 # attribute to the class
                 next;
             }
         }
         else {
-            # add it, although it could be overriden 
-            $other->add_attribute(
-                $attribute_name,
-                %{$self->get_attribute($attribute_name)}
-            );
+            # NOTE:
+            # this is kinda ugly ...
+            if ($other->isa('Moose::Meta::Class')) {
+                $other->_process_attribute(
+                    $attribute_name,
+                    %{$self->get_attribute($attribute_name)}
+                );
+            }
+            else {
+                $other->add_attribute(
+                    $attribute_name,
+                    $self->get_attribute($attribute_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)) {
+            $other->get_method($method_name)->body != $self->get_method($method_name)->body) {
             # see if we are composing into a role
-            if ($other->isa('Moose::Meta::Role')) { 
-                # method conflicts between roles result 
+            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 
+                # 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 
+                # 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__/;
+                $other->Moose::Meta::Class::remove_method($method_name)
+                    if $other->name =~ /__COMPOSITE_ROLE_SANDBOX__/;
             }
             else {
                 next;
             }
         }
         else {
-            # add it, although it could be overriden 
+            # add it, although it could be overriden
             $other->alias_method(
                 $method_name,
                 $self->get_method($method_name)
             );
         }
-    }    
-    
+    }
+}
+
+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 
+            # 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 " . 
+            # 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 
+                # if it is a class, then we
                 # just ignore this here ...
                 next;
             }
         }
         else {
-            # if no local method is found, then we 
+            # 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 
+            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). " . 
+                    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 {   
+                else {
                     # if there is no conflict,
-                    # just add it to the role  
+                    # just add it to the role
                     $other->add_override_method_modifier(
-                        $method_name, 
+                        $method_name,
                         $self->get_override_method_modifier($method_name)
-                    );                    
+                    );
                 }
             }
             else {
-                # if this is not a role, then we need to 
+                # 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 
+                # 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;
+                my ($package) = Class::MOP::get_code_info($method);
                 # if it is a class, we just add it
                 $other->add_override_method_modifier($method_name, $method, $package);
             }
         }
-    }    
-    
-    foreach my $method_name ($self->get_method_modifier_list('before')) {
-        $other->add_before_method_modifier(
-            $method_name,
-            $_
-        ) foreach $self->get_before_method_modifiers($method_name);
-    }    
-    
-    foreach my $method_name ($self->get_method_modifier_list('after')) {
-        $other->add_after_method_modifier(
-            $method_name,
-            $_
-        ) foreach $self->get_after_method_modifiers($method_name);
-    }    
-    
-    foreach my $method_name ($self->get_method_modifier_list('around')) {
-        $other->add_around_method_modifier(
-            $method_name,
-            $_
-        ) foreach $self->get_around_method_modifiers($method_name);
-    }    
-    
-    $other->add_role($self);
+    }
 }
 
-sub combine {
-    my ($class, @roles) = @_;
-    
-    my $combined = $class->new(
-        ':role_meta' => Moose::Meta::Class->create_anon_class()
-    );
-    
-    foreach my $role (@roles) {
-        $role->apply($combined);
+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);
     }
-    
-    $combined->_clean_up_required_methods;   
-    
-    return $combined;
 }
 
-package Moose::Meta::Role::Method;
-
-use strict;
-use warnings;
-
-our $VERSION = '0.01';
+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:
+## 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 => 'Collection::Array',
+#     reader    => 'get_roles',
+#     isa       => 'ArrayRef[Moose::Meta::Roles]',
+#     default   => sub { [] },
+#     provides  => {
+#         'push' => 'add_role',
+#     }
+# );
+# 
+# has 'excluded_roles_map' => (
+#     metaclass => 'Collection::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 'attribute_map' => (
+#     metaclass => 'Collection::Hash',
+#     reader    => 'get_attribute_map',
+#     isa       => 'HashRef[Str]',    
+#     provides => {
+#         # 'set'  => 'add_attribute' # has some special crap in it
+#         'get'    => 'get_attribute',
+#         'keys'   => 'get_attribute_list',
+#         'exists' => 'has_attribute',
+#         # Not exactly delete, cause it sets multiple
+#         'delete' => 'remove_attribute',    
+#     }
+# );
+# 
+# has 'required_methods' => (
+#     metaclass => 'Collection::Hash',
+#     reader    => 'get_required_methods_map',
+#     isa       => 'HashRef[Str]',
+#     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 => 'Collection::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 => 'Collection::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 => 'Collection::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 => 'Collection::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 base 'Class::MOP::Method';
 
 1;
 
@@ -525,9 +763,9 @@ 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 
+Please see L<Moose::Role> for more information about roles.
+For the most part, this has no user-serviceable parts inside
+this module. It's API is still subject to some change (although
 probably not that much really).
 
 =head1 METHODS
@@ -574,10 +812,16 @@ probably not that much really).
 
 =item B<get_excluded_roles_map>
 
+=item B<calculate_all_roles>
+
 =back
 
 =over 4
 
+=item B<method_metaclass>
+
+=item B<find_method_by_name>
+
 =item B<get_method>
 
 =item B<has_method>
@@ -586,6 +830,12 @@ probably not that much really).
 
 =item B<get_method_list>
 
+=item B<get_method_map>
+
+=item B<update_package_cache_flag>
+
+=item B<reset_package_cache_flag>
+
 =back
 
 =over 4
@@ -670,7 +920,7 @@ probably not that much really).
 
 =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.
 
@@ -680,11 +930,11 @@ Stevan Little E<lt>stevan@iinteractive.comE<gt>
 
 =head1 COPYRIGHT AND LICENSE
 
-Copyright 2006 by Infinity Interactive, Inc.
+Copyright 2006, 2007 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
\ No newline at end of file
+=cut