From: Fabien Wernli Date: Tue, 9 Feb 2010 20:53:48 +0000 (+0000) Subject: should work now X-Git-Tag: v0.08120~3^2~9 X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?p=dbsrgits%2FDBIx-Class.git;a=commitdiff_plain;h=dd2600c6c6fc85f4d5e53224af9e290f90afbf57 should work now --- diff --git a/lib/DBIx/Class/Storage/DBI.pm b/lib/DBIx/Class/Storage/DBI.pm index 4333c4e..f9e4bc0 100644 --- a/lib/DBIx/Class/Storage/DBI.pm +++ b/lib/DBIx/Class/Storage/DBI.pm @@ -45,6 +45,7 @@ __PACKAGE__->sql_maker_class('DBIx::Class::SQLAHacks'); # Each of these methods need _determine_driver called before itself # in order to function reliably. This is a purely DRY optimization my @rdbms_specific_methods = qw/ + deployment_statements sqlt_type build_datetime_parser datetime_parser_type diff --git a/lib/DBIx/Class/Storage/DBI/Oracle/Generic.pm b/lib/DBIx/Class/Storage/DBI/Oracle/Generic.pm index 792679a..8626419 100644 --- a/lib/DBIx/Class/Storage/DBI/Oracle/Generic.pm +++ b/lib/DBIx/Class/Storage/DBI/Oracle/Generic.pm @@ -25,9 +25,35 @@ versions before 9. =cut +my %ora_reserved = map { $_, 1 } qw( + ACCESS ADD ALL ALTER AND ANY AS ASC AUDIT BETWEEN BY CHAR CHECK + CLUSTER COLUMN COMMENT COMPRESS CONNECT CREATE CURRENT DATE DECIMAL + DEFAULT DELETE DESC DISTINCT DROP ELSE EXCLUSIVE EXISTS FILE FLOAT + FOR FROM GRANT GROUP HAVING IDENTIFIED IMMEDIATE IN INCREMENT + INDEX INITIAL INSERT INTEGER INTERSECT INTO IS LEVEL LIKE LOCK + LONG MAXEXTENTS MINUS MLSLABEL MODE MODIFY NOAUDIT NOCOMPRESS NOT + NOWAIT NULL NUMBER OF OFFLINE ON ONLINE OPTION OR ORDER PCTFREE + PRIOR PRIVILEGES PUBLIC RAW RENAME RESOURCE REVOKE ROW ROWID ROWNUM + ROWS SELECT SESSION SET SHARE SIZE SMALLINT START SUCCESSFUL SYNONYM + SYSDATE TABLE THEN TO TRIGGER UID UNION UNIQUE UPDATE USER VALIDATE + VALUES VARCHAR VARCHAR2 VIEW WHENEVER WHERE WITH +); + use base qw/DBIx::Class::Storage::DBI/; use mro 'c3'; +sub deployment_statements { + my $self = shift;; + 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; + + $self->next::method($schema, $type, $version, $dir, $sqltargs, @rest); +} + sub _dbh_last_insert_id { my ($self, $dbh, $source, @columns) = @_; my @ids = (); @@ -42,8 +68,8 @@ 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 $name_sep = $self->schema->storage->{'_sql_maker_opts'}->{'name_sep'}; # look up the correct sequence automatically my $sql = q{ @@ -65,6 +91,7 @@ sub _dbh_get_autoinc_seq { 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+)/ ) { @@ -75,9 +102,11 @@ sub _dbh_get_autoinc_seq { AND t.status = 'ENABLED' }; $sth = $dbh->prepare($sql); - my $table_name = $quote_char ? "$quote_char$table$quote_char" : uc($table); - die $table_name; - $sth->execute( uc($schema), $table_name ); + 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); @@ -165,7 +194,7 @@ names to uppercase sub columns_info_for { my ($self, $table) = @_; - $self->next::method(uc($table)); + $self->next::method($table); } =head2 datetime_parser_type