Changeset 7279 for trunk/src/includes/topics/template.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/topics/template.php (modified) (1 diff)

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