0
@@ -1,5 +1,14 @@
0
 *SVN*
0
 
0
+* Added change_table for migrations (Jeff Dean) [#71]. Example:
0
+
0
+    change_table :videos do |t|
0
+      t.timestamps                          # adds created_at, updated_at
0
+      t.belongs_to :goat                    # add goat_id integer
0
+      t.string :name, :email, :limit => 20  # adds name and email both with a 20 char limit
0
+      t.remove :name, :email                # removes the name and email columns
0
+    end
0
+
0
 * Fixed has_many :through .create with no parameters caused a "can't dup NilClass" error (Steven Soroka) [#85]
0
 
0
 * Added block-setting of attributes for Base.create like Base.new already has (Adam Meehan) [#39]

0
@@ -164,6 +164,28 @@ A short rundown of the major features:
0
     ActiveRecord::Base.logger = Log4r::Logger.new("Application Log")
0
 
0
 
0
+* Database agnostic schema management with Migrations
0
+
0
+    class AddSystemSettings < ActiveRecord::Migration
0
+      def self.up
0
+        create_table :system_settings do |t|
0
+          t.string  :name
0
+          t.string  :label
0
+          t.text  :value
0
+          t.string  :type
0
+          t.integer  :position
0
+        end
0
+
0
+        SystemSetting.create :name => "notice", :label => "Use notice?", :value => 1
0
+      end
0
+
0
+      def self.down
0
+        drop_table :system_settings
0
+      end
0
+    end
0
+
0
+  {Learn more}[link:classes/ActiveRecord/Migration.html]
0
+
0
 == Simple example (1/2): Defining tables and classes (using MySQL)
0
 
0
 Data definitions are specified only in the database. Active Record queries the database for 

0
@@ -27,7 +27,10 @@ you can do so with:
0
 
0
    rake test_mysql TEST=test/cases/base_test.rb
0
    
0
-That'll run the base suite using the MySQL-Ruby adapter.
0
+That'll run the base suite using the MySQL-Ruby adapter.  Some tests rely on the schema
0
+being initialized - you can initialize the schema with:
0
+
0
+  rake test_mysql TEST=test/cases/aaa_create_tables_test.rb
0
 
0
 
0
 

0
@@ -469,5 +469,195 @@ module ActiveRecord
0
           @base.native_database_types
0
         end
0
     end
0
+
0
+    # Represents a SQL table in an abstract way for updating a table.
0
+    # Also see TableDefinition and SchemaStatements#create_table
0
+    #
0
+    # Available transformations are:
0
+    #
0
+    #   change_table :table do |t|
0
+    #     t.column
0
+    #     t.index
0
+    #     t.timestamps
0
+    #     t.change
0
+    #     t.change_default
0
+    #     t.rename
0
+    #     t.references
0
+    #     t.belongs_to
0
+    #     t.string
0
+    #     t.text
0
+    #     t.integer
0
+    #     t.float
0
+    #     t.decimal
0
+    #     t.datetime
0
+    #     t.timestamp
0
+    #     t.time
0
+    #     t.date
0
+    #     t.binary
0
+    #     t.boolean
0
+    #     t.remove
0
+    #     t.remove_references
0
+    #     t.remove_belongs_to
0
+    #     t.remove_index
0
+    #     t.remove_timestamps
0
+    #   end
0
+    #
0
+    class Table
0
+      def initialize(table_name, base)
0
+        @table_name = table_name
0
+        @base = base
0
+      end
0
+
0
+      # Adds a new column to the named table.
0
+      # See TableDefinition#column for details of the options you can use.
0
+      # ===== Examples
0
+      # ====== Creating a simple columns
0
+      #  t.column(:name, :string)
0
+      def column(column_name, type, options = {})
0
+        @base.add_column(@table_name, column_name, type, options)
0
+      end
0
+
0
+      # Adds a new index to the table.  +column_name+ can be a single Symbol, or
0
+      # an Array of Symbols.  See SchemaStatements#add_index
0
+      #
0
+      # ===== Examples
0
+      # ====== Creating a simple index
0
+      #  t.index(:name)
0
+      # ====== Creating a unique index
0
+      #  t.index([:branch_id, :party_id], :unique => true)
0
+      # ====== Creating a named index
0
+      #  t.index([:branch_id, :party_id], :unique => true, :name => 'by_branch_party')
0
+      def index(column_name, options = {})
0
+        @base.add_index(@table_name, column_name, options)
0
+      end
0
+
0
+      # Adds timestamps (created_at and updated_at) columns to the table.  See SchemaStatements#timestamps
0
+      # ===== Examples
0
+      #  t.timestamps
0
+      def timestamps
0
+        @base.add_timestamps(@table_name)
0
+      end
0
+
0
+      # Changes the column's definition according to the new options.
0
+      # See TableDefinition#column for details of the options you can use.
0
+      # ===== Examples
0
+      #  t.change(:name, :string, :limit => 80)
0
+      #  t.change(:description, :text)
0
+      def change(column_name, type, options = {})
0
+        @base.change_column(@table_name, column_name, type, options)
0
+      end
0
+
0
+      # Sets a new default value for a column.  See
0
+      # ===== Examples
0
+      #  t.change_default(:qualification, 'new')
0
+      #  t.change_default(:authorized, 1)
0
+      def change_default(column_name, default)
0
+        @base.change_column_default(@table_name, column_name, default)
0
+      end
0
+
0
+      # Removes the column(s) from the table definition.
0
+      # ===== Examples
0
+      #  t.remove(:qualification)
0
+      #  t.remove(:qualification, :experience)
0
+      #  t.removes(:qualification, :experience)
0
+      def remove(*column_names)
0
+        @base.remove_column(@table_name, column_names)
0
+      end
0
+
0
+      # Remove the given index from the table.
0
+      #
0
+      # Remove the suppliers_name_index in the suppliers table.
0
+      #   t.remove_index :name
0
+      # Remove the index named accounts_branch_id_index in the accounts table.
0
+      #   t.remove_index :column => :branch_id
0
+      # Remove the index named accounts_branch_id_party_id_index in the accounts table.
0
+      #   t.remove_index :column => [:branch_id, :party_id]
0
+      # Remove the index named by_branch_party in the accounts table.
0
+      #   t.remove_index :name => :by_branch_party
0
+      def remove_index(options = {})
0
+        @base.remove_index(@table_name, options)
0
+      end
0
+
0
+      # Removes the timestamp columns (created_at and updated_at) from the table.
0
+      # ===== Examples
0
+      #  t.remove_timestamps
0
+      def remove_timestamps
0
+        @base.remove_timestamps(@table_name)
0
+      end
0
+
0
+      # Renames a column.
0
+      # ===== Example
0
+      #  t.rename(:description, :name)
0
+      def rename(column_name, new_column_name)
0
+        @base.rename_column(@table_name, column_name, new_column_name)
0
+      end
0
+
0
+      # Adds a reference.  Optionally adds a +type+ column.  <tt>reference</tt>,
0
+      # <tt>references</tt> and <tt>belongs_to</tt> are all acceptable
0
+      # ===== Example
0
+      #  t.references(:goat)
0
+      #  t.references(:goat, :polymorphic => true)
0
+      #  t.references(:goat)
0
+      #  t.belongs_to(:goat)
0
+      def references(*args)
0
+        options = args.extract_options!
0
+        polymorphic = options.delete(:polymorphic)
0
+        args.each do |col|
0
+          @base.add_column(@table_name, "#{col}_id", :integer, options)
0
+          @base.add_column(@table_name, "#{col}_type", :string, polymorphic.is_a?(Hash) ? polymorphic : options) unless polymorphic.nil?
0
+        end
0
+      end
0
+      alias :belongs_to :references
0
+
0
+      # Adds a reference.  Optionally removes a +type+ column.  <tt>remove_reference</tt>,
0
+      # <tt>remove_references</tt> and <tt>remove_belongs_to</tt> are all acceptable
0
+      # ===== Example
0
+      #  t.remove_reference(:goat)
0
+      #  t.remove_reference(:goat, :polymorphic => true)
0
+      #  t.remove_references(:goat)
0
+      #  t.remove_belongs_to(:goat)
0
+      def remove_references(*args)
0
+        options = args.extract_options!
0
+        polymorphic = options.delete(:polymorphic)
0
+        args.each do |col|
0
+          @base.remove_column(@table_name, "#{col}_id")
0
+          @base.remove_column(@table_name, "#{col}_type") unless polymorphic.nil?
0
+        end
0
+      end
0
+      alias :remove_belongs_to  :remove_references
0
+
0
+      # Adds a column or columns of a specified type
0
+      # ===== Example
0
+      #  t.string(:goat)
0
+      #  t.string(:goat, :sheep)
0
+      %w( string text integer float decimal datetime timestamp time date binary boolean ).each do |column_type|
0
+        class_eval <<-EOV
0
+          def #{column_type}(*args)
0
+            options = args.extract_options!
0
+            column_names = args
0
+
0
+            column_names.each do |name|
0
+              column = ColumnDefinition.new(@base, name, '#{column_type}')
0
+              if options[:limit]
0
+                column.limit = options[:limit]
0
+              elsif native['#{column_type}'.to_sym].is_a?(Hash)
0
+                column.limit = native['#{column_type}'.to_sym][:limit]
0
+              end
0
+              column.precision = options[:precision]
0
+              column.scale = options[:scale]
0
+              column.default = options[:default]
0
+              column.null = options[:null]
0
+              @base.add_column(@table_name, name, column.sql_type, options)
0
+            end
0
+          end
0
+        EOV
0
+      end
0
+
0
+      private
0
+        def native
0
+          @base.native_database_types
0
+        end
0
+    end
0
+
0
   end
0
 end

0
@@ -104,6 +104,67 @@ module ActiveRecord
0
         execute create_sql
0
       end
0
 
0
+      # A block for changing columns in +table+.
0
+      #
0
+      # === Example
0
+      #  # change_table() yields a Table instance
0
+      #  change_table(:suppliers) do |t|
0
+      #    t.column :name, :string, :limit => 60
0
+      #    # Other column alterations here
0
+      #  end
0
+      #
0
+      # ===== Examples
0
+      # ====== Add a column
0
+      #  change_table(:suppliers) do |t|
0
+      #    t.column :name, :string, :limit => 60
0
+      #  end
0
+      #
0
+      # ====== Add 2 integer columns
0
+      #  change_table(:suppliers) do |t|
0
+      #    t.integer :width, :height, :null => false, :default => 0
0
+      #  end
0
+      #
0
+      # ====== Add created_at/updated_at columns
0
+      #  change_table(:suppliers) do |t|
0
+      #    t.timestamps
0
+      #  end
0
+      #
0
+      # ====== Add a foreign key column
0
+      #  change_table(:suppliers) do |t|
0
+      #    t.references :company
0
+      #  end
0
+      #
0
+      # Creates a <tt>company_id(integer)</tt> column
0
+      #
0
+      # ====== Add a polymorphic foreign key column
0
+      #  change_table(:suppliers) do |t|
0
+      #    t.belongs_to :company, :polymorphic => true
0
+      #  end
0
+      #
0
+      # Creates <tt>company_type(varchar)</tt> and <tt>company_id(integer)</tt> columns
0
+      #
0
+      # ====== Remove a column
0
+      #  change_table(:suppliers) do |t|
0
+      #    t.remove :company
0
+      #  end
0
+      #
0
+      # ====== Remove a column
0
+      #  change_table(:suppliers) do |t|
0
+      #    t.remove :company_id
0
+      #    t.remove :width, :height
0
+      #  end
0
+      #
0
+      # ====== Remove an index
0
+      #  change_table(:suppliers) do |t|
0
+      #    t.remove_index :company_id
0
+      #  end
0
+      #
0
+      # See also Table for details on
0
+      # all of the various column transformation
0
+      def change_table(table_name)
0
+        yield Table.new(table_name, self)
0
+      end
0
+      
0
       # Renames a table.
0
       # ===== Example
0
       #  rename_table('octopuses', 'octopi')
0
@@ -124,13 +185,17 @@ module ActiveRecord
0
         execute(add_column_sql)
0
       end
0
 
0
-      # Removes the column from the table definition.
0
+      # Removes the column(s) from the table definition.
0
       # ===== Examples
0
       #  remove_column(:suppliers, :qualification)
0
-      def remove_column(table_name, column_name)
0
-        execute "ALTER TABLE #{quote_table_name(table_name)} DROP #{quote_column_name(column_name)}"
0
+      #  remove_columns(:suppliers, :qualification, :experience)
0
+      def remove_column(table_name, *column_names)
0
+        column_names.flatten.each do |column_name|
0
+          execute "ALTER TABLE #{quote_table_name(table_name)} DROP #{quote_column_name(column_name)}"
0
+        end
0
       end
0
-
0
+      alias :remove_columns :remove_column
0
+      
0
       # Changes the column's definition according to the new options.
0
       # See TableDefinition#column for details of the options you can use.
0
       # ===== Examples

0
@@ -219,11 +219,14 @@ module ActiveRecord
0
         execute "VACUUM"
0
       end
0
 
0
-      def remove_column(table_name, column_name) #:nodoc:
0
-        alter_table(table_name) do |definition|
0
-          definition.columns.delete(definition[column_name])
0
+      def remove_column(table_name, *column_names) #:nodoc:
0
+        column_names.flatten.each do |column_name|
0
+          alter_table(table_name) do |definition|
0
+            definition.columns.delete(definition[column_name])
0
+          end
0
         end
0
       end
0
+      alias :remove_columns :remove_column
0
 
0
       def change_column_default(table_name, column_name, default) #:nodoc:
0
         alter_table(table_name) do |definition|

0
@@ -1077,7 +1077,7 @@ if ActiveRecord::Base.connection.supports_migrations?
0
 
0
       protected
0
       def with_new_table
0
-        Person.connection.create_table :delete_me do |t|
0
+        Person.connection.create_table :delete_me, :force => true do |t|
0
           yield t
0
         end
0
       ensure
0
@@ -1086,4 +1086,210 @@ if ActiveRecord::Base.connection.supports_migrations?
0
 
0
     end # SexyMigrationsTest
0
   end # uses_mocha
0
+
0
+  uses_mocha 'ChangeTable migration tests' do
0
+    class ChangeTableMigrationsTest < ActiveRecord::TestCase
0
+      def setup
0
+        @connection = Person.connection
0
+        @connection.create_table :delete_me, :force => true do |t|
0
+        end
0
+      end
0
+
0
+      def teardown
0
+        Person.connection.drop_table :delete_me rescue nil
0
+      end
0
+
0
+      def test_references_column_type_adds_id
0
+        with_change_table do |t|
0
+          @connection.expects(:add_column).with(:delete_me, 'customer_id', :integer, {})
0
+          t.references :customer
0
+        end
0
+      end
0
+
0
+      def test_remove_references_column_type_removes_id
0
+        with_change_table do |t|
0
+          @connection.expects(:remove_column).with(:delete_me, 'customer_id')
0
+          t.remove_references :customer
0
+        end
0
+      end
0
+
0
+      def test_add_belongs_to_works_like_add_references
0
+        with_change_table do |t|
0
+          @connection.expects(:add_column).with(:delete_me, 'customer_id', :integer, {})
0
+          t.belongs_to :customer
0
+        end
0
+      end
0
+
0
+      def test_remove_belongs_to_works_like_remove_references
0
+        with_change_table do |t|
0
+          @connection.expects(:remove_column).with(:delete_me, 'customer_id')
0
+          t.remove_belongs_to :customer
0
+        end
0
+      end
0
+
0
+      def test_references_column_type_with_polymorphic_adds_type
0
+        with_change_table do |t|
0
+          @connection.expects(:add_column).with(:delete_me, 'taggable_type', :string, {})
0
+          @connection.expects(:add_column).with(:delete_me, 'taggable_id', :integer, {})
0
+          t.references :taggable, :polymorphic => true
0
+        end
0
+      end
0
+
0
+      def test_remove_references_column_type_with_polymorphic_removes_type
0
+        with_change_table do |t|
0
+          @connection.expects(:remove_column).with(:delete_me, 'taggable_type')
0
+          @connection.expects(:remove_column).with(:delete_me, 'taggable_id')
0
+          t.remove_references :taggable, :polymorphic => true
0
+        end
0
+      end
0
+
0
+      def test_references_column_type_with_polymorphic_and_options_null_is_false_adds_table_flag
0
+        with_change_table do |t|
0
+          @connection.expects(:add_column).with(:delete_me, 'taggable_type', :string, {:null => false})
0
+          @connection.expects(:add_column).with(:delete_me, 'taggable_id', :integer, {:null => false})
0
+          t.references :taggable, :polymorphic => true, :null => false
0
+        end
0
+      end
0
+
0
+      def test_remove_references_column_type_with_polymorphic_and_options_null_is_false_removes_table_flag
0
+        with_change_table do |t|
0
+          @connection.expects(:remove_column).with(:delete_me, 'taggable_type')
0
+          @connection.expects(:remove_column).with(:delete_me, 'taggable_id')
0
+          t.remove_references :taggable, :polymorphic => true, :null => false
0
+        end
0
+      end
0
+
0
+      def test_timestamps_creates_updated_at_and_created_at
0
+        with_change_table do |t|
0
+          @connection.expects(:add_timestamps).with(:delete_me)
0
+          t.timestamps
0
+        end
0
+      end
0
+
0
+      def test_remove_timestamps_creates_updated_at_and_created_at
0
+        with_change_table do |t|
0
+          @connection.expects(:remove_timestamps).with(:delete_me)
0
+          t.remove_timestamps
0
+        end
0
+      end
0
+
0
+      def string_column
0
+        if current_adapter?(:PostgreSQLAdapter)
0
+          "character varying(255)"
0
+        else
0
+          'varchar(255)'
0
+        end
0
+      end
0
+
0
+      def integer_column
0
+        if current_adapter?(:SQLite3Adapter) || current_adapter?(:SQLiteAdapter) || current_adapter?(:PostgreSQLAdapter)
0
+          "integer"
0
+        else
0
+          'int(11)'
0
+        end
0
+      end
0
+
0
+      def test_integer_creates_integer_column
0
+        with_change_table do |t|
0
+          @connection.expects(:add_column).with(:delete_me, :foo, integer_column, {})
0
+          @connection.expects(:add_column).with(:delete_me, :bar, integer_column, {})
0
+          t.integer :foo, :bar
0
+        end
0
+      end
0
+
0
+      def test_string_creates_string_column
0
+        with_change_table do |t|
0
+          @connection.expects(:add_column).with(:delete_me, :foo, string_column, {})
0
+          @connection.expects(:add_column).with(:delete_me, :bar, string_column, {})
0
+          t.string :foo, :bar
0
+        end
0
+      end
0
+
0
+      def test_column_creates_column
0
+        with_change_table do |t|
0
+          @connection.expects(:add_column).with(:delete_me, :bar, :integer, {})
0
+          t.column :bar, :integer
0
+        end
0
+      end
0
+
0
+      def test_column_creates_column_with_options
0
+        with_change_table do |t|
0
+          @connection.expects(:add_column).with(:delete_me, :bar, :integer, {:null => false})
0
+          t.column :bar, :integer, :null => false
0
+        end
0
+      end
0
+
0
+      def test_index_creates_index
0
+        with_change_table do |t|
0
+          @connection.expects(:add_index).with(:delete_me, :bar, {})
0
+          t.index :bar
0
+        end
0
+      end
0
+
0
+      def test_index_creates_index_with_options
0
+        with_change_table do |t|
0
+          @connection.expects(:add_index).with(:delete_me, :bar, {:unique => true})
0
+          t.index :bar, :unique => true
0
+        end
0
+      end
0
+
0
+      def test_change_changes_column
0
+        with_change_table do |t|
0
+          @connection.expects(:change_column).with(:delete_me, :bar, :string, {})
0
+          t.change :bar, :string
0
+        end
0
+      end
0
+
0
+      def test_change_changes_column_with_options
0
+        with_change_table do |t|
0
+          @connection.expects(:change_column).with(:delete_me, :bar, :string, {:null => true})
0
+          t.change :bar, :string, :null => true
0
+        end
0
+      end
0
+
0
+      def test_change_default_changes_column
0
+        with_change_table do |t|
0
+          @connection.expects(:change_column_default).with(:delete_me, :bar, :string)
0
+          t.change_default :bar, :string
0
+        end
0
+      end
0
+
0
+      def test_remove_drops_single_column
0
+        with_change_table do |t|
0
+          @connection.expects(:remove_column).with(:delete_me, [:bar])
0
+          t.remove :bar
0
+        end
0
+      end
0
+
0
+      def test_remove_drops_multiple_columns
0
+        with_change_table do |t|
0
+          @connection.expects(:remove_column).with(:delete_me, [:bar, :baz])
0
+          t.remove :bar, :baz
0
+        end
0
+      end
0
+
0
+      def test_remove_index_removes_index_with_options
0
+        with_change_table do |t|
0
+          @connection.expects(:remove_index).with(:delete_me, {:unique => true})
0
+          t.remove_index :unique => true
0
+        end
0
+      end
0
+
0
+      def test_rename_renames_column
0
+        with_change_table do |t|
0
+          @connection.expects(:rename_column).with(:delete_me, :bar, :baz)
0
+          t.rename :bar, :baz
0
+        end
0
+      end
0
+
0
+      protected
0
+      def with_change_table
0
+        Person.connection.change_table :delete_me do |t|
0
+          yield t
0
+        end
0
+      end
0
+
0
+    end # ChangeTable test
0
+  end # uses_mocha
0
+
0
 end