diff options
| author | Eugen Rochko <eugen@zeonfederated.com> | 2017-03-30 16:06:27 +0200 |
|---|---|---|
| committer | Eugen Rochko <eugen@zeonfederated.com> | 2017-03-30 16:06:27 +0200 |
| commit | 03fb6c16ecc3c36104185507d601af87edecc655 (patch) | |
| tree | f0200405e9dd5c6c57f5b43bb56b3399255e891c | |
| parent | 87513b31e004bedc41e6479c51ea2b9db841d30e (diff) | |
Fix up null values on latest migration, add notes
| -rw-r--r-- | db/migrate/20170330021336_add_counter_caches.rb | 14 | ||||
| -rw-r--r-- | db/schema.rb | 10 |
2 files changed, 14 insertions, 10 deletions
diff --git a/db/migrate/20170330021336_add_counter_caches.rb b/db/migrate/20170330021336_add_counter_caches.rb index 5da9a87ea..cf064b9e1 100644 --- a/db/migrate/20170330021336_add_counter_caches.rb +++ b/db/migrate/20170330021336_add_counter_caches.rb @@ -1,9 +1,13 @@ class AddCounterCaches < ActiveRecord::Migration[5.0] def change - add_column :statuses, :favourites_count, :integer - add_column :statuses, :reblogs_count, :integer - add_column :accounts, :statuses_count, :integer - add_column :accounts, :followers_count, :integer - add_column :accounts, :following_count, :integer + add_column :statuses, :favourites_count, :integer, null: false, default: 0 + add_column :statuses, :reblogs_count, :integer, null: false, default: 0 + add_column :accounts, :statuses_count, :integer, null: false, default: 0 + add_column :accounts, :followers_count, :integer, null: false, default: 0 + add_column :accounts, :following_count, :integer, null: false, default: 0 end end + +# To make the new fields contain correct data: +# update statuses set favourites_count = (select count(*) from favourites where favourites.status_id = statuses.id), reblogs_count = (select count(*) from statuses as reblogs where reblogs.reblog_of_id = statuses.id); +# update accounts set statuses_count = (select count(*) from statuses where account_id = accounts.id), followers_count = (select count(*) from follows where target_account_id = accounts.id), following_count = (select count(*) from follows where account_id = accounts.id); diff --git a/db/schema.rb b/db/schema.rb index 52437ca57..7675ed1a9 100644 --- a/db/schema.rb +++ b/db/schema.rb @@ -44,9 +44,9 @@ ActiveRecord::Schema.define(version: 20170330021336) do t.boolean "suspended", default: false, null: false t.boolean "locked", default: false, null: false t.string "header_remote_url", default: "", null: false - t.integer "statuses_count" - t.integer "followers_count" - t.integer "following_count" + t.integer "statuses_count", default: 0, null: false + t.integer "followers_count", default: 0, null: false + t.integer "following_count", default: 0, null: false t.index "(((setweight(to_tsvector('simple'::regconfig, (display_name)::text), 'A'::\"char\") || setweight(to_tsvector('simple'::regconfig, (username)::text), 'B'::\"char\")) || setweight(to_tsvector('simple'::regconfig, (COALESCE(domain, ''::character varying))::text), 'C'::\"char\")))", name: "search_index", using: :gin t.index "lower((username)::text), lower((domain)::text)", name: "index_accounts_on_username_and_domain_lower", using: :btree t.index ["username", "domain"], name: "index_accounts_on_username_and_domain", unique: true, using: :btree @@ -223,8 +223,8 @@ ActiveRecord::Schema.define(version: 20170330021336) do t.integer "application_id" t.text "spoiler_text", default: "", null: false t.boolean "reply", default: false - t.integer "favourites_count" - t.integer "reblogs_count" + t.integer "favourites_count", default: 0, null: false + t.integer "reblogs_count", default: 0, null: false t.index ["account_id"], name: "index_statuses_on_account_id", using: :btree t.index ["in_reply_to_id"], name: "index_statuses_on_in_reply_to_id", using: :btree t.index ["reblog_of_id"], name: "index_statuses_on_reblog_of_id", using: :btree |