activerecord-oracle_enhanced-adapter 1.7.11 → 1.8.0.beta1

data/lib/active_record/connection_adapters/oracle_enhanced/database_statements.rb

@@ -16,10 +16,10 @@ module ActiveRecord
           reload_type_map
         end
 
-        def exec_query(sql, name = 'SQL', binds = [], prepare: false)
+        def exec_query(sql, name = "SQL", binds = [], prepare: false)
           type_casted_binds = binds.map { |attr| type_cast(attr.value_for_database) }
 
-          log(sql, name, binds) do
+          log(sql, name, binds, type_casted_binds) do
             cursor = nil
             cached = false
             if without_prepared_statement?(binds)
@@ -38,14 +38,14 @@ module ActiveRecord
 
             cursor.exec
 
-            if name == 'EXPLAIN' and sql =~ /^EXPLAIN/
+            if (name == "EXPLAIN") && sql =~ /^EXPLAIN/
               res = true
             else
               columns = cursor.get_col_names.map do |col_name|
                 @connection.oracle_downcase(col_name)
               end
               rows = []
-              fetch_options = {:get_lob_value => (name != 'Writable Large Object')}
+              fetch_options = { get_lob_value: (name != "Writable Large Object") }
               while row = cursor.fetch(fetch_options)
                 rows << row
               end
@@ -69,17 +69,11 @@ module ActiveRecord
           sql = "EXPLAIN PLAN FOR #{to_sql(arel, binds)}"
           return if sql =~ /FROM all_/
           if ORACLE_ENHANCED_CONNECTION == :jdbc
-            exec_query(sql, 'EXPLAIN', binds)
+            exec_query(sql, "EXPLAIN", binds)
           else
-            exec_query(sql, 'EXPLAIN')
+            exec_query(sql, "EXPLAIN")
           end
-          select_values("SELECT * FROM TABLE(DBMS_XPLAN.DISPLAY)", 'EXPLAIN').join("\n")
-        end
-
-        # Returns an array of arrays containing the field values.
-        # Order is the same as that returned by #columns.
-        def select_rows(sql, name = nil, binds = [])
-          exec_query(sql, name, binds).rows
+          select_values("SELECT * FROM TABLE(DBMS_XPLAN.DISPLAY)", "EXPLAIN").join("\n")
         end
 
         # New method in ActiveRecord 3.1
@@ -92,11 +86,17 @@ module ActiveRecord
           super
         end
 
+        def insert(arel, name = nil, pk = nil, id_value = nil, sequence_name = nil, binds = [])
+          pk = nil if id_value
+          super
+        end
+
         # New method in ActiveRecord 3.1
-        def exec_insert(sql, name, binds, pk = nil, sequence_name = nil)
+        def exec_insert(sql, name = nil, binds = [], pk = nil, sequence_name = nil)
+          sql, binds = sql_for_insert(sql, pk, nil, sequence_name, binds)
           type_casted_binds = binds.map { |attr| type_cast(attr.value_for_database) }
 
-          log(sql, name, binds) do
+          log(sql, name, binds, type_casted_binds) do
             returning_id_col = returning_id_index = nil
             if without_prepared_statement?(binds)
               cursor = @connection.prepare(sql)
@@ -129,19 +129,19 @@ module ActiveRecord
         end
 
         # New method in ActiveRecord 3.1
-        def exec_update(sql, name, binds)
+        def exec_update(sql, name = nil, binds = [])
           type_casted_binds = binds.map { |attr| type_cast(attr.value_for_database) }
 
-          log(sql, name, binds) do
+          log(sql, name, binds, type_casted_binds) do
             cached = false
             if without_prepared_statement?(binds)
               cursor = @connection.prepare(sql)
             else
-              cursor = if @statements.key?(sql)
-                         @statements[sql]
-                       else
-                         @statements[sql] = @connection.prepare(sql)
-                       end
+              if @statements.key?(sql)
+                cursor = @statements[sql]
+              else
+                cursor = @statements[sql] = @connection.prepare(sql)
+              end
 
               cursor.bind_params(type_casted_binds)
 
@@ -202,7 +202,7 @@ module ActiveRecord
         # Returns default sequence name for table.
         # Will take all or first 26 characters of table name and append _seq suffix
         def default_sequence_name(table_name, primary_key = nil)
-          table_name.to_s.gsub((/(^|\.)([\w$-]{1,#{sequence_name_length-4}})([\w$-]*)$/), '\1\2_seq')
+          table_name.to_s.gsub((/(^|\.)([\w$-]{1,#{sequence_name_length - 4}})([\w$-]*)$/), '\1\2_seq')
         end
 
         # Inserts the given fixture into the table. Overridden to properly handle lobs.
@@ -220,6 +220,13 @@ module ActiveRecord
             write_lobs(table_name, klass, fixture, klass.lob_columns)
           end
         end
+
+        # Oracle Database does not support this feature
+        # Refer https://community.oracle.com/ideas/13845 and consider to vote
+        # if you need this feature.
+        def empty_insert_statement_value
+          raise NotImplementedError
+        end
       end
     end
   end

data/lib/active_record/connection_adapters/oracle_enhanced/database_tasks.rb

@@ -1,21 +1,21 @@
-require 'active_record/base'
+require "active_record/base"
 
 module ActiveRecord
   module ConnectionAdapters
     class OracleEnhancedAdapter
       class DatabaseTasks
-        delegate :connection, :establish_connection, :to => ActiveRecord::Base
+        delegate :connection, :establish_connection, to: ActiveRecord::Base
 
         def initialize(config)
           @config = config
         end
 
         def create
-          system_password = ENV.fetch('ORACLE_SYSTEM_PASSWORD') {
+          system_password = ENV.fetch("ORACLE_SYSTEM_PASSWORD") {
             print "Please provide the SYSTEM password for your Oracle installation (set ORACLE_SYSTEM_PASSWORD to avoid this prompt)\n>"
             $stdin.gets.strip
           }
-          establish_connection(@config.merge('username' => 'SYSTEM', 'password' => system_password))
+          establish_connection(@config.merge("username" => "SYSTEM", "password" => system_password))
           begin
             connection.execute "CREATE USER #{@config['username']} IDENTIFIED BY #{@config['password']}"
           rescue => e
@@ -39,18 +39,18 @@ module ActiveRecord
 
         def purge
           drop
-          connection.execute('PURGE RECYCLEBIN') rescue nil
+          connection.execute("PURGE RECYCLEBIN") rescue nil
         end
 
-        def structure_dump(filename)
+        def structure_dump(filename, extra_flags)
           establish_connection(@config)
-          File.open(filename, 'w:utf-8') { |f| f << connection.structure_dump }
-          if @config['structure_dump'] == 'db_stored_code'
-             File.open(filename, 'a') { |f| f << connection.structure_dump_db_stored_code }
+          File.open(filename, "w:utf-8") { |f| f << connection.structure_dump }
+          if @config["structure_dump"] == "db_stored_code"
+            File.open(filename, "a") { |f| f << connection.structure_dump_db_stored_code }
           end
         end
 
-        def structure_load(filename)
+        def structure_load(filename, extra_flags)
           establish_connection(@config)
           connection.execute_structure_dump(File.read(filename))
         end

data/lib/active_record/connection_adapters/oracle_enhanced/jdbc_connection.rb

@@ -2,27 +2,25 @@ begin
   require "java"
   require "jruby"
 
-  # ojdbc7.jar, ojdbc6.jar or ojdbc5.jar file should be in application ./lib directory or in load path or in ENV['PATH']
+  # ojdbc7.jar or ojdbc6.jar file should be in application ./lib directory or in load path or in ENV['PATH']
 
   java_version = java.lang.System.getProperty("java.version")
-  ojdbc_jars = if java_version =~ /^1.5/
-    %w(ojdbc5.jar)
-  elsif java_version =~ /^1.6/
-    %w(ojdbc6.jar)
-  elsif java_version >= '1.7'
-    # Oracle 11g client ojdbc6.jar is also compatible with Java 1.7
-    # Oracle 12c client provides new ojdbc7.jar
-    %w(ojdbc7.jar ojdbc6.jar)
-  else
-    nil
+  # Dropping Java SE 6(1.6) or older version without deprecation cycle.
+  # Rails 5.0 already requires CRuby 2.2.2 or higher and JRuby 9.0 supporging CRuby 2.2 requires Java SE 7.
+  if java_version < "1.7"
+    raise "ERROR: Java SE 6 or older version is not supported. Upgrade Java version to Java SE 7 or higher"
   end
 
-  if ojdbc_jars && ENV_JAVA['java.class.path'] !~ Regexp.new(ojdbc_jars.join('|'))
+  # Oracle 11g client ojdbc6.jar is also compatible with Java 1.7
+  # Oracle 12c client provides new ojdbc7.jar
+  ojdbc_jars = %w(ojdbc7.jar ojdbc6.jar)
+
+  if ENV_JAVA["java.class.path"] !~ Regexp.new(ojdbc_jars.join("|"))
     # On Unix environment variable should be PATH, on Windows it is sometimes Path
-    env_path = (ENV["PATH"] || ENV["Path"] || '').split(File::PATH_SEPARATOR)
+    env_path = (ENV["PATH"] || ENV["Path"] || "").split(File::PATH_SEPARATOR)
     # Look for JDBC driver at first in lib subdirectory (application specific JDBC file version)
     # then in Ruby load path and finally in environment PATH
-    ['./lib'].concat($LOAD_PATH).concat(env_path).detect do |dir|
+    ["./lib"].concat($LOAD_PATH).concat(env_path).detect do |dir|
       # check any compatible JDBC driver in the priority order
       ojdbc_jars.any? do |ojdbc_jar|
         if File.exists?(file_path = File.join(dir, ojdbc_jar))
@@ -43,7 +41,7 @@ begin
 
 rescue LoadError, NameError
   # JDBC driver is unavailable.
-  raise LoadError, "ERROR: ActiveRecord oracle_enhanced adapter could not load Oracle JDBC driver. Please install #{ojdbc_jars ? ojdbc_jars.join(' or ') : "Oracle JDBC"} library."
+  raise LoadError, "ERROR: ActiveRecord oracle_enhanced adapter could not load Oracle JDBC driver. Please install #{ojdbc_jars.join(' or ') } library."
 end
 
 module ActiveRecord
@@ -74,7 +72,7 @@ module ActiveRecord
 
           # tomcat needs first lookup method, oc4j (and maybe other application servers) need second method
           begin
-            env = ctx.lookup('java:/comp/env')
+            env = ctx.lookup("java:/comp/env")
             ds = env.lookup(jndi)
           rescue
             ds = ctx.lookup(jndi)
@@ -102,13 +100,13 @@ module ActiveRecord
           # to_s needed if username, password or database is specified as number in database.yml file
           username = config[:username] && config[:username].to_s
           password = config[:password] && config[:password].to_s
-          database = config[:database] && config[:database].to_s || 'XE'
+          database = config[:database] && config[:database].to_s || "XE"
           host, port = config[:host], config[:port]
           privilege = config[:privilege] && config[:privilege].to_s
 
           # connection using TNS alias, or connection-string from DATABASE_URL
-          using_tns_alias = !host && !config[:url] && ENV['TNS_ADMIN']
-          if database && (using_tns_alias || host == 'connection-string')
+          using_tns_alias = !host && !config[:url] && ENV["TNS_ADMIN"]
+          if database && (using_tns_alias || host == "connection-string")
             url = "jdbc:oracle:thin:@#{database}"
           else
             unless database.match(/^(\:|\/)/)
@@ -120,7 +118,7 @@ module ActiveRecord
 
           prefetch_rows = config[:prefetch_rows] || 100
           # get session time_zone from configuration or from TZ environment variable
-          time_zone = config[:time_zone] || ENV['TZ'] || java.util.TimeZone.default.getID
+          time_zone = config[:time_zone] || ENV["TZ"] || java.util.TimeZone.default.getID
 
           properties = java.util.Properties.new
           properties.put("user", username)
@@ -144,7 +142,7 @@ module ActiveRecord
           # @raw_connection.setDefaultRowPrefetch(prefetch_rows) if prefetch_rows
         end
 
-        cursor_sharing = config[:cursor_sharing] || 'force'
+        cursor_sharing = config[:cursor_sharing] || "force"
         exec "alter session set cursor_sharing = #{cursor_sharing}"
 
         # Initialize NLS parameters
@@ -327,7 +325,7 @@ module ActiveRecord
           @raw_statement = raw_statement
         end
 
-        def bind_params( *bind_vars )
+        def bind_params(*bind_vars)
           index = 1
           bind_vars.flatten.each do |var|
             if Hash === var
@@ -339,17 +337,7 @@ module ActiveRecord
           end
         end
 
-        def bind_param(position, value, column = nil)
-          if column
-            ActiveSupport::Deprecation.warn(<<-MSG.squish)
-              *******************************************************
-              Passing a column to `bind_param` will be deprecated.
-              `type_casted_binds` should be already type casted
-              so that `bind_param` should not need to know column.
-              *******************************************************
-            MSG
-          end
-
+        def bind_param(position, value)
           case value
           when Integer
             @raw_statement.setLong(position, value)
@@ -376,14 +364,10 @@ module ActiveRecord
             # TODO: Really needed or not
             @raw_statement.setTimestamp(position, value)
           when NilClass
-            if column && column.object_type?
-              @raw_statement.setNull(position, java.sql.Types::STRUCT, column.sql_type)
-            else
-              # TODO: currently nil is always bound as NULL with VARCHAR type.
-              # When nils will actually be used by ActiveRecord as bound parameters
-              # then need to pass actual column type.
-              @raw_statement.setNull(position, java.sql.Types::VARCHAR)
-            end
+            # TODO: currently nil is always bound as NULL with VARCHAR type.
+            # When nils will actually be used by ActiveRecord as bound parameters
+            # then need to pass actual column type.
+            @raw_statement.setNull(position, java.sql.Types::VARCHAR)
           else
             raise ArgumentError, "Don't know how to bind variable with type #{value.class}"
           end
@@ -411,21 +395,21 @@ module ActiveRecord
         end
 
         def column_types
-          @column_types ||= (1..metadata.getColumnCount).map{|i| metadata.getColumnTypeName(i).to_sym}
+          @column_types ||= (1..metadata.getColumnCount).map { |i| metadata.getColumnTypeName(i).to_sym }
         end
 
         def column_names
-          @column_names ||= (1..metadata.getColumnCount).map{|i| metadata.getColumnName(i)}
+          @column_names ||= (1..metadata.getColumnCount).map { |i| metadata.getColumnName(i) }
         end
         alias :get_col_names :column_names
 
-        def fetch(options={})
+        def fetch(options = {})
           if @raw_result_set.next
             get_lob_value = options[:get_lob_value]
             row_values = []
             column_types.each_with_index do |column_type, i|
               row_values <<
-                @connection.get_ruby_value_from_result_set(@raw_result_set, i+1, column_type, get_lob_value)
+                @connection.get_ruby_value_from_result_set(@raw_result_set, i + 1, column_type, get_lob_value)
             end
             row_values
           else
@@ -469,14 +453,14 @@ module ActiveRecord
 
         cols_types_index = (1..column_count).map do |i|
           col_name = oracle_downcase(metadata.getColumnName(i))
-          next if col_name == 'raw_rnum_'
+          next if col_name == "raw_rnum_"
           column_hash[col_name] = nil
           [col_name, metadata.getColumnTypeName(i).to_sym, i]
         end
         cols_types_index.delete(nil)
 
         rows = []
-        get_lob_value = !(name == 'Writable Large Object')
+        get_lob_value = !(name == "Writable Large Object")
 
         while rset.next
           hash = column_hash.dup
@@ -496,7 +480,7 @@ module ActiveRecord
         if is_binary
           lob.setBytes(1, value.to_java_bytes)
         else
-          lob.setString(1,value)
+          lob.setString(1, value)
         end
       end
 
@@ -521,19 +505,13 @@ module ActiveRecord
           else
             BigDecimal.new(d.stringValue)
           end
-        when :BINARY_FLOAT
-          rset.getFloat(i)
         when :VARCHAR2, :CHAR, :LONG, :NVARCHAR2, :NCHAR
           rset.getString(i)
         when :DATE
           if dt = rset.getDATE(i)
             d = dt.dateValue
             t = dt.timeValue
-            if OracleEnhancedAdapter.emulate_dates && t.hours == 0 && t.minutes == 0 && t.seconds == 0
-              Date.new(d.year + 1900, d.month + 1, d.date)
-            else
-              Time.send(Base.default_timezone, d.year + 1900, d.month + 1, d.date, t.hours, t.minutes, t.seconds)
-            end
+            Time.send(Base.default_timezone, d.year + 1900, d.month + 1, d.date, t.hours, t.minutes, t.seconds)
           else
             nil
           end
@@ -547,7 +525,7 @@ module ActiveRecord
           get_lob_value ? lob_to_ruby_value(rset.getBlob(i)) : rset.getBlob(i)
         when :RAW
           raw_value = rset.getRAW(i)
-          raw_value && raw_value.getBytes.to_a.pack('C*')
+          raw_value && raw_value.getBytes.to_a.pack("C*")
         else
           nil
         end
@@ -555,22 +533,22 @@ module ActiveRecord
 
       private
 
-      def lob_to_ruby_value(val)
-        case val
-        when ::Java::OracleSql::CLOB
-          if val.isEmptyLob
-            nil
-          else
-            val.getSubString(1, val.length)
-          end
-        when ::Java::OracleSql::BLOB
-          if val.isEmptyLob
-            nil
-          else
-            String.from_java_bytes(val.getBytes(1, val.length))
+        def lob_to_ruby_value(val)
+          case val
+          when ::Java::OracleSql::CLOB
+            if val.isEmptyLob
+              nil
+            else
+              val.getSubString(1, val.length)
+            end
+          when ::Java::OracleSql::BLOB
+            if val.isEmptyLob
+              nil
+            else
+              String.from_java_bytes(val.getBytes(1, val.length))
+            end
           end
         end
-      end
     end
   end
 end

data/lib/active_record/connection_adapters/oracle_enhanced/jdbc_quoting.rb

@@ -15,10 +15,7 @@ module ActiveRecord
           when Date, DateTime
             Java::oracle.sql.DATE.new(value.strftime("%Y-%m-%d %H:%M:%S"))
           when Time
-            Java::java.sql.Timestamp.new(value.year-1900, value.month-1, value.day, value.hour, value.min, value.sec, value.usec * 1000)
-          when Java::JavaSql::Timestamp
-            # Returning value as it is likely this value was already type casted from Time to Java::JavaSql::Timestamp
-            value
+            Java::java.sql.Timestamp.new(value.year - 1900, value.month - 1, value.day, value.hour, value.min, value.sec, value.usec * 1000)
           else
             super
           end

data/lib/active_record/connection_adapters/oracle_enhanced/oci_connection.rb

@@ -1,4 +1,4 @@
-require 'delegate'
+require "delegate"
 
 begin
   require "oci8"
@@ -11,7 +11,7 @@ end
 
 # check ruby-oci8 version
 required_oci8_version = [2, 2, 0]
-oci8_version_ints = OCI8::VERSION.scan(/\d+/).map{|s| s.to_i}
+oci8_version_ints = OCI8::VERSION.scan(/\d+/).map { |s| s.to_i }
 if (oci8_version_ints <=> required_oci8_version) < 0
   raise LoadError, "ERROR: ruby-oci8 version #{OCI8::VERSION} is too old. Please install ruby-oci8 version #{required_oci8_version.join('.')} or later."
 end
@@ -98,9 +98,9 @@ module ActiveRecord
       # and return :insert_id value
       def exec_with_returning(sql)
         cursor = @raw_connection.parse(sql)
-        cursor.bind_param(':insert_id', nil, Integer)
+        cursor.bind_param(":insert_id", nil, Integer)
         cursor.exec
-        cursor[':insert_id']
+        cursor[":insert_id"]
       ensure
         cursor.close rescue nil
       end
@@ -115,7 +115,7 @@ module ActiveRecord
           @raw_cursor = raw_cursor
         end
 
-        def bind_params( *bind_vars )
+        def bind_params(*bind_vars)
           index = 1
           bind_vars.flatten.each do |var|
             if Hash === var
@@ -127,30 +127,16 @@ module ActiveRecord
           end
         end
 
-        def bind_param(position, value, column = nil)
-          if column
-            ActiveSupport::Deprecation.warn(<<-MSG.squish)
-              *******************************************************
-              Passing a column to `bind_param` will be deprecated.
-              `type_casted_binds` should be already type casted
-              so that `bind_param` should not need to know column.
-              *******************************************************
-            MSG
-          end
-
-          if column && column.object_type?
-            @raw_cursor.bind_param(position, value, :named_type, column.sql_type)
+        def bind_param(position, value)
+          case value
+          when ActiveRecord::OracleEnhanced::Type::Raw
+            @raw_cursor.bind_param(position, ActiveRecord::ConnectionAdapters::OracleEnhanced::Quoting.encode_raw(value))
+          when ActiveModel::Type::Decimal
+            @raw_cursor.bind_param(position, BigDecimal.new(value.to_s))
+          when NilClass
+            @raw_cursor.bind_param(position, nil, String)
           else
-            case value
-            when ActiveRecord::OracleEnhanced::Type::Raw
-              @raw_cursor.bind_param(position, ActiveRecord::ConnectionAdapters::OracleEnhanced::Quoting.encode_raw(value))
-            when ActiveModel::Type::Decimal
-              @raw_cursor.bind_param(position, BigDecimal.new(value.to_s))
-            when NilClass
-              @raw_cursor.bind_param(position, nil, String)
-            else
-              @raw_cursor.bind_param(position, value)
-            end
+            @raw_cursor.bind_param(position, value)
           end
         end
 
@@ -170,7 +156,7 @@ module ActiveRecord
           @raw_cursor.get_col_names
         end
 
-        def fetch(options={})
+        def fetch(options = {})
           if row = @raw_cursor.fetch
             get_lob_value = options[:get_lob_value]
             row.map do |col|
@@ -194,13 +180,13 @@ module ActiveRecord
         # Ignore raw_rnum_ which is used to simulate LIMIT and OFFSET
         cursor.get_col_names.each do |col_name|
           col_name = oracle_downcase(col_name)
-          cols << col_name unless col_name == 'raw_rnum_'
+          cols << col_name unless col_name == "raw_rnum_"
         end
         # Reuse the same hash for all rows
         column_hash = {}
-        cols.each {|c| column_hash[c] = nil}
+        cols.each { |c| column_hash[c] = nil }
         rows = []
-        get_lob_value = !(name == 'Writable Large Object')
+        get_lob_value = !(name == "Writable Large Object")
 
         while row = cursor.fetch
           hash = column_hash.dup
@@ -223,7 +209,7 @@ module ActiveRecord
 
       def describe(name)
         # fall back to SELECT based describe if using database link
-        return super if name.to_s.include?('@')
+        return super if name.to_s.include?("@")
         quoted_name = ActiveRecord::ConnectionAdapters::OracleEnhanced::Quoting.valid_table_name?(name) ? name : "\"#{name}\""
         @raw_connection.describe(quoted_name)
       rescue OCIException => e
@@ -261,17 +247,13 @@ module ActiveRecord
           if get_lob_value
             data = value.read || ""     # if value.read returns nil, then we have an empty_clob() i.e. an empty string
             # In Ruby 1.9.1 always change encoding to ASCII-8BIT for binaries
-            data.force_encoding('ASCII-8BIT') if data.respond_to?(:force_encoding) && value.is_a?(OCI8::BLOB)
+            data.force_encoding("ASCII-8BIT") if data.respond_to?(:force_encoding) && value.is_a?(OCI8::BLOB)
             data
           else
             value
           end
         when Time, DateTime
-          if OracleEnhancedAdapter.emulate_dates && date_without_time?(value)
-            value.to_date
-          else
-            create_time_with_default_timezone(value)
-          end
+          create_time_with_default_timezone(value)
         else
           value
         end
@@ -283,32 +265,32 @@ module ActiveRecord
 
       private
 
-      def date_without_time?(value)
-        case value
-        when OraDate
-          value.hour == 0 && value.minute == 0 && value.second == 0
-        else
-          value.hour == 0 && value.min == 0 && value.sec == 0
+        def date_without_time?(value)
+          case value
+          when OraDate
+            value.hour == 0 && value.minute == 0 && value.second == 0
+          else
+            value.hour == 0 && value.min == 0 && value.sec == 0
+          end
         end
-      end
 
-      def create_time_with_default_timezone(value)
-        year, month, day, hour, min, sec, usec = case value
-        when Time
-          [value.year, value.month, value.day, value.hour, value.min, value.sec, value.usec]
-        when OraDate
-          [value.year, value.month, value.day, value.hour, value.minute, value.second, 0]
-        else
-          [value.year, value.month, value.day, value.hour, value.min, value.sec, 0]
-        end
-        # code from Time.time_with_datetime_fallback
-        begin
-          Time.send(Base.default_timezone, year, month, day, hour, min, sec, usec)
-        rescue
-          offset = Base.default_timezone.to_sym == :local ? ::DateTime.local_offset : 0
-          ::DateTime.civil(year, month, day, hour, min, sec, offset)
+        def create_time_with_default_timezone(value)
+          year, month, day, hour, min, sec, usec = case value
+                                                   when Time
+                                                     [value.year, value.month, value.day, value.hour, value.min, value.sec, value.usec]
+                                                   when OraDate
+                                                     [value.year, value.month, value.day, value.hour, value.minute, value.second, 0]
+          else
+                                                     [value.year, value.month, value.day, value.hour, value.min, value.sec, 0]
+          end
+          # code from Time.time_with_datetime_fallback
+          begin
+            Time.send(Base.default_timezone, year, month, day, hour, min, sec, usec)
+          rescue
+            offset = Base.default_timezone.to_sym == :local ? ::DateTime.local_offset : 0
+            ::DateTime.civil(year, month, day, hour, min, sec, offset)
+          end
         end
-      end
     end
 
     # The OracleEnhancedOCIFactory factors out the code necessary to connect and
@@ -324,16 +306,16 @@ module ActiveRecord
         privilege = config[:privilege] && config[:privilege].to_sym
         async = config[:allow_concurrency]
         prefetch_rows = config[:prefetch_rows] || 100
-        cursor_sharing = config[:cursor_sharing] || 'force'
+        cursor_sharing = config[:cursor_sharing] || "force"
         # get session time_zone from configuration or from TZ environment variable
-        time_zone = config[:time_zone] || ENV['TZ']
+        time_zone = config[:time_zone] || ENV["TZ"]
 
         # using a connection string via DATABASE_URL
-        connection_string = if host == 'connection-string'
+        connection_string = if host == "connection-string"
           database
         # connection using host, port and database name
         elsif host || port
-          host ||= 'localhost'
+          host ||= "localhost"
           host = "[#{host}]" if host =~ /^[^\[].*:/  # IPv6
           port ||= 1521
           database = "/#{database}" unless database.match(/^\//)
@@ -372,8 +354,8 @@ class OCI8 #:nodoc:
   def describe(name)
     info = describe_table(name.to_s)
     raise %Q{"DESC #{name}" failed} if info.nil?
-    if info.respond_to? :obj_link and info.obj_link
-      [info.obj_schema, info.obj_name, '@' + info.obj_link]
+    if info.respond_to?(:obj_link) && info.obj_link
+      [info.obj_schema, info.obj_name, "@" + info.obj_link]
     else
       [info.obj_schema, info.obj_name]
     end
@@ -402,7 +384,7 @@ class OCI8EnhancedAutoRecover < DelegateClass(OCI8) #:nodoc:
     @active = true
     @config = config
     @factory = factory
-    @connection  = @factory.new_connection @config
+    @connection = @factory.new_connection @config
     super @connection
   end