Skip to:

Opened 17 months ago

Closed 8 months ago

#3249 closed enhancement (fixed)

User IDs filter to be notified should pass the reply ID ad the topic ID

Reported by: satollo Owned by: johnjamesjacoby
Milestone: 2.6 Priority: normal
Severity: normal Version:
Component: API - Subscriptions Keywords: needs-patch


It would be very useful if the filter bbp_topic_subscription_user_ids (on bbPress 2.5 function bbp_notify_topic_subscribers on common/functions.php) could pass even the reply, topic (and maybe forum) IDs.

It'll let the filters to decide if someone should not be notified for a specif topic (for example I use it to not send to the same user more than one notification a day for a topic IF she does not visit the topic after a notification).

$user_ids = apply_filters( 'bbp_topic_subscription_user_ids', $user_ids, $reply_id, $topic_id );

Thank you, Stefano.

Change History (2)

#1 @johnjamesjacoby
10 months ago

  • Component changed from General to API - Subscriptions
  • Keywords needs-patch added
  • Milestone changed from Awaiting Review to 2.6
  • Owner set to johnjamesjacoby
  • Status changed from new to assigned

#2 @johnjamesjacoby
8 months ago

  • Resolution set to fixed
  • Status changed from assigned to closed

In 6916:

Subscriptions: pass additional IDs into user ID filters.

This commit adds forum, topic, and reply IDs to the filters commonly used to filter the user IDs that emails are being sent to, providing a quick opportunity to peek at them easily and make decisions based on what they are at the time.

Fixes #3249. Props satollo.

Note: See TracTickets for help on using tickets.