Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | Make change_table use object of current database adapter | Nishant Modak | 2014-01-07 | 1 | -4/+6 | |
| | | | | | | | | | - Earlier, change_table was creating database-agnostic object. - After this change, it will create correct object based on current database adapter. - This will ensure that create_table and change_table will get same objects. - This makes update_table_definition method public and nodoc. - Fixes #13577 and #13503 | |||||
* | Make 'enable_extension' revertible | Eric Tipton | 2013-08-10 | 1 | -1/+2 | |
| | | | | | If 'enable_extension' is used in a migration's 'change' method, use 'disable_extension' on down migration (and vice-versa). | |||||
* | raise IrreversibleMigration if no column given | Neeraj Singh | 2013-05-06 | 1 | -1/+4 | |
| | | | | | | | | | | | | | | | fixes #10419 Following code should raise IrreversibleMigration. But the code was failing since options is an array and not a hash. def change change_table :users do |t| t.remove_index [:name, :email] end end Fix was to check if the options is a Hash before operating on it. | |||||
* | Fixes for PR [#8267] | Marc-Andre Lafortune | 2012-12-22 | 1 | -1/+2 | |
| | | | | | | | | * Fix Migration#reversible by not using `transaction`. * Adapt mysql adapter to updated api for remove_column * Update test after aedcd683684d08eaf30623a4b48ce31a31426372 | |||||
* | Remove duplicated methods in command recorder and duplicated test name | Carlos Antonio da Silva | 2012-12-21 | 1 | -3/+3 | |
| | ||||||
* | Make execute, change_column and remove_columns methods actually irreversible | Marc-Andre Lafortune | 2012-12-21 | 1 | -1/+2 | |
| | | | | [#8267] | |||||
* | Make change_table reversible when possible [#8267] | Marc-Andre Lafortune | 2012-12-21 | 1 | -1/+5 | |
| | ||||||
* | Factorize methods that are easily reversible [#8267] | Marc-Andre Lafortune | 2012-12-21 | 1 | -34/+20 | |
| | ||||||
* | Make remove_index reversible [#8267] | Marc-Andre Lafortune | 2012-12-21 | 1 | -4/+10 | |
| | ||||||
* | Differentiate between remove_column and remove_columns. Make remove_column ↵ | Marc-Andre Lafortune | 2012-12-21 | 1 | -2/+7 | |
| | | | | | | reversible. [#8267] | |||||
* | Make drop_table reversible [#8267] | Marc-Andre Lafortune | 2012-12-21 | 1 | -3/+10 | |
| | ||||||
* | Add drop_join_table [#8267] | Marc-Andre Lafortune | 2012-12-21 | 1 | -3/+6 | |
| | ||||||
* | Add Migration#reversible for reversible data operations [#8267] | Marc-Andre Lafortune | 2012-12-21 | 1 | -1/+5 | |
| | ||||||
* | Allow reverting of migration commands with Migration#revert [#8267] | Marc-Andre Lafortune | 2012-12-21 | 1 | -17/+36 | |
| | ||||||
* | Split ridiculously long line | Marc-Andre Lafortune | 2012-12-21 | 1 | -1/+4 | |
| | ||||||
* | Coerce strings in create_join_table. | Steve Klabnik | 2012-11-21 | 1 | -1/+1 | |
| | | | | | | | If you accidentally pass a string and a symbol, this breaks. So we coerce them both to strings. Fixes #7715 | |||||
* | Create default join table name using a Symbol | Carlos Antonio da Silva | 2012-07-19 | 1 | -1/+1 | |
| | | | | | Lets stick with a symbol in join table name, it was removed in 993e1643650b7b0939437f86b7b4cda0d92b0b8f and broke the build. | |||||
* | Add fkey attributes to `join_table` migration generator | Aleksey Magusev | 2012-07-19 | 1 | -1/+1 | |
| | ||||||
* | Add join table migration generator | Aleksey Magusev | 2012-07-18 | 1 | -4/+2 | |
| | | | | | | | | | | | | | For instance, running rails g migration CreateMediaJoinTable artists musics:uniq will create a migration with create_join_table :artists, :musics do |t| # t.index [:artist_id, :music_id] t.index [:music_id, :artist_id], unique: true end | |||||
* | Make references statements reversible | Aleksey Magusev | 2012-07-03 | 1 | -1/+13 | |
| | ||||||
* | Add `create_join_table` migration helper to create HABTM join tables | Rafael Mendonça França | 2012-01-27 | 2 | -2/+27 | |
| | ||||||
* | Fix GH #4285. Remove options when we record calling creat_table | kennyj | 2012-01-04 | 1 | -1/+1 | |
| | ||||||
* | Fixed bad options order in ↵ | jbbarth | 2011-08-28 | 1 | -1/+1 | |
| | | | | ActiveRecord::Migration::CommandRecorder#invert_rename_index | |||||
* | Merge branch 'master' of git://github.com/lifo/docrails | Xavier Noria | 2011-07-05 | 1 | -1/+1 | |
|\ | ||||||
| * | fix typo in method name | Vijay Dev | 2011-06-25 | 1 | -1/+1 | |
| | | ||||||
| * | Revert "add missing methods supported by reversible migrations using the ↵ | Vijay Dev | 2011-06-25 | 1 | -4/+1 | |
| | | | | | | | | | | | | | | | | | | change method" This reverts commit 7f91eebae361f2f3a1558f6d899b372524509a2d. Reason: Incorrect documenting. The newly added methods to the list are not actually supported by reversible migrations. | |||||
| * | add missing methods supported by reversible migrations using the change method | Vijay Dev | 2011-06-24 | 1 | -1/+4 | |
| | | ||||||
* | | Reversing the changes done in c278a2c while still resolving #1857. | Vijay Dev | 2011-07-02 | 1 | -9/+5 | |
| | | | | | | | | | | | | | | | | | | The changes broke bulk migration tests and were fixed in 4d256bc6; however that brought back the issue of #1857 and so this commit goes back to the original scenario and just adds change_table to the list of methods which are to be recorded in the CommandRecorder. The method_missing now delegates all calls to the underlying connection as before. | |||||
* | | CommandRecorder should delegate in method_missing where possible. Fixes some ↵ | Jon Leighton | 2011-06-30 | 1 | -1/+5 | |
| | | | | | | | | tests in migration_test.rb under mysql. The problem was introduced in c278a2c5e109204ec8a47fcbfdfc327aad7996ce. | |||||
* | | record unsupported methods in CommandRecorder instead of letting the ↵ | Vijay Dev | 2011-06-25 | 1 | -4/+4 | |
|/ | | | | unsupported methods go through to the underlying db causing errors like duplicate columns to occur when rolling back migrations | |||||
* | Merge branch 'master' of git://github.com/lifo/docrails | Xavier Noria | 2011-06-06 | 1 | -6/+6 | |
|\ | | | | | | | | | Conflicts: activerecord/RUNNING_UNIT_TESTS | |||||
| * | comment the recorder methods | Vijay Dev | 2011-06-07 | 1 | -3/+3 | |
| | | ||||||
| * | styling changes | Vijay Dev | 2011-06-02 | 1 | -3/+3 | |
| | | ||||||
* | | Merge pull request #1229 from workmad3/master | Aaron Patterson | 2011-05-27 | 1 | -2/+4 | |
|\ \ | |/ |/| | Fix for Issue #1205 | |||||
| * | Using .try to test for the existence of a method option in a nil-resistent ↵ | David Workman | 2011-05-23 | 1 | -5/+2 | |
| | | | | | | | | manner. Inlined the determination of the options hash for reversing using a ternary operator. Shortens the method in a way that keeps the code neat | |||||
| * | Neatened up the invert_add_index method as per suggeston | David Workman | 2011-05-23 | 1 | -2/+3 | |
| | | ||||||
| * | Simple fix for correctly inverting an add_index migration when a name has ↵ | David Workman | 2011-05-23 | 1 | -2/+6 | |
| | | | | | | | | been provided | |||||
* | | Remove extra white spaces on ActiveRecord docs. | Sebastian Martinez | 2011-05-23 | 1 | -3/+3 | |
|/ | ||||||
* | Add :bulk => true option to change_table | Pratik Naik | 2011-01-31 | 1 | -4/+16 | |
| | ||||||
* | adding documentation for reversible migrations | Aaron Patterson | 2010-11-19 | 1 | -1/+11 | |
| | ||||||
* | invertable migrations are working | Aaron Patterson | 2010-11-19 | 1 | -1/+1 | |
| | ||||||
* | command recorder will record commands sent to a delegate object | Aaron Patterson | 2010-11-19 | 1 | -3/+13 | |
| | ||||||
* | updating documentation | Aaron Patterson | 2010-11-19 | 1 | -1/+7 | |
| | ||||||
* | IrreversibleMigration is raised if we cannot invert the command | Aaron Patterson | 2010-11-19 | 1 | -1/+5 | |
| | ||||||
* | commands are reversed | Aaron Patterson | 2010-11-19 | 1 | -1/+1 | |
| | ||||||
* | add and remove timestamps can be inverted | Aaron Patterson | 2010-11-19 | 1 | -0/+8 | |
| | ||||||
* | inverting add_index | Aaron Patterson | 2010-11-19 | 1 | -0/+5 | |
| | ||||||
* | partial implementation of the command recorder | Aaron Patterson | 2010-11-19 | 1 | -0/+48 | |