Skip to:
Content

BuddyPress.org

Opened 12 years ago

Closed 10 years ago

#6110 closed enhancement (fixed)

Provide user ID for bp_notifications_get_unread_notification_count filter

Reported by: tw2113 Owned by: tw2113
Priority: normal Milestone: 2.7
Component: Toolbar & Notifications Version:
Severity: normal Keywords: good-first-bug
Cc:

Description

Right now, we simply provide the count for the bp_notifications_get_unread_notification_count filter, but that's not enough to conditionally filter the count. It'd be best to also provide the ID of the user whose count is being filtered.

Attachments (1)

6110-userid-param.diff​ (892 bytes ) - added by tw2113 10 years ago.

Download all attachments as: .zip

Change History (8)

#1 @tw2113
12 years ago

My mistake, bp_notifications_get_unread_notification_count is the function name, the filter needing the user ID is bp_notifications_get_total_notification_count

#2 @DJPaul
12 years ago

  • Component Core → Notifications
  • Keywords good-first-bug added
  • Milestone Awaiting Review → Future Release

@tw2113
10 years ago

#3 @tw2113
10 years ago

  • Owner set to tw2113
  • Status new → assigned

Easy fix. Assigning to self, pending approval.

#4 @tw2113
10 years ago

  • Milestone Future Release → 2.7

#5 @tw2113
10 years ago

This is an easy one and shouldn't break any compatibility, but still want at least one a-okay.

#6 @DJPaul
10 years ago

Yep, as it's harmless.

#7 @tw2113
10 years ago

  • Resolution → fixed
  • Status assigned → closed

In 11167:

Pass user ID value to bp_notifications_get_unread_notification_count filter for better conditional checking.

Fixes #6110.

Note: See TracTickets for help on using tickets.