Opened 10 years ago
Last modified 9 years ago
#5504 assigned defect (bug)
BuddyPress, in some cases, marks some notifications as read without the user has read them
Reported by: | SGr33n | Owned by: | imath |
---|---|---|---|
Milestone: | Awaiting Contributions | Priority: | normal |
Severity: | normal | Version: | |
Component: | Toolbar & Notifications | Keywords: | needs-patch |
Cc: |
Description
Hi,
In the case the user has more than 20 new_at_mention notifications, the admin bar will show e.g. "You have 60 new mentions". If you click on it, even if you will see just 20 notifications, all 60 will mark as read.
Change History (4)
#1
@
10 years ago
- Component changed from Core to Notifications
- Keywords needs-patch added
- Milestone changed from Awaiting Review to 2.1
#2
@
10 years ago
- Milestone changed from 2.1 to 2.2
Not going to have time to investigate this for 2.1.
#4
@
9 years ago
- Owner set to imath
- Status changed from new to assigned
Hi @SGr33n thanks for your feedback.
I think i see what's happening here. When you click on the admin bar, you display the page site.url/members/username/activity/mentions. In this page only 20 activities are displayed because the default activity per_page
loop parameter is 20. But when this screen is displayed, we are using a function to clear all new_at_mention notifications using bp_activity_remove_screen_notifications()
I have an idea about a possible "fix", will come back later on this ticket.
Sounds like a carryover from before Notifications was broken into a separate component.