miteruzo/btrc-hub
1
0
Fork 0
Code Issues 69 Pull Requests 1 Releases Wiki Activity

feat: 投稿とタグのリレーション・テーブルについて論理削除と履歴を追加（#84） #148

Merged
みてるぞ merged 5 commits from feature/084 into main 2025-12-13 16:34:47 +09:00
Conversation 2 Commits 5 Files Changed 8
+173 -56
みてるぞ commented 2025-10-13 02:31:11 +09:00
Owner
Copy Link
Copy Source

主要変更点

  • post_tags テーブルに対して論理削除コラム追加
  • 投稿更新時に外されたタグの関聯を論理削除するロジックに変更
  • 投稿追加、更新時に追加されたタグの関聯に操作者情報を登録するロジックに変更
  • ニコニコ同期時に外されたタグの関聯を論理削除するロジックに変更

テスト観点

  • 投稿追加時に post_tags.created_user_id に操作者が登録されること
  • タグ編輯時に追加したタグの post_tags.created_user_id に操作者が登録されること
  • タグ編輯時に削除したタグの post_tags.discarded_at に削除日時が登録されること
  • タグ編輯時に削除したタグの post_tags.deleted_user_id に操作者が登録されること
  • タグ編輯時に削除したことのあるタグを追加し，過去の削除レコードとは別に post_tags レコードが生えること
  • タグ編輯時に削除したことのあるタグを追加し，直後にそのタグを削除した際に同じ参照の post_tags レコードが 2 本生えること
  • ニコニコ同期時の新規動画に追加された nico タグの操作者が NULL であること
  • ニコニコ同期時の新規動画に追加された nico タグの連携タグの操作者が NULL であること
  • ニコニコ同期時の更新動画に追加された nico タグの操作者が NULL であること
  • ニコニコ同期時の更新動画に追加された nico タグの連携タグの操作者が NULL であること
  • ニコニコ同期時の更新動画に削除された nico タグの操作者が NULL であること
  • ニコニコ同期時の更新動画に削除された nico タグに削除日時が記録されること
  • 投稿追加時に親タグの post_tags.created_user_id に操作者が登録されること
  • タグ編輯時に追加したタグの親の post_tags.created_user_id に操作者が登録されること

テスト補足

ニコニコ同期のテストは，以下の形式の JSON を返す Python プログラムをもって API とみなし実施するとよい：

{ "id": number
, "code": string
, "title": string
, "description": string
, "tags": string[]
, "uploaded_at": string
, "deleted_at": null
}[]
### 主要変更点 - `post_tags` テーブルに対して論理削除コラム追加 - 投稿更新時に外されたタグの関聯を論理削除するロジックに変更 - 投稿追加、更新時に追加されたタグの関聯に操作者情報を登録するロジックに変更 - ニコニコ同期時に外されたタグの関聯を論理削除するロジックに変更 ### テスト観点 - [x] 投稿追加時に `post_tags.created_user_id` に操作者が登録されること - [x] タグ編輯時に追加したタグの `post_tags.created_user_id` に操作者が登録されること - [x] タグ編輯時に削除したタグの `post_tags.discarded_at` に削除日時が登録されること - [x] タグ編輯時に削除したタグの `post_tags.deleted_user_id` に操作者が登録されること - [x] タグ編輯時に削除したことのあるタグを追加し，過去の削除レコードとは別に `post_tags` レコードが生えること - [x] タグ編輯時に削除したことのあるタグを追加し，直後にそのタグを削除した際に同じ参照の `post_tags` レコードが 2 本生えること - [x] ニコニコ同期時の新規動画に追加された nico タグの操作者が NULL であること - [x] ニコニコ同期時の新規動画に追加された nico タグの連携タグの操作者が NULL であること - [x] ニコニコ同期時の更新動画に追加された nico タグの操作者が NULL であること - [x] ニコニコ同期時の更新動画に追加された nico タグの連携タグの操作者が NULL であること - [x] ニコニコ同期時の更新動画に削除された nico タグの操作者が NULL であること - [x] ニコニコ同期時の更新動画に削除された nico タグに削除日時が記録されること - [x] 投稿追加時に親タグの `post_tags.created_user_id` に操作者が登録されること - [x] タグ編輯時に追加したタグの親の `post_tags.created_user_id` に操作者が登録されること #### テスト補足 ニコニコ同期のテストは，以下の形式の JSON を返す Python プログラムをもって API とみなし実施するとよい： ``` { "id": number , "code": string , "title": string , "description": string , "tags": string[] , "uploaded_at": string , "deleted_at": null }[]
みてるぞ self-assigned this 2025-10-13 02:31:11 +09:00
みてるぞ changed title from feat: 投稿とタグのリレーション・テーブルについて論理削除と履歴を追加（q） to feat: 投稿とタグのリレーション・テーブルについて論理削除と履歴を追加（#84） 2025-10-13 02:31:24 +09:00
みてるぞ reviewed 2025-12-10 22:02:44 +09:00
backend/app/models/tag.rb Outdated
@@ -2,2 +2,2 @@
has_many :post_tags, dependent: :destroy
has_many :posts, through: :post_tags
has_many :post_tags, dependent: :delete_all, inverse_of: :tag
has_many :active_post_tags, -> { kept }, class_name: 'PostTag', :inverse_of: :tag
みてるぞ commented 2025-12-10 22:02:44 +09:00
Author
Owner
Copy Link
Copy Source

:inverse_of: :tag が文法エラー．

`:inverse_of: :tag` が文法エラー．
miteruzo marked this conversation as resolved
みてるぞ changed target branch from test to main 2025-12-10 22:55:24 +09:00
みてるぞ reviewed 2025-12-12 00:20:36 +09:00
backend/db/migrate/20251011200300_add_discard_to_post_tags.rb Outdated
@@ -0,0 +1,18 @@
class AddDiscardToPostTags < ActiveRecord::Migration[8.0]
def change
みてるぞ commented 2025-12-12 00:20:36 +09:00
Author
Owner
Copy Link
Copy Source

change ではなく up にしたい．
down は設けず，不可逆な作りにしたはぅが安心．

`change` ではなく `up` にしたい． `down` は設けず，不可逆な作りにしたはぅが安心．
miteruzo marked this conversation as resolved
みてるぞ merged commit 9a656a9e6e into main 2025-12-13 16:34:46 +09:00
みてるぞ started working 2026-01-31 01:45:30 +09:00
みてるぞ worked for 33min 15s 2026-01-31 02:18:45 +09:00
Sign in to join this conversation.
Reviewers
No Reviewers
みてるぞ
Labels
Clear labels
area/backend
area/frontend
duplicate
invalid
P0
P1
P2
P3
status/blocked
status/in-progress
status/ready
status/review
type/bug
type/enhancement
type/task
wontfix
No Label
Milestone
No items
No Milestone
Assignees
Clear assignees
miteruzo (みてるぞ)
No Assignees みてるぞ
1 Participants
Notifications
Total Time Spent: 33 minutes
みてるぞ
33 minutes
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: miteruzo/btrc-hub#148
Delete Branch "feature/084"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?