Skip to:
Content

Opened 5 years ago

Closed 4 years ago

#2651 closed enhancement (fixed)

Add topic_id parameter to bbp_get_topic_subscribers filter

Reported by: DJPaul Owned by: 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 5 years ago.

Download all attachments as: .zip

Change History (3)

@DJPaul
5 years ago

#1 @johnjamesjacoby
4 years ago

  • Milestone changed from Awaiting Review to 2.6

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

#2 @johnjamesjacoby
4 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.