From 04bb267f6ff6dc6feaba163a028e15982428c5bc Mon Sep 17 00:00:00 2001 From: jingquan huang Date: Fri, 21 Jun 2019 18:53:04 +0800 Subject: [PATCH] =?UTF-8?q?=E5=A4=84=E7=90=86=E6=97=A0=E6=95=88=E7=9A=84?= =?UTF-8?q?=E6=95=B0=E6=8D=AE?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- ...rb => 20190621105144_sync_ivalid_users.rb} | 68 +++++++++---------- 1 file changed, 34 insertions(+), 34 deletions(-) rename db/migrate/{20190505092009_delete_ivalid_data.rb => 20190621105144_sync_ivalid_users.rb} (60%) diff --git a/db/migrate/20190505092009_delete_ivalid_data.rb b/db/migrate/20190621105144_sync_ivalid_users.rb similarity index 60% rename from db/migrate/20190505092009_delete_ivalid_data.rb rename to db/migrate/20190621105144_sync_ivalid_users.rb index 98f13d9e4..3520ca84d 100644 --- a/db/migrate/20190505092009_delete_ivalid_data.rb +++ b/db/migrate/20190621105144_sync_ivalid_users.rb @@ -1,34 +1,34 @@ -class DeleteIvalidData < ActiveRecord::Migration[5.2] - def change - users = User.find_by_sql("select count(*) as user_count, login from users group by login having user_count>1") - users.each do |user| - valid_users = User.where(login: user.login) - valid_users.each do |valid_user| - unless valid_user.lastname.present? - valid_user.delete - end - end - end - - mail_users = User.find_by_sql("select count(*) as user_count, mail from users where mail is not null group by mail having user_count>1") - mail_users.each do |mail_user| - valid_mail_users = User.where(mail: mail_user.mail) - valid_mail_users.each do |valid_mail_user| - unless valid_mail_user.lastname.present? - valid_mail_user.delete - end - end - end - - - phone_users = User.find_by_sql("select count(*) as user_count, phone from users where phone is not null group by phone having user_count>1") - phone_users.each do |phone_user| - valid_phone_users = User.where(phone: phone_user.phone) - valid_phone_users.each do |valid_phone_user| - unless valid_phone_user.lastname.present? - valid_phone_user.delete - end - end - end - end -end +class SyncIvalidUsers < ActiveRecord::Migration[5.2] + def change + users = User.where.not(id: 2).find_by_sql("select count(*) as user_count, login from users group by login having user_count>1") + users.each do |user| + valid_users = User.where(login: user.login) + valid_users.each do |valid_user| + unless valid_user.lastname.present? + valid_user.delete + end + end + end + + mail_users = User.where.not(id: 2).find_by_sql("select count(*) as user_count, mail from users where mail is not null group by mail having user_count>1") + mail_users.each do |mail_user| + valid_mail_users = User.where(mail: mail_user.mail) + valid_mail_users.each do |valid_mail_user| + unless valid_mail_user.lastname.present? + valid_mail_user.delete + end + end + end + + + phone_users = User.where.not(id: 2).find_by_sql("select count(*) as user_count, phone from users where phone is not null group by phone having user_count>1") + phone_users.each do |phone_user| + valid_phone_users = User.where(phone: phone_user.phone) + valid_phone_users.each do |valid_phone_user| + unless valid_phone_user.lastname.present? + valid_phone_user.delete + end + end + end + end +end