Changeset 3946 for branches/plugin/bbp-admin/bbp-admin.php

Timestamp:

06/12/2012 05:36:00 AM (14 years ago)

Author:

johnjamesjacoby

Message:

Capabilities & Settings:

• Rename bbp_admin_show_ui() to bbp_current_user_can_see().
• Add detailed support for hiding admin settings sections.
• Remove BBP_Admin UI override flags in lieu of new method.
• See #1846.

File:

• branches/plugin/bbp-admin/bbp-admin.php (modified) (11 diffs)

branches/plugin/bbp-admin/bbp-admin.php

@@ -45 +45 @@
     public $styles_url = '';
 
-    /** Tools *****************************************************************/
-
-    /**
-     * @var bool Enable screens in Tools area
-     */
-    public $enable_tools = false;
-
-    /** Settings **************************************************************/
-
-    /**
-     * @var bool Enable screens in Settings area
-     */
-    public $enable_settings = false;
-
     /** Capability ************************************************************/
 
@@ -182 +168 @@
     public function admin_menus() {
 
-        // Are tools enabled?
-        if ( is_super_admin() || ! empty( $this->enable_tools ) ) {
-
-            $hooks = array();
-
-            // These are later removed in admin_head
-            $hooks[] = add_management_page(
-                __( 'Repair Forums', 'bbpress' ),
-                __( 'Forum Repair',  'bbpress' ),
-                $this->minimum_capability,
-                'bbp-repair',
-                'bbp_admin_repair'
-            );
-            $hooks[] = add_management_page(
-                __( 'Import Forums', 'bbpress' ),
-                __( 'Forum Import',  'bbpress' ),
-                $this->minimum_capability,
-                'bbp-converter',
-                'bbp_converter_settings'
-            );
-            $hooks[] = add_management_page(
-                __( 'Reset Forums', 'bbpress' ),
-                __( 'Forum Reset',  'bbpress' ),
-                $this->minimum_capability,
-                'bbp-reset',
-                'bbp_admin_reset'
-            );
+        $hooks = array();
+
+        // These are later removed in admin_head
+        if ( bbp_current_user_can_see( 'bbp_tools_page' ) ) {
+            if ( bbp_current_user_can_see( 'bbp_tools_repair_page' ) ) {
+                $hooks[] = add_management_page(
+                    __( 'Repair Forums', 'bbpress' ),
+                    __( 'Forum Repair',  'bbpress' ),
+                    $this->minimum_capability,
+                    'bbp-repair',
+                    'bbp_admin_repair'
+                );
+            }
+
+            if ( bbp_current_user_can_see( 'bbp_tools_import_page' ) ) {
+                $hooks[] = add_management_page(
+                    __( 'Import Forums', 'bbpress' ),
+                    __( 'Forum Import',  'bbpress' ),
+                    $this->minimum_capability,
+                    'bbp-converter',
+                    'bbp_converter_settings'
+                );
+            }
+
+            if ( bbp_current_user_can_see( 'bbp_tools_reset_page' ) ) {
+                $hooks[] = add_management_page(
+                    __( 'Reset Forums', 'bbpress' ),
+                    __( 'Forum Reset',  'bbpress' ),
+                    $this->minimum_capability,
+                    'bbp-reset',
+                    'bbp_admin_reset'
+                );
+            }
 
             // Fudge the highlighted subnav item when on a bbPress admin page
@@ -226 +218 @@
 
         // Are settings enabled?
-        if ( is_super_admin() || ! empty( $this->enable_settings ) ) {
+        if ( bbp_current_user_can_see( 'bbp_settings_page' ) ) {
             add_options_page(
                 __( 'Forums',  'bbpress' ),
@@ -281 +273 @@
 
         $section = 'bbp_settings_main';
-        if ( bbp_admin_show_ui( $section ) ) {
+        if ( bbp_current_user_can_see( $section ) ) {
 
             // Add the main section
@@ -328 +320 @@
 
         $section = 'bbp_settings_theme_compat';
-        if ( bbp_admin_show_ui( $section ) ) {
+        if ( bbp_current_user_can_see( $section ) ) {
 
             // Add the per page section
@@ -341 +333 @@
 
         $section = 'bbp_settings_per_page';
-        if ( bbp_admin_show_ui( $section ) ) {
+        if ( bbp_current_user_can_see( $section ) ) {
 
             // Add the per page section
@@ -358 +350 @@
 
         $section = 'bbp_settings_per_page_rss';
-        if ( bbp_admin_show_ui( $section ) ) {
+        if ( bbp_current_user_can_see( $section ) ) {
 
             // Add the per page section
@@ -375 +367 @@
 
         $section = 'bbp_settings_root_slugs';
-        if ( bbp_admin_show_ui( $section ) ) {
+        if ( bbp_current_user_can_see( $section ) ) {
 
             // Add the per page section
@@ -397 +389 @@
 
         $section = 'bbp_settings_single_slugs';
-        if ( bbp_admin_show_ui( $section ) ) {
+        if ( bbp_current_user_can_see( $section ) ) {
 
             // Add the per page section
@@ -436 +428 @@
 
         $section = 'bbp_settings_buddypress';
-        if ( bbp_admin_show_ui( $section ) ) {
+        if ( bbp_current_user_can_see( $section ) ) {
 
             // Add the per page section
@@ -453 +445 @@
 
         $section = 'bbp_settings_akismet';
-        if ( bbp_admin_show_ui( $section ) ) {
+        if ( bbp_current_user_can_see( $section ) ) {
 
             // Add the per page section