mirror of
https://github.com/discourse/discourse.git
synced 2025-02-23 06:13:27 +08:00

This commit fixes an issue where when some actions were done (deleting/recovering post, moving posts) we updated the topic_users.bookmarked column to the wrong value. This was happening because the SyncTopicUserBookmarked job was not taking into account Topic level bookmarks, so if there was a Topic bookmark and no Post bookmarks for a user in the topic, they would have topic_users.bookmarked set to false, which meant the bookmark would no longer show in the /bookmarks list. To reproduce before the fix: * Bookmark a topic and don’t bookmark any posts within * Delete or recover any post in the topic c.f. https://meta.discourse.org/t/disappearing-bookmarks-and-expected-behavior-of-bookmarks/264670/36