pmacs-activerecord-oracle_enhanced-adapter 1.4.2.rc1 → 1.5.5.1

data/lib/active_record/connection_adapters/oracle_enhanced_schema_creation.rb

@@ -0,0 +1,89 @@
+module ActiveRecord
+  module ConnectionAdapters
+    class OracleEnhancedAdapter < AbstractAdapter
+      class SchemaCreation < AbstractAdapter::SchemaCreation
+        private
+
+        def visit_ColumnDefinition(o)
+          if o.type.to_sym == :virtual
+            sql_type = type_to_sql(o.default[:type], o.limit, o.precision, o.scale) if o.default[:type]
+            "#{quote_column_name(o.name)} #{sql_type} AS (#{o.default[:as]})"
+          else
+            super
+          end
+        end
+
+        def visit_TableDefinition(o)
+          tablespace = tablespace_for(:table, o.options[:tablespace])
+          create_sql = "CREATE#{' GLOBAL TEMPORARY' if o.temporary} TABLE "
+          create_sql << "#{quote_table_name(o.name)} ("
+          create_sql << o.columns.map { |c| accept c }.join(', ')
+          create_sql << ")"
+          unless o.temporary
+            create_sql << " ORGANIZATION #{o.options[:organization]}" if o.options[:organization]
+            create_sql << "#{tablespace}"
+          end
+          create_sql << " #{o.options[:options]}"
+          create_sql
+        end
+
+        def tablespace_for(obj_type, tablespace_option, table_name=nil, column_name=nil)
+          tablespace_sql = ''
+          if tablespace = (tablespace_option || default_tablespace_for(obj_type))
+            tablespace_sql << if [:blob, :clob].include?(obj_type.to_sym)
+              " LOB (#{quote_column_name(column_name)}) STORE AS #{column_name.to_s[0..10]}_#{table_name.to_s[0..14]}_ls (TABLESPACE #{tablespace})"
+            else
+              " TABLESPACE #{tablespace}"
+            end
+          end
+          tablespace_sql
+        end
+
+        def default_tablespace_for(type)
+          (ActiveRecord::ConnectionAdapters::OracleEnhancedAdapter.default_tablespaces[type] || 
+           ActiveRecord::ConnectionAdapters::OracleEnhancedAdapter.default_tablespaces[native_database_types[type][:name]]) rescue nil
+        end
+
+        def foreign_key_definition(to_table, options = {})
+          @conn.foreign_key_definition(to_table, options)
+        end
+
+        def add_column_options!(sql, options)
+          type = options[:type] || ((column = options[:column]) && column.type)
+          type = type && type.to_sym
+          # handle case of defaults for CLOB columns, which would otherwise get "quoted" incorrectly
+          if options_include_default?(options)
+            if type == :text
+              sql << " DEFAULT #{@conn.quote(options[:default])}"
+            else
+              # from abstract adapter
+              sql << " DEFAULT #{@conn.quote(options[:default], options[:column])}"
+            end
+          end
+          # must explicitly add NULL or NOT NULL to allow change_column to work on migrations
+          if options[:null] == false
+            sql << " NOT NULL"
+          elsif options[:null] == true
+            sql << " NULL" unless type == :primary_key
+          end
+          # add AS expression for virtual columns
+          if options[:as].present?
+            sql << " AS (#{options[:as]})"
+          end
+        end
+
+        # This method does not exist in SchemaCreation at Rails 4.0
+        # It can be removed only when Oracle enhanced adapter supports Rails 4.1 and higher
+        def options_include_default?(options)
+          options.include?(:default) && !(options[:null] == false && options[:default].nil?)
+        end
+
+      end
+      
+      def schema_creation
+          SchemaCreation.new self
+      end
+
+    end
+  end
+end

data/lib/active_record/connection_adapters/oracle_enhanced_schema_definitions.rb

@@ -13,8 +13,8 @@ module ActiveRecord
     module OracleEnhancedColumnDefinition
        def self.included(base) #:nodoc:
         base.class_eval do
-          alias_method_chain :to_sql, :virtual_columns
-          alias to_s :to_sql
+          #alias_method_chain :to_sql, :virtual_columns
+          #alias to_s :to_sql
         end
        end
 
@@ -42,10 +42,6 @@ module ActiveRecord
           include OracleEnhancedTableDefinition
         end
 
