Changeset 7293

Timestamp:

04/16/2025 06:05:04 PM (2 months ago)

Author:

johnjamesjacoby

Message:

Replies: Fix a bug that causes replies to not appear in some topics.

In WordPress 6.8, WP_Query now enforces a consistent alphabetical order for array values. This change breaks the _bbp_has_replies_where() function, which previously relied on a strict comparison against post_type.

See: ​https://core.trac.wordpress.org/ticket/59516

This commit replaces the strict comparison with an array_diff() check to account for the new behavior. It also optimizes the surrounding logic to minimize repeated calls to $query->get().

Props vortfu, arl1nd.

Committed to trunk for 2.7.

Fixes #3631.

File:

• trunk/src/includes/replies/functions.php (modified) (3 diffs)

trunk/src/includes/replies/functions.php

@@ -2087 +2087 @@
     }
 
-    // Bail if no post_parent to replace
-    if ( ! is_numeric( $query->get( 'post_parent' ) ) ) {
-        return $where;
-    }
-
-    // Bail if not a topic and reply query
-    if ( array( bbp_get_topic_post_type(), bbp_get_reply_post_type() ) !== $query->get( 'post_type' ) ) {
-        return $where;
-    }
-
     // Bail if including specific post ID's
     if ( $query->get( 'post__in' ) ) {
@@ -2102 +2092 @@
     }
 
+    // Get post_parent from query (used to get the topic ID below)
+    $post_parent = $query->get( 'post_parent' );
+
+    // Bail if post_parent is default '' (uses is_numeric() because WordPress does internally)
+    if ( ! is_numeric( $post_parent ) ) {
+        return $where;
+    }
+
+    // Get post_type from query, define array to diff against
+    $queried_types = (array) $query->get( 'post_type' );
+    $post_types    = array( bbp_get_topic_post_type(), bbp_get_reply_post_type() );
+
+    // Bail if query is not already for both topic and reply post types
+    if ( array_diff( $post_types, $queried_types ) ) {
+        return $where;
+    }
+
     /** Proceed ***************************************************************/
 
@@ -2108 +2115 @@
 
     // Get the topic ID from the post_parent, set in bbp_has_replies()
-    $topic_id   = bbp_get_topic_id( $query->get( 'post_parent' ) );
+    $topic_id   = bbp_get_topic_id( $post_parent );
 
     // The texts to search for