DRY up: Extract #remove_index override into Module

Signed-off-by: Alex Coles <alex@alexbcoles.com>

lib/schema_plus/indexes.rb

@@ -1,6 +1,7 @@
 require 'schema_plus/core'
 require 'its-it'
 
+require_relative 'indexes/remove_if_exists'
 require_relative 'indexes/active_record/base'
 require_relative 'indexes/active_record/connection_adapters/abstract_adapter'
 require_relative 'indexes/active_record/connection_adapters/postgresql_adapter'

lib/schema_plus/indexes/active_record/connection_adapters/abstract_adapter.rb

@@ -2,17 +2,7 @@ module SchemaPlus::Indexes
   module ActiveRecord
     module ConnectionAdapters
       module AbstractAdapter
-
-        # Extends rails' remove_index to include this options:
-        #   :if_exists
-        def remove_index(table_name, *args)
-          options = args.extract_options!
-          if_exists = options.delete(:if_exists)
-          args << options if options.any?
-          return if if_exists and not index_name_exists?(table_name, options[:name] || index_name(table_name, *args), false)
-          super table_name, *args
-        end
-
+        include ::SchemaPlus::Indexes::RemoveIfExists
       end
     end
   end

lib/schema_plus/indexes/active_record/connection_adapters/postgresql_adapter.rb

@@ -2,17 +2,7 @@ module SchemaPlus::Indexes
   module ActiveRecord
     module ConnectionAdapters
       module PostgresqlAdapter
-
-        # Extends rails' remove_index to include this options:
-        #   :if_exists
-        def remove_index(table_name, *args)
-          options = args.extract_options!
-          if_exists = options.delete(:if_exists)
-          args << options if options.any?
-          return if if_exists and not index_name_exists?(table_name, options[:name] || index_name(table_name, *args), false)
-          super table_name, *args
-        end
-
+        include ::SchemaPlus::Indexes::RemoveIfExists
       end
     end
   end

lib/schema_plus/indexes/active_record/connection_adapters/sqlite3_adapter.rb

@@ -2,17 +2,7 @@ module SchemaPlus::Indexes
   module ActiveRecord
     module ConnectionAdapters
       module Sqlite3Adapter
-
-        # Extends rails' remove_index to include this options:
-        #   :if_exists
-        def remove_index(table_name, *args)
-          options = args.extract_options!
-          if_exists = options.delete(:if_exists)
-          args << options if options.any?
-          return if if_exists and not index_name_exists?(table_name, options[:name] || index_name(table_name, *args), false)
-          super table_name, *args
-        end
-
+        include ::SchemaPlus::Indexes::RemoveIfExists
       end
     end
   end

lib/schema_plus/indexes/remove_if_exists.rb

@@ -0,0 +1,13 @@
+module SchemaPlus::Indexes
+  module RemoveIfExists
+    # Extends rails' remove_index to include this options:
+    #   :if_exists
+    def remove_index(table_name, *args)
+      options = args.extract_options!
+      if_exists = options.delete(:if_exists)
+      args << options if options.any?
+      return if if_exists && !index_name_exists?(table_name, options[:name] || index_name(table_name, *args), false)
+      super table_name, *args
+    end
+  end
+end