Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Add migration compatibility class for Rails 5.1 | Rafael Mendonça França | 2016-05-10 | 1 | -1/+4 |
| | |||||
* | Migrations: move version-finding responsibility | yui-knk | 2016-04-29 | 1 | -0/+10 |
| | | | | | | | `ActiveRecord::Migration` needn't know about migration version compatibility lookup. Delegate it to the Compatibility module. Signed-off-by: Jeremy Daer <jeremydaer@gmail.com> | ||||
* | Update compatibility.rb | Bogdan | 2016-03-26 | 1 | -1/+1 |
| | |||||
* | Correctly show deprecation warning for incompatible migrations | Prathamesh Sonpatki | 2016-02-12 | 1 | -1/+1 |
| | |||||
* | Intervene on change_table as well as create_table | Matthew Draper | 2016-01-30 | 1 | -0/+13 |
| | |||||
* | Pare back default `index` option for the migration generator | Prathamesh Sonpatki | 2016-01-24 | 1 | -0/+12 |
| | | | | | | | | | | - Using `references` or `belongs_to` in migrations will always add index for the referenced column by default, without adding `index:true` option to generated migration file. - Users can opt out of this by passing `index: false`. - Legacy migrations won't be affected by this change. They will continue to run as they were before. - Fixes #18146 | ||||
* | fix remove_index for postgresql when running legacy migrations | Lachlan Sylvester | 2016-01-06 | 1 | -2/+3 |
| | |||||
* | Support removing custom-names indexes when only specifying column names | Grey Baker | 2015-12-15 | 1 | -0/+25 |
| | |||||
* | Ignore index name in `index_exists?` when not passed a name to check for | Grey Baker | 2015-12-15 | 1 | -0/+11 |
| | |||||
* | In 4.2 migrations, `timestamps` defaulted to `null: true` | Matthew Draper | 2015-12-15 | 1 | -0/+24 |
| | | | | | .. it also showed a deprecation warning, but we obviously needn't retain that. | ||||
* | Add migration versioning via Migration subclasses | Matthew Draper | 2015-12-15 | 1 | -0/+30 |
| | |||||
* | adding basic change_column and remove_columns in CommandRecorder ↵ | Gaurav Sharma | 2015-09-29 | 1 | -0/+2 |
| | | | | documentations [ci skip] | ||||
* | Merge pull request #21412 from yui-knk/feature/irreversible_migration_error_msg | Yves Senn | 2015-09-04 | 1 | -1/+6 |
|\ | | | | | | | Add detailed error message to `IrreversibleMigration` | ||||
| * | Add detailed error message to `IrreversibleMigration` | yui-knk | 2015-08-30 | 1 | -1/+6 |
| | | |||||
| * | Add detailed error message to `IrreversibleMigration` | yui-knk | 2015-08-28 | 1 | -1/+1 |
| | | |||||
| * | Add detailed error message to `IrreversibleMigration` | yui-knk | 2015-08-28 | 1 | -1/+1 |
| | | |||||
* | | Merge pull request #21432 from yui-knk/fix/what_change_method_can_reverse | Sean Griffin | 2015-08-30 | 1 | -1/+13 |
|\ \ | | | | | | | [ci skip] Update what methods `Migration#change` can reverse | ||||
| * | | [ci skip] Update what methods `Migration#change` can reverse | yui-knk | 2015-08-31 | 1 | -1/+13 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Documentations and comments about what methods `Migration#change` can reverse is out of date. For example `change_column_default` is now reversible by this [commit](https://github.com/rails/rails/pull/20018). * Comments about `CommandRecorder` dose not match with Rails Guide. For example `add_foreign_key` is listed only on Rails Guide. | ||||
* | | | Merge pull request #21429 from yui-knk/fix/revert_disable_extension | Sean Griffin | 2015-08-30 | 1 | -1/+1 |
|\ \ \ | |/ / |/| | | Make revert of `disable_extension` to work | ||||
| * | | Make revert of `disable_extension` to work | yui-knk | 2015-08-30 | 1 | -1/+1 |
| |/ | | | | | | | | | | | This is fix of #11826 which miss to add `disable_extension` to `ReversibleAndIrreversibleMethods`. So `CommandRecorder#method_missing` catches `change_column_default` and @delegate's method is called. | ||||
* | | Merge branch 'master' of github.com:rails/docrails | Vijay Dev | 2015-08-30 | 1 | -1/+1 |
|\ \ | |/ |/| | |||||
| * | [ci skip] Capitalize comments | yui-knk | 2015-08-29 | 1 | -1/+1 |
| | | |||||
* | | Make `change_column_default` to work | yui-knk | 2015-08-25 | 1 | -8/+8 |
|/ | | | | | | | | | | | | This is fix of #20018 which removes `change_column_default` from array, so `CommandRecorder#method_missing` catches `change_column_default` and @delegate's method is called. This PR * fix this bug * define `ReversibleAndIrreversibleMethods` const making clear which this array means to prevent these miss | ||||
* | Add reversible syntax for change_column_default | Prem Sichanugrist | 2015-06-26 | 1 | -1/+11 |
| | | | | | | | | | | | | | Passing `:from` and `:to` to `change_column_default` makes this command reversible as user has defined its previous state. So, instead of having the migration command as: change_column_default(:posts, :state, "draft") They can write it as: change_column_default(:posts, :state, from: nil, to: "draft") | ||||
* | make `remove_index :table, :column` reversible. | Yves Senn | 2015-06-15 | 1 | -7/+9 |
| | | | | | | | | | This used to raise a `IrreversibleMigration` error (since #10437). However since `remove_index :table, :column` is probably the most basic use-case we should make it reversible again. Conflicts: activerecord/CHANGELOG.md | ||||
* | Add an invert method for remove_foreign_key | Aster Ryan | 2015-06-11 | 1 | -0/+10 |
| | |||||
* | Merge pull request #16305 from tomkadwill/documented_change_table | Rafael Mendonça França | 2014-07-28 | 1 | -1/+1 |
|\ | | | | | Added documentation for change_table | ||||
| * | Added nodoc to change_table | Tom Kadwill | 2014-07-28 | 1 | -1/+1 |
| | | |||||
* | | create_join_table uses same logic as HABTM reflections | Stefan Kanev | 2014-07-18 | 1 | -1/+1 |
| | | | | | | | | | | | | | | | | Before this change, create_join_table would not remove the common prefix in the join table name, unlike ActiveRecord::Reflections. A HABTM between Music::Artist and Music::Record would use a table music_artists_records, while create_join table would create music_artists_music_records. | ||||
* | | fk: review corrections: indent, visibility, syntax, wording. | Yves Senn | 2014-06-26 | 1 | -3/+3 |
| | | |||||
* | | fk: infere column name from table names. | Yves Senn | 2014-06-26 | 1 | -1/+3 |
| | | | | | | | | This allows to create and remove foreign keys without specifying a column. | ||||
* | | fk: make `add_foreign_key` reversible. | Yves Senn | 2014-06-26 | 1 | -1/+16 |
|/ | |||||
* | Drop the correct index after reverting a migration | Hubert Dąbrowski | 2014-02-13 | 1 | -1/+6 |
| | | | | | | Previously when reverting a migration which added a named index it would instead drop a corresponding index with matching columns but without a name. | ||||
* | make `change_column_null` reversible. Closes #13576. | Yves Senn | 2014-01-08 | 1 | -1/+6 |
| | | | | Closes #13623. | ||||
* | 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 |