From bedb879913903e7b2548db7d67a57111b30fa267 Mon Sep 17 00:00:00 2001 From: Jeremy Kemper Date: Mon, 12 Mar 2007 05:44:01 +0000 Subject: Oracle: fix quoted primary keys and datetime overflow. Closes #7798. git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@6388 5ecf4fe2-1ee6-0310-87b1-e25e094e27de --- .../active_record/connection_adapters/oracle_adapter.rb | 17 +++++++++++++---- 1 file changed, 13 insertions(+), 4 deletions(-) (limited to 'activerecord/lib/active_record') diff --git a/activerecord/lib/active_record/connection_adapters/oracle_adapter.rb b/activerecord/lib/active_record/connection_adapters/oracle_adapter.rb index 0dc0513356..8b555b1466 100644 --- a/activerecord/lib/active_record/connection_adapters/oracle_adapter.rb +++ b/activerecord/lib/active_record/connection_adapters/oracle_adapter.rb @@ -162,7 +162,7 @@ begin # camelCase column names need to be quoted; not that anyone using Oracle # would really do this, but handling this case means we pass the test... def quote_column_name(name) #:nodoc: - name =~ /[A-Z]/ ? "\"#{name}\"" : name + name.to_s =~ /[A-Z]/ ? "\"#{name}\"" : name end def quote_string(s) #:nodoc: @@ -507,10 +507,19 @@ begin when OCI8::LOB name == 'Writable Large Object' ? row[i]: row[i].read when OraDate - if emulate_dates && (row[i].hour == 0 && row[i].minute == 0 && row[i].second == 0) - row[i].to_date + d = row[i] + if emulate_dates && (d.hour == 0 && d.minute == 0 && d.second == 0) + d.to_date else - row[i].to_time rescue row[i].to_datetime + # see string_to_time; Time overflowing to DateTime, respecting the default timezone + time_array = [d.year, d.month, d.day, d.hour, d.minute, d.second] + begin + Time.send(Base.default_timezone, *time_array) + rescue + zone_offset = if Base.default_timezone == :local then DateTime.now.offset else 0 end + # Append zero calendar reform start to account for dates skipped by calendar reform + DateTime.new(*time_array[0..5] << zone_offset << 0) rescue nil + end end else row[i] end unless col == 'raw_rnum_' -- cgit v1.2.3