Lesezeichen existieren noch für Themen, auf die Sie keinen Zugriff mehr haben

zogstrip · 24. Dezember 2025 um 10:52

Schön gemacht @JammyDodger

Das wird behoben durch

github.com/discourse/discourse

FIX: filter bookmark count by access permissions

main ← fix-bookmark-count

opened 10:52AM - 24 Dec 25 UTC

ZogStriP

+320 -290

Previously, `UserSummary#bookmark_count` used a simple `Bookmark.where(user:).co…unt` which included bookmarks for content the user could no longer access (e.g., topics moved to private categories, deleted posts, or PMs they were removed from). This caused a mismatch where users saw "4 bookmarks" in their profile stats but an empty bookmark page. Added `BookmarkQuery#count_all` that reuses the same access-filtered UNION query as `list_all`, ensuring the count reflects only accessible bookmarks. Also refactored `BookmarkQuery` for clarity: - Extracted `build_list_queries` to share query building between `count_all` and `list_all` - Extracted helper methods in `unread_notifications`: `fetch_reminder_notifications`, `load_deleted_bookmarkables`, `can_see_notification_bookmark?` - Removed verbose comments and debug SQL formatting Ref - https://meta.discourse.org/t/391822

Thema		Antworten	Aufrufe
Bookmark issue when access to topic is lost (PMs) Bug	5	1130	8. März 2017
Disappearing bookmarks and expected behavior of /bookmarks Bug bookmarks	23	2044	7. November 2023
Bookmarks are not removed even if the topic/post is moved to a PM/private category Bug bookmarks	2	374	8. April 2024
Reporting few issues with bookmarks Bug	2	758	29. April 2020
Moderators still able to access other users bookmarks Bug	1	2597	3. April 2016

Lesezeichen existieren noch für Themen, auf die Sie keinen Zugriff mehr haben

Verwandte Themen