-        base::ColumnDefinition.class_eval do
-          include OracleEnhancedColumnDefinition
-        end
-        
         # Available starting from ActiveRecord 2.1
         base::Table.class_eval do
           include OracleEnhancedTable
@@ -64,8 +60,6 @@ module ActiveRecord
       def self.included(base) #:nodoc:
         base.class_eval do
           alias_method_chain :references, :foreign_keys
-          alias_method_chain :to_sql, :foreign_keys
-
           alias_method_chain :column, :virtual_columns
         end
       end
@@ -110,11 +104,15 @@ module ActiveRecord
       # Note: If no name is specified, the database driver creates one for you!
       def references_with_foreign_keys(*args)
         options = args.extract_options!
+        index_options = options[:index]
         fk_options = options.delete(:foreign_key)
 
         if fk_options && !options[:polymorphic]
           fk_options = {} if fk_options == true
-          args.each { |to_table| foreign_key(to_table, fk_options) }
+          args.each do |to_table| 
+            foreign_key(to_table, fk_options) 
+            add_index(to_table, "#{to_table}_id", index_options.is_a?(Hash) ? index_options : nil) if index_options
+          end
         end
 
         references_without_foreign_keys(*(args << options))
@@ -133,7 +131,8 @@ module ActiveRecord
       # ====== Defining the column of the +to_table+.
       #  t.foreign_key(:people, :column => :sender_id, :primary_key => :person_id)
       def foreign_key(to_table, options = {})
-        if @base.respond_to?(:supports_foreign_keys?) && @base.supports_foreign_keys?
+        #TODO
+        if ActiveRecord::Base.connection.supports_foreign_keys?
           to_table = to_table.to_s.pluralize if ActiveRecord::Base.pluralize_table_names
           foreign_keys << ForeignKey.new(@base, to_table, options)
         else
@@ -141,20 +140,9 @@ module ActiveRecord
         end
       end
     
-      def to_sql_with_foreign_keys #:nodoc:
-        sql = to_sql_without_foreign_keys
-        sql << ', ' << (foreign_keys * ', ') unless foreign_keys.blank?
-        sql
+      def foreign_keys
+        @foreign_keys ||= []
       end
-
-      def lob_columns
-        columns.select(&:lob?)
-      end
-    
-      private
-        def foreign_keys
-          @foreign_keys ||= []
-        end
     end
 
     module OracleEnhancedTable
@@ -212,6 +200,7 @@ module ActiveRecord
       def references_with_foreign_keys(*args)
         options = args.extract_options!
         polymorphic = options[:polymorphic]
+        index_options = options[:index]
         fk_options = options.delete(:foreign_key)
 
         references_without_foreign_keys(*(args << options))
@@ -219,7 +208,10 @@ module ActiveRecord
         args.extract_options!
         if fk_options && !polymorphic
           fk_options = {} if fk_options == true
-          args.each { |to_table| foreign_key(to_table, fk_options) }
+          args.each do |to_table| 
+            foreign_key(to_table, fk_options) 
+            add_index(to_table, "#{to_table}_id", index_options.is_a?(Hash) ? index_options : nil) if index_options
+          end
         end
       end
     end

data/lib/active_record/connection_adapters/oracle_enhanced_schema_dumper.rb

@@ -13,10 +13,10 @@ module ActiveRecord #:nodoc:
       private
 
       def ignore_table?(table)
-        [ActiveRecord::Migrator.proper_table_name('schema_migrations'), ignore_tables].flatten.any? do |ignored|
+        ['schema_migrations', ignore_tables].flatten.any? do |ignored|
           case ignored
-          when String; table == ignored
-          when Regexp; table =~ ignored
+          when String; remove_prefix_and_suffix(table) == ignored
+          when Regexp; remove_prefix_and_suffix(table) =~ ignored
           else
             raise StandardError, 'ActiveRecord::SchemaDumper.ignore_tables accepts an array of String and / or Regexp values.'
           end
@@ -51,7 +51,7 @@ module ActiveRecord #:nodoc:
         if @connection.respond_to?(:has_primary_key_trigger?) && @connection.has_primary_key_trigger?(table_name)
           pk, pk_seq = @connection.pk_and_sequence_for(table_name)
           stream.print "  add_primary_key_trigger #{table_name.inspect}"
-          stream.print ", :primary_key => \"#{pk}\"" if pk != 'id'
+          stream.print ", primary_key: \"#{pk}\"" if pk != 'id'
           stream.print "\n\n"
         end
       end
@@ -61,24 +61,24 @@ module ActiveRecord #:nodoc:
           add_foreign_key_statements = foreign_keys.map do |foreign_key|
             statement_parts = [ ('add_foreign_key ' + foreign_key.from_table.inspect) ]
             statement_parts << foreign_key.to_table.inspect
-            
+
             if foreign_key.options[:columns].size == 1
               column = foreign_key.options[:columns].first
               if column != "#{foreign_key.to_table.singularize}_id"
-                statement_parts << (':column => ' + column.inspect)
+                statement_parts << ('column: ' + column.inspect)
               end
-              
+
               if foreign_key.options[:references].first != 'id'
-                statement_parts << (':primary_key => ' + foreign_key.options[:primary_key].inspect)
+                statement_parts << ('primary_key: ' + foreign_key.options[:references].first.inspect)
               end
             else
-              statement_parts << (':columns => ' + foreign_key.options[:columns].inspect)
+              statement_parts << ('columns: ' + foreign_key.options[:columns].inspect)
             end
 
-            statement_parts << (':name => ' + foreign_key.options[:name].inspect)
+            statement_parts << ('name: ' + foreign_key.options[:name].inspect)
             
             unless foreign_key.options[:dependent].blank?
-              statement_parts << (':dependent => ' + foreign_key.options[:dependent].inspect)
+              statement_parts << ('dependent: ' + foreign_key.options[:dependent].inspect)
             end
 
             '  ' + statement_parts.join(', ')
@@ -97,7 +97,7 @@ module ActiveRecord #:nodoc:
             table_name = syn.table_name
             table_name = "#{syn.table_owner}.#{table_name}" if syn.table_owner
             table_name = "#{table_name}@#{syn.db_link}" if syn.db_link
-            stream.print "  add_synonym #{syn.name.inspect}, #{table_name.inspect}, :force => true"
+            stream.print "  add_synonym #{syn.name.inspect}, #{table_name.inspect}, force: true"
             stream.puts
           end
           stream.puts unless syns.empty?
@@ -118,9 +118,9 @@ module ActiveRecord #:nodoc:
               # use table.inspect as it will remove prefix and suffix
               statement_parts = [ ('add_index ' + table.inspect) ]
               statement_parts << index.columns.inspect
-              statement_parts << (':name => ' + index.name.inspect)
-              statement_parts << ':unique => true' if index.unique
-              statement_parts << ':tablespace => ' + index.tablespace.inspect if index.tablespace
+              statement_parts << ('name: ' + index.name.inspect)
+              statement_parts << 'unique: true' if index.unique
+              statement_parts << 'tablespace: ' + index.tablespace.inspect if index.tablespace
             when 'CTXSYS.CONTEXT'
               if index.statement_parameters
                 statement_parts = [ ('add_context_index ' + table.inspect) ]
@@ -128,7 +128,7 @@ module ActiveRecord #:nodoc:
               else
                 statement_parts = [ ('add_context_index ' + table.inspect) ]
                 statement_parts << index.columns.inspect
-                statement_parts << (':name => ' + index.name.inspect)
+                statement_parts << ('name: ' + index.name.inspect)
               end
             else
               # unrecognized index type
@@ -157,42 +157,30 @@ module ActiveRecord #:nodoc:
           tbl.print "  create_table #{table.inspect}"
           
           # addition to make temporary option work
-          tbl.print ", :temporary => true" if @connection.temporary_table?(table)
+          tbl.print ", temporary: true" if @connection.temporary_table?(table)
           
           if columns.detect { |c| c.name == pk }
             if pk != 'id'
-              tbl.print %Q(, :primary_key => "#{pk}")
+              tbl.print %Q(, primary_key: "#{pk}")
             end
           else
-            tbl.print ", :id => false"
+            tbl.print ", id: false"
           end
-          tbl.print ", :force => true"
+          tbl.print ", force: true"
           tbl.puts " do |t|"
 
           # then dump all non-primary key columns
           column_specs = columns.map do |column|
             raise StandardError, "Unknown type '#{column.sql_type}' for column '#{column.name}'" if @types[column.type].nil?
             next if column.name == pk
-            spec = {}
-            spec[:name]      = column.name.inspect
-            spec[:type]      = column.virtual? ? 'virtual' : column.type.to_s
-            spec[:type]      = column.virtual? ? 'virtual' : column.type.to_s
-            spec[:virtual_type] = column.type.inspect if column.virtual?
-            spec[:limit]     = column.limit.inspect if column.limit != @types[column.type][:limit] && column.type != :decimal
-            spec[:precision] = column.precision.inspect if !column.precision.nil?
-            spec[:scale]     = column.scale.inspect if !column.scale.nil?
-            spec[:null]      = 'false' if !column.null
-            spec[:as]        = column.virtual_column_data_default if column.virtual?
-            spec[:default]   = default_string(column.default) if column.has_default? && !column.virtual?
-            (spec.keys - [:name, :type]).each do |k|
-              key_s = (k == :virtual_type ? ":type => " : "#{k.inspect} => ")
-              spec[k] = key_s + spec[k]
-            end
-            spec
+            @connection.column_spec(column, @types)
           end.compact
 
           # find all migration keys used in this table
-          keys = [:name, :limit, :precision, :scale, :default, :null, :as, :virtual_type] & column_specs.map(&:keys).flatten
+          # 
+          # TODO `& column_specs.map(&:keys).flatten` should be executed 
+          # in migration_keys_with_oracle_enhanced
+          keys = @connection.migration_keys & column_specs.map(&:keys).flatten
 
           # figure out the lengths for each column based on above keys
           lengths = keys.map{ |key| column_specs.map{ |spec| spec[key] ? spec[key].length + 2 : 0 }.max }
@@ -230,7 +218,10 @@ module ActiveRecord #:nodoc:
         
         stream
       end
-      
+
+      def remove_prefix_and_suffix(table)
+        table.gsub(/^(#{ActiveRecord::Base.table_name_prefix})(.+)(#{ActiveRecord::Base.table_name_suffix})$/,  "\\2")
+      end 
       
       # remove table name prefix and suffix when doing #inspect (which is used in tables method)
       module TableInspect #:nodoc:

data/lib/active_record/connection_adapters/oracle_enhanced_schema_statements.rb

@@ -1,3 +1,4 @@
+# -*- coding: utf-8 -*-
 require 'digest/sha1'
 
 module ActiveRecord
@@ -22,11 +23,11 @@ module ActiveRecord
       #   end
       #
       # Create primary key trigger (so that you can skip primary key value in INSERT statement).
-      # By default trigger name will be "table_name_pkt", you can override the name with 
+      # By default trigger name will be "table_name_pkt", you can override the name with
       # :trigger_name option (but it is not recommended to override it as then this trigger will
       # not be detected by ActiveRecord model and it will still do prefetching of sequence value).
       # Example:
-      # 
+      #
       #   create_table :users, :primary_key_trigger => true do |t|
       #     # ...
       #   end
@@ -37,12 +38,13 @@ module ActiveRecord
       #     t.string      :first_name, :comment => “Given name”
       #     t.string      :last_name, :comment => “Surname”
       #   end
-      
-      def create_table(name, options = {}, &block)
+
+      def create_table(name, options = {})
         create_sequence = options[:id] != false
         column_comments = {}
-        
-        table_definition = TableDefinition.new(self)
+        temporary = options.delete(:temporary)
+        additional_options = options
+        table_definition = create_table_definition name, temporary, additional_options
         table_definition.primary_key(options[:primary_key] || Base.get_primary_key(name.to_s.singularize)) unless options[:id] == false
 
         # store that primary key was defined in create_table block
@@ -68,7 +70,7 @@ module ActiveRecord
           end
         end
 
-        result = block.call(table_definition) if block
+        yield table_definition if block_given? 
         create_sequence = create_sequence || table_definition.create_sequence
         column_comments = table_definition.column_comments if table_definition.column_comments
         tablespace = tablespace_for(:table, options[:tablespace])
@@ -77,35 +79,38 @@ module ActiveRecord
           drop_table(name, options)
         end
 
-        create_sql = "CREATE#{' GLOBAL TEMPORARY' if options[:temporary]} TABLE "
-        create_sql << quote_table_name(name)
-        create_sql << " (#{table_definition.to_sql})"
-        unless options[:temporary]
-          create_sql << " ORGANIZATION #{options[:organization]}" if options[:organization]
-          create_sql << tablespace
-          table_definition.lob_columns.each{|cd| create_sql << tablespace_for(cd.sql_type.downcase.to_sym, nil, name, cd.name)}
-        end
-        create_sql << " #{options[:options]}"
-        execute create_sql
-        
+        execute schema_creation.accept table_definition
+
         create_sequence_and_trigger(name, options) if create_sequence
-        
+
         add_table_comment name, options[:comment]
         column_comments.each do |column_name, comment|
           add_comment name, column_name, comment
         end
-        
+        table_definition.indexes.each_pair { |c,o| add_index name, c, o }
+
+        unless table_definition.foreign_keys.nil?
+          table_definition.foreign_keys.each do |foreign_key|
+            add_foreign_key(table_definition.name, foreign_key.to_table, foreign_key.options)
+          end
+        end
+      end
+
+      def create_table_definition(name, temporary, options)
+        TableDefinition.new native_database_types, name, temporary, options
       end
 
-      def rename_table(name, new_name) #:nodoc:
+      def rename_table(table_name, new_name) #:nodoc:
         if new_name.to_s.length > table_name_length
           raise ArgumentError, "New table name '#{new_name}' is too long; the limit is #{table_name_length} characters"
         end
         if "#{new_name}_seq".to_s.length > sequence_name_length
           raise ArgumentError, "New sequence name '#{new_name}_seq' is too long; the limit is #{sequence_name_length} characters"
         end
-        execute "RENAME #{quote_table_name(name)} TO #{quote_table_name(new_name)}"
-        execute "RENAME #{quote_table_name("#{name}_seq")} TO #{quote_table_name("#{new_name}_seq")}"
+        execute "RENAME #{quote_table_name(table_name)} TO #{quote_table_name(new_name)}"
+        execute "RENAME #{quote_table_name("#{table_name}_seq")} TO #{quote_table_name("#{new_name}_seq")}" rescue nil
+
+        rename_table_indexes(table_name, new_name)
       end
 
       def drop_table(name, options = {}) #:nodoc:
@@ -114,30 +119,74 @@ module ActiveRecord
         execute "DROP SEQUENCE #{quote_table_name(seq_name)}" rescue nil
       ensure
         clear_table_columns_cache(name)
+        self.all_schema_indexes = nil
+      end
+
+      def initialize_schema_migrations_table
+        sm_table = ActiveRecord::Migrator.schema_migrations_table_name
+
+        unless table_exists?(sm_table)
+          index_name = "#{Base.table_name_prefix}unique_schema_migrations#{Base.table_name_suffix}"
+          if index_name.length > index_name_length
+            truncate_to    = index_name_length - index_name.to_s.length - 1
+            truncated_name = "unique_schema_migrations"[0..truncate_to]
+            index_name     = "#{Base.table_name_prefix}#{truncated_name}#{Base.table_name_suffix}"
+          end
+
+          create_table(sm_table, :id => false) do |schema_migrations_table|
+            schema_migrations_table.column :version, :string, :null => false
+          end
+          add_index sm_table, :version, :unique => true, :name => index_name
+
+          # Backwards-compatibility: if we find schema_info, assume we've
+          # migrated up to that point:
+          si_table = Base.table_name_prefix + 'schema_info' + Base.table_name_suffix
+          if table_exists?(si_table)
+            ActiveSupport::Deprecation.warn "Usage of the schema table `#{si_table}` is deprecated. Please switch to using `schema_migrations` table"
+
+            old_version = select_value("SELECT version FROM #{quote_table_name(si_table)}").to_i
+            assume_migrated_upto_version(old_version)
+            drop_table(si_table)
+          end
+        end
       end
 
       # clear cached indexes when adding new index
       def add_index(table_name, column_name, options = {}) #:nodoc:
         column_names = Array(column_name)
-        index_name   = index_name(table_name, :column => column_names)
+        index_name   = index_name(table_name, column: column_names)
 
         if Hash === options # legacy support, since this param was a string
+          options.assert_valid_keys(:unique, :order, :name, :where, :length, :internal, :tablespace, :options, :using)
+
           index_type = options[:unique] ? "UNIQUE" : ""
           index_name = options[:name].to_s if options.key?(:name)
           tablespace = tablespace_for(:index, options[:tablespace])
+          max_index_length = options.fetch(:internal, false) ? index_name_length : allowed_index_name_length
+          additional_options = options[:options]
         else
+          if options
+            message = "Passing a string as third argument of `add_index` is deprecated and will" +
+              " be removed in Rails 4.1." +
+              " Use add_index(#{table_name.inspect}, #{column_name.inspect}, unique: true) instead"
+
+            ActiveSupport::Deprecation.warn message
+          end
+
           index_type = options
+          additional_options = nil
+          max_index_length = allowed_index_name_length
         end
 
-        if index_name.to_s.length > index_name_length
-          raise ArgumentError, "Index name '#{index_name}' on table '#{table_name}' is too long; the limit is #{index_name_length} characters"
+        if index_name.to_s.length > max_index_length
+          raise ArgumentError, "Index name '#{index_name}' on table '#{table_name}' is too long; the limit is #{max_index_length} characters"
         end
         if index_name_exists?(table_name, index_name, false)
           raise ArgumentError, "Index name '#{index_name}' on table '#{table_name}' already exists"
         end
         quoted_column_names = column_names.map { |e| quote_column_name_or_expression(e) }.join(", ")
 
-        execute "CREATE #{index_type} INDEX #{quote_column_name(index_name)} ON #{quote_table_name(table_name)} (#{quoted_column_names})#{tablespace} #{options[:options]}"
+        execute "CREATE #{index_type} INDEX #{quote_column_name(index_name)} ON #{quote_table_name(table_name)} (#{quoted_column_names})#{tablespace} #{additional_options}"
       ensure
         self.all_schema_indexes = nil
       end
@@ -147,6 +196,14 @@ module ActiveRecord
       def remove_index(table_name, options = {}) #:nodoc:
         index_name = index_name(table_name, options)
         unless index_name_exists?(table_name, index_name, true)
+          # sometimes options can be String or Array with column names
+          options = {} unless options.is_a?(Hash)
+          if options.has_key? :name
+            options_without_column = options.dup
+            options_without_column.delete :column
+            index_name_without_column = index_name(table_name, options_without_column)
+            return index_name_without_column if index_name_exists?(table_name, index_name_without_column, false)
+          end
           raise ArgumentError, "Index name '#{index_name}' on table '#{table_name}' does not exist"
         end
         remove_index!(table_name, index_name)
@@ -166,10 +223,10 @@ module ActiveRecord
         options = {} unless options.is_a?(Hash)
         identifier_max_length = options[:identifier_max_length] || index_name_length
         return default_name if default_name.length <= identifier_max_length
-        
+
         # remove 'index', 'on' and 'and' keywords
         shortened_name = "i_#{table_name}_#{Array(options[:column]) * '_'}"
-        
+
         # leave just first three letters from each word
         if shortened_name.length > identifier_max_length
           shortened_name = shortened_name.split('_').map{|w| w[0,3]}.join('_')
@@ -221,6 +278,8 @@ module ActiveRecord
         add_column_sql << tablespace_for((type_to_sql(type).downcase.to_sym), nil, table_name, column_name) if type
 
         execute(add_column_sql)
+
+        create_sequence_and_trigger(table_name, options) if type && type.to_sym == :primary_key
       ensure
         clear_table_columns_cache(table_name)
       end
@@ -266,25 +325,17 @@ module ActiveRecord
 
       def rename_column(table_name, column_name, new_column_name) #:nodoc:
         execute "ALTER TABLE #{quote_table_name(table_name)} RENAME COLUMN #{quote_column_name(column_name)} to #{quote_column_name(new_column_name)}"
+        self.all_schema_indexes = nil
+        rename_column_indexes(table_name, column_name, new_column_name)
       ensure
         clear_table_columns_cache(table_name)
       end
 
-      def remove_column(table_name, *column_names) #:nodoc:
-        major, minor = ActiveRecord::VERSION::MAJOR, ActiveRecord::VERSION::MINOR
-        is_deprecated = (major == 3 and minor >= 2) or major > 3
-
-        if column_names.flatten! and is_deprecated
-          message = 'Passing array to remove_columns is deprecated, please use ' +
-            'multiple arguments, like: `remove_columns(:posts, :foo, :bar)`'
-          ActiveSupport::Deprecation.warn message, caller
-        end
-
-        column_names.each do |column_name|
-          execute "ALTER TABLE #{quote_table_name(table_name)} DROP COLUMN #{quote_column_name(column_name)}"
-        end
+      def remove_column(table_name, column_name, type = nil, options = {}) #:nodoc:
+        execute "ALTER TABLE #{quote_table_name(table_name)} DROP COLUMN #{quote_column_name(column_name)}"
       ensure
         clear_table_columns_cache(table_name)
+        self.all_schema_indexes = nil
       end
 
       def add_comment(table_name, column_name, comment) #:nodoc: