Commit c3659ef2e50ff238cecb62e5e4b89f6d94c54be2
1 parent
ed60b6fc
Exists in
master
and in
4 other branches
Fix merge request migration for postgres
Showing
1 changed file
with
5 additions
and
5 deletions
Show diff stats
db/migrate/20130218141327_convert_closed_to_state_in_merge_request.rb
1 | 1 | class ConvertClosedToStateInMergeRequest < ActiveRecord::Migration |
2 | 2 | def up |
3 | 3 | MergeRequest.transaction do |
4 | - MergeRequest.where("closed = 1 AND merged = 1").update_all("state = 'merged'") | |
5 | - MergeRequest.where("closed = 1 AND merged = 0").update_all("state = 'closed'") | |
6 | - MergeRequest.where("closed = 0").update_all("state = 'opened'") | |
4 | + MergeRequest.where(closed: true, merged: true).update_all("state = 'merged'") | |
5 | + MergeRequest.where(closed: true, merged: true).update_all("state = 'closed'") | |
6 | + MergeRequest.where(closed: false).update_all("state = 'opened'") | |
7 | 7 | end |
8 | 8 | end |
9 | 9 | |
10 | 10 | def down |
11 | 11 | MergeRequest.transaction do |
12 | - MergeRequest.where(state: :closed).update_all("closed = 1") | |
13 | - MergeRequest.where(state: :merged).update_all("closed = 1, merged = 1") | |
12 | + MergeRequest.where(state: :closed).update_all(closed: true) | |
13 | + MergeRequest.where(state: :merged).update_all(closed: true, merged: true) | |
14 | 14 | end |
15 | 15 | end |
16 | 16 | end | ... | ... |