X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=lib%2FDBIx%2FClass%2FStorage%2FDBI%2FOracle%2FGeneric.pm;h=3e74d427e67a5151c5cab32ce1c4e262054eb43c;hb=ed7ab0f4ce1a9118ea6285ee562ef003085a6b64;hp=65df78b30d8a3c3f1506f7564da8744503f0110e;hpb=a4433d8e55074f96911ede2a922fced8b96743de;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 65df78b..3e74d42 100644 --- a/lib/DBIx/Class/Storage/DBI/Oracle/Generic.pm +++ b/lib/DBIx/Class/Storage/DBI/Oracle/Generic.pm @@ -2,6 +2,9 @@ package DBIx::Class::Storage::DBI::Oracle::Generic; use strict; use warnings; +use Scope::Guard (); +use Context::Preserve (); +use Try::Tiny; =head1 NAME @@ -33,15 +36,15 @@ sub deployment_statements { my ($schema, $type, $version, $dir, $sqltargs, @rest) = @_; $sqltargs ||= {}; - my $quote_char = $self->schema->storage->{'_sql_maker_opts'}->{'quote_char'}; - $sqltargs->{quote_table_names} = 0 unless $quote_char; - $sqltargs->{quote_field_names} = 0 unless $quote_char; + my $quote_char = $self->schema->storage->sql_maker->quote_char; + $sqltargs->{quote_table_names} = $quote_char ? 1 : 0; + $sqltargs->{quote_field_names} = $quote_char ? 1 : 0; - my $oracle_version = eval { $self->_get_dbh->get_info(18) }; + my $oracle_version = eval { $self->_get_dbh->get_info(18) }; $sqltargs->{producer_args}{oracle_version} = $oracle_version; - $self->next::method($schema, $type, $version, $dir, $sqltargs, @rest); + $self->next::method($schema, $type, $version, $dir, $sqltargs, @rest); } sub _dbh_last_insert_id { @@ -58,54 +61,42 @@ sub _dbh_last_insert_id { sub _dbh_get_autoinc_seq { my ($self, $dbh, $source, $col) = @_; - # check if quoting is on - my $quote_char = $self->schema->storage->{'_sql_maker_opts'}->{'quote_char'}; + my $sql_maker = $self->sql_maker; - # look up the correct sequence automatically - my $sql = q{ - SELECT trigger_body FROM ALL_TRIGGERS t - WHERE t.table_name = ? - AND t.triggering_event = 'INSERT' - AND t.status = 'ENABLED' - }; + my $source_name; + if ( ref $source->name eq 'SCALAR' ) { + $source_name = ${$source->name}; + } + else { + $source_name = $source->name; + } + $source_name = uc($source_name) unless $sql_maker->quote_char; # trigger_body is a LONG local $dbh->{LongReadLen} = 64 * 1024 if ($dbh->{LongReadLen} < 64 * 1024); - my $sth; + # disable default bindtype + local $sql_maker->{bindtype} = 'normal'; + + # look up the correct sequence automatically + my ( $schema, $table ) = $source_name =~ /(\w+)\.(\w+)/; + my ($sql, @bind) = $sql_maker->select ( + 'ALL_TRIGGERS', + ['trigger_body'], + { + $schema ? (owner => $schema) : (), + table_name => $table || $source_name, + triggering_event => 'INSERT', + status => 'ENABLED', + }, + ); + my $sth = $dbh->prepare($sql); + $sth->execute (@bind); - my $source_name; - if ( ref $source->name ne 'SCALAR' ) { - $source_name = $source->name; - } - else { - $source_name = ${$source->name}; - } - $source_name = uc($source_name) unless $quote_char; - - # check for fully-qualified name (eg. SCHEMA.TABLENAME) - if ( my ( $schema, $table ) = $source_name =~ /(\w+)\.(\w+)/ ) { - $sql = q{ - SELECT trigger_body FROM ALL_TRIGGERS t - WHERE t.owner = ? AND t.table_name = ? - AND t.triggering_event = 'INSERT' - AND t.status = 'ENABLED' - }; - $sth = $dbh->prepare($sql); - my $table_name = $self -> sql_maker -> _quote($table); - #my $schema_name = $self -> sql_maker -> _quote($schema); - my $schema_name = uc($schema); - - $sth->execute( $schema_name, $table_name ); - } - else { - $sth = $dbh->prepare($sql); - $sth->execute( $source_name ); - } while (my ($insert_trigger) = $sth->fetchrow_array) { return $1 if $insert_trigger =~ m!("?\w+"?)\.nextval!i; # col name goes here??? } - $self->throw_exception("Unable to find a sequence INSERT trigger on table '" . $source->name . "'."); + $self->throw_exception("Unable to find a sequence INSERT trigger on table '$source_name'."); } sub _sequence_fetch { @@ -120,12 +111,16 @@ sub _ping { my $dbh = $self->_dbh or return 0; local $dbh->{RaiseError} = 1; + local $dbh->{PrintError} = 0; - eval { - $dbh->do("select 1 from dual"); + my $rc = 1; + try { + $dbh->do('select 1 from dual'); + } catch { + $rc = 0; }; - return $@ ? 0 : 1; + return $rc; } sub _dbh_execute { @@ -138,14 +133,16 @@ sub _dbh_execute { RETRY: { do { - eval { + my $exception; + try { if ($wantarray) { @res = $self->next::method(@_); } else { $res[0] = $self->next::method(@_); } + } catch { + $exception = shift; }; - $exception = $@; if ($exception =~ /ORA-01003/) { # ORA-01003: no statement parsed (someone changed the table somehow, # invalidating your cursor.) @@ -159,7 +156,7 @@ sub _dbh_execute { $self->throw_exception($exception) if $exception; - wantarray ? @res : $res[0] + $wantarray ? @res : $res[0] } =head2 get_autoinc_seq @@ -202,10 +199,10 @@ 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. +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. @@ -350,6 +347,35 @@ sub relname_to_table_alias { return $new_alias; } +=head2 with_deferred_fk_checks + +Runs a coderef between: + + alter session set constraints = deferred + ... + alter session set constraints = immediate + +to defer foreign key checks. + +Constraints must be declared C for this to work. + +=cut + +sub with_deferred_fk_checks { + my ($self, $sub) = @_; + + my $txn_scope_guard = $self->txn_scope_guard; + + $self->_do_query('alter session set constraints = deferred'); + + my $sg = Scope::Guard->new(sub { + $self->_do_query('alter session set constraints = immediate'); + }); + + return Context::Preserve::preserve_context(sub { $sub->() }, + after => sub { $txn_scope_guard->commit }); +} + =head1 AUTHOR See L.