トピックをカテゴリ間でサイレントに移動しようとすると、意図せず大量のメール/通知が届く

zogstrip · 2025 年 12 月 11 日午前 11:39

アハハ！問題を見つけたようです

FIX: silent option not working for bulk category moves via PostRevisor

Commit by

ZogStriP - DEV: improve topics_bulk_action_spec clarity and coverage

main ← dev-improve-topics-bulk-action-specs

The `silent` option for bulk category changes was not being respected when `crea…te_revision_on_bulk_topic_moves` was enabled. This caused users watching the destination category to receive notifications even when the "Perform this action silently" checkbox was checked. The bug was introduced in commit 7a099ebb6a which added `silent: @silent` to the `track_topic_field(:category_id)` block. However, since blocks capture `self` at definition time (the class, not the instance), `@silent` was always nil in that context. The fix passes the `silent` value through the `TopicChanges` object which is already available to the block as `tc`. Also extends test coverage to verify both topic watchers and category watchers are properly silenced in both code paths. Ref - meta/t/390993

@silent オプションが適切に伝播していなかったため、「このアクションをサイレントで実行する」チェックボックスがオンになっていても、移動先カテゴリをウォッチしているユーザーが通知を受け取っていました。

トピック		返信	表示
Bulk editing topic categories should not trigger thousands of email notifications Feature completed , bulk-actions	13	1380	2025 年 7 月 31 日
Moving old posts triggers email notifications Support	9	922	2021 年 12 月 5 日
Any way to NOT send emails when a topic category is changed? Support	14	2121	2019 年 4 月 26 日
Ability to move topics to new categories and suppress notifications Feature	10	298	2025 年 11 月 24 日
UI Bulk operations & Notifications Feature	1	751	2023 年 5 月 18 日

トピックをカテゴリ間でサイレントに移動しようとすると、意図せず大量のメール/通知が届く

関連トピック