Skip to:
Content

BuddyPress.org

Opened 12 years ago

Last modified 11 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
Priority: normal Milestone: Awaiting Contributions
Component: Toolbar & Notifications Version:
Severity: normal 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
12 years ago

  • Component CoreNotifications
  • Keywords needs-patch added
  • Milestone Awaiting Review2.1

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

#2 @DJPaul
12 years ago

  • Milestone 2.12.2

Not going to have time to investigate this for 2.1.

#3 @DJPaul
12 years ago

  • Milestone 2.2Future Release

#4 @imath
11 years ago

  • Owner set to imath
  • Status newassigned

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 11 years ago by imath (previous) (diff)
Note: See TracTickets for help on using tickets.