Changeset 7279 for trunk/src/includes/common/functions.php

Timestamp:

07/05/2024 05:54:33 PM (12 months ago)

Author:

johnjamesjacoby

Message:

Common: prefer bbp_get_topic_permalink() over get_permalink()

This change ensures that features like Subscriptions & Pagination (inside of topics list loops) have: an opportunity to use the anticipated function path, and access to the right filters when including links back to them.

This fixes a potential bug where filtering a topic permalink would not happen inside of emails or pagination in the loop-topic template part.

See #3603.

File:

• trunk/src/includes/common/functions.php (modified) (2 diffs)

trunk/src/includes/common/functions.php

@@ -1265 +1265 @@
     /** Topic *****************************************************************/
 
-    // Bail if topic is not public (includes closed)
+    // Bail if topic is not public (both open/closed are public)
     if ( ! bbp_is_topic_public( $topic_id ) ) {
         return false;
@@ -1311 +1311 @@
     $topic_author_name = wp_specialchars_decode( strip_tags( $topic_author_name ), ENT_QUOTES );
     $topic_content     = wp_specialchars_decode( strip_tags( bbp_get_topic_content( $topic_id ) ), ENT_QUOTES );
-    $topic_url         = get_permalink( $topic_id );
+    $topic_url         = bbp_get_topic_permalink( $topic_id );
 
     // For plugins to filter messages per reply/topic/user