X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=lib%2FDBIx%2FClass%2FAccessorGroup.pm;h=4d7e0469096d10caf215aa52e884703cee4e08b2;hb=a6b68a60b376e918a6058f37cb1115ba3163a59b;hp=a7e85be061135179abe0d0adb9056d533100dca8;hpb=c687b87e860c97257542dda2b14c0137f6fbc583;p=dbsrgits%2FDBIx-Class.git diff --git a/lib/DBIx/Class/AccessorGroup.pm b/lib/DBIx/Class/AccessorGroup.pm index a7e85be..4d7e046 100644 --- a/lib/DBIx/Class/AccessorGroup.pm +++ b/lib/DBIx/Class/AccessorGroup.pm @@ -3,120 +3,27 @@ package DBIx::Class::AccessorGroup; use strict; use warnings; -use NEXT; +use base qw/Class::Accessor::Grouped/; -sub mk_group_accessors { - my($self, $group, @fields) = @_; - - $self->_mk_group_accessors('make_group_accessor', $group, @fields); -} - - -{ - no strict 'refs'; - no warnings 'redefine'; - - sub _mk_group_accessors { - my($self, $maker, $group, @fields) = @_; - my $class = ref $self || $self; - - # So we don't have to do lots of lookups inside the loop. - $maker = $self->can($maker) unless ref $maker; - - foreach my $field (@fields) { - if( $field eq 'DESTROY' ) { - require Carp; - &Carp::carp("Having a data accessor named DESTROY in ". - "'$class' is unwise."); - } - - my $name = $field; - - ($name, $field) = @$field if ref $field; - - my $accessor = $self->$maker($group, $field); - my $alias = "_${name}_accessor"; - - #warn "$class $group $field $alias"; - - *{$class."\:\:$name"} = $accessor; - #unless defined &{$class."\:\:$field"} - - *{$class."\:\:$alias"} = $accessor; - #unless defined &{$class."\:\:$alias"} - } - } -} - -sub mk_group_ro_accessors { - my($self, $group, @fields) = @_; - - $self->_mk_group_accessors('make_group_ro_accessor', $group, @fields); -} - -sub mk_group_wo_accessors { - my($self, $group, @fields) = @_; - - $self->_mk_group_accessors('make_group_wo_accessor', $group, @fields); -} - -sub make_group_accessor { - my ($class, $group, $field) = @_; - - my $set = "set_$group"; - my $get = "get_$group"; +1; - # Build a closure around $field. - return sub { - my $self = shift; +=head1 NAME - if(@_) { - return $self->$set($field, @_); - } - else { - return $self->$get($field); - } - }; -} +DBIx::Class::AccessorGroup - See Class::Accessor::Grouped -sub make_group_ro_accessor { - my($class, $group, $field) = @_; +=head1 SYNOPSIS - my $get = "get_$group"; +=head1 DESCRIPTION - return sub { - my $self = shift; +This class now exists in its own right on CPAN as Class::Accessor::Grouped - if(@_) { - my $caller = caller; - require Carp; - Carp::croak("'$caller' cannot alter the value of '$field' on ". - "objects of class '$class'"); - } - else { - return $self->$get($field); - } - }; -} +=head1 AUTHORS -sub make_group_wo_accessor { - my($class, $group, $field) = @_; +Matt S. Trout - my $set = "set_$group"; +=head1 LICENSE - return sub { - my $self = shift; +You may distribute this code under the same terms as Perl itself. - unless (@_) { - my $caller = caller; - require Carp; - Carp::croak("'$caller' cannot access the value of '$field' on ". - "objects of class '$class'"); - } - else { - return $self->$set($field, @_); - } - }; -} +=cut -1;