Skip to:
Content

BuddyPress.org

Opened 13 years ago

Closed 12 years ago

#5315 closed enhancement (fixed)

Add ability to query both read and unread notifications

Reported by: rodrigorznd Owned by: boonebgorges
Priority: normal Milestone: 2.0
Component: Toolbar & Notifications Version: 1.9
Severity: normal Keywords: has-patch 2nd-opinion needs-unit-tests needs-testing
Cc:

Description (last modified by johnjamesjacoby)

Attached patch allows loading read and unread notifications in the same query.

Attachments (1)

buddypress.patch​ (801 bytes ) - added by rodrigorznd 13 years ago.

Download all attachments as: .zip

Change History (6)

@rodrigorznd
13 years ago

#1 @johnjamesjacoby
13 years ago

  • Description modified (diff)
  • Milestone Awaiting Review → 2.0
  • Summary Loading read and unread notifications → Add ability to query both read and unread notifications

#2 @DJPaul
12 years ago

Needs testing, and unit tests would be extra nice to ensure we don't break this in future, but I don't see any harm in adding this patch.

#3 @DJPaul
12 years ago

  • Keywords 2nd-opinion needs-unit-tests needs-testing added

#4 @boonebgorges
12 years ago

+1 DJPaul. The unit tests will take someone about ten minutes to write; just copy one of the existing tests and make some minor mods.

#5 @boonebgorges
12 years ago

  • Owner set to boonebgorges
  • Resolution → fixed
  • Status new → closed

In 8167:

Allow all notifications to be fetched by passing 'both' to is_new param of BP_Notifications_Notification::get()

Fixes #5315

Props rodrigorznd

Note: See TracTickets for help on using tickets.