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

Timestamp:

09/08/2014 03:05:19 PM (12 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)