diff options
| author | Tim Connor <tim@wasabi.local> | 2010-09-17 16:31:25 +1200 |
|---|---|---|
| committer | José Valim <jose.valim@gmail.com> | 2010-09-18 20:49:31 +0200 |
| commit | 37de59eacf8f6478e866309615af7381d41a5a14 (patch) | |
| tree | e57a203a9e4400bd3e6acf78bac889b11929e531 /activerecord/lib/active_record/connection_adapters/abstract/schema_statements.rb | |
| parent | 55d0d57bfc72c0bdbc81ae5d95c99729f16899af (diff) | |
| download | rails-37de59eacf8f6478e866309615af7381d41a5a14.tar.gz rails-37de59eacf8f6478e866309615af7381d41a5a14.tar.bz2 rails-37de59eacf8f6478e866309615af7381d41a5a14.zip | |
Fix issue with remove_index and add unit test [#5645 state:resolved]
Diffstat (limited to 'activerecord/lib/active_record/connection_adapters/abstract/schema_statements.rb')
| -rw-r--r-- | activerecord/lib/active_record/connection_adapters/abstract/schema_statements.rb | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/activerecord/lib/active_record/connection_adapters/abstract/schema_statements.rb b/activerecord/lib/active_record/connection_adapters/abstract/schema_statements.rb index 63d76472fa..310423bb20 100644 --- a/activerecord/lib/active_record/connection_adapters/abstract/schema_statements.rb +++ b/activerecord/lib/active_record/connection_adapters/abstract/schema_statements.rb @@ -363,8 +363,7 @@ module ActiveRecord def remove_index(table_name, options = {}) index_name = index_name(table_name, options) unless index_name_exists?(table_name, index_name, true) - @logger.warn("Index name '#{index_name}' on table '#{table_name}' does not exist. Skipping.") - return + raise ArgumentError, "Index name '#{index_name}' on table '#{table_name}' does not exist" end remove_index!(table_name, index_name) end |