X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=lib%2FDBIx%2FClass%2FStorage%2FDBI%2FOracle%2FGeneric.pm;h=925d7f980c0829f58c353b817f3f806cd5d3f169;hb=1816be4f51709d94945380c0a65de80e5606b162;hp=87715ed1611e1f730c1517bd0a0be1ddd5ba14e4;hpb=c2481821ea4d4e988a3586a126a103aa676bec47;p=dbsrgits%2FDBIx-Class.git diff --git a/lib/DBIx/Class/Storage/DBI/Oracle/Generic.pm b/lib/DBIx/Class/Storage/DBI/Oracle/Generic.pm index 87715ed..925d7f9 100644 --- a/lib/DBIx/Class/Storage/DBI/Oracle/Generic.pm +++ b/lib/DBIx/Class/Storage/DBI/Oracle/Generic.pm @@ -1,12 +1,11 @@ package DBIx::Class::Storage::DBI::Oracle::Generic; -# -*- mode: cperl; cperl-indent-level: 2 -*- use strict; use warnings; =head1 NAME -DBIx::Class::Storage::DBI::Oracle::Generic - Automatic primary key class for Oracle +DBIx::Class::Storage::DBI::Oracle::Generic - Oracle Support for DBIx::Class =head1 SYNOPSIS @@ -24,12 +23,11 @@ This class implements autoincrements for Oracle. =cut -use Carp::Clan qw/^DBIx::Class/; -use Scalar::Util (); +use base qw/DBIx::Class::Storage::DBI/; +use mro 'c3'; -use base qw/DBIx::Class::Storage::DBI::MultiDistinctEmulation/; - -# __PACKAGE__->load_components(qw/PK::Auto/); +# For ORA_BLOB => 113, ORA_CLOB => 112 +use DBD::Oracle qw( :ora_types ); sub _dbh_last_insert_id { my ($self, $dbh, $source, @columns) = @_; @@ -54,7 +52,7 @@ sub _dbh_get_autoinc_seq { }; # trigger_body is a LONG - $dbh->{LongReadLen} = 64 * 1024 if ($dbh->{LongReadLen} < 64 * 1024); + local $dbh->{LongReadLen} = 64 * 1024 if ($dbh->{LongReadLen} < 64 * 1024); my $sth; @@ -85,28 +83,18 @@ sub _sequence_fetch { return $id; } -sub connected { +sub _ping { my $self = shift; - - if ($self->SUPER::connected(@_)) { - my $dbh = $self->_dbh; - - my $ping_sth = $dbh->prepare_cached("select 1 from dual"); - - local $dbh->{RaiseError} = 1; - eval { - $ping_sth->execute; - $ping_sth->finish; - }; - - if ($@) { - return 0; - } else { - return 1; - } - } - - return 0; + + my $dbh = $self->_dbh or return 0; + + local $dbh->{RaiseError} = 1; + + eval { + $dbh->do("select 1 from dual"); + }; + + return $@ ? 0 : 1; } sub _dbh_execute { @@ -114,28 +102,28 @@ sub _dbh_execute { my ($dbh, $op, $extra_bind, $ident, $bind_attributes, @args) = @_; my $wantarray = wantarray; - my @res; - my $exception; - - my $try = 2; - - while ($try--) { - eval { - if ($wantarray) { - @res = $self->SUPER::_dbh_execute(@_); + + my (@res, $exception, $retried); + + RETRY: { + do { + eval { + if ($wantarray) { + @res = $self->next::method(@_); + } else { + $res[0] = $self->next::method(@_); + } + }; + $exception = $@; + if ($exception =~ /ORA-01003/) { + # ORA-01003: no statement parsed (someone changed the table somehow, + # invalidating your cursor.) + my ($sql, $bind) = $self->_prep_for_execute($op, $extra_bind, $ident, \@args); + delete $dbh->{CachedKids}{$sql}; } else { - $res[0] = $self->SUPER::_dbh_execute(@_); + last RETRY; } - }; - $exception = $@; - if ($exception =~ /ORA-01003/) { -# ORA-01003: no statement parsed (someone changed the table somehow, -# invalidating your cursor.) - my ($sql, $bind) = $self->_prep_for_execute($op, $extra_bind, $ident, \@args); - delete $dbh->{CachedKids}{$sql}; - } else { - last; - } + } while (not $retried++); } $self->throw_exception($exception) if $exception; @@ -151,7 +139,7 @@ Returns the sequence name for an autoincrement column sub get_autoinc_seq { my ($self, $source, $col) = @_; - + $self->dbh_do('_dbh_get_autoinc_seq', $source, $col); } @@ -177,9 +165,95 @@ L. sub datetime_parser_type { return "DateTime::Format::Oracle"; } +=head2 connect_call_datetime_setup + +Used as: + + on_connect_call => 'datetime_setup' + +In L to set the session nls date, and +timestamp values for use with L and the +necessary environment variables for L, which is used +by it. + +Maximum allowable precision is used, unless the environment variables have +already been set. + +These are the defaults used: + + $ENV{NLS_DATE_FORMAT} ||= 'YYYY-MM-DD HH24:MI:SS'; + $ENV{NLS_TIMESTAMP_FORMAT} ||= 'YYYY-MM-DD HH24:MI:SS.FF'; + $ENV{NLS_TIMESTAMP_TZ_FORMAT} ||= 'YYYY-MM-DD HH24:MI:SS.FF TZHTZM'; + +To get more than second precision with L +for your timestamps, use something like this: + + use Time::HiRes 'time'; + my $ts = DateTime->from_epoch(epoch => time); + +=cut + +sub connect_call_datetime_setup { + my $self = shift; + + my $date_format = $ENV{NLS_DATE_FORMAT} ||= 'YYYY-MM-DD HH24:MI:SS'; + my $timestamp_format = $ENV{NLS_TIMESTAMP_FORMAT} ||= + 'YYYY-MM-DD HH24:MI:SS.FF'; + my $timestamp_tz_format = $ENV{NLS_TIMESTAMP_TZ_FORMAT} ||= + 'YYYY-MM-DD HH24:MI:SS.FF TZHTZM'; + + $self->_do_query("alter session set nls_date_format = '$date_format'"); + $self->_do_query( +"alter session set nls_timestamp_format = '$timestamp_format'"); + $self->_do_query( +"alter session set nls_timestamp_tz_format='$timestamp_tz_format'"); +} + +=head2 source_bind_attributes + +Handle LOB types in Oracle. Under a certain size (4k?), you can get away +with the driver assuming your input is the deprecated LONG type if you +encode it as a hex string. That ain't gonna fly at larger values, where +you'll discover you have to do what this does. + +This method had to be overridden because we need to set ora_field to the +actual column, and that isn't passed to the call (provided by Storage) to +bind_attribute_by_data_type. + +According to L, the ora_field isn't always necessary, but +adding it doesn't hurt, and will save your bacon if you're modifying a +table with more than one LOB column. + +=cut + +sub source_bind_attributes +{ + my $self = shift; + my($source) = @_; + + my %bind_attributes; + + foreach my $column ($source->columns) { + my $data_type = $source->column_info($column)->{data_type} || ''; + next unless $data_type; + + my %column_bind_attrs = $self->bind_attribute_by_data_type($data_type); + + if ($data_type =~ /^[BC]LOB$/i) { + $column_bind_attrs{'ora_type'} + = uc($data_type) eq 'CLOB' ? ORA_CLOB : ORA_BLOB; + $column_bind_attrs{'ora_field'} = $column; + } + + $bind_attributes{$column} = \%column_bind_attrs; + } + + return \%bind_attributes; +} + sub _svp_begin { my ($self, $name) = @_; - + $self->dbh->do("SAVEPOINT $name"); } @@ -193,11 +267,9 @@ sub _svp_rollback { $self->dbh->do("ROLLBACK TO SAVEPOINT $name") } -=head1 AUTHORS - -Andy Grundman +=head1 AUTHOR -Scott Connelly +See L. =head1 LICENSE