Topic and reply URLs in notification messages

Hi GreyNomad,

Thanks for your contribution. I’m about to release version 1.6 with the following filters that you can hook into with your changes. That way you won’t lose them next time there’s a plugin update:

312: $url     = apply_filters( 'bbpnns_topic_url', bbp_get_topic_permalink($post_id) );

320: $url     = apply_filters( 'bbpnns_reply_url', bbp_get_reply_permalink($post_id) );

331: $topic_reply = apply_filters( 'bbpnns_topic_reply', bbp_get_reply_url($post_id) );

You can hook into them using add_filter() in your functions.php file or customization plugin.

Thanks… I’ll have to figure out how to do that. I don’t know much about WP plugin internals. I’m balls to the wall getting the event site up right now, I’ll look into that when the dust settles and I get my brain back.

~~ Grey

I missed adding a couple of variables in the previous release. Please update to 1.6.1 and follow the steps below.

In your theme’s functions.php file, add the following:

function my_topic_url( $url, $post_id, $title )
{
    return bbp_get_topics_url() . $title;
}
add_filter( 'bbpnns_topic_url', 'my_topic_url', 10, 3);

function my_reply_url( $url, $post_id, $title )
{
    return bbp_get_reply_url($post_id);
}
add_filter( 'bbpnns_reply_url', 'my_reply_url', 10, 3);

Thanks, you rock!

In case anyone is going to use this… I found out that using the topic title can result in broken links because of spaces and other link-breaking characters. It needs to use the topic slug instead. The revised code for returning the topic URL is:

return bbp_get_topics_url() . sanitize_title($title) . ‘/’;

The complete code:

function my_topic_url( $url, $post_id, $title )
{
    return bbp_get_topics_url() . sanitize_title($title) . '/';
}
add_filter( 'bbpnns_topic_url', 'my_topic_url', 10, 3);

function my_reply_url( $url, $post_id, $title )
{
    return bbp_get_reply_url($post_id);
}
add_filter( 'bbpnns_reply_url', 'my_reply_url', 10, 3);