Los marcadores siguen existiendo para temas a los que ya no tienes acceso

zogstrip · 24 Diciembre, 2025 10:52

¡Bien hecho @JammyDodger !

Eso será solucionado por

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

Tema		Respuestas	Vistas
Bookmark issue when access to topic is lost (PMs) Bug	6	1109	11 Marzo 2017
Disappearing bookmarks and expected behavior of /bookmarks Bug bookmarks	24	1897	11 Noviembre 2023
Bookmarks are not removed even if the topic/post is moved to a PM/private category Bug bookmarks	2	360	8 Abril 2024
Moderators still able to access other users bookmarks Bug	2	2581	4 Abril 2016
Reporting few issues with bookmarks Bug	3	746	29 Abril 2020

Los marcadores siguen existiendo para temas a los que ya no tienes acceso

Temas relacionados