Skip to:
Content

bbPress.org

Opened 11 years ago

Closed 10 years ago

#2651 closed enhancement (fixed)

Add topic_id parameter to bbp_get_topic_subscribers filter

Reported by: djpaul's profile DJPaul Owned by: johnjamesjacoby's profile johnjamesjacoby
Milestone: 2.6 Priority: normal
Severity: normal Version: 2.5.4
Component: API - Subscriptions Keywords: has-patch
Cc:

Description

We're needing to use the bbp_get_topic_subscribers filter to change the results (we need to query a different database table), but the $topic_id isn't passed to the filter, making it pretty hard to reliably figure out what topic you're trying to check subscriptions for.

Attached patch just adds the parameter to the filter.

Attachments (1)

2651.01.patch (472 bytes) - added by DJPaul 11 years ago.

Download all attachments as: .zip

Change History (3)

@DJPaul
11 years ago

#1 @johnjamesjacoby
10 years ago

  • Milestone changed from Awaiting Review to 2.6

A few other filters could use similar treatment. Moving to 2.6.

#2 @johnjamesjacoby
10 years ago

  • Owner set to johnjamesjacoby
  • Resolution set to fixed
  • Status changed from new to closed

In 5486:

Pass $topic_id and $forum_id to relevant favorites and subscriptions filters. Hat tip DJPaul. Fixes #2651.

Note: See TracTickets for help on using tickets.