X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=lib%2FDBIx%2FClass%2FInflateColumn.pm;h=2c6a955bbebceddaa7cb546a55a754bb2f78ee6e;hb=2d0b795a54a018d5c9cf2593cf83045962cd9b93;hp=5dec97d6d0bfb32f26becec0eed76a331b693f8f;hpb=b82c8a28bb4cb7d704496b8ce3966565d255d5b3;p=dbsrgits%2FDBIx-Class.git diff --git a/lib/DBIx/Class/InflateColumn.pm b/lib/DBIx/Class/InflateColumn.pm index 5dec97d..2c6a955 100644 --- a/lib/DBIx/Class/InflateColumn.pm +++ b/lib/DBIx/Class/InflateColumn.pm @@ -2,7 +2,6 @@ package DBIx::Class::InflateColumn; use strict; use warnings; -use Scalar::Util qw/blessed/; use base qw/DBIx::Class::Row/; @@ -26,7 +25,7 @@ for the database. It can be used, for example, to automatically convert to and from L objects for your date and time fields. There's a -conveniece component to actually do that though, try +convenience component to actually do that though, try L. It will handle all types of references except scalar references. It @@ -37,7 +36,7 @@ deal with, to allow such settings as C< \'year + 1'> and C< \'DEFAULT' > to work. If you want to filter plain scalar values and replace them with -something else, contribute a filtering component. +something else, see L. =head1 METHODS @@ -74,12 +73,19 @@ used in the database layer. sub inflate_column { my ($self, $col, $attrs) = @_; + + my $colinfo = $self->column_info($col); + + $self->throw_exception("InflateColumn does not work with FilterColumn") + if $self->isa('DBIx::Class::FilterColumn') && + defined $colinfo->{_filter_info}; + $self->throw_exception("No such column $col to inflate") unless $self->has_column($col); $self->throw_exception("inflate_column needs attr hashref") unless ref $attrs eq 'HASH'; - $self->column_info($col)->{_inflate_info} = $attrs; - my $acc = $self->column_info($col)->{accessor}; + $colinfo->{_inflate_info} = $attrs; + my $acc = $colinfo->{accessor}; $self->mk_group_accessors('inflated_column' => [ (defined $acc ? $acc : $col), $col]); return 1; } @@ -114,7 +120,7 @@ sub _deflated_column { Fetch a column value in its inflated state. This is directly analogous to L in that it only fetches a -column already retreived from the database, and then inflates it. +column already retrieved from the database, and then inflates it. Throws an exception if the column requested is not an inflated column. =cut @@ -146,9 +152,9 @@ sub set_inflated_column { $self->set_column($col, $self->_deflated_column($col, $inflated)); # if (blessed $inflated) { if (ref $inflated && ref($inflated) ne 'SCALAR') { - $self->{_inflated_column}{$col} = $inflated; + $self->{_inflated_column}{$col} = $inflated; } else { - delete $self->{_inflated_column}{$col}; + delete $self->{_inflated_column}{$col}; } return $inflated; }