X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=lib%2FCatalyst%2FClassData.pm;h=3dde73af36f55da543e11861e94ce61df294b748;hb=c1ec627ecb058c863d93f1f3b0b4697047bc8adf;hp=2b004122bb2186f1672bd23c198cb7bc1ac79b7c;hpb=875d811063ef237d5cc41e9258fdbf1b567e403d;p=catagits%2FCatalyst-Runtime.git diff --git a/lib/Catalyst/ClassData.pm b/lib/Catalyst/ClassData.pm index 2b00412..3dde73a 100644 --- a/lib/Catalyst/ClassData.pm +++ b/lib/Catalyst/ClassData.pm @@ -2,6 +2,7 @@ package Catalyst::ClassData; use Moose::Role; use Class::MOP; +use Class::MOP::Object; use Scalar::Util 'blessed'; sub mk_classdata { @@ -11,9 +12,9 @@ sub mk_classdata { my $slot = '$'.$attribute; my $accessor = sub { - my $meta = $_[0]->meta; my $pkg = ref $_[0] || $_[0]; - if(@_ > 1){ + my $meta = $pkg->Class::MOP::Object::meta(); + if (@_ > 1){ $meta->namespace->{$attribute} = \$_[1]; return $_[1]; } @@ -28,6 +29,13 @@ sub mk_classdata { return ${$v}; } else { foreach my $super ( $meta->linearized_isa ) { + # If there is a code symbol for this attr in a parent class, + # then copy it into our package. Is this the correct + # fix for C::D::I back-compat? (t0m) + my $parent_symbol = *{"${super}::${attribute}"}{CODE} ? \&{"${super}::${attribute}"} : undef; + if (defined $parent_symbol) { + *{"${pkg}::${attribute}"} = $parent_symbol; + } # tighter version of same after # my $super_meta = Moose::Meta::Class->initialize($super); my $v = ${"${super}::"}{$attribute} ? *{"${super}::${attribute}"}{SCALAR} : undef; @@ -42,7 +50,7 @@ sub mk_classdata { confess("Failed to create accessor: $@ ") unless ref $accessor eq 'CODE'; - my $meta = $class->meta; + my $meta = $class->Class::MOP::Object::meta(); my $immutable_options; if( $meta->is_immutable ){ $immutable_options = $meta->get_immutable_options; @@ -52,7 +60,7 @@ sub mk_classdata { $meta->add_method($alias, $accessor); $meta->add_method($attribute, $accessor); if(defined $immutable_options){ - $meta->make_immutable($immutable_options); + $meta->make_immutable(%{ $immutable_options }); } $class->$attribute($_[2]) if(@_ > 2); return $accessor;