Modernise use of find and update_all in database migrations
[rails.git] / db / migrate / 044_create_user_roles.rb
index 6544bbba1dc3e0fcdf310f6de655678896f35100..0db3ea77e5fc01976419aa297ddad97b3b5b6152 100644 (file)
@@ -11,7 +11,7 @@ class CreateUserRoles < ActiveRecord::Migration
     end
     add_column :user_roles, :role, :user_role_enum, :null => false
 
-    User.all(:conditions => ['administrator = ?', true]).each do |user|
+    User.where(:administrator => true).each do |user|
       UserRole.create(:user_id => user.id, :role => "administrator")
     end
     remove_column :users, :administrator
@@ -21,7 +21,7 @@ class CreateUserRoles < ActiveRecord::Migration
 
   def self.down
     add_column :users, :administrator, :boolean, :default => false, :null => false
-    UserRole.all(:conditions => ['role = ?', "administrator"]).each do |role|
+    UserRole.where(:role => "administrator").each do |role|
       user = User.find(role.user_id)
       user.administrator = true
       user.save!