Changeset 9997 for trunk/src/bp-groups/classes/class-bp-group-extension.php

Timestamp:

07/05/2015 05:36:58 PM (10 years ago)

Author:

dcavins

Message:

Refactor bp_core_new_subnav_item().

Use new functions bp_core_create_subnav_link()
and bp_core_register_subnav_screen_function()
within bp_core_new_subnav_item().
Switch group extension usage to take advantage of new granularity.

See #6503.

File:

• trunk/src/bp-groups/classes/class-bp-group-extension.php (modified) (3 diffs)

trunk/src/bp-groups/classes/class-bp-group-extension.php

@@ -735 +735 @@
         }
 
+        // If the user can see the nav item, we create it.
         $user_can_see_nav_item = $this->user_can_see_nav_item();
 
@@ -740 +741 @@
             $group_permalink = bp_get_group_permalink( groups_get_current_group() );
 
-            bp_core_new_subnav_item( array(
+            bp_core_create_subnav_link( array(
                 'name'            => ! $this->nav_item_name ? $this->name : $this->nav_item_name,
                 'slug'            => $this->slug,
@@ -749 +750 @@
                 'screen_function' => array( &$this, '_display_hook' ),
                 'user_has_access' => $user_can_see_nav_item,
+                'no_access_url'   => $group_permalink,
+            ) );
+        }
+
+        // If the user can visit the screen, we register it.
+        $user_can_visit = $this->user_can_visit();
+
+        if ( $user_can_visit ) {
+            $group_permalink = bp_get_group_permalink( groups_get_current_group() );
+
+            bp_core_register_subnav_screen_function( array(
+                'slug'            => $this->slug,
+                'parent_slug'     => bp_get_current_group_slug(),
+                'screen_function' => array( &$this, '_display_hook' ),
+                'user_has_access' => $user_can_visit,
                 'no_access_url'   => $group_permalink,
             ) );