Skip to:
Content

BuddyPress.org

Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#7141 closed defect (bug) (fixed)

Querying notifications by string format fails

Reported by: r-a-y's profile r-a-y Owned by: r-a-y's profile r-a-y
Milestone: 2.6.1 Priority: normal
Severity: normal Version: 2.6.0
Component: Toolbar & Notifications Keywords: has-patch
Cc:

Description

See:
https://buddypress.org/support/topic/buddypress-2-6-0-espejo-is-now-available/#post-255213

Problem was introduced by me! Forgot to pass the $format variable as 'string'.

Attachments (1)

7141.01.patch (864 bytes) - added by r-a-y 9 years ago.

Download all attachments as: .zip

Change History (4)

@r-a-y
9 years ago

This ticket was mentioned in Slack in #buddypress by dcavins. View the logs.


9 years ago

#2 @r-a-y
9 years ago

  • Owner set to r-a-y
  • Resolution set to fixed
  • Status changed from new to closed

In 10910:

Notifications: Fix issue with querying notifications by string format.

If someone is using:
bp_notifications_get_notifications_for_user( $user_id, 'string' )

This functionality broke in #7066.

Commit fixes this issue and includes unit tests.

Fixes #7141 (2.6-branch).

#3 @r-a-y
9 years ago

In 10911:

Notifications: Fix issue with querying notifications by string format.

If someone is using:
bp_notifications_get_notifications_for_user( $user_id, 'string' )

This functionality broke in #7066.

Commit fixes this issue and includes unit tests.

Fixes #7141 (trunk).

Note: See TracTickets for help on using tickets.