diff options
| author | Takeshi Umeda <noel.yoshiba@gmail.com> | 2022-10-22 21:30:59 +0900 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2022-10-22 14:30:59 +0200 |
| commit | 53e86747e49dea5e4314bc3fedef4d69ba8f338e (patch) | |
| tree | 8852115d6d4d094e006e0d5c96501c42984dea61 /db | |
| parent | 1d34eff63f65dd0809af172cf3b1ed5e24c62eaf (diff) | |
Fix duplicate featured tags (#19403)
* Fix duplicate featured tags * Add unique tag name validator * Fix error message
Diffstat (limited to 'db')
| -rw-r--r-- | db/migrate/20221021055441_add_index_featured_tags_on_account_id_and_tag_id.rb | 19 | ||||
| -rw-r--r-- | db/schema.rb | 4 |
2 files changed, 21 insertions, 2 deletions
diff --git a/db/migrate/20221021055441_add_index_featured_tags_on_account_id_and_tag_id.rb b/db/migrate/20221021055441_add_index_featured_tags_on_account_id_and_tag_id.rb new file mode 100644 index 000000000..74d7673f7 --- /dev/null +++ b/db/migrate/20221021055441_add_index_featured_tags_on_account_id_and_tag_id.rb @@ -0,0 +1,19 @@ +class AddIndexFeaturedTagsOnAccountIdAndTagId < ActiveRecord::Migration[6.1] + disable_ddl_transaction! + + def up + duplicates = FeaturedTag.connection.select_all('SELECT string_agg(id::text, \',\') AS ids FROM featured_tags GROUP BY account_id, tag_id HAVING count(*) > 1').to_ary + + duplicates.each do |row| + FeaturedTag.where(id: row['ids'].split(',')[0...-1]).destroy_all + end + + add_index :featured_tags, [:account_id, :tag_id], unique: true, algorithm: :concurrently + remove_index :featured_tags, [:account_id], algorithm: :concurrently + end + + def down + add_index :featured_tags, [:account_id], algorithm: :concurrently + remove_index :featured_tags, [:account_id, :tag_id], unique: true, algorithm: :concurrently + end +end diff --git a/db/schema.rb b/db/schema.rb index 3972f777a..ed00d9e74 100644 --- a/db/schema.rb +++ b/db/schema.rb @@ -10,7 +10,7 @@ # # It's strongly recommended that you check this file into your version control system. -ActiveRecord::Schema.define(version: 2022_10_12_181003) do +ActiveRecord::Schema.define(version: 2022_10_21_055441) do # These are extensions that must be enabled in order to support this database enable_extension "plpgsql" @@ -442,7 +442,7 @@ ActiveRecord::Schema.define(version: 2022_10_12_181003) do t.datetime "last_status_at" t.datetime "created_at", null: false t.datetime "updated_at", null: false - t.index ["account_id"], name: "index_featured_tags_on_account_id" + t.index ["account_id", "tag_id"], name: "index_featured_tags_on_account_id_and_tag_id", unique: true t.index ["tag_id"], name: "index_featured_tags_on_tag_id" end |