Rephrase the OCI description for clarity

[rails.git] / db / migrate / 023_add_changesets.rb

diff --git a/db/migrate/023_add_changesets.rb b/db/migrate/023_add_changesets.rb
index 5bee215122e23f21c415a77ad8471821d0a915c1..ff6ce2bb4356db29f5680b3d871edde95699572b 100644 (file)
--- a/db/migrate/023_add_changesets.rb
+++ b/db/migrate/023_add_changesets.rb
@@ -1,48 +1,45 @@
-require 'migrate'
+class AddChangesets < ActiveRecord::Migration[4.2]
+  @conv_user_tables = %w[current_nodes current_relations current_ways nodes relations ways]
 
-class AddChangesets < ActiveRecord::Migration
-  @@conv_user_tables = ['current_nodes',
-  'current_relations', 'current_ways', 'nodes', 'relations', 'ways' ]
-  
   def self.up
-    create_table "changesets", innodb_table do |t|
-      t.column "id",             :bigint_pk,              :null => false
-      t.column "user_id",        :bigint,   :limit => 20, :null => false
-      t.column "created_at",     :datetime,               :null => false
-      t.column "open",           :boolean,                :null => false, :default => true
-      t.column "min_lat",        :integer,                :null => true
-      t.column "max_lat",        :integer,                :null => true
-      t.column "min_lon",        :integer,                :null => true
-      t.column "max_lon",        :integer,                :null => true
+    create_table "changesets", :id => false do |t|
+      t.column "id",             :bigserial, :primary_key => true, :null => false
+      t.column "user_id",        :bigint, :null => false
+      t.column "created_at",     :datetime, :null => false
+      t.column "open",           :boolean, :null => false, :default => true
+      t.column "min_lat",        :integer, :null => true
+      t.column "max_lat",        :integer, :null => true
+      t.column "min_lon",        :integer, :null => true
+      t.column "max_lon",        :integer, :null => true
     end
 
-    create_table "changeset_tags", innodb_table do |t|
-      t.column "id", :bigint, :limit => 64, :null => false
+    create_table "changeset_tags", :id => false do |t|
+      t.column "id", :bigint, :null => false
       t.column "k",  :string, :default => "", :null => false
       t.column "v",  :string, :default => "", :null => false
     end
 
     add_index "changeset_tags", ["id"], :name => "changeset_tags_id_idx"
-    
+
     #
-    # Initially we will have one changeset for every user containing 
-    # all edits up to the API change,  
+    # Initially we will have one changeset for every user containing
+    # all edits up to the API change,
     # 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, false from users;"
+    execute "INSERT INTO changesets (id, user_id, created_at, open)" \
+            "SELECT id, id, creation_time, false from users;"
 
-    @@conv_user_tables.each { |tbl|
+    @conv_user_tables.each do |tbl|
       rename_column tbl, :user_id, :changeset_id
-      #foreign keys too
-      add_foreign_key tbl, [:changeset_id], :changesets, [:id]
-    }
+      # foreign keys too
+      add_foreign_key tbl, :changesets, :name => "#{tbl}_changeset_id_fkey"
+    end
   end
 
   def self.down
     # It's not easy to generate the user ids from the changesets
     raise ActiveRecord::IrreversibleMigration
-    #drop_table "changesets"
-    #drop_table "changeset_tags"
+    # drop_table "changesets"
+    # drop_table "changeset_tags"
   end
 end