X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=lib%2FDBIx%2FClass%2FStorage%2FDBI%2FPg.pm;h=bd28e02232cf584a26a8e3c43c4d4db844cc4b51;hb=9c0df5f32b68e23c670c89ce6cdbff60b4bd0ed0;hp=6faaf4c3aa8c5c90e163ff17d2c15fb9cf2e33a9;hpb=0ceac5e1cf9d39d6bf733f36a2461ba3f2fc0023;p=dbsrgits%2FDBIx-Class.git diff --git a/lib/DBIx/Class/Storage/DBI/Pg.pm b/lib/DBIx/Class/Storage/DBI/Pg.pm index 6faaf4c..bd28e02 100644 --- a/lib/DBIx/Class/Storage/DBI/Pg.pm +++ b/lib/DBIx/Class/Storage/DBI/Pg.pm @@ -3,44 +3,103 @@ package DBIx::Class::Storage::DBI::Pg; use strict; use warnings; +use DBD::Pg qw(:pg_types); + use base qw/DBIx::Class::Storage::DBI/; # __PACKAGE__->load_components(qw/PK::Auto/); +# Warn about problematic versions of DBD::Pg +warn "DBD::Pg 1.49 is strongly recommended" + if ($DBD::Pg::VERSION < 1.49); + +sub _dbh_last_insert_id { + my ($self, $dbh, $seq) = @_; + $dbh->last_insert_id(undef, undef, undef, undef, {sequence => $seq}); +} + sub last_insert_id { - my ($self, $source) = @_; - $self->get_autoinc_seq($source) unless $source->{_autoinc_seq}; - $self->_dbh->last_insert_id(undef,undef,undef,undef, - {sequence => $source->{_autoinc_seq}}); + my ($self,$source,$col) = @_; + my $seq = ($source->column_info($col)->{sequence} ||= $self->get_autoinc_seq($source,$col)); + $self->throw_exception("could not fetch primary key for " . $source->name . ", could not " + . "get autoinc sequence for $col (check that table and column specifications are correct " + . "and in the correct case)") unless defined $seq; + $self->dbh_do('_dbh_last_insert_id', $seq); } -sub get_autoinc_seq { - my ($self,$source) = @_; - - # return the user-defined sequence if known - my $result_class = $source->result_class; - if ($result_class->can('sequence') and $result_class->sequence) { - return ($source->{_autoinc_seq} = $result_class->sequence); +sub _dbh_get_autoinc_seq { + my ($self, $dbh, $schema, $table, @pri) = @_; + + while (my $col = shift @pri) { + my $info = $dbh->column_info(undef,$schema,$table,$col)->fetchrow_hashref; + if(defined $info->{COLUMN_DEF} and + $info->{COLUMN_DEF} =~ /^nextval\(+'([^']+)'::(?:text|regclass)\)/) { + my $seq = $1; + # may need to strip quotes -- see if this works + return $seq =~ /\./ ? $seq : $info->{TABLE_SCHEM} . "." . $seq; + } } - + return; +} + +sub get_autoinc_seq { + my ($self,$source,$col) = @_; + my @pri = $source->primary_columns; - my $dbh = $self->_dbh; my ($schema,$table) = $source->name =~ /^(.+)\.(.+)$/ ? ($1,$2) : (undef,$source->name); - while (my $col = shift @pri) { - my $info = $dbh->column_info(undef,$schema,$table,$col)->fetchrow_arrayref; - if (defined $info->[12] and $info->[12] =~ - /^nextval\('"?([^"']+)"?'::(?:text|regclass)\)/) - { - $source->{_autoinc_seq} = $1; - last; - } + + $self->dbh_do('_dbh_get_autoinc_seq', $schema, $table, @pri); +} + +sub sqlt_type { + return 'PostgreSQL'; +} + +sub datetime_parser_type { return "DateTime::Format::Pg"; } + +sub bind_attribute_by_data_type { + my ($self,$data_type) = @_; + + my $bind_attributes = { + bytea => { pg_type => DBD::Pg::PG_BYTEA }, + }; + + if( defined $bind_attributes->{$data_type} ) { + return $bind_attributes->{$data_type}; + } + else { + return; } } +sub _sequence_fetch { + my ( $self, $type, $seq ) = @_; + my ($id) = $self->dbh->selectrow_array("SELECT nextval('${seq}')"); + return $id; +} + +sub _svp_begin { + my ($self, $name) = @_; + + $self->dbh->pg_savepoint($name); +} + +sub _svp_release { + my ($self, $name) = @_; + + $self->dbh->pg_release($name); +} + +sub _svp_rollback { + my ($self, $name) = @_; + + $self->dbh->pg_rollback_to($name); +} + 1; -=head1 NAME +=head1 NAME DBIx::Class::Storage::DBI::Pg - Automatic primary key class for PostgreSQL