From 73167fb4f26b34c1da9c9bd9c12d175442a8e56d Mon Sep 17 00:00:00 2001 From: Jean-Philippe Lang Date: Sat, 6 Nov 2010 10:57:56 +0000 Subject: [PATCH] Rewrites UpdateMailNotificationValues migration to avoid model validations and failures. git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@4368 e93f8b46-1217-0410-a6f0-8f06a7374b81 --- ...9193813_update_mail_notification_values.rb | 19 +++---------------- 1 file changed, 3 insertions(+), 16 deletions(-) diff --git a/db/migrate/20100129193813_update_mail_notification_values.rb b/db/migrate/20100129193813_update_mail_notification_values.rb index 35ad91709..fe4bb10cd 100644 --- a/db/migrate/20100129193813_update_mail_notification_values.rb +++ b/db/migrate/20100129193813_update_mail_notification_values.rb @@ -1,22 +1,9 @@ # Patch the data from a boolean change. class UpdateMailNotificationValues < ActiveRecord::Migration def self.up - User.record_timestamps = false - User.all.each do |u| - u.mail_notification = if u.mail_notification =~ /\A(1|t)\z/ - # User set for all email (t is for sqlite) - 'all' - else - # User wants to recieve notifications on specific projects? - if u.memberships.count(:conditions => {:mail_notification => true}) > 0 - 'selected' - else - 'only_my_events' - end - end - u.save! - end - User.record_timestamps = true + User.update_all("mail_notification = 'all'", "mail_notification IN ('1', 't')") + User.update_all("mail_notification = 'selected'", "EXISTS (SELECT 1 FROM #{Member.table_name} WHERE #{Member.table_name}.mail_notification = #{connection.quoted_true} AND #{Member.table_name}.user_id = #{User.table_name}.id)") + User.update_all("mail_notification = 'only_my_events'", "mail_notification NOT IN ('all', 'selected')") end def self.down -- 2.39.5