nocebo/diaspora
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix migration for invitation-cleanup

Browse source 
delete invitations-table (and constraints) before deleting the users
This commit is contained in:
Benjamin Neff 2016-08-13 18:41:57 +02:00
parent 3942dca08f
commit 6248e945c2
1 changed files with 7 additions and 16 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

23
db/migrate/20160810230114_cleanup_invitation_columns_from_users.rb
View file

@ -12,7 +12,7 @@ class CleanupInvitationColumnsFromUsers < ActiveRecord::Migration
remove_index :users, column: :invitation_token, name: :index_users_on_invitation_token remove_index :users, column: :invitation_token, name: :index_users_on_invitation_token
remove_index :users, column: :email, name: :index_users_on_email, length: 191 remove_index :users, column: :email, name: :index_users_on_email, length: 191
username_not_null cleanup_invitations
remove_column :users, :invitation_token, :string, limit: 60 remove_column :users, :invitation_token, :string, limit: 60
remove_column :users, :invitation_sent_at, :datetime remove_column :users, :invitation_sent_at, :datetime
@ -22,21 +22,6 @@ class CleanupInvitationColumnsFromUsers < ActiveRecord::Migration
remove_column :users, :invited_by_type, :string remove_column :users, :invited_by_type, :string
add_index :users, :email, name: :index_users_on_email, unique: true, length: 191 add_index :users, :email, name: :index_users_on_email, unique: true, length: 191
cleanup_invitations
end
def username_not_null
reversible do |dir|
dir.up do
User.delete_all(username: nil)
change_column :users, :username, :string, null: false
end
dir.down do
change_column :users, :username, :string, null: true
end
end
end end
def cleanup_invitations def cleanup_invitations
@ -47,9 +32,15 @@ class CleanupInvitationColumnsFromUsers < ActiveRecord::Migration
# reset negative invitation counters # reset negative invitation counters
new_counter = AppConfig.settings.enable_registrations? ? AppConfig["settings.invitations.count"] : 0 new_counter = AppConfig.settings.enable_registrations? ? AppConfig["settings.invitations.count"] : 0
InvitationCode.where("count < 0").update_all(count: new_counter) InvitationCode.where("count < 0").update_all(count: new_counter)
# remove old invitation-users
User.delete_all(username: nil)
change_column :users, :username, :string, null: false
end end
dir.down do dir.down do
change_column :users, :username, :string, null: true
create_invitations_table create_invitations_table
end end
end end