Skip to:
Content

BuddyPress.org

Opened 6 years ago

Last modified 4 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 @boonebgorges
6 years ago

  • Component changed from Core to Notifications
  • Keywords needs-patch added
  • Milestone changed from Awaiting Review to 2.1

Sounds like a carryover from before Notifications was broken into a separate component.

#2 @DJPaul
5 years ago

  • Milestone changed from 2.1 to 2.2

Not going to have time to investigate this for 2.1.

#3 @DJPaul
5 years ago

  • Milestone changed from 2.2 to Future Release

#4 @imath
4 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.

Last edited 4 years ago by imath (previous) (diff)
Note: See TracTickets for help on using tickets.