@@ -79,6 +79,7 @@ if ActiveRecord::Base.connection.supports_migrations? # Note: changed index name from "key" to "key_idx" since "key" is a Firebird reserved word # OpenBase does not have named indexes. You must specify a single column name unless current_adapter?(:OpenBaseAdapter) + Person.update_all "#{Person.connection.quote_column_name 'key'}=#{Person.connection.quote_column_name 'id'}" #some databases (including sqlite2 won't add a unique index if existing data non unique) assert_nothing_raised { Person.connection.add_index("people", ["key"], :name => "key_idx", :unique => true) } assert_nothing_raised { Person.connection.remove_index("people", :name => "key_idx", :unique => true) } end activerecord/test/cases/migration_test.rb bf1b1e0925085811f0b58bb4093e678438ea0236 Frederick Cheung frederick.cheung@gmail.com http://github.com/rails/rails/commit/ae51013c3f7b8a8579fcb99d889ed80e9dd75797 ae51013c3f7b8a8579fcb99d889ed80e9dd75797 2008-04-22T19:24:20-07:00 2008-04-22T19:23:53-07:00 Provide data for the key column otherwise adding a unique index will fail. [#30 state:closed] fc347139364aead58aeaae74a915e71ac07b25d4 Michael Koziarski michael@koziarski.com