aboutsummaryrefslogtreecommitdiffstats
path: root/activerecord/lib
diff options
context:
space:
mode:
authorRafael Mendonça França <rafaelmfranca@gmail.com>2013-05-06 12:23:56 -0700
committerRafael Mendonça França <rafaelmfranca@gmail.com>2013-05-06 12:23:56 -0700
commit4be5bce125a19a5bc0a60b33e08fdaf2066fa4d9 (patch)
treee9d684ba94974f8ebb2780c51d1b8178575e9dc4 /activerecord/lib
parent32b4abddfa1f01009e0a9eb2a8820715b2fae78a (diff)
parent3771e4d51122e1ec22728029bae00f121d5d4e3b (diff)
downloadrails-4be5bce125a19a5bc0a60b33e08fdaf2066fa4d9.tar.gz
rails-4be5bce125a19a5bc0a60b33e08fdaf2066fa4d9.tar.bz2
rails-4be5bce125a19a5bc0a60b33e08fdaf2066fa4d9.zip
Merge pull request #10437 from neerajdotname/10419
raise IrreversibleMigration if no column given
Diffstat (limited to 'activerecord/lib')
-rw-r--r--activerecord/lib/active_record/migration/command_recorder.rb5
1 files changed, 4 insertions, 1 deletions
diff --git a/activerecord/lib/active_record/migration/command_recorder.rb b/activerecord/lib/active_record/migration/command_recorder.rb
index 79c55045ba..9782a48055 100644
--- a/activerecord/lib/active_record/migration/command_recorder.rb
+++ b/activerecord/lib/active_record/migration/command_recorder.rb
@@ -144,7 +144,10 @@ module ActiveRecord
def invert_remove_index(args)
table, options = *args
- raise ActiveRecord::IrreversibleMigration, "remove_index is only reversible if given a :column option." unless options && options[:column]
+
+ unless options && options.is_a?(Hash) && options[:column]
+ raise ActiveRecord::IrreversibleMigration, "remove_index is only reversible if given a :column option."
+ end
options = options.dup
[:add_index, [table, options.delete(:column), options]]