Changeset 7279

Timestamp:

07/05/2024 05:54:33 PM (9 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.

Location:

trunk/src/includes

Files:

• common/functions.php (modified) (2 diffs)
• topics/template.php (modified) (1 diff)

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

trunk/src/includes/topics/template.php

@@ -809 +809 @@
         $has_slug = bbp_get_topic( $r['topic_id'] )->post_name;
 
+        // Get the topic permalink
+        $topic_permalink = bbp_get_topic_permalink( $r['topic_id'] );
+
         // If pretty permalinks are enabled, make our pagination pretty
         $base = ! empty( $has_slug ) && bbp_use_pretty_urls() && bbp_is_topic_public( $r['topic_id'] )
-            ? trailingslashit( get_permalink( $r['topic_id'] ) ) . user_trailingslashit( bbp_get_paged_slug() . '/%#%/' )
-            : add_query_arg( 'paged', '%#%', get_permalink( $r['topic_id'] ) );
+            ? trailingslashit( $topic_permalink ) . user_trailingslashit( bbp_get_paged_slug() . '/%#%/' )
+            : add_query_arg( 'paged', '%#%', $topic_permalink );
 
         // Get total and add 1 if topic is included in the reply loop