| @@ -1,22 +1,22 | |||
|
|
1 | 1 | class AddUniqueIndexOnMembers < ActiveRecord::Migration |
|
|
2 | 2 | def self.up |
|
|
3 | 3 | # Clean and reassign MemberRole rows if needed |
|
|
4 | 4 | MemberRole.delete_all("member_id NOT IN (SELECT id FROM #{Member.table_name})") |
|
|
5 | 5 | MemberRole.update_all("member_id =" + |
|
|
6 | 6 | " (SELECT min(m2.id) FROM #{Member.table_name} m1, #{Member.table_name} m2" + |
|
|
7 | 7 | " WHERE m1.user_id = m2.user_id AND m1.project_id = m2.project_id" + |
|
|
8 | 8 | " AND m1.id = #{MemberRole.table_name}.member_id)") |
|
|
9 | 9 | # Remove duplicates |
|
|
10 | 10 | Member.connection.select_values("SELECT m.id FROM #{Member.table_name} m" + |
|
|
11 | 11 | " WHERE m.id > (SELECT min(m1.id) FROM #{Member.table_name} m1 WHERE m1.user_id = m.user_id AND m1.project_id = m.project_id)").each do |i| |
|
|
12 | 12 | Member.delete_all(["id = ?", i]) |
|
|
13 | 13 | end |
|
|
14 | ||
|
|
14 | ||
|
|
15 | 15 | # Then add a unique index |
|
|
16 | 16 | add_index :members, [:user_id, :project_id], :unique => true |
|
|
17 | 17 | end |
|
|
18 | 18 | |
|
|
19 | 19 | def self.down |
|
|
20 | 20 | remove_index :members, [:user_id, :project_id] |
|
|
21 | 21 | end |
|
|
22 | 22 | end |
General Comments 0
You need to be logged in to leave comments.
Login now