From aedb546964cc6b6e456369bf17a3d917089f7017 Mon Sep 17 00:00:00 2001 From: Robert Speicher Date: Mon, 22 Jun 2015 11:36:11 -0400 Subject: Correct AddDefaultOtpRequiredForLoginValue migration Also MySQL complains when you change a field to be not null if any existing records already have a null value, so this updates those rows. --- db/migrate/20150620233230_add_default_otp_required_for_login_value.rb | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/db/migrate/20150620233230_add_default_otp_required_for_login_value.rb b/db/migrate/20150620233230_add_default_otp_required_for_login_value.rb index c209fa4e80f..8eed8678b2f 100644 --- a/db/migrate/20150620233230_add_default_otp_required_for_login_value.rb +++ b/db/migrate/20150620233230_add_default_otp_required_for_login_value.rb @@ -1,9 +1,11 @@ class AddDefaultOtpRequiredForLoginValue < ActiveRecord::Migration def up + execute %q{UPDATE users SET otp_required_for_login = FALSE WHERE otp_required_for_login IS NULL} + change_column :users, :otp_required_for_login, :boolean, default: false, null: false end def down - change_column :users, :otp_required_for_login, :boolean, default: nil + change_column :users, :otp_required_for_login, :boolean, null: true end end -- cgit v1.2.1