Boolean correctness in migrations
authorAndy Allan <gravitystorm@gmail.com>
Sat, 8 Nov 2008 15:01:14 +0000 (15:01 +0000)
committerAndy Allan <gravitystorm@gmail.com>
Sat, 8 Nov 2008 15:01:14 +0000 (15:01 +0000)
db/migrate/013_add_email_valid.rb
db/migrate/015_add_user_visible.rb
db/migrate/021_add_changesets.rb

index b8af4bf..2192383 100644 (file)
@@ -1,7 +1,7 @@
 class AddEmailValid < ActiveRecord::Migration
   def self.up
     add_column "users", "email_valid", :boolean, :default => false, :null => false
-    User.update_all("email_valid = active")
+    User.update_all(:email_valid => true)
   end
 
   def self.down
index d870dff..869f24c 100644 (file)
@@ -1,7 +1,7 @@
 class AddUserVisible < ActiveRecord::Migration
   def self.up
     add_column "users", "visible", :boolean, :default => true, :null => false
-    User.update_all("visible = 1")
+    User.update_all(:visible => true)
   end
 
   def self.down
index 9b769c7..e0cf390 100644 (file)
@@ -28,7 +28,7 @@ class AddChangesets < ActiveRecord::Migration
     # all the changesets will have the id of the user that made them.
     # We need to generate a changeset for each user in the database
     execute "INSERT INTO changesets (id, user_id, created_at, open)" + 
-      "SELECT id, id, creation_time, 0 from users;"
+      "SELECT id, id, creation_time, false from users;"
 
     @@conv_user_tables.each { |tbl|
       rename_column tbl, :user_id, :changeset_id