Changeset 4971 for trunk/includes/admin/settings.php

Timestamp:

05/28/2013 11:12:36 PM (13 years ago)

Author:

johnjamesjacoby

Message:

Allow the search feature to be toggled on and off. Introduces new setting and supporting function to get and override the option easily in third party plugins. Fixes #2342.

File:

• trunk/includes/admin/settings.php (modified) (4 diffs)

trunk/includes/admin/settings.php

@@ -85 +85 @@
                 /** User Section ******************************************************/
 
-                'bbp_settings_user' => array(
+                'bbp_settings_users' => array(
 
                         // Edit lock setting
@@ -158 +158 @@
                                 'title'             => __( 'Topic tags', 'bbpress' ),
                                 'callback'          => 'bbp_admin_setting_callback_topic_tags',
+                                'sanitize_callback' => 'intval',
+                                'args'              => array()
+                        ),
+
+                        // Allow topic tags
+                        '_bbp_allow_search' => array(
+                                'title'             => __( 'Search', 'bbpress' ),
+                                'callback'          => 'bbp_admin_setting_callback_search',
                                 'sanitize_callback' => 'intval',
                                 'args'              => array()
@@ -178 +186 @@
                         ),
 
-                        // Allow threadde replies
-                        '_bbp_allow_threaded_replies' => array(
-                                'sanitize_callback' => 'intval',
-                                'args'              => array()
-                        ),
-
                         // Set reply threading level
                         '_bbp_thread_replies_depth' => array(
                                 'title'             => __( 'Reply Threading', 'bbpress' ),
                                 'callback'          => 'bbp_admin_setting_callback_thread_replies_depth',
+                                'sanitize_callback' => 'intval',
+                                'args'              => array()
+                        ),
+
+                        // Allow threadde replies
+                        '_bbp_allow_threaded_replies' => array(
                                 'sanitize_callback' => 'intval',
                                 'args'              => array()
@@ -588 +596 @@
         <input id="_bbp_allow_topic_tags" name="_bbp_allow_topic_tags" type="checkbox" id="_bbp_allow_topic_tags" value="1" <?php checked( bbp_allow_topic_tags( true ) ); bbp_maybe_admin_setting_disabled( '_bbp_allow_topic_tags' ); ?> />
         <label for="_bbp_allow_topic_tags"><?php esc_html_e( 'Allow topics to have tags', 'bbpress' ); ?></label>
+
+<?php
+}
+
+/**
+ * Allow forum wide search
+ *
+ * @since bbPress (r4970)
+ *
+ * @uses checked() To display the checked attribute
+ */
+function bbp_admin_setting_callback_search() {
+?>
+
+        <input id="_bbp_allow_search" name="_bbp_allow_search" type="checkbox" id="_bbp_allow_search" value="1" <?php checked( bbp_allow_search( true ) ); bbp_maybe_admin_setting_disabled( '_bbp_allow_search' ); ?> />
+        <label for="_bbp_allow_search"><?php esc_html_e( 'Allow forum wide search', 'bbpress' ); ?></label>
 
 <?php