diff options
| author | Oscar Del Ben <info@oscardelben.com> | 2012-05-18 08:36:27 -0700 |
|---|---|---|
| committer | Oscar Del Ben <info@oscardelben.com> | 2012-05-18 09:24:45 -0700 |
| commit | 925f2e9ab78d997a364e7e6abae10de81a073bb9 (patch) | |
| tree | 2d6b639303b76b1ea84f649b4075ef98388abf8c /activerecord/lib/rails | |
| parent | aeac0bf620ab4504aceccf5cc3997c7a4ed96791 (diff) | |
| download | rails-925f2e9ab78d997a364e7e6abae10de81a073bb9.tar.gz rails-925f2e9ab78d997a364e7e6abae10de81a073bb9.tar.bz2 rails-925f2e9ab78d997a364e7e6abae10de81a073bb9.zip | |
Refactor migration generator
Diffstat (limited to 'activerecord/lib/rails')
| -rw-r--r-- | activerecord/lib/rails/generators/active_record/migration/templates/migration.rb | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/activerecord/lib/rails/generators/active_record/migration/templates/migration.rb b/activerecord/lib/rails/generators/active_record/migration/templates/migration.rb index b9b5ec7956..b1a0f83b5f 100644 --- a/activerecord/lib/rails/generators/active_record/migration/templates/migration.rb +++ b/activerecord/lib/rails/generators/active_record/migration/templates/migration.rb @@ -12,10 +12,7 @@ class <%= migration_class_name %> < ActiveRecord::Migration def up <% attributes.each do |attribute| -%> <%- if migration_action -%> - <%= migration_action %>_column :<%= table_name %>, :<%= attribute.name %><% if migration_action == 'add' %>, :<%= attribute.type %><%= attribute.inject_options %><% end %> - <%- if attribute.has_index? && migration_action == 'add' -%> - add_index :<%= table_name %>, :<%= attribute.index_name %><%= attribute.inject_index_options %> - <%- end -%> + <%= migration_action %>_column :<%= table_name %>, :<%= attribute.name %> <%- end -%> <%- end -%> end @@ -23,8 +20,8 @@ class <%= migration_class_name %> < ActiveRecord::Migration def down <% attributes.reverse.each do |attribute| -%> <%- if migration_action -%> - <%= migration_action == 'add' ? 'remove' : 'add' %>_column :<%= table_name %>, :<%= attribute.name %><% if migration_action == 'remove' %>, :<%= attribute.type %><%= attribute.inject_options %><% end %> - <%- if attribute.has_index? && migration_action == 'remove' -%> + add_column :<%= table_name %>, :<%= attribute.name %>, :<%= attribute.type %><%= attribute.inject_options %> + <%- if attribute.has_index? -%> add_index :<%= table_name %>, :<%= attribute.index_name %><%= attribute.inject_index_options %> <%- end -%> <%- end -%> |