From: Jean-Philippe Lang <jp_lang@yahoo.fr>
Date: Sat, 6 Nov 2010 10:57:56 +0000 (+0000)
Subject: Rewrites UpdateMailNotificationValues migration to avoid model validations and failures.
X-Git-Tag: 1.1.0~207
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=73167fb4f26b34c1da9c9bd9c12d175442a8e56d;p=redmine.git

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
---

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