Description:
fix migration to work with rails 4 and ruby 2.3
Commit status:
[Not Reviewed]
References:
Diff options:
Comments:
0 Commit comments
0 Inline Comments
Unresolved TODOs:
There are no unresolved TODOs
r666:0327c11ce255 - - 8 files changed: 9 inserted, 10 deleted
@@ -1,16 +1,16 | |||||
|
1 | class AddLanguageExt < ActiveRecord::Migration |
|
1 | class AddLanguageExt < ActiveRecord::Migration |
|
2 | def self.up |
|
2 | def self.up |
|
3 | add_column :languages, :ext, :string, :limit => 10 |
|
3 | add_column :languages, :ext, :string, :limit => 10 |
|
4 |
|
4 | ||
|
5 | Language.reset_column_information |
|
5 | Language.reset_column_information |
|
6 |
- langs = Language. |
|
6 | + langs = Language.all |
|
7 | langs.each do |l| |
|
7 | langs.each do |l| |
|
8 | l.ext = l.name |
|
8 | l.ext = l.name |
|
9 | l.save |
|
9 | l.save |
|
10 | end |
|
10 | end |
|
11 | end |
|
11 | end |
|
12 |
|
12 | ||
|
13 | def self.down |
|
13 | def self.down |
|
14 | remove_column :languages, :ext |
|
14 | remove_column :languages, :ext |
|
15 | end |
|
15 | end |
|
16 | end |
|
16 | end |
@@ -1,17 +1,17 | |||||
|
1 | class AddStatusToTasks < ActiveRecord::Migration |
|
1 | class AddStatusToTasks < ActiveRecord::Migration |
|
2 | def self.up |
|
2 | def self.up |
|
3 | add_column :tasks, :status, :integer |
|
3 | add_column :tasks, :status, :integer |
|
4 | add_column :tasks, :updated_at, :datetime |
|
4 | add_column :tasks, :updated_at, :datetime |
|
5 |
|
5 | ||
|
6 | Task.reset_column_information |
|
6 | Task.reset_column_information |
|
7 |
- Task. |
|
7 | + Task.all.each do |task| |
|
8 | task.status_complete |
|
8 | task.status_complete |
|
9 | task.save |
|
9 | task.save |
|
10 | end |
|
10 | end |
|
11 | end |
|
11 | end |
|
12 |
|
12 | ||
|
13 | def self.down |
|
13 | def self.down |
|
14 | remove_column :tasks, :updated_at |
|
14 | remove_column :tasks, :updated_at |
|
15 | remove_column :tasks, :status |
|
15 | remove_column :tasks, :status |
|
16 | end |
|
16 | end |
|
17 | end |
|
17 | end |
@@ -1,25 +1,24 | |||||
|
1 | class AddNumberToSubmissions < ActiveRecord::Migration |
|
1 | class AddNumberToSubmissions < ActiveRecord::Migration |
|
2 | def self.up |
|
2 | def self.up |
|
3 | add_column :submissions, :number, :integer |
|
3 | add_column :submissions, :number, :integer |
|
4 |
|
4 | ||
|
5 | # add number field for all records |
|
5 | # add number field for all records |
|
6 | Submission.reset_column_information |
|
6 | Submission.reset_column_information |
|
7 |
|
7 | ||
|
8 | last_user_id = nil |
|
8 | last_user_id = nil |
|
9 | last_problem_id = nil |
|
9 | last_problem_id = nil |
|
10 | current_number = 0 |
|
10 | current_number = 0 |
|
11 |
|
11 | ||
|
12 | - Submission.find(:all, |
|
12 | + Submission.order('user_id, problem_id, submitted_at').each do |submission| |
|
13 | - :order => 'user_id, problem_id, submitted_at').each do |submission| |
|
||
|
14 | if submission.user_id==last_user_id and submission.problem_id==last_problem_id |
|
13 | if submission.user_id==last_user_id and submission.problem_id==last_problem_id |
|
15 | current_number += 1 |
|
14 | current_number += 1 |
|
16 | else |
|
15 | else |
|
17 | current_number = 1 |
|
16 | current_number = 1 |
|
18 | end |
|
17 | end |
|
19 | submission.number = current_number |
|
18 | submission.number = current_number |
|
20 | submission.save |
|
19 | submission.save |
|
21 |
|
20 | ||
|
22 | last_user_id = submission.user_id |
|
21 | last_user_id = submission.user_id |
|
23 | last_problem_id = submission.problem_id |
|
22 | last_problem_id = submission.problem_id |
|
24 | end |
|
23 | end |
|
25 |
|
24 |
@@ -1,22 +1,22 | |||||
|
1 | class AddSiteToUserAndAddDefaultSite < ActiveRecord::Migration |
|
1 | class AddSiteToUserAndAddDefaultSite < ActiveRecord::Migration |
|
2 | def self.up |
|
2 | def self.up |
|
3 | default_site = Site.new({:name => 'default', |
|
3 | default_site = Site.new({:name => 'default', |
|
4 | :started => false}) |
|
4 | :started => false}) |
|
5 | default_site.save! |
|
5 | default_site.save! |
|
6 |
|
6 | ||
|
7 | add_column :users, :site_id, :integer |
|
7 | add_column :users, :site_id, :integer |
|
8 | User.reset_column_information |
|
8 | User.reset_column_information |
|
9 |
|
9 | ||
|
10 |
- User. |
|
10 | + User.all.each do |user| |
|
11 |
|
11 | ||
|
12 | class << user |
|
12 | class << user |
|
13 | def valid? |
|
13 | def valid? |
|
14 | true |
|
14 | true |
|
15 | end |
|
15 | end |
|
16 | end |
|
16 | end |
|
17 |
|
17 | ||
|
18 | user.site_id = default_site.id |
|
18 | user.site_id = default_site.id |
|
19 | user.save |
|
19 | user.save |
|
20 | end |
|
20 | end |
|
21 | end |
|
21 | end |
|
22 |
|
22 |
@@ -1,33 +1,33 | |||||
|
1 | class RefactorProblemBodyToDescription < ActiveRecord::Migration |
|
1 | class RefactorProblemBodyToDescription < ActiveRecord::Migration |
|
2 | def self.up |
|
2 | def self.up |
|
3 | add_column :problems, :description_id, :integer |
|
3 | add_column :problems, :description_id, :integer |
|
4 | Problem.reset_column_information |
|
4 | Problem.reset_column_information |
|
5 |
|
5 | ||
|
6 |
- Problem. |
|
6 | + Problem.all.each do |problem| |
|
7 | if problem.body!=nil |
|
7 | if problem.body!=nil |
|
8 | description = Description.new |
|
8 | description = Description.new |
|
9 | description.body = problem.body |
|
9 | description.body = problem.body |
|
10 | description.markdowned = false |
|
10 | description.markdowned = false |
|
11 | description.save |
|
11 | description.save |
|
12 | problem.description_id = description.id |
|
12 | problem.description_id = description.id |
|
13 | problem.save |
|
13 | problem.save |
|
14 | end |
|
14 | end |
|
15 | end |
|
15 | end |
|
16 |
|
16 | ||
|
17 | remove_column :problems, :body |
|
17 | remove_column :problems, :body |
|
18 | end |
|
18 | end |
|
19 |
|
19 | ||
|
20 | def self.down |
|
20 | def self.down |
|
21 | add_column :problems, :body, :text |
|
21 | add_column :problems, :body, :text |
|
22 | Problem.reset_column_information |
|
22 | Problem.reset_column_information |
|
23 |
|
23 | ||
|
24 |
- Problem. |
|
24 | + Problem.all.each do |problem| |
|
25 | if problem.description_id != nil |
|
25 | if problem.description_id != nil |
|
26 | problem.body = Description.find(problem.description_id).body |
|
26 | problem.body = Description.find(problem.description_id).body |
|
27 | problem.save |
|
27 | problem.save |
|
28 | end |
|
28 | end |
|
29 | end |
|
29 | end |
|
30 |
|
30 | ||
|
31 | remove_column :problems, :description_id |
|
31 | remove_column :problems, :description_id |
|
32 | end |
|
32 | end |
|
33 | end |
|
33 | end |
@@ -1,15 +1,15 | |||||
|
1 | class AddTestAllowedToProblems < ActiveRecord::Migration |
|
1 | class AddTestAllowedToProblems < ActiveRecord::Migration |
|
2 | def self.up |
|
2 | def self.up |
|
3 | add_column :problems, :test_allowed, :boolean |
|
3 | add_column :problems, :test_allowed, :boolean |
|
4 | Problem.reset_column_information |
|
4 | Problem.reset_column_information |
|
5 |
|
5 | ||
|
6 |
- Problem. |
|
6 | + Problem.all.each do |problem| |
|
7 | problem.test_allowed = true |
|
7 | problem.test_allowed = true |
|
8 | problem.save |
|
8 | problem.save |
|
9 | end |
|
9 | end |
|
10 | end |
|
10 | end |
|
11 |
|
11 | ||
|
12 | def self.down |
|
12 | def self.down |
|
13 | remove_column :problems, :test_allowed |
|
13 | remove_column :problems, :test_allowed |
|
14 | end |
|
14 | end |
|
15 | end |
|
15 | end |
@@ -1,19 +1,19 | |||||
|
1 | class AddActivatedToUsers < ActiveRecord::Migration |
|
1 | class AddActivatedToUsers < ActiveRecord::Migration |
|
2 | def self.up |
|
2 | def self.up |
|
3 | add_column :users, :activated, :boolean, :default => 0 |
|
3 | add_column :users, :activated, :boolean, :default => 0 |
|
4 |
|
4 | ||
|
5 | User.reset_column_information |
|
5 | User.reset_column_information |
|
6 |
|
6 | ||
|
7 |
- User. |
|
7 | + User.all.each do |user| |
|
8 |
|
8 | ||
|
9 | # disable validation |
|
9 | # disable validation |
|
10 | class <<user |
|
10 | class <<user |
|
11 | def valid? |
|
11 | def valid? |
|
12 | return true |
|
12 | return true |
|
13 | end |
|
13 | end |
|
14 | end |
|
14 | end |
|
15 |
|
15 | ||
|
16 | user.activated = true |
|
16 | user.activated = true |
|
17 | user.save |
|
17 | user.save |
|
18 | end |
|
18 | end |
|
19 | end |
|
19 | end |
@@ -2,22 +2,22 | |||||
|
2 | def self.up |
|
2 | def self.up |
|
3 | # language.common_ext is a comma-separated list of common file |
|
3 | # language.common_ext is a comma-separated list of common file |
|
4 | # extensions. |
|
4 | # extensions. |
|
5 | add_column :languages, :common_ext, :string |
|
5 | add_column :languages, :common_ext, :string |
|
6 |
|
6 | ||
|
7 | # updating table information |
|
7 | # updating table information |
|
8 | Language.reset_column_information |
|
8 | Language.reset_column_information |
|
9 | common_ext = { |
|
9 | common_ext = { |
|
10 | 'c' => 'c', |
|
10 | 'c' => 'c', |
|
11 | 'cpp' => 'cpp,cc', |
|
11 | 'cpp' => 'cpp,cc', |
|
12 | 'pas' => 'pas' |
|
12 | 'pas' => 'pas' |
|
13 | } |
|
13 | } |
|
14 |
- Language. |
|
14 | + Language.all.each do |lang| |
|
15 | lang.common_ext = common_ext[lang.name] |
|
15 | lang.common_ext = common_ext[lang.name] |
|
16 | lang.save |
|
16 | lang.save |
|
17 | end |
|
17 | end |
|
18 | end |
|
18 | end |
|
19 |
|
19 | ||
|
20 | def self.down |
|
20 | def self.down |
|
21 | remove_column :languages, :common_ext |
|
21 | remove_column :languages, :common_ext |
|
22 | end |
|
22 | end |
|
23 | end |
|
23 | end |
You need to be logged in to leave comments.
Login now