sub last_insert_id {
my ($self,$source,$col) = @_;
- my $seq = ($source->column_info($col)->{sequence} ||= $self->get_autoinc_seq($source,$col))
+ my $seq = ( $source->column_info($col)->{sequence} ||= $self->dbh_do('_dbh_get_autoinc_seq', $source, $col) )
or $self->throw_exception( "could not determine sequence for "
. $source->name
. ".$col, please consider adding a "
}
-# get the postgres search path, and cache it
-sub _get_pg_search_path {
- my ($self,$dbh) = @_;
- # cache the search path as ['schema','schema',...] in the storage
- # obj
- $self->{_pg_search_path} ||= do {
- my @search_path;
- my ($sp_string) = $dbh->selectrow_array('SHOW search_path');
- while( $sp_string =~ s/("[^"]+"|[^,]+),?// ) {
- unless( defined $1 and length $1 ) {
- $self->throw_exception("search path sanity check failed: '$1'")
- }
- push @search_path, $1;
- }
- \@search_path
- };
-}
-
sub _dbh_get_autoinc_seq {
- my ($self, $dbh, $schema, $table, $col) = @_;
+ my ($self, $dbh, $source, $col) = @_;
+
+ my $schema;
+ my $table = $source->name;
+ # deref table name if it needs it
+ $table = $$table
+ if ref $table eq 'SCALAR';
- my @where = ( 'c.relname = ?', 'a.attname = ?' );
- my @bind = ($table, $col);
- if( defined $schema && length $schema ) {
- push @where, 'n.nspname = ?';
- push @bind, $schema;
- } else {
- push @where, 'pg_catalog.pg_table_is_visible(c.oid)';
+ # parse out schema name if present
+ if( $table =~ /^(.+)\.(.+)$/ ) {
+ ( $schema, $table ) = ( $1, $2 );
}
- my $where = join ' AND ', @where;
+
+ my $sqlmaker = $self->sql_maker;
+ local $sqlmaker->{bindtype} = 'normal';
+
+ my ($where, @bind) = $sqlmaker->where ({
+ 'a.attnum' => {'>', 0},
+ 'c.relname' => $table,
+ 'a.attname' => $col,
+ -not_bool => 'a.attisdropped',
+ (defined $schema && length $schema)
+ ? ( 'n.nspname' => $schema )
+ : ( -bool => \'pg_catalog.pg_table_is_visible(c.oid)' )
+ });
my ($seq_expr) = $dbh->selectrow_array(<<EOS,undef,@bind);
+
SELECT
(SELECT pg_catalog.pg_get_expr(d.adbin, d.adrelid)
FROM pg_catalog.pg_attrdef d
WHERE d.adrelid = a.attrelid AND d.adnum = a.attnum AND a.atthasdef)
FROM pg_catalog.pg_class c
- JOIN pg_catalog.pg_namespace n ON n.oid = c.relnamespace
+ LEFT JOIN pg_catalog.pg_namespace n ON n.oid = c.relnamespace
JOIN pg_catalog.pg_attribute a ON a.attrelid = c.oid
-WHERE
- $where
- AND a.attnum > 0 AND NOT a.attisdropped
+$where
+
EOS
- $seq_expr =~ /^nextval\(+'([^']+)'::(?:text|regclass)\)/i
- or $self->throw_exception("could not parse sequence expression '$seq_expr'");
+ defined $seq_expr and length $seq_expr
+ or $self->throw_exception( "no sequence found for $table.$col, check table definition, "
+ . "or explicitly set the 'sequence' for this column in the "
+ . $source->source_name
+ . " class"
+ );
+
+ unless ( $seq_expr =~ /^nextval\(+'([^']+)'::(?:text|regclass)\)/i ){
+ $seq_expr = '' unless defined $seq_expr;
+ $schema = $schema . "." if defined $schema && length $schema;
+ $self->throw_exception("could not parse nextval expression for $schema$table.$col: '$seq_expr'");
+ }
return $1;
}
sub get_autoinc_seq {
my ($self,$source,$col) = @_;
- my $schema;
- my $table = $source->name;
-
- if (ref $table eq 'SCALAR') {
- $table = $$table;
- }
- elsif ($table =~ /^(.+)\.(.+)$/) {
- ($schema, $table) = ($1, $2);
- }
-
- $self->dbh_do('_dbh_get_autoinc_seq', $schema, $table, $col);
}
sub sqlt_type {