Changeset 6630 for trunk/src/templates/default/bbpress/form-reply-search.php

Timestamp:

07/19/2017 05:23:30 PM (9 years ago)

Author:

johnjamesjacoby

Message:

Search: move bbp_allow_search() checks into form template parts.

This allows the inclusion of those forms without needing to wrap each inclusion statement in its own feature check, making search a more atomic and isolated feature within bbPress.

It also should conclude the development of the user content search functionality.

Because this involves some template-part changes to the "Default" template pack, users who have opted to override these parts in their own themes will not receive these updates (nor should they.) This is a known consequence of including template parts in themes, so while still not an ideal system, will not result in any breakage for existing sites.

File:

• trunk/src/templates/default/bbpress/form-reply-search.php (modified) (2 diffs)

trunk/src/templates/default/bbpress/form-reply-search.php

@@ -2 +2 @@
 
 /**
- * Search 
+ * Search
  *
  * @package bbPress
@@ -11 +11 @@
 defined( 'ABSPATH' ) || exit;
 
-?>
+if ( bbp_allow_search() ) : ?>
 
-<form role="search" method="get" id="bbp-reply-search-form">
-    <div>
-        <label class="screen-reader-text hidden" for="ts"><?php esc_html_e( 'Search replies:', 'bbpress' ); ?></label>
-        <input type="text" value="<?php bbp_search_terms(); ?>" name="ts" id="rs" />
-        <input class="button" type="submit" id="bbp_search_submit" value="<?php esc_attr_e( 'Search', 'bbpress' ); ?>" />
+    <div class="bbp-search-form">
+        <form role="search" method="get" id="bbp-reply-search-form">
+            <div>
+                <label class="screen-reader-text hidden" for="rs"><?php esc_html_e( 'Search replies:', 'bbpress' ); ?></label>
+                <input type="text" value="<?php bbp_search_terms(); ?>" name="rs" id="rs" />
+                <input class="button" type="submit" id="bbp_search_submit" value="<?php esc_attr_e( 'Search', 'bbpress' ); ?>" />
+            </div>
+        </form>
     </div>
-</form>
+
+<?php endif;