activerecord-oracle_enhanced-adapter 1.7.3 → 1.8.0

data/lib/active_record/oracle_enhanced/type/boolean.rb

@@ -2,20 +2,16 @@ module ActiveRecord
   module OracleEnhanced
     module Type
       class Boolean < ActiveModel::Type::Boolean # :nodoc:
-        # Add 'N' as FALSE_VALUES
-        FALSE_VALUES = [false, 0, '0', 'f', 'F', 'false', 'FALSE', 'off', 'OFF', 'n', 'N'].to_set
-
         private
 
-        def cast_value(value)
-          # Not calling super to use its own `FALSE_VALUES`
-          if value == ''
-            nil
-          else
-            !FALSE_VALUES.include?(value)
+          def cast_value(value)
+            # Kind of adding 'n' and 'N' to  `FALSE_VALUES`
+            if ["n", "N"].include?(value)
+              false
+            else
+              super
+            end
           end
-        end
-
       end
     end
   end

data/lib/active_record/oracle_enhanced/type/integer.rb

@@ -1,13 +1,12 @@
-#TODO Need to consider namespace change since paremt class moved to ActiveModel
-module ActiveRecord 
+module ActiveRecord
   module OracleEnhanced
     module Type
       class Integer < ActiveModel::Type::Integer # :nodoc:
         private
 
-        def max_value
-          ("9"*38).to_i
-        end
+          def max_value
+            ("9" * 38).to_i
+          end
       end
     end
   end

data/lib/active_record/oracle_enhanced/type/json.rb

@@ -0,0 +1,8 @@
+module ActiveRecord
+  module OracleEnhanced
+    module Type
+      class Json < ActiveRecord::Type::Internal::AbstractJson
+      end
+    end
+  end
+end

data/lib/active_record/oracle_enhanced/type/national_character_string.rb

@@ -1,10 +1,9 @@
-require 'active_model/type/string'
+require "active_model/type/string"
 
 module ActiveRecord
   module OracleEnhanced
     module Type
       class NationalCharacterString < ActiveRecord::OracleEnhanced::Type::String # :nodoc:
-
         def serialize(value)
           return unless value
           Data.new(super)

data/lib/active_record/oracle_enhanced/type/raw.rb

@@ -1,6 +1,5 @@
-require 'active_model/type/string'
+require "active_model/type/string"
 
-#TODO Need to consider namespace change since paremt class moved to ActiveModel
 module ActiveRecord
   module OracleEnhanced
     module Type
@@ -14,11 +13,10 @@ module ActiveRecord
           if value.nil?
             super
           else
-            value = value.unpack('C*')
+            value = value.unpack("C*")
             value.map { |x| "%02X" % x }.join
           end
         end
-
       end
     end
   end

data/lib/active_record/oracle_enhanced/type/string.rb

@@ -1,4 +1,4 @@
-require 'active_model/type/string'
+require "active_model/type/string"
 
 module ActiveRecord
   module OracleEnhanced
@@ -15,13 +15,12 @@ module ActiveRecord
 
         def changed_in_place?(raw_old_value, new_value)
           if raw_old_value.nil?
-            new_value = nil if new_value == ''
+            new_value = nil if new_value == ""
             raw_old_value != new_value
           else
             super
           end
         end
-
       end
     end
   end

data/lib/active_record/oracle_enhanced/type/text.rb

@@ -1,10 +1,9 @@
-require 'active_model/type/string'
+require "active_model/type/string"
 
 module ActiveRecord
   module OracleEnhanced
     module Type
-      class Text < ActiveModel::Type::Text # :nodoc:
-
+      class Text < ActiveRecord::Type::Text # :nodoc:
         def changed_in_place?(raw_old_value, new_value)
           #TODO: Needs to find a way not to cast here.
           raw_old_value = cast(raw_old_value)
@@ -25,7 +24,6 @@ module ActiveRecord
             @value
           end
         end
-
       end
     end
   end

data/lib/active_record/oracle_enhanced/type/timestamptz.rb

@@ -0,0 +1,23 @@
+module ActiveRecord
+  module OracleEnhanced
+    module Type
+      class TimestampTz < ActiveRecord::Type::DateTime
+        def type
+          :timestamptz
+        end
+
+        class Data < DelegateClass(::Time) # :nodoc:
+        end
+
+        def serialize(value)
+          case value = super
+          when ::Time
+            Data.new(value)
+          else
+            value
+          end
+        end
+      end
+    end
+  end
+end

data/lib/activerecord-oracle_enhanced-adapter.rb

@@ -1,20 +1,21 @@
-module ActiveRecord
-  module ConnectionAdapters
-    class OracleEnhancedRailtie < ::Rails::Railtie
-      rake_tasks do
-        load 'active_record/connection_adapters/oracle_enhanced/database_tasks.rb'
-      end
+if defined?(Rails)
+  module ActiveRecord
+    module ConnectionAdapters
+      class OracleEnhancedRailtie < ::Rails::Railtie
+        rake_tasks do
+          load "active_record/connection_adapters/oracle_enhanced/database_tasks.rb"
+        end
 
-      ActiveSupport.on_load(:active_record) do
-        require 'active_record/connection_adapters/oracle_enhanced_adapter'
+        ActiveSupport.on_load(:active_record) do
+          require "active_record/connection_adapters/oracle_enhanced_adapter"
 
-        # Cache column descriptions between requests in test and production environments
-        if Rails.env.test? || Rails.env.production?
-          ActiveRecord::ConnectionAdapters::OracleEnhancedAdapter.cache_columns = true
-        end
+          # Cache column descriptions between requests in test and production environments
+          if Rails.env.test? || Rails.env.production?
+            ActiveRecord::ConnectionAdapters::OracleEnhancedAdapter.cache_columns = true
+          end
 
+        end
       end
-
     end
   end
 end