Changeset 5467 for trunk/src/includes/replies/template.php

Timestamp:

09/08/2014 03:05:19 PM (10 years ago)

Author:

johnjamesjacoby

Message:

Only add the s argument to queries if rs or ts are set in their respective topics & replies queries.

See #2607, ​#WP11330, ​[WP28623] ​[WP28612], ​[WP28804].

File:

• trunk/src/includes/replies/template.php (modified) (2 diffs)

trunk/src/includes/replies/template.php

@@ -125 +125 @@
 
     // Other defaults
-    $default_reply_search   = !empty( $_REQUEST['rs'] ) ? $_REQUEST['rs']    : false;
     $default_post_parent    = ( bbp_is_single_topic() ) ? bbp_get_topic_id() : 'any';
     $default_post_type      = ( bbp_is_single_topic() && bbp_show_lead_topic() ) ? bbp_get_reply_post_type() : array( bbp_get_topic_post_type(), bbp_get_reply_post_type() );
@@ -140 +139 @@
         'hierarchical'        => $default_thread_replies,    // Hierarchical replies
         'ignore_sticky_posts' => true,                       // Stickies not supported
-        's'                   => $default_reply_search,      // Maybe search
     );
+
+    // Only add 's' arg if searching for replies
+    // See https://bbpress.trac.wordpress.org/ticket/2607
+    if ( ! empty( $_REQUEST['rs'] ) ) {
+        $default['s'] = $_REQUEST['rs'];
+    }
 
     // What are the default allowed statuses (based on user caps)