Ticket #6534: 6534.05.patch

src/bp-core/bp-core-buddybar.php

@@ -13 +13 @@
 defined( 'ABSPATH' ) || exit;
 
 /**
- * Add an item to the main BuddyPress navigation array.
+ * Add an item to the primary navigation of the specified component.
  *
- * @since 1.1.0
+ * @since 2.6.0
  *
  * @param array|string $args {
  *     Array describing the new nav item.
@@ -32 +32 @@
  *     @type bool|string $default_subnav_slug     Optional. The slug of the default subnav item to select when the nav
  *                                                item is clicked.
  * }
+ * @param string       $component                 The component the navigation is attached to. Defaults to 'members'.
  * @return bool|null Returns false on failure.
  */
-function bp_core_new_nav_item( $args = '' ) {
+
+function bp_core_add_nav_item( $args, $component = 'members' ) {
+        if ( ! bp_is_active( $component ) ) {
+                return;
+        }
 
         $defaults = array(
                 'name'                    => false, // Display name for the nav item.
@@ -49 +54 @@
 
         $r = wp_parse_args( $args, $defaults );
 
-        // First, add the nav item link to the bp_nav array.
-        $created = bp_core_create_nav_link( $r );
+        // First, add the nav item link to the primary nav for the component.
+        $nav_item = bp_core_create_nav_link( $r );
+
+        /**
+         * Fires after a link is added to the main BuddyPress navigation array.
+         *
+         * @since 2.4.0
+         *
+         * @param array $r        Parsed arguments for the nav item.
+         * @param array $args     Originally passed in arguments for the nav item.
+         * @param array $defaults Default arguments for a nav item.
+         */
+        do_action( 'bp_core_create_nav_link', $r, $args, $defaults );
 
         // To mimic the existing behavior, if bp_core_create_nav_link()
         // returns false, we make an early exit and don't attempt to register
         // the screen function.
-        if ( false === $created ) {
+        if ( false === $nav_item ) {
                 return false;
         }
 
         // Then, hook the screen function for the added nav item.
-        $hooked = bp_core_register_nav_screen_function( $r );
+        $hooked = bp_core_register_nav_screen_function( $nav_item );
         if ( false === $hooked ){
                 return false;
         }
 
+        // Add the item to the nav
+        buddypress()->{$component}->nav->add_nav( $nav_item );
+
         /**
          * Fires after adding an item to the main BuddyPress navigation array.
          * Note that, when possible, the more specific action hooks
@@ -132 +151 @@
                 $r['item_css_id'] = $r['slug'];
         }
 
-        $bp->bp_nav[$r['slug']] = array(
+        return array(
                 'name'                    => $r['name'],
                 'slug'                    => $r['slug'],
                 'link'                    => trailingslashit( bp_loggedin_user_domain() . $r['slug'] ),
@@ -142 +161 @@
                 'screen_function'         => &$r['screen_function'],
                 'default_subnav_slug'     => $r['default_subnav_slug']
         );
-
-        /**
-         * Fires after a link is added to the main BuddyPress navigation array.
-         *
-         * @since 2.4.0
-         *
-         * @param array $r        Parsed arguments for the nav item.
-         * @param array $args     Originally passed in arguments for the nav item.
-         * @param array $defaults Default arguments for a nav item.
-         */
-        do_action( 'bp_core_create_nav_link', $r, $args, $defaults );
 }
 
 /**
@@ -290 +298 @@
 
         $r = wp_parse_args( $args, $defaults );
 
-        if ( $function = $bp->bp_nav[$r['parent_slug']]['screen_function'] ) {
+        // This is specific to Members, it's not available in Groups.
+        $parent_nav = $bp->members->nav->get_primary( array( 'slug' => $r['parent_slug'] ), false );
+
+        if ( ! $parent_nav ) {
+                return ;
+        }
+
+        $parent_nav = reset( $parent_nav );
+
+        if ( ! empty( $parent_nav->screen_function ) ) {
                 // Remove our screen hook if screen function is callable.
-                if ( is_callable( $function ) ) {
-                        remove_action( 'bp_screens', $function, 3 );
+                if ( is_callable( $parent_nav->screen_function ) ) {
+                        remove_action( 'bp_screens', $parent_nav->screen_function, 3 );
                 }
         }
 
-        $bp->bp_nav[$r['parent_slug']]['screen_function'] = &$r['screen_function'];
+        // Edit the screen function for the parent nav
+        $bp->members->nav->edit_nav( array(
+                'screen_function'     => &$r['screen_function'],
+                'default_subnav_slug' => $r['subnav_slug'],
+        ), $parent_nav->slug );
 
-        if ( bp_is_current_component( $r['parent_slug'] ) ) {
+        if ( bp_is_current_component( $parent_nav->slug ) ) {
 
                 // The only way to tell whether to set the subnav is to peek at the unfiltered_uri
                 // Find the component.
-                $component_uri_key = array_search( $r['parent_slug'], $bp->unfiltered_uri );
+                $component_uri_key = array_search( $parent_nav->slug, $bp->unfiltered_uri );
 
                 if ( false !== $component_uri_key ) {
                         if ( ! empty( $bp->unfiltered_uri[$component_uri_key + 1] ) ) {
@@ -339 +360 @@
 }
 
 /**
- * Sort the navigation menu items.
- *
- * The sorting is split into a separate function because it can only happen
- * after all plugins have had a chance to register their navigation items.
- *
- * @since 1.0.0
- *
- * @return bool|null Returns false on failure.
- */
-function bp_core_sort_nav_items() {
-        $bp = buddypress();
-
-        if ( empty( $bp->bp_nav ) || ! is_array( $bp->bp_nav ) ) {
-                return false;
-        }
-
-        $temp = array();
-
-        foreach ( (array) $bp->bp_nav as $slug => $nav_item ) {
-                if ( empty( $temp[$nav_item['position']] ) ) {
-                        $temp[$nav_item['position']] = $nav_item;
-                } else {
-                        // Increase numbers here to fit new items in.
-                        do {
-                                $nav_item['position']++;
-                        } while ( ! empty( $temp[$nav_item['position']] ) );
-
-                        $temp[$nav_item['position']] = $nav_item;
-                }
-        }
-
-        ksort( $temp );
-        $bp->bp_nav = &$temp;
-}
-add_action( 'wp_head',    'bp_core_sort_nav_items' );
-add_action( 'admin_head', 'bp_core_sort_nav_items' );
-
-/**
- * Add a subnav item to the BuddyPress navigation.
+ * Add an item to secondary navigation of the specified component.
  *
- * @since 1.1.0
+ * @since 2.6.0
  *
  * @param array|string $args {
  *     Array describing the new subnav item.
@@ -403 +386 @@
  *     @type bool        $show_in_admin_bar Optional. Whether the nav item should be added into the group's "Edit"
  *                                          Admin Bar menu for group admins. Default: false.
  * }
+ * @param  string       $component          The component the navigation is attached to. Defaults to 'members'.
  * @return bool|null Returns false on failure.
  */
-function bp_core_new_subnav_item( $args = '' ) {
+function bp_core_add_subnav_item( $args, $component = 'members' ) {
+        if ( ! bp_is_active( $component ) ) {
+                return;
+        }
 
         // First, add the subnav item link to the bp_options_nav array.
-        $created = bp_core_create_subnav_link( $args );
+        $subnav_item = bp_core_create_subnav_link( $args, $component );
 
         // To mimic the existing behavior, if bp_core_create_subnav_link()
         // returns false, we make an early exit and don't attempt to register
         // the screen function.
-        if ( false === $created ) {
+        if ( false === $subnav_item ) {
                 return false;
         }
 
         // Then, hook the screen function for the added subnav item.
-        $hooked = bp_core_register_subnav_screen_function( $args );
+        $hooked = bp_core_register_subnav_screen_function( $subnav_item, $component );
         if ( false === $hooked ) {
                 return false;
         }
@@ -428 +415 @@
  * Add a subnav link to the BuddyPress navigation.
  *
  * @since 2.4.0
+ * @since 2.6.0 Adds the $component parameter.
  *
  * @param array|string $args {
  *     Array describing the new subnav item.
@@ -455 +443 @@
  *                                          the group's "Edit" Admin Bar menu for group admins.
  *                                          Default: false.
  * }
+ * @param  string       $component          The component the navigation is attached to. Defaults to 'members'.
  * @return bool|null Returns false on failure.
  */
-function bp_core_create_subnav_link( $args = '' ) {
+function bp_core_create_subnav_link( $args = '', $component = 'members' ) {
         $bp = buddypress();
 
         $r = wp_parse_args( $args, array(
@@ -483 +472 @@
         if ( empty( $r['link'] ) ) {
                 $r['link'] = trailingslashit( $r['parent_url'] . $r['slug'] );
 
+                $parent_nav = $bp->{$component}->nav->get_primary( array( 'slug' => $r['parent_slug'] ), false );
+
                 // If this sub item is the default for its parent, skip the slug.
-                if ( ! empty( $bp->bp_nav[$r['parent_slug']]['default_subnav_slug'] ) && $r['slug'] == $bp->bp_nav[$r['parent_slug']]['default_subnav_slug'] ) {
+                if ( ! empty( $parent_nav->default_subnav_slug ) && $r['slug'] === $parent_nav->default_subnav_slug ) {
                         $r['link'] = trailingslashit( $r['parent_url'] );
                 }
         }
@@ -502 +493 @@
                 'name'              => $r['name'],
                 'link'              => $r['link'],
                 'slug'              => $r['slug'],
+                'parent_slug'       => $r['parent_slug'],
                 'css_id'            => $r['item_css_id'],
                 'position'          => $r['position'],
                 'user_has_access'   => $r['user_has_access'],
@@ -510 +502 @@
                 'show_in_admin_bar' => (bool) $r['show_in_admin_bar'],
         );
 
-        $bp->bp_options_nav[$r['parent_slug']][$r['slug']] = $subnav_item;
+        buddypress()->{$component}->nav->add_nav( $subnav_item );
+
+        return $subnav_item;
 }
 
 /**
  * Register a screen function, whether or not a related subnav link exists.
  *
  * @since 2.4.0
+ * @since 2.6.0 Adds the $component parameter.
  *
  * @param array|string $args {
  *     Array describing the new subnav item.
@@ -541 +536 @@
  *                                       the group's "Edit" Admin Bar menu for group admins.
  *                                       Default: false.
  * }
+ * @param  string       $component          The component the navigation is attached to. Defaults to 'members'.
  * @return bool|null Returns false on failure.
  */
-function bp_core_register_subnav_screen_function( $args = '' ) {
+function bp_core_register_subnav_screen_function( $args = '', $component = 'members' ) {
         $bp = buddypress();
 
         $r = wp_parse_args( $args, array(
@@ -568 +564 @@
          *       (b) there is no current_action (ie, this is the default subnav for the parent nav)
          *       and this subnav item is the default for the parent item (which we check by
          *       comparing this subnav item's screen function with the screen function of the
-         *       parent nav item in $bp->bp_nav). This condition only arises when viewing a
+         *       parent nav item in the component's primary nav). This condition only arises when viewing a
          *       user, since groups should always have an action set.
          */
 
@@ -577 +573 @@
                 return;
         }
 
+        $parent_nav = $bp->{$component}->nav->get_primary( array( 'slug' => $r['parent_slug'] ), false );
+
         // If we *do* meet condition (2), then the added subnav item is currently being requested.
-        if ( ( bp_current_action() && bp_is_current_action( $r['slug'] ) ) || ( bp_is_user() && ! bp_current_action() && ( $r['screen_function'] == $bp->bp_nav[$r['parent_slug']]['screen_function'] ) ) ) {
+        if ( ( bp_current_action() && bp_is_current_action( $r['slug'] ) ) || ( bp_is_user() && ! bp_current_action() && ! empty( $parent_nav->screen_function ) && $r['screen_function'] == $parent_nav->screen_function ) ) {
 
                 // If this is for site admins only and the user is not one, don't create the subnav item.
                 if ( ! empty( $r['site_admin_only'] ) && ! bp_current_user_can( 'bp_moderate' ) ) {
                         return false;
                 }
 
-                $hooked = bp_core_maybe_hook_new_subnav_screen_function( $r );
+                $hooked = bp_core_maybe_hook_new_subnav_screen_function( $r, $component );
 
                 // If redirect args have been returned, perform the redirect now.
                 if ( ! empty( $hooked['status'] ) && 'failure' === $hooked['status'] && isset( $hooked['redirect_args'] ) ) {
@@ -598 +596 @@
  * For a given subnav item, either hook the screen function or generate redirect arguments, as necessary.
  *
  * @since 2.1.0
+ * @since 2.6.0 Adds the $component parameter.
  *
- * @param array $subnav_item The subnav array added to bp_options_nav in `bp_core_new_subnav_item()`.
+ * @param array  $subnav_item The subnav array added to the secondary navigation of the component in `bp_core_add_subnav_item()`.
+ * @param string $component   The component the navigation is attached to. Defaults to 'members'.
  * @return array
  */
-function bp_core_maybe_hook_new_subnav_screen_function( $subnav_item ) {
+function bp_core_maybe_hook_new_subnav_screen_function( $subnav_item, $component = 'members' ) {
         $retval = array(
                 'status' => '',
         );
@@ -645 +645 @@
                         // redirect URL.
                         } elseif ( bp_is_user() ) {
 
+                                $parent_nav_default = $bp->{$component}->nav->get_primary( array( 'slug' => $bp->default_component ), false );
+
                                 // Redirect to the displayed user's default
                                 // component, as long as that component is
                                 // publicly accessible.
-                                if ( bp_is_my_profile() || ! empty( $bp->bp_nav[ $bp->default_component ]['show_for_displayed_user'] ) ) {
+                                if ( bp_is_my_profile() || ! empty( $parent_nav_default->show_for_displayed_user ) ) {
                                         $message     = __( 'You do not have access to this page.', 'buddypress' );
                                         $redirect_to = bp_displayed_user_domain();
 
@@ -691 +693 @@
 }
 
 /**
- * Sort all subnavigation arrays.
+ * Check whether a given nav item has subnav items.
  *
- * @since 1.1.0
+ * @since 1.5.0
+ * @since 2.6.0 Adds the $component parameter.
  *
- * @return bool|null Returns false on failure.
+ * @param string $nav_item  The slug of the top-level nav item whose subnav items you're checking.
+ *                          Default: the current component slug.
+ * @param string $component The component the navigation is attached to. Defaults to 'members'.
+ * @return bool $has_subnav True if the nav item is found and has subnav items; false otherwise.
  */
-function bp_core_sort_subnav_items() {
+function bp_nav_item_has_subnav( $nav_item = '', $component = 'members' ) {
         $bp = buddypress();
 
-        if ( empty( $bp->bp_options_nav ) || !is_array( $bp->bp_options_nav ) )
+        if ( ! isset( $bp->{$component}->nav ) ) {
                 return false;
+        }
 
-        foreach ( (array) $bp->bp_options_nav as $parent_slug => $subnav_items ) {
-                if ( !is_array( $subnav_items ) )
-                        continue;
-
-                foreach ( (array) $subnav_items as $subnav_item ) {
-                        if ( empty( $temp[$subnav_item['position']]) )
-                                $temp[$subnav_item['position']] = $subnav_item;
-                        else {
-                                // Increase numbers here to fit new items in.
-                                do {
-                                        $subnav_item['position']++;
-                                } while ( !empty( $temp[$subnav_item['position']] ) );
+        if ( ! $nav_item ) {
+                $nav_item = bp_current_component();
 
-                                $temp[$subnav_item['position']] = $subnav_item;
-                        }
+                if ( bp_is_group() ) {
+                        $nav_item = bp_current_item();
                 }
-                ksort( $temp );
-                $bp->bp_options_nav[$parent_slug] = &$temp;
-                unset( $temp );
         }
-}
-add_action( 'wp_head',    'bp_core_sort_subnav_items' );
-add_action( 'admin_head', 'bp_core_sort_subnav_items' );
-
-/**
- * Check whether a given nav item has subnav items.
- *
- * @since 1.5.0
- *
- * @param string $nav_item The slug of the top-level nav item whose subnav items you're checking.
- *                         Default: the current component slug.
- * @return bool $has_subnav True if the nav item is found and has subnav items; false otherwise.
- */
-function bp_nav_item_has_subnav( $nav_item = '' ) {
-        $bp = buddypress();
-
-        if ( !$nav_item )
-                $nav_item = bp_current_component();
 
-        $has_subnav = isset( $bp->bp_options_nav[$nav_item] ) && count( $bp->bp_options_nav[$nav_item] ) > 0;
+        $has_subnav = (bool) $bp->{$component}->nav->get_secondary( array( 'parent_slug' => $nav_item ), false );
 
         /**
          * Filters whether or not a given nav item has subnav items.
@@ -756 +732 @@
 }
 
 /**
- * Remove a nav item from the navigation array.
+ * Deletes an item from the primary navigation of the specified component.
  *
- * @since 1.0.0
+ * @since 2.6.0
  *
- * @param int $parent_id The slug of the parent navigation item.
- * @return bool Returns false on failure, ie if the nav item can't be found.
+ * @param string $slug      The slug of the primary navigation item.
+ * @param string $component The component the navigation is attached to. Defaults to 'members'.
+ * @return bool Returns false on failure, True on success.
  */
-function bp_core_remove_nav_item( $parent_id ) {
+function bp_core_delete_nav_item( $slug, $component = 'members' ) {
         $bp = buddypress();
 
-        // Unset subnav items for this nav item.
-        if ( isset( $bp->bp_options_nav[$parent_id] ) && is_array( $bp->bp_options_nav[$parent_id] ) ) {
-                foreach( (array) $bp->bp_options_nav[$parent_id] as $subnav_item ) {
-                        bp_core_remove_subnav_item( $parent_id, $subnav_item['slug'] );
-                }
+        if ( ! isset( $bp->{$component}->nav ) ) {
+                return false;
         }
 
-        if ( empty( $bp->bp_nav[ $parent_id ] ) )
+        $screen_functions = $bp->{$component}->nav->delete_nav( $slug );
+
+        if ( ! is_array( $screen_functions ) ) {
                 return false;
+        }
 
-        if ( $function = $bp->bp_nav[$parent_id]['screen_function'] ) {
+        foreach ( $screen_functions as $screen_function ) {
                 // Remove our screen hook if screen function is callable.
-                if ( is_callable( $function ) ) {
-                        remove_action( 'bp_screens', $function, 3 );
+                if ( is_callable( $screen_function ) ) {
+                        remove_action( 'bp_screens', $screen_function, 3 );
                 }
         }
 
-        unset( $bp->bp_nav[$parent_id] );
+        return true;
 }
 
 /**
- * Remove a subnav item from the navigation array.
+ * Deletes an item from the secondary navigation of the specified component.
  *
- * @since 1.0.0
+ * @since 2.6.0
  *
- * @param string $parent_id The slug of the parent navigation item.
- * @param string $slug      The slug of the subnav item to be removed.
+ * @param string $slug        The slug of the secondary item to be removed.
+ * @param string $parent_slug The slug of the primary navigation item.
+ * @param string $component   The component the navigation is attached to. Defaults to 'members'.
+ * @return bool Returns false on failure, True on success.
  */
-function bp_core_remove_subnav_item( $parent_id, $slug ) {
+function bp_core_delete_subnav_item( $slug, $parent_slug, $component = 'members' ) {
         $bp = buddypress();
 
-        $screen_function = isset( $bp->bp_options_nav[$parent_id][$slug]['screen_function'] )
-                ? $bp->bp_options_nav[$parent_id][$slug]['screen_function']
-                : false;
+        if ( ! isset( $bp->{$component}->nav ) ) {
+                return false;
+        }
 
-        if ( ! empty( $screen_function ) ) {
-                // Remove our screen hook if screen function is callable.
-                if ( is_callable( $screen_function ) ) {
-                        remove_action( 'bp_screens', $screen_function, 3 );
-                }
+        $screen_functions = $bp->{$component}->nav->delete_nav( $slug, $parent_slug );
+
+        if ( ! is_array( $screen_functions ) ) {
+                return false;
         }
 
-        unset( $bp->bp_options_nav[$parent_id][$slug] );
+        $screen_function = reset( $screen_functions );
 
-        if ( isset( $bp->bp_options_nav[$parent_id] ) && !count( $bp->bp_options_nav[$parent_id] ) )
-                unset($bp->bp_options_nav[$parent_id]);
+        // Remove our screen hook if screen function is callable.
+        if ( is_callable( $screen_function ) ) {
+                remove_action( 'bp_screens', $screen_function, 3 );
+        }
+
+        return true;
 }
 
 /**
  * Clear all subnav items from a specific nav item.
  *
  * @since 1.0.0
+ * @since 2.6.0 Adds the $component parameter.
  *
  * @param string $parent_slug The slug of the parent navigation item.
+ * @param string $component   The component the navigation is attached to. Defaults to 'members'.
  */
-function bp_core_reset_subnav_items( $parent_slug ) {
+function bp_core_reset_subnav_items( $parent_slug, $component = 'members' ) {
         $bp = buddypress();
 
-        unset( $bp->bp_options_nav[$parent_slug] );
+        if ( ! isset( $bp->{$component}->nav ) ) {
+                return;
+        }
+
+        $subnav_items = $bp->{$component}->nav->get_secondary( array( 'parent_slug' => $parent_slug ), false );
+
+        if ( ! $subnav_items ) {
+                return;
+        }
+
+        foreach( $subnav_items as $subnav_item ) {
+                $bp->{$component}->nav->delete_nav( $subnav_item->slug, $parent_slug );
+        }
 }
 
 

src/bp-core/bp-core-catchuri.php

@@ -684 +684 @@
  *
  * @see BP_Members_Component::setup_globals() where
  *      $bp->canonical_stack['base_url'] and ['component'] may be set.
- * @see bp_core_new_nav_item() where $bp->canonical_stack['action'] may be set.
+ * @see bp_core_add_nav_item() where $bp->canonical_stack['action'] may be set.
  * @uses bp_get_canonical_url()
  * @uses bp_get_requested_url()
  */

src/bp-core/bp-core-classes.php

@@ -30 +30 @@
 require dirname( __FILE__ ) . '/classes/class-bp-email.php';
 require dirname( __FILE__ ) . '/classes/class-bp-email-delivery.php';
 require dirname( __FILE__ ) . '/classes/class-bp-phpmailer.php';
+require dirname( __FILE__ ) . '/classes/class-bp-core-nav.php';
+require dirname( __FILE__ ) . '/classes/class-bp-core-nav-item.php';
+
+if ( buddypress()->do_nav_backcompat ) {
+        require dirname( __FILE__ ) . '/classes/class-bp-core-bp-nav-backcompat.php';
+        require dirname( __FILE__ ) . '/classes/class-bp-core-bp-options-nav-backcompat.php';
+}

src/bp-core/bp-core-functions.php

@@ -2443 +2443 @@
                 return buddypress()->wp_nav_menu_items->loggedin;
         }
 
-        // Pull up a list of items registered in BP's top-level nav array.
-        $bp_menu_items = buddypress()->bp_nav;
-
-        // Alphabetize.
-        $bp_menu_items = bp_alpha_sort_by_key( $bp_menu_items, 'name' );
+        // Pull up a list of items registered in BP's primary nav for the member.
+        $bp_menu_items = buddypress()->members->nav->get_primary();
 
         // Some BP nav menu items will not be represented in bp_nav, because
         // they are not real BP components. We add them manually here.
-        $bp_menu_items[] = array(
+        $bp_menu_items[] = (object) array(
                 'name' => __( 'Log Out', 'buddypress' ),
                 'slug' => 'logout',
                 'link' => wp_logout_url(),
@@ -2467 +2464 @@
         foreach ( $bp_menu_items as $bp_item ) {
 
                 // Remove <span>number</span>.
-                $item_name = _bp_strip_spans_from_title( $bp_item['name'] );
+                $item_name = _bp_strip_spans_from_title( $bp_item->name );
 
-                $page_args[ $bp_item['slug'] ] = (object) array(
+                $page_args[ $bp_item->slug ] = (object) array(
                         'ID'             => -1,
                         'post_title'     => $item_name,
                         'post_author'    => 0,
                         'post_date'      => 0,
-                        'post_excerpt'   => $bp_item['slug'],
+                        'post_excerpt'   => $bp_item->slug,
                         'post_type'      => 'page',
                         'post_status'    => 'publish',
                         'comment_status' => 'closed',
-                        'guid'           => $bp_item['link']
+                        'guid'           => $bp_item->link
                 );
         }
 
@@ -2575 +2572 @@
  * @since 1.9.0
  *
  * @param string $slug The slug of the nav item: login, register, or one of the
- *                     slugs from buddypress()->bp_nav.
+ *                     slugs from the members navigation.
  * @return string $nav_item_url The URL generated for the current user.
  */
 function bp_nav_menu_get_item_url( $slug ) {

src/bp-core/bp-core-template.php

@@ -13 +13 @@
 /**
  * Output the "options nav", the secondary-level single item navigation menu.
  *
- * Uses the $bp->bp_options_nav global to render out the sub navigation for the
+ * Uses the component's nav global to render out the sub navigation for the
  * current component. Each component adds to its sub navigation array within
  * its own setup_nav() function.
  *
@@ -40 +40 @@
         $component_index = !empty( $bp->displayed_user ) ? bp_current_component() : bp_get_root_slug( bp_current_component() );
         $selected_item   = bp_current_action();
 
+        /**
+         * Use the members Nav
+         */
         if ( ! bp_is_single_item() ) {
-                if ( !isset( $bp->bp_options_nav[$component_index] ) || count( $bp->bp_options_nav[$component_index] ) < 1 ) {
+                // Set the parent slug if not provided
+                if ( empty( $parent_slug ) ) {
+                        $parent_slug = $component_index;
+                }
+
+                $secondary_nav_items = $bp->members->nav->get_secondary( array( 'parent_slug' => $parent_slug ) );
+
+                if ( ! $secondary_nav_items ) {
                         return false;
-                } else {
-                        $the_index = $component_index;
                 }
+
+        /**
+         * Try to use the component's Nav for the single item.
+         */
         } else {
                 $current_item = bp_current_item();
+                $single_item_component = bp_current_component();
 
+                // Adjust the selected nav item for the current single item if needed
                 if ( ! empty( $parent_slug ) ) {
                         $current_item  = $parent_slug;
                         $selected_item = bp_action_variable( 0 );
                 }
 
-                if ( !isset( $bp->bp_options_nav[$current_item] ) || count( $bp->bp_options_nav[$current_item] ) < 1 ) {
-                        return false;
+                // The nav is not defined by the parent component, try to look in the members one
+                if ( ! isset( $bp->{$single_item_component}->nav ) ) {
+                        $secondary_nav_items = $bp->members->nav->get_secondary( array( 'parent_slug' => $current_item ) );
                 } else {
-                        $the_index = $current_item;
+                        $secondary_nav_items = $bp->{$single_item_component}->nav->get_secondary( array( 'parent_slug' => $current_item ) );
+                }
+
+                if ( ! $secondary_nav_items ) {
+                        return false;
                 }
         }
 
         // Loop through each navigation item.
-        foreach ( (array) $bp->bp_options_nav[$the_index] as $subnav_item ) {
-                if ( empty( $subnav_item['user_has_access'] ) ) {
+        foreach ( $secondary_nav_items as $subnav_item ) {
+                if ( empty( $subnav_item->user_has_access ) ) {
                         continue;
                 }
 
                 // If the current action or an action variable matches the nav item id, then add a highlight CSS class.
-                if ( $subnav_item['slug'] == $selected_item ) {
+                if ( $subnav_item->slug === $selected_item ) {
                         $selected = ' class="current selected"';
                 } else {
                         $selected = '';
@@ -88 +107 @@
                  * @param array  $subnav_item   Submenu array item being displayed.
                  * @param string $selected_item Current action.
                  */
-                echo apply_filters( 'bp_get_options_nav_' . $subnav_item['css_id'], '<li id="' . esc_attr( $subnav_item['css_id'] . '-' . $list_type . '-li' ) . '" ' . $selected . '><a id="' . esc_attr( $subnav_item['css_id'] ) . '" href="' . esc_url( $subnav_item['link'] ) . '">' . $subnav_item['name'] . '</a></li>', $subnav_item, $selected_item );
+                echo apply_filters( 'bp_get_options_nav_' . $subnav_item->css_id, '<li id="' . esc_attr( $subnav_item->css_id . '-' . $list_type . '-li' ) . '" ' . $selected . '><a id="' . esc_attr( $subnav_item->css_id ) . '" href="' . esc_url( $subnav_item->link ) . '">' . $subnav_item->name . '</a></li>', $subnav_item, $selected_item );
         }
 }
 
@@ -3010 +3029 @@
                 // Set empty subnav name.
                 $component_subnav_name = '';
 
+                if ( ! empty( $bp->members->nav ) ) {
+                        $primary_nav_item = $bp->members->nav->get_primary( array( 'slug' => $component_id ), false );
+                        $primary_nav_item = reset( $primary_nav_item );
+                }
+
                 // Use the component nav name.
-                if ( ! empty( $bp->bp_nav[$component_id] ) ) {
-                        $component_name = _bp_strip_spans_from_title( $bp->bp_nav[ $component_id ]['name'] );
+                if ( ! empty( $primary_nav_item->name ) ) {
+                        $component_name = _bp_strip_spans_from_title( $primary_nav_item->name );
 
                 // Fall back on the component ID.
                 } elseif ( ! empty( $bp->{$component_id}->id ) ) {
                         $component_name = ucwords( $bp->{$component_id}->id );
                 }
 
-                // Append action name if we're on a member component sub-page.
-                if ( ! empty( $bp->bp_options_nav[ $component_id ] ) && ! empty( $bp->canonical_stack['action'] ) ) {
-                        $component_subnav_name = wp_filter_object_list( $bp->bp_options_nav[ $component_id ], array( 'slug' => bp_current_action() ), 'and', 'name' );
+                if ( ! empty( $bp->members->nav ) ) {
+                        $secondary_nav_item = $bp->members->nav->get_secondary( array(
+                                'parent_slug' => $component_id,
+                                'slug'        => bp_current_action()
+                        ), false );
 
-                        if ( ! empty( $component_subnav_name ) ) {
-                                $component_subnav_name = array_shift( $component_subnav_name );
+                        if ( $secondary_nav_item ) {
+                                $secondary_nav_item = reset( $secondary_nav_item );
                         }
                 }
 
+                // Append action name if we're on a member component sub-page.
+                if ( ! empty( $secondary_nav_item->name ) && ! empty( $bp->canonical_stack['action'] ) ) {
+                        $component_subnav_name = $secondary_nav_item->name;
+                }
+
                 // If on the user profile's landing page, just use the fullname.
                 if ( bp_is_current_component( $bp->default_component ) && ( bp_get_requested_url() === bp_displayed_user_domain() ) ) {
                         $bp_title_parts[] = $displayed_user_name;
@@ -3045 +3076 @@
                         }
                 }
 
-        // A single group.
-        } elseif ( bp_is_active( 'groups' ) && ! empty( $bp->groups->current_group ) && ! empty( $bp->bp_options_nav[ $bp->groups->current_group->slug ] ) ) {
-                $subnav      = isset( $bp->bp_options_nav[ $bp->groups->current_group->slug ][ bp_current_action() ]['name'] ) ? $bp->bp_options_nav[ $bp->groups->current_group->slug ][ bp_current_action() ]['name'] : '';
-                $bp_title_parts = array( $bp->bp_options_title, $subnav );
-
-        // A single item from a component other than groups.
+        // A single item from a component other than members.
         } elseif ( bp_is_single_item() ) {
-                $bp_title_parts = array( $bp->bp_options_title, $bp->bp_options_nav[ bp_current_item() ][ bp_current_action() ]['name'] );
+                $component_id = bp_current_component();
+
+                if ( ! empty( $bp->{$component_id}->nav ) ) {
+                        $secondary_nav_item = $bp->{$component_id}->nav->get_secondary( array(
+                                'parent_slug' => bp_current_item(),
+                                'slug'        => bp_current_action()
+                        ), false );
+
+                        if ( $secondary_nav_item ) {
+                                $secondary_nav_item = reset( $secondary_nav_item );
+                        }
+                }
+
+                $single_item_subnav = '';
+
+                if ( ! empty( $secondary_nav_item->name ) ) {
+                        $single_item_subnav = $secondary_nav_item->name;
+                }
+
+                $bp_title_parts = array( $bp->bp_options_title, $single_item_subnav );
 
         // An index or directory.
         } elseif ( bp_is_directory() ) {
@@ -3428 +3473 @@
  * Get the items registered in the primary and secondary BuddyPress navigation menus.
  *
  * @since 1.7.0
+ * @since 2.6.0 Add the component parameter to be able to get the menu items for any single item
  *
  * @return array A multidimensional array of all navigation items.
  */
-function bp_get_nav_menu_items() {
-        $menus = $selected_menus = array();
-
-        // Get the second level menus.
-        foreach ( (array) buddypress()->bp_options_nav as $parent_menu => $sub_menus ) {
-
-                // The root menu's ID is "xprofile", but the Profile submenus are using "profile". See BP_Core::setup_nav().
-                if ( 'profile' === $parent_menu ) {
-                        $parent_menu = 'xprofile';
-                }
+function bp_get_nav_menu_items( $component = 'members' ) {
+        $bp    = buddypress();
+        $menus = array();
 
-                // Sort the items in this menu's navigation by their position property.
-                $second_level_menus = (array) $sub_menus;
-                usort( $second_level_menus, '_bp_nav_menu_sort' );
-
-                // Iterate through the second level menus.
-                foreach( $second_level_menus as $sub_nav ) {
-
-                        // Skip items we don't have access to.
-                        if ( empty( $sub_nav['user_has_access'] ) ) {
-                                continue;
-                        }
-
-                        // Add this menu.
-                        $menu         = new stdClass;
-                        $menu->class  = array( 'menu-child' );
-                        $menu->css_id = $sub_nav['css_id'];
-                        $menu->link   = $sub_nav['link'];
-                        $menu->name   = $sub_nav['name'];
-                        $menu->parent = $parent_menu;  // Associate this sub nav with a top-level menu.
-
-                        // If we're viewing this item's screen, record that we need to mark its parent menu to be selected.
-                        if ( $sub_nav['slug'] == bp_current_action() ) {
-                                $menu->class[]    = 'current-menu-item';
-                                $selected_menus[] = $parent_menu;
-                        }
-
-                        $menus[] = $menu;
-                }
+        if ( ! isset( $bp->{$component}->nav ) ) {
+                return $menus;
         }
 
-        // Get the top-level menu parts (Friends, Groups, etc) and sort by their position property.
-        $top_level_menus = (array) buddypress()->bp_nav;
-        usort( $top_level_menus, '_bp_nav_menu_sort' );
-
-        // Iterate through the top-level menus.
-        foreach ( $top_level_menus as $nav ) {
-
-                // Skip items marked as user-specific if you're not on your own profile.
-                if ( empty( $nav['show_for_displayed_user'] ) && ! bp_core_can_edit_settings()  ) {
-                        continue;
-                }
-
+        // Get the item nav and build the menus
+        foreach ( $bp->{$component}->nav->get_item_nav() as $nav_menu ) {
                 // Get the correct menu link. See https://buddypress.trac.wordpress.org/ticket/4624.
-                $link = bp_loggedin_user_domain() ? str_replace( bp_loggedin_user_domain(), bp_displayed_user_domain(), $nav['link'] ) : trailingslashit( bp_displayed_user_domain() . $nav['link'] );
+                $link = bp_loggedin_user_domain() ? str_replace( bp_loggedin_user_domain(), bp_displayed_user_domain(), $nav_menu->link ) : trailingslashit( bp_displayed_user_domain() . $nav_menu->link );
 
                 // Add this menu.
                 $menu         = new stdClass;
                 $menu->class  = array( 'menu-parent' );
-                $menu->css_id = $nav['css_id'];
-                $menu->link   = $link;
-                $menu->name   = $nav['name'];
+                $menu->css_id = $nav_menu->css_id;
+                $menu->link   = $nav_menu->link;
+                $menu->name   = $nav_menu->name;
                 $menu->parent = 0;
 
-                // Check if we need to mark this menu as selected.
-                if ( in_array( $nav['css_id'], $selected_menus ) ) {
-                        $menu->class[] = 'current-menu-parent';
+                if ( ! empty( $nav_menu->children ) ) {
+                        $submenus = array();
+
+                        foreach( $nav_menu->children as $sub_menu ) {
+                                $submenu = new stdClass;
+                                $submenu->class  = array( 'menu-child' );
+                                $submenu->css_id = $sub_menu->css_id;
+                                $submenu->link   = $sub_menu->link;
+                                $submenu->name   = $sub_menu->name;
+                                $submenu->parent = $nav_menu->slug;
+
+                                // If we're viewing this item's screen, record that we need to mark its parent menu to be selected.
+                                if ( $sub_menu->slug == bp_current_action() ) {
+                                        $menu->class[]    = 'current-menu-parent';
+                                        $submenu->class[] = 'current-menu-item';
+                                }
+
+                                $submenus[] = $submenu;
+                        }
                 }
 
                 $menus[] = $menu;
+
+                if ( ! empty( $submenus ) ) {
+                        $menus = array_merge( $menus, $submenus );
+                }
         }
 
         /**

src/bp-core/classes/class-bp-component.php

@@ -473 +473 @@
          * Set up component navigation.
          *
          * @since 1.5.0
+         * @since 2.6.0 Use bp_core_add_nav_item() instead of bp_core_new_nav_item() (deprecated)
+         *                              Use bp_core_add_subnav_item() instead of bp_core_new_subnav_item() (deprecated)
          *
-         * @see bp_core_new_nav_item() For a description of the $main_nav
+         * @see bp_core_add_nav_item() For a description of the $main_nav
          *      parameter formatting.
-         * @see bp_core_new_subnav_item() For a description of how each item
+         * @see bp_core_add_subnav_item() For a description of how each item
          *      in the $sub_nav parameter array should be formatted.
          *
-         * @param array $main_nav Optional. Passed directly to bp_core_new_nav_item().
+         * @param array $main_nav Optional. Passed directly to bp_core_add_nav_item().
          *                        See that function for a description.
          * @param array $sub_nav  Optional. Multidimensional array, each item in
-         *                        which is passed to bp_core_new_subnav_item(). See that
+         *                        which is passed to bp_core_add_subnav_item(). See that
          *                        function for a description.
          */
         public function setup_nav( $main_nav = array(), $sub_nav = array() ) {
 
                 // No sub nav items without a main nav item.
                 if ( !empty( $main_nav ) ) {
-                        bp_core_new_nav_item( $main_nav );
+                        bp_core_add_nav_item( $main_nav, 'members' );
 
                         // Sub nav items are not required.
                         if ( !empty( $sub_nav ) ) {
                                 foreach( (array) $sub_nav as $nav ) {
-                                        bp_core_new_subnav_item( $nav );
+                                        bp_core_add_subnav_item( $nav, 'members' );
                                 }
                         }
                 }

new file src/bp-core/classes/class-bp-core-bp-nav-backcompat.php

@@ +1 @@
+<?php
+/**
+ * Backward compatibility for the $bp->bp_nav global.
+ *
+ * @since 2.6.0
+ */
+
+// Exit if accessed directly.
+defined( 'ABSPATH' ) || exit;
+
+/**
+ * bp_nav backward compatibility class.
+ *
+ * This class is used to provide backward compatibility for extensions that access and modify
+ * the $bp->bp_nav global.
+ *
+ * @since 2.6.0
+ */
+class BP_Core_BP_Nav_BackCompat implements ArrayAccess {
+        public $backcompat_nav = array();
+        public $component;
+
+        public function __construct( $backcompat_nav = array() ) {
+                foreach ( $backcompat_nav as $key => $value ) {
+                        if ( is_array( $value ) ) {
+                                $this->backcompat_nav[ $key ] = new self( $value );
+                        } else {
+                                $this->backcompat_nav[ $key ] = $value;
+                        }
+                }
+        }
+
+        public function offsetSet( $offset, $value ) {
+                _doing_it_wrong(
+                        'bp_nav',
+                        __( 'These globals should not be used directly and are deprecated. Please use the BuddyPress nav functions instead.', 'buddypress' ),
+                        '2.6.0'
+                );
+
+                $bp = buddypress();
+
+                if ( is_array( $value ) ) {
+                        $value = new self( $value );
+                }
+
+                if ( $offset !== null ) {
+                        // Temporarily set the backcompat_nav.
+                        $this->backcompat_nav[ $offset ] = $value;
+
+                        $args = $this->to_array();
+                        if ( isset( $args['parent_slug'] ) ) {
+                                $this->get_component_nav( $args['parent_slug'] )->edit_nav( $args, $args['slug'], $args['parent_slug'] );
+                        } elseif ( isset( $args['slug'] ) ) {
+                                $bp->members->nav->edit_nav( $args, $args['slug'] );
+                        }
+                }
+        }
+
+        public function offsetGet( $offset ) {
+                _doing_it_wrong(
+                        'bp_nav',
+                        __( 'These globals should not be used directly and are deprecated. Please use the BuddyPress nav functions instead.', 'buddypress' ),
+                        '2.6.0'
+                );
+
+                if ( ! isset( $this->backcompat_nav[ $offset ] ) ) {
+                        $nav = $this->get_nav( $offset );
+                        if ( $nav && isset( $nav[ $offset ] ) ) {
+                                $this->backcompat_nav[ $offset ] = new self( $nav[ $offset ] );
+                        }
+                }
+
+                return $this->backcompat_nav[ $offset ];
+        }
+
+        public function offsetExists( $offset ) {
+                _doing_it_wrong(
+                        'bp_nav',
+                        __( 'These globals should not be used directly and are deprecated. Please use the BuddyPress nav functions instead.', 'buddypress' ),
+                        '2.6.0'
+                );
+
+                if ( isset( $this->backcompat_nav[ $offset ] ) ) {
+                        return true;
+                }
+
+                $nav = $this->get_nav( $offset );
+                if ( $nav && isset( $nav[ $offset ] ) ) {
+                        return true;
+                }
+
+                return false;
+        }
+
+        public function offsetUnset( $offset ) {
+                _doing_it_wrong(
+                        'bp_nav',
+                        __( 'These globals should not be used directly and are deprecated. Please use the BuddyPress nav functions instead.', 'buddypress' ),
+                        '2.6.0'
+                );
+
+                // For top-level nav items, the backcompat nav hasn't yet been initialized.
+                if ( ! isset( $this->backcompat_nav[ $offset ] ) ) {
+                        buddypress()->members->nav->delete_nav( $offset );
+                        unset( $this->backcompat_nav[ $offset ] );
+                }
+        }
+
+        public function get_nav( $offset ) {
+                $bp = buddypress();
+
+                $component_nav = $this->get_component_nav( $offset );
+                $primary_nav   = $component_nav->get_primary( array( 'slug' => $offset ), false );
+
+                $nav = array();
+
+                if ( empty( $primary_nav ) ) {
+                        return $nav;
+                }
+
+                foreach ( $primary_nav as $item ) {
+                        $nav[ $item->slug ] = (array) $item;
+                }
+
+                return $nav;
+        }
+
+        public function set_component( $component ) {
+                $this->component = $component;
+        }
+
+        public function get_component( $offset = '' ) {
+                if ( ! isset( $this->component ) ) {
+                        if ( bp_is_active( 'groups' ) && $offset === bp_get_current_group_slug() ) {
+                                $this->component = 'groups';
+                        } else {
+                                $this->component = 'members';
+                        }
+                }
+
+                return $this->component;
+        }
+
+        protected function to_array() {
+                return $this->backcompat_nav;
+        }
+
+        /**
+         * Get the BP_Core_Nav object corresponding to the component, based on a nav item name.
+         *
+         * The way bp_nav was previously organized makes it impossible to know for sure which component's nav is
+         * being referenced by a given nav item name. We guess in the following manner:
+         *   - If we're looking at a group, and the nav item name (`$offset`) is the same as the slug of the current
+         *     group, we assume that the proper component nav is 'groups'.
+         *   - Otherwise, fall back on 'members'.
+         *
+         * @since 2.6.0
+         *
+         * @param string $offset Nav item name.
+         * @return BP_Core_Nav
+         */
+        protected function get_component_nav( $offset = '' ) {
+                $component = $this->get_component( $offset );
+
+                $bp = buddypress();
+                if ( ! isset( $bp->{$component}->nav ) ) {
+                        return false;
+                }
+
+                return $bp->{$component}->nav;
+        }
+}

new file src/bp-core/classes/class-bp-core-bp-options-nav-backcompat.php

@@ +1 @@
+<?php
+/**
+ * Backward compatibility for the $bp->bp_options_nav global.
+ *
+ * @since 2.6.0
+ */
+
+// Exit if accessed directly.
+defined( 'ABSPATH' ) || exit;
+
+/**
+ * bp_options_nav backward compatibility class.
+ *
+ * This class is used to provide backward compatibility for extensions that access and modify
+ * the $bp->bp_options_nav global.
+ *
+ * @since 2.6.0
+ */
+class BP_Core_BP_Options_Nav_BackCompat extends BP_Core_BP_Nav_BackCompat {
+        protected $parent_slug = '';
+
+        public function offsetGet( $offset ) {
+                _doing_it_wrong(
+                        'bp_nav',
+                        __( 'These globals should not be used directly and are deprecated. Please use the BuddyPress nav functions instead.', 'buddypress' ),
+                        '2.6.0'
+                );
+
+                if ( empty( $this->backcompat_nav[ $offset ] ) ) {
+                        $nav = $this->get_nav( $offset );
+                        if ( $nav ) {
+                                $subnavs = $this->get_component_nav( $offset )->get_secondary( array( 'parent_slug' => $offset ) );
+                                $subnav_keyed = array();
+                                foreach ( $subnavs as $subnav ) {
+                                        $subnav_keyed[ $subnav->slug ] = (array) $subnav;
+                                }
+
+                                $subnav_object = new self( $subnav_keyed );
+                                $subnav_object->set_component( $this->get_component() );
+                                $subnav_object->set_parent_slug( $offset );
+
+                                $this->backcompat_nav[ $offset ] = $subnav_object;
+                        }
+                }
+                return $this->backcompat_nav[ $offset ];
+        }
+
+        public function offsetUnset( $offset ) {
+                _doing_it_wrong(
+                        'bp_nav',
+                        __( 'These globals should not be used directly and are deprecated. Please use the BuddyPress nav functions instead.', 'buddypress' ),
+                        '2.6.0'
+                );
+
+                $this->get_component_nav( $offset )->delete_nav( $offset, $this->get_parent_slug() );
+
+                // Clear the cached nav.
+                unset( $this->backcompat_nav[ $offset ] );
+        }
+
+        public function get_parent_slug() {
+                return $this->parent_slug;
+        }
+
+        public function set_parent_slug( $slug ) {
+                $this->parent_slug = $slug;
+        }
+}

new file src/bp-core/classes/class-bp-core-nav-item.php

@@ +1 @@
+<?php
+
+// Exit if accessed directly.
+defined( 'ABSPATH' ) || exit;
+
+if ( class_exists( 'ArrayObject' ) ) :
+
+/**
+ * Navigation item.
+ *
+ * @since 2.6.0
+ */
+class BP_Core_Nav_Item extends ArrayObject {
+        public function __construct( $data ) {
+                parent::__construct( $data, ArrayObject::ARRAY_AS_PROPS );
+        }
+}
+
+else :
+
+/**
+ * Navigation item.
+ *
+ * @since 2.6.0
+ */
+class BP_Core_Nav_Item {
+        public function __construct( $data ) {
+                foreach ( $data as $key => $value ) {
+                        $this->key = $value;
+                }
+        }
+}
+
+endif;

new file src/bp-core/classes/class-bp-core-nav.php

@@ +1 @@
+<?php
+/**
+ * Core component class.
+ *
+ * @package BuddyPress
+ * @subpackage Core
+ * @since  2.6.0
+ */
+
+// Exit if accessed directly.
+defined( 'ABSPATH' ) || exit;
+
+/**
+ * BuddyPress Nav.
+ *
+ * This class is used to build our component's primary and secondary navigations
+ *
+ * @since 2.6.0
+ */
+class BP_Core_Nav {
+        /**
+         * An associative array containing the nav object items for the Item ID
+         *
+         * @var array
+         */
+        protected $nav;
+
+        /**
+         * The Current Item ID
+         *
+         * @var int
+         */
+        private $item_id;
+
+        /**
+         * Init the Nav for the given Item ID
+         *
+         * @since 2.6.0
+         *
+         * @param int $item_id The item ID to build the nav for. Optional.
+         *                     Default: The displayed user ID
+         */
+        public function __construct( $item_id = 0 ) {
+                if ( empty( $item_id ) ) {
+                        $this->item_id = (int) bp_displayed_user_id();
+                } else {
+                        $this->item_id = (int) $item_id;
+                }
+
+                $this->nav[ $this->item_id ] = array();
+        }
+
+        /**
+         * Checks if a nav item is set.
+         *
+         * @since 2.6.0
+         *
+         * @param  string  $key The requested nav slug
+         * @return bool    True if the nav item is set, false otherwise.
+         */
+        public function __isset( $key ) {
+                return isset( $this->nav[ $this->item_id ][ $key ] );
+        }
+
+        /**
+         * Gets a nav item.
+         *
+         * @since 2.6.0
+         *
+         * @param  string  $key The requested nav slug.
+         * @return mixed   The value corresponding to the requested nav item.
+         */
+        public function __get( $key ) {
+                if ( ! isset( $this->nav[ $this->item_id ][ $key ] ) ) {
+                        $this->nav[ $this->item_id ][ $key ] = null;
+                }
+
+                return $this->nav[ $this->item_id ][ $key ];
+        }
+
+        /**
+         * Sets a nav item.
+         *
+         * @since 2.6.0
+         *
+         * @param  string  $key   The requested nav slug.
+         * @param  mixed   $value The value of the nav item.
+         */
+        public function __set( $key, $value ) {
+                if ( is_array( $value ) ) {
+                        $value['primary'] = true;
+                }
+
+                $this->nav[ $this->item_id ][ $key ] = new BP_Core_Nav_Item( $value );
+        }
+
+        /**
+         * Gets a specific Raw Item Nav or all items.
+         *
+         * @since 2.6.0
+         *
+         * @param  string $key The nav item slug to get. Optional.
+         * @return mixed       An array of nav item(s), null if not found.
+         */
+        public function get( $key = '' ) {
+                $return = null;
+
+                // Return the requested nav item attribute
+                if ( ! empty( $key ) ) {
+                        if ( ! isset( $this->nav[ $this->item_id ][ $key ] ) ) {
+                                $return = null;
+                        } else {
+                                $return = $this->nav[ $this->item_id ][ $key ];
+                        }
+
+                // Return all nav item attributes
+                } else {
+                        $return = $this->nav[ $this->item_id ];
+                }
+
+                return $return;
+        }
+
+        /**
+         * Adds a new nav item.
+         *
+         * @since 2.6.0
+         *
+         * @param  array $args The nav item's arguments.
+         */
+        public function add_nav( $args ) {
+                if ( empty( $args['slug'] ) ) {
+                        return false;
+                }
+
+                // We have a child and the parent exists
+                if ( ! empty( $args['parent_slug'] ) ) {
+                        $slug              = $args['parent_slug'] . '/' . $args['slug'];
+                        $args['secondary'] = true;
+
+                // This is a parent
+                } else {
+                        $slug            = $args['slug'];
+                        $args['primary'] = true;
+                }
+
+                // Add to the nav
+                $this->nav[ $this->item_id ][ $slug ] = new BP_Core_Nav_Item( $args );
+        }
+
+        /**
+         * Edits a nav item.
+         *
+         * @since 2.6.0
+         *
+         * @param  array  $args        The nav item's arguments.
+         * @param  string $slug        The slug of the nav item.
+         * @param  string $parent_slug The Slug of the Parent nav item (Required to edit a child).
+         */
+        public function edit_nav( $args = array(), $slug = '', $parent_slug = '' ) {
+                if ( empty( $slug ) ) {
+                        return false;
+                }
+
+                // We're editing a parent!
+                if ( empty( $parent_slug ) ) {
+                        $nav_items = $this->get_primary( array( 'slug' => $slug ), false );
+
+                        if ( ! $nav_items ) {
+                                return false;
+                        }
+
+                        $nav_item = reset( $nav_items );
+                        $this->nav[ $this->item_id ][ $slug ] = new BP_Core_Nav_Item( wp_parse_args( $args, (array) $nav_item ) );
+
+                        // Return the edited object
+                        return $this->nav[ $this->item_id ][ $slug ];
+
+                // We're editing a child
+                } else {
+                        $sub_items = $this->get_secondary( array( 'parent_slug' => $parent_slug, 'slug' => $slug ), false );
+
+                        if ( ! $sub_items ) {
+                                return false;
+                        }
+
+                        $sub_item = reset( $sub_items );
+
+                        $params = wp_parse_args( $args, (array) $sub_item );
+
+                        // When we have parents, it's for life, we can't change them!
+                        if ( empty( $params['parent_slug'] ) || $parent_slug !== $params['parent_slug'] ) {
+                                return false;
+                        }
+
+                        $this->nav[ $this->item_id ][ $parent_slug . '/' . $slug ] = new BP_Core_Nav_Item( $params );
+
+                        // Return the edited object
+                        return $this->nav[ $this->item_id ][ $parent_slug . '/' . $slug ];
+                }
+        }
+
+        /**
+         * Unset an item or a subitem of the nav.
+         *
+         * @since 2.6.0
+         *
+         * @param  string $key     The slug of the main item.
+         * @param  string $sub_key The slug of the sub item.
+         */
+        public function delete_nav( $slug, $parent_slug = '' ) {
+                if ( empty( $slug ) ) {
+                        return false;
+                }
+
+                // We're deleting a child
+                if ( ! empty( $parent_slug ) ) {
+
+                        // Validate the subnav
+                        $sub_items = $this->get_secondary( array( 'parent_slug' => $parent_slug, 'slug' => $slug ), false );
+
+                        if ( ! $sub_items ) {
+                                return false;
+                        }
+
+                        $sub_item = reset( $sub_items );
+
+                        if ( empty( $sub_item->slug ) ) {
+                                return false;
+                        }
+
+                        // Delete the child
+                        unset( $this->nav[ $this->item_id ][ $parent_slug . '/' . $slug ] );
+
+                        // Return the deleted item's screen function
+                        return array( $sub_item->screen_function );
+
+                // We're deleting a parent
+                } else {
+                        // Validate the nav
+                        $nav_items = $this->get_primary( array( 'slug' => $slug ), false );
+
+                        if ( ! $nav_items ) {
+                                return false;
+                        }
+
+                        $nav_item = reset( $nav_items );
+
+                        if ( empty( $nav_item->slug ) ) {
+                                return false;
+                        }
+
+                        $screen_functions = array( $nav_item->screen_function );
+
+                        // Life's unfair, children won't survive the parent :(
+                        $sub_items = $this->get_secondary( array( 'parent_slug' => $nav_item->slug ), false );
+
+                        if ( ! empty( $sub_items ) ) {
+                                foreach ( $sub_items as $sub_item ) {
+                                        $screen_functions[] = $sub_item->screen_function;
+
+                                        // Delete the child
+                                        unset( $this->nav[ $this->item_id ][ $nav_item->slug . '/' . $sub_item->slug ] );
+                                }
+                        }
+
+                        // Delete the parent
+                        unset( $this->nav[ $this->item_id ][ $nav_item->slug ] );
+
+                        // Return the deleted item's screen functions
+                        return array_unique( $screen_functions );
+                }
+        }
+
+        /**
+         * Sorts a list of nav items.
+         *
+         * @since 2.6.0
+         *
+         * @param  array $items The nav item's to sort.
+         */
+        public function sort_nav( $items ) {
+                $sorted = array();
+
+                foreach ( $items as $item ) {
+                        // Default position
+                        $position = 99;
+
+                        if ( isset( $item->position ) ) {
+                                $position = (int) $item->position;
+                        }
+
+                        // If position is already taken, move to the first next available
+                        if ( isset( $sorted[ $position ] ) ) {
+                                $sorted_keys = array_keys( $sorted );
+
+                                do {
+                                        $position += 1;
+                                } while ( in_array( $position, $sorted_keys ) );
+                        }
+
+                        $sorted[ $position ] = $item;
+                }
+
+                ksort( $sorted );
+                return $sorted;
+        }
+
+        /**
+         * Gets the primary nav items.
+         *
+         * @since 2.6.0
+         *
+         * @param  array $args Filters to select the specific primary items.
+         * @param  bool  $sort True to sort the nav items. False otherwise.
+         * @return array       The list of primary objects nav
+         */
+        public function get_primary( $args = array(), $sort = true ) {
+                $params = wp_parse_args( $args, array( 'primary' => true ) );
+
+                // This parameter is not overridable
+                if ( empty( $params['primary'] ) ) {
+                        return false;
+                }
+
+                $primary_nav = wp_list_filter( $this->nav[ $this->item_id ], $params );
+
+                if ( ! $primary_nav ) {
+                        return false;
+                }
+
+                if ( true !== $sort ) {
+                        return $primary_nav;
+                }
+
+                return $this->sort_nav( $primary_nav );
+        }
+
+        /**
+         * Gets the secondary nav items.
+         *
+         * @since 2.6.0
+         *
+         * @param  array $args Filters to select the specific secondary items.
+         * @param  bool  $sort True to sort the nav items. False otherwise.
+         * @return array       The list of secondary objects nav
+         */
+        public function get_secondary( $args = array(), $sort = true ) {
+                $params = wp_parse_args( $args, array( 'parent_slug' => '' ) );
+
+                // No need to search children if the parent is not set
+                if ( empty( $params['parent_slug'] ) && empty( $params['secondary'] ) ) {
+                        return false;
+                }
+
+                $secondary_nav = wp_list_filter( $this->nav[ $this->item_id ], $params );
+
+                if ( ! $secondary_nav ) {
+                        return false;
+                }
+
+                if ( true !== $sort ) {
+                        return $secondary_nav;
+                }
+
+                return $this->sort_nav( $secondary_nav );
+        }
+
+        /**
+         * Gets a nested list of visible nav items.
+         *
+         * @since 2.6.0
+         *
+         * @return array The list of visible objects nav
+         */
+        public function get_item_nav() {
+                $primary_nav_items = $this->get_primary( array( 'show_for_displayed_user' => true ) );
+
+                if ( $primary_nav_items ) {
+                        foreach( $primary_nav_items as $key_nav => $primary_nav ) {
+                                // Try to get the children
+                                $children = $this->get_secondary( array( 'parent_slug' => $primary_nav->slug, 'user_has_access' => true ) );
+
+                                if ( $children ) {
+                                        $primary_nav_items[ $key_nav ] = clone( $primary_nav );
+                                        $primary_nav_items[ $key_nav ]->children = $children;
+                                }
+                        }
+                }
+
+                return $primary_nav_items;
+        }
+}

src/bp-core/deprecated/2.6.php

@@ -21 +21 @@
 
         <?php
 }
+
+/**
+ * Add an item to the main BuddyPress navigation array.
+ *
+ * @since 1.1.0
+ * @deprecated 2.6.0
+ *
+ * @param array|string $args {
+ *     Array describing the new nav item.
+ *     @type string      $name                    Display name for the nav item.
+ *     @type string      $slug                    Unique URL slug for the nav item.
+ *     @type bool|string $item_css_id             Optional. 'id' attribute for the nav item. Default: the value of `$slug`.
+ *     @type bool        $show_for_displayed_user Optional. Whether the nav item should be visible when viewing a
+ *                                                member profile other than your own. Default: true.
+ *     @type bool        $site_admin_only         Optional. Whether the nav item should be visible only to site admins
+ *                                                (those with the 'bp_moderate' cap). Default: false.
+ *     @type int         $position                Optional. Numerical index specifying where the item should appear in
+ *                                                the nav array. Default: 99.
+ *     @type callable    $screen_function         The callback function that will run when the nav item is clicked.
+ *     @type bool|string $default_subnav_slug     Optional. The slug of the default subnav item to select when the nav
+ *                                                item is clicked.
+ * }
+ * @return bool|null Returns false on failure.
+ */
+function bp_core_new_nav_item( $args = '' ) {
+        $e = new Exception;
+        $trace = $e->getTraceAsString();
+
+        _deprecated_function( __FUNCTION__, '2.6', "bp_core_add_nav_item() instead.\n===\nTrace:\n" . $trace . "\n===\n" );
+        return bp_core_add_nav_item( $args );
+}
+
+/**
+ * Sort the navigation menu items.
+ *
+ * The sorting is split into a separate function because it can only happen
+ * after all plugins have had a chance to register their navigation items.
+ *
+ * @since 1.0.0
+ * @deprecated 2.6.0
+ *
+ * @return bool|null Returns false on failure.
+ */
+function bp_core_sort_nav_items() {
+        _deprecated_function( __FUNCTION__, '2.6' );
+}
+
+/**
+ * Add a subnav item to the BuddyPress navigation.
+ *
+ * @since 1.1.0
+ * @deprecated 2.6.0
+ *
+ * @param array|string $args {
+ *     Array describing the new subnav item.
+ *     @type string      $name              Display name for the subnav item.
+ *     @type string      $slug              Unique URL slug for the subnav item.
+ *     @type string      $parent_slug       Slug of the top-level nav item under which the new subnav item should
+ *                                          be added.
+ *     @type string      $parent_url        URL of the parent nav item.
+ *     @type bool|string $item_css_id       Optional. 'id' attribute for the nav item. Default: the value of `$slug`.
+ *     @type bool        $user_has_access   Optional. True if the logged-in user has access to the subnav item,
+ *                                          otherwise false. Can be set dynamically when registering the subnav;
+ *                                          eg, use `bp_is_my_profile()` to restrict access to profile owners only.
+ *                                          Default: true.
+ *     @type bool        $site_admin_only   Optional. Whether the nav item should be visible only to site admins
+ *                                          (those with the 'bp_moderate' cap). Default: false.
+ *     @type int         $position          Optional. Numerical index specifying where the item should appear in the
+ *                                          subnav array. Default: 90.
+ *     @type callable    $screen_function   The callback function that will run when the nav item is clicked.
+ *     @type string      $link              Optional. The URL that the subnav item should point to. Defaults to a value
+ *                                          generated from the `$parent_url` + `$slug`.
+ *     @type bool        $show_in_admin_bar Optional. Whether the nav item should be added into the group's "Edit"
+ *                                          Admin Bar menu for group admins. Default: false.
+ * }
+ * @return bool|null Returns false on failure.
+ */
+function bp_core_new_subnav_item( $args = '' ) {
+        $e = new Exception;
+        $trace = $e->getTraceAsString();
+
+        _deprecated_function( __FUNCTION__, '2.6', "bp_core_add_subnav_item() instead.\n===\nTrace:\n" . $trace . "\n===\n" );
+        return bp_core_add_subnav_item( $args );
+}
+
+/**
+ * Sort all subnavigation arrays.
+ *
+ * @since 1.1.0
+ * @deprecated 2.6.0
+ *
+ * @return bool|null Returns false on failure.
+ */
+function bp_core_sort_subnav_items() {
+        _deprecated_function( __FUNCTION__, '2.6' );
+}
+
+/**
+ * Remove a nav item from the navigation array.
+ *
+ * @since 1.0.0
+ * @deprecated 2.6.0
+ *
+ * @param int $parent_id The slug of the parent navigation item.
+ * @return bool Returns false on failure, ie if the nav item can't be found.
+ */
+function bp_core_remove_nav_item( $parent_id ) {
+        $bp = buddypress();
+
+        $e = new Exception;
+        $trace = $e->getTraceAsString();
+
+        _deprecated_function( __FUNCTION__, '2.6', "bp_core_delete_nav_item() instead.\n===\nTrace:\n" . $trace . "\n===\n" );
+
+        if ( bp_is_active( 'groups' ) && isset( $bp->groups->nav ) ) {
+
+                if ( $bp->groups->nav->get_primary( array( 'slug' => $parent_id ) ) ) {
+                        return bp_core_delete_nav_item( $parent_id, 'groups' );
+                }
+        }
+
+        return bp_core_delete_nav_item( $parent_id );
+}
+
+/**
+ * Remove a subnav item from the navigation array.
+ *
+ * @since 1.0.0
+ * @deprecated 2.6.0
+ *
+ * @param string $parent_id The slug of the parent navigation item.
+ * @param string $slug      The slug of the subnav item to be removed.
+ */
+function bp_core_remove_subnav_item( $parent_id, $slug ) {
+        $bp = buddypress();
+
+        $e = new Exception;
+        $trace = $e->getTraceAsString();
+
+        _deprecated_function( __FUNCTION__, '2.6', "bp_core_delete_subnav_item() instead.\n===\nTrace:\n" . $trace . "\n===\n" );
+
+        if ( bp_is_active( 'groups' ) && isset( $bp->groups->nav ) ) {
+
+                if ( $bp->groups->nav->get_primary( array( 'slug' => $parent_id ) ) ) {
+                        return bp_core_delete_subnav_item( $slug, $parent_id, 'groups' );
+                }
+        }
+
+        return bp_core_delete_subnav_item( $slug, $parent_id );
+}

src/bp-groups/bp-groups-adminbar.php

@@ -47 +47 @@
         ) );
 
         // Index of the Manage tabs parent slug.
-        $nav_index = $bp->groups->current_group->slug . '_manage';
+        $secondary_nav_items = $bp->groups->nav->get_secondary( array( 'parent_slug' => $bp->groups->current_group->slug . '_manage' ) );
 
         // Check if current group has Manage tabs.
-        if ( empty( $bp->bp_options_nav[ $nav_index ] ) ) {
+        if ( ! $secondary_nav_items ) {
                 return;
         }
 
         // Build the Group Admin menus.
-        foreach ( $bp->bp_options_nav[ $nav_index ] as $menu ) {
+        foreach ( $secondary_nav_items as $menu ) {
                 /**
                  * Should we add the current manage link in the Group's "Edit" Admin Bar menu ?
                  *
@@ -63 +63 @@
                  * to also add the link to the "edit screen" of their group component. To do so, set the
                  * the 'show_in_admin_bar' argument of your edit screen to true
                  */
-                if ( $menu['show_in_admin_bar'] ) {
-                        $title = sprintf( _x( 'Edit Group %s', 'Group WP Admin Bar manage links', 'buddypress' ), $menu['name'] );
+                if ( $menu->show_in_admin_bar ) {
+                        $title = sprintf( _x( 'Edit Group %s', 'Group WP Admin Bar manage links', 'buddypress' ), $menu->name );
 
                         // Title is specific for delete.
-                        if ( 'delete-group' == $menu['slug'] ) {
-                                $title = sprintf( _x( '%s Group', 'Group WP Admin Bar delete link', 'buddypress' ), $menu['name'] );
+                        if ( 'delete-group' == $menu->slug ) {
+                                $title = sprintf( _x( '%s Group', 'Group WP Admin Bar delete link', 'buddypress' ), $menu->name );
                         }
 
                         $wp_admin_bar->add_menu( array(
                                 'parent' => $bp->group_admin_menu_id,
-                                'id'     => $menu['slug'],
+                                'id'     => $menu->slug,
                                 'title'  => $title,
-                                'href'   => bp_get_groups_action_link( 'admin/' . $menu['slug'] )
+                                'href'   => bp_get_groups_action_link( 'admin/' . $menu->slug )
                         ) );
                 }
         }

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

@@ -737 +737 @@
                                 'screen_function' => array( &$this, '_display_hook' ),
                                 'user_has_access' => $user_can_see_nav_item,
                                 'no_access_url'   => $group_permalink,
-                        ) );
+                        ), 'groups' );
                 }
 
                 // If the user can visit the screen, we register it.
@@ -752 +752 @@
                                 'screen_function' => array( &$this, '_display_hook' ),
                                 'user_has_access' => $user_can_visit,
                                 'no_access_url'   => $group_permalink,
-                        ) );
+                        ), 'groups' );
 
                         // When we are viewing the extension display page, set the title and options title.
                         if ( bp_is_current_action( $this->slug ) ) {
@@ -967 +967 @@
                 }
 
                 // Add the tab to the manage navigation.
-                bp_core_new_subnav_item( $subnav_args );
+                bp_core_add_subnav_item( $subnav_args, 'groups' );
 
                 // Catch the edit screen and forward it to the plugin template.
                 if ( bp_is_groups_component() && bp_is_current_action( 'admin' ) && bp_is_action_variable( $screen['slug'], 0 ) ) {

src/bp-groups/classes/class-bp-groups-component.php

@@ -253 +253 @@
                         // Check once if the current group has a custom front template.
                         $this->current_group->front_template = bp_groups_get_front_template( $this->current_group );
 
+                        // Backward compatibility for plugins modifying the legacy bp_options_nav global property.
+                        if ( buddypress()->do_nav_backcompat ) {
+                                $bp->bp_options_nav = new BP_Core_BP_Options_Nav_BackCompat();
+                        }
+
+                        // Initialize the nav for the groups component.
+                        $this->nav = new BP_Core_Nav( $this->current_group->id );
+
                 // Set current_group to 0 to prevent debug errors.
                 } else {
                         $this->current_group = 0;
@@ -496 +504 @@
                         // Reset sub nav.
                         $sub_nav = array();
 
-                        // Add 'Groups' to the main navigation.
-                        $main_nav = array(
+                        /**
+                         * Set the Current Group's main navigation
+                         *
+                         * Since 2.6.0 it's now using its own navigation by passing the
+                         * component's id as the second argument of bp_core_add_nav_item()
+                         */
+                        bp_core_add_nav_item( array(
                                 'name'                => __( 'Memberships', 'buddypress' ),
                                 'slug'                => $this->current_group->slug,
                                 'position'            => -1, // Do not show in BuddyBar.
                                 'screen_function'     => 'groups_screen_group_home',
                                 'default_subnav_slug' => $this->default_extension,
                                 'item_css_id'         => $this->id
-                        );
+                        ), 'groups' );
 
                         $group_link = bp_get_group_permalink( $this->current_group );
 
@@ -675 +688 @@
                                 ), $default_params );
                         }
 
-                        parent::setup_nav( $main_nav, $sub_nav );
+                        foreach( $sub_nav as $nav ) {
+                                /**
+                                 * Set the Current Group's sub navigation
+                                 *
+                                 * Since 2.6.0 it's now using its own sub navigation by passing it
+                                 * as the second argument of bp_core_add_subnav_item()
+                                 */
+                                bp_core_add_subnav_item( $nav, 'groups' );
+                        }
                 }
 
                 if ( isset( $this->current_group->user_has_access ) ) {

src/bp-loader.php

@@ -72 +72 @@
         /**
          * @var array The canonical URI stack.
          * @see bp_redirect_canonical()
-         * @see bp_core_new_nav_item()
+         * @see bp_core_add_nav_item()
          */
         public $canonical_stack = array();
 
@@ -109 +109 @@
          */
         public $do_autoload = false;
 
+        /**
+         * Whether to load backward compatibility classes for navigation globals.
+         *
+         * @since 2.6.0
+         * @var bool
+         */
+        public $do_nav_backcompat = false;
+
         /** Option Overload *******************************************************/
 
         /**
@@ -434 +442 @@
                 /** Post types and taxonomies *****************************************/
                 $this->email_post_type     = apply_filters( 'bp_email_post_type', 'bp-email' );
                 $this->email_taxonomy_type = apply_filters( 'bp_email_tax_type', 'bp-email-type' );
+
+                /** Navigation backward compatibility *********************************/
+                if ( interface_exists( 'ArrayAccess', false ) ) {
+                        $this->do_nav_backcompat = true;
+                }
         }
 
         /**

src/bp-members/bp-members-template.php

@@ -1290 +1290 @@
         $bp = buddypress();
 
         // Loop through each navigation item.
-        foreach( (array) $bp->bp_nav as $nav_item ) {
+        foreach( (array) $bp->members->nav->get_primary() as $nav_item ) {
 
                 $selected = '';
 
                 // If the current component matches the nav item id, then add a highlight CSS class.
-                if ( !bp_is_directory() && !empty( $bp->active_components[bp_current_component()] ) && $bp->active_components[bp_current_component()] == $nav_item['css_id'] ) {
+                if ( !bp_is_directory() && !empty( $bp->active_components[bp_current_component()] ) && $bp->active_components[bp_current_component()] == $nav_item->css_id ) {
                         $selected = ' class="current selected"';
                 }
 
@@ -1307 +1307 @@
                         $selected = '';
 
                         if ( bp_is_active( 'friends' ) ) {
-                                if ( $nav_item['css_id'] == $bp->friends->id ) {
+                                if ( $nav_item->css_id == $bp->friends->id ) {
                                         if ( friends_check_friendship( bp_loggedin_user_id(), bp_displayed_user_id() ) ) {
                                                 $selected = ' class="current selected"';
                                         }
@@ -1316 +1316 @@
                 }
 
                 // Echo out the final list item.
-                echo apply_filters_ref_array( 'bp_get_loggedin_user_nav_' . $nav_item['css_id'], array( '<li id="li-nav-' . $nav_item['css_id'] . '" ' . $selected . '><a id="my-' . $nav_item['css_id'] . '" href="' . $nav_item['link'] . '">' . $nav_item['name'] . '</a></li>', &$nav_item ) );
+                echo apply_filters_ref_array( 'bp_get_loggedin_user_nav_' . $nav_item->css_id, array( '<li id="li-nav-' . $nav_item->css_id . '" ' . $selected . '><a id="my-' . $nav_item->css_id . '" href="' . $nav_item->link . '">' . $nav_item->name . '</a></li>', &$nav_item ) );
         }
 
         // Always add a log out list item to the end of the navigation.
@@ -1333 +1333 @@
 function bp_get_displayed_user_nav() {
         $bp = buddypress();
 
-        foreach ( (array) $bp->bp_nav as $user_nav_item ) {
-                if ( empty( $user_nav_item['show_for_displayed_user'] ) && !bp_is_my_profile() )
+        foreach ( $bp->members->nav->get_primary() as $user_nav_item ) {
+                if ( empty( $user_nav_item->show_for_displayed_user ) && ! bp_is_my_profile() ) {
                         continue;
+                }
 
                 $selected = '';
-                if ( bp_is_current_component( $user_nav_item['slug'] ) ) {
+                if ( bp_is_current_component( $user_nav_item->slug ) ) {
                         $selected = ' class="current selected"';
                 }
 
                 if ( bp_loggedin_user_domain() ) {
-                        $link = str_replace( bp_loggedin_user_domain(), bp_displayed_user_domain(), $user_nav_item['link'] );
+                        $link = str_replace( bp_loggedin_user_domain(), bp_displayed_user_domain(), $user_nav_item->link );
                 } else {
-                        $link = trailingslashit( bp_displayed_user_domain() . $user_nav_item['link'] );
+                        $link = trailingslashit( bp_displayed_user_domain() . $user_nav_item->link );
                 }
 
                 /**
@@ -1359 +1360 @@
                  * @param array  $user_nav_item Array holding parts used to construct tab list item.
                  *                              Passed by reference.
                  */
-                echo apply_filters_ref_array( 'bp_get_displayed_user_nav_' . $user_nav_item['css_id'], array( '<li id="' . $user_nav_item['css_id'] . '-personal-li" ' . $selected . '><a id="user-' . $user_nav_item['css_id'] . '" href="' . $link . '">' . $user_nav_item['name'] . '</a></li>', &$user_nav_item ) );
+                echo apply_filters_ref_array( 'bp_get_displayed_user_nav_' . $user_nav_item->css_id, array( '<li id="' . $user_nav_item->css_id . '-personal-li" ' . $selected . '><a id="user-' . $user_nav_item->css_id . '" href="' . $link . '">' . $user_nav_item->name . '</a></li>', &$user_nav_item ) );
         }
 }
 

src/bp-members/classes/class-bp-members-component.php

@@ -150 +150 @@
                 // The domain for the user currently being displayed.
                 $bp->displayed_user->domain   = bp_core_get_user_domain( bp_displayed_user_id() );
 
+                // Backward compatibility for plugins modifying the legacy bp_nav and bp_options_nav global properties.
+                if ( buddypress()->do_nav_backcompat ) {
+                        $bp->bp_nav         = new BP_Core_BP_Nav_BackCompat();
+                        $bp->bp_options_nav = new BP_Core_BP_Options_Nav_BackCompat();
+                }
+
+                // Initialize the nav for the members component.
+                $this->nav = new BP_Core_Nav();
+
                 /** Signup ***********************************************************
                  */
 

src/bp-messages/bp-messages-screens.php

@@ -132 +132 @@
         $class    = ( 0 === $count ) ? 'no-count' : 'count';
         $nav_name = sprintf( __( 'Messages <span class="%s">%s</span>', 'buddypress' ), esc_attr( $class ), bp_core_number_format( $count ) );
 
-        $bp->bp_nav[ $bp->messages->slug ]['name'] = $nav_name;
+        // Edit the Navigation name.
+        $bp->members->nav->edit_nav( array(
+                'name' => $nav_name,
+        ), $bp->messages->slug );
 
         /**
          * Fires right before the loading of the Messages view screen template file.

src/bp-settings/bp-settings-actions.php

@@ -488 +488 @@
  */
 function bp_settings_remove_email_subnav() {
         if ( ! has_action( 'bp_notification_settings' ) ) {
-                bp_core_remove_subnav_item( BP_SETTINGS_SLUG, 'notifications' );
+                bp_core_delete_subnav_item( 'notifications', BP_SETTINGS_SLUG );
         }
 }
 add_action( 'bp_actions', 'bp_settings_remove_email_subnav' );

src/bp-xprofile/classes/class-bp-xprofile-component.php

@@ -287 +287 @@
                 // Get the settings slug.
                 $settings_slug = bp_get_settings_slug();
 
-                bp_core_new_subnav_item( array(
+                bp_core_add_subnav_item( array(
                         'name'            => _x( 'Profile Visibility', 'Profile settings sub nav', 'buddypress' ),
                         'slug'            => 'profile',
                         'parent_url'      => trailingslashit( $user_domain . $settings_slug ),
@@ -295 +295 @@
                         'screen_function' => 'bp_xprofile_screen_settings',
                         'position'        => 30,
                         'user_has_access' => bp_core_can_edit_settings()
-                ) );
+                ), 'members' );
         }
 
         /**

new file tests/phpunit/testcases/core/nav/globalAccess.php

@@ +1 @@
+<?php
+
+/**
+ * @group core
+ * @group nav
+ * @ticket BP6534
+ * @expectedIncorrectUsage bp_nav
+ */
+class BP_Core_Nav_GlobalAccess extends BP_UnitTestCase {
+        protected $bp_nav;
+        protected $bp_options_nav;
+
+        public function setUp() {
+                parent::setUp();
+                $this->bp_nav = buddypress()->bp_nav;
+                $this->bp_options_nav = buddypress()->bp_options_nav;
+        }
+
+        public function tearDown() {
+                buddypress()->bp_nav = $this->bp_nav;
+                buddypress()->bp_options_nav = $this->bp_options_nav;
+                parent::tearDown();
+        }
+
+        protected function create_nav_items() {
+                $this->setExpectedDeprecated( 'bp_core_new_nav_item' );
+                $this->setExpectedDeprecated( 'bp_core_new_subnav_item' );
+
+                bp_core_new_nav_item( array(
+                        'name'                => 'Foo',
+                        'slug'                => 'foo',
+                        'position'            => 25,
+                        'screen_function'     => 'foo_screen_function',
+                        'default_subnav_slug' => 'foo-subnav'
+                ) );
+
+                bp_core_new_subnav_item( array(
+                        'name'            => 'Foo Subnav',
+                        'slug'            => 'foo-subnav',
+                        'parent_url'      => 'example.com/foo',
+                        'parent_slug'     => 'foo',
+                        'screen_function' => 'foo_screen_function',
+                        'position'        => 10
+                ) );
+        }
+
+        /**
+         * Create a group, set up nav item, and go to the group.
+         */
+        protected function set_up_group() {
+                $g = $this->factory->group->create( array(
+                        'slug' => 'testgroup',
+                ) );
+
+                $group = groups_get_group( array( 'group_id' => $g ) );
+                $group_permalink = bp_get_group_permalink( $group );
+
+                $this->go_to( $group_permalink );
+
+                // @todo There is a backward compatibility break with anyone using
+                // 'parent_slug' = group_slug to add a group subnav using
+                // `bp_core_new_subnav_item()`
+                bp_core_add_subnav_item( array(
+                        'name'            => 'Foo',
+                        'slug'            => 'foo',
+                        'parent_url'      => $group_permalink,
+                        'parent_slug'     => 'testgroup',
+                        'screen_function' => 'foo_screen_function',
+                        'position'        => 10
+                ), 'groups' );
+        }
+
+        public function test_bp_nav_isset() {
+                $this->create_nav_items();
+
+                $bp = buddypress();
+
+                $this->assertTrue( isset( $bp->bp_nav ) );
+                $this->assertTrue( isset( $bp->bp_nav['foo'] ) );
+                $this->assertTrue( isset( $bp->bp_nav['foo']['name'] ) );
+        }
+
+        public function test_bp_nav_unset() {
+                $this->create_nav_items();
+
+                $bp = buddypress();
+
+                // No support for this - it would create a malformed nav item.
+                /*
+                unset( $bp->bp_nav['foo']['css_id'] );
+                $this->assertFalse( isset( $bp->bp_nav['foo']['css_id'] ) );
+                */
+
+                unset( $bp->bp_nav['foo'] );
+                $this->assertFalse( isset( $bp->bp_nav['foo'] ) );
+        }
+
+        public function test_bp_nav_get() {
+                $this->create_nav_items();
+
+                $bp = buddypress();
+
+                $foo = $bp->bp_nav['foo'];
+                $this->assertSame( 'Foo', $foo['name'] );
+
+                $this->assertSame( 'Foo', $bp->bp_nav['foo']['name'] );
+        }
+
+        public function test_bp_nav_set() {
+                $this->create_nav_items();
+
+                $bp = buddypress();
+
+                $bp->bp_nav['foo']['name'] = 'Bar';
+
+                $nav = bp_get_nav_menu_items();
+
+                foreach ( $nav as $_nav ) {
+                        if ( 'foo' === $_nav->css_id ) {
+                                $found = $_nav;
+                                break;
+                        }
+                }
+
+                $this->assertSame( 'Bar', $found->name );
+        }
+
+        public function test_bp_options_nav_isset() {
+                $this->create_nav_items();
+
+                $bp = buddypress();
+
+                $this->assertTrue( isset( $bp->bp_options_nav ) );
+                $this->assertTrue( isset( $bp->bp_options_nav['foo'] ) );
+                $this->assertTrue( isset( $bp->bp_options_nav['foo']['foo-subnav'] ) );
+                $this->assertTrue( isset( $bp->bp_options_nav['foo']['foo-subnav']['name'] ) );
+        }
+
+        public function test_bp_options_nav_unset() {
+                $this->create_nav_items();
+
+                $bp = buddypress();
+
+                // No support for this - it would create a malformed nav item.
+                /*
+                unset( $bp->bp_options_nav['foo']['foo-subnav']['user_has_access'] );
+                $this->assertFalse( isset( $bp->bp_options_nav['foo']['foo-subnav']['user_has_access'] ) );
+                */
+
+                unset( $bp->bp_options_nav['foo']['foo-subnav'] );
+                $this->assertFalse( isset( $bp->bp_options_nav['foo']['foo-subnav'] ) );
+
+                // Make sure the parent nav hasn't been wiped out.
+                $this->assertTrue( isset( $bp->bp_options_nav['foo'] ) );
+
+                unset( $bp->bp_options_nav['foo'] );
+                $this->assertFalse( isset( $bp->bp_options_nav['foo'] ) );
+        }
+
+        public function test_bp_options_nav_get() {
+                $this->create_nav_items();
+
+                $bp = buddypress();
+
+                $foo_subnav = $bp->bp_options_nav['foo']['foo-subnav'];
+                $this->assertSame( 'Foo Subnav', $foo_subnav['name'] );
+
+                $this->assertSame( 'Foo Subnav', $bp->bp_options_nav['foo']['foo-subnav']['name'] );
+        }
+
+        public function test_bp_options_nav_set() {
+                $this->create_nav_items();
+
+                $bp = buddypress();
+
+                $bp->bp_options_nav['foo']['foo-subnav']['name'] = 'Bar';
+                $nav = bp_get_nav_menu_items();
+
+                foreach ( $nav as $_nav ) {
+                        if ( 'foo-subnav' === $_nav->css_id ) {
+                                $found = $_nav;
+                                break;
+                        }
+                }
+
+                $this->assertSame( 'Bar', $found->name );
+
+                $subnav = array(
+                        'name' => 'Bar',
+                        'css_id' => 'bar-id',
+                        'link' => 'bar-link',
+                        'slug' => 'bar-slug',
+                        'user_has_access' => true,
+                );
+                $bp->bp_options_nav['foo']['foo-subnav'] = $subnav;
+                $nav = bp_get_nav_menu_items();
+
+                foreach ( $nav as $_nav ) {
+                        if ( 'bar-id' === $_nav->css_id ) {
+                                $found = $_nav;
+                                break;
+                        }
+                }
+
+                $this->assertSame( 'Bar', $found->name );
+        }
+
+        /**
+         * @group groups
+         */
+        public function test_bp_options_nav_isset_group_nav() {
+                $this->set_up_group();
+
+                $bp = buddypress();
+
+                $this->assertTrue( isset( $bp->bp_options_nav ) );
+                $this->assertTrue( isset( $bp->bp_options_nav['testgroup'] ) );
+                $this->assertTrue( isset( $bp->bp_options_nav['testgroup']['foo'] ) );
+                $this->assertTrue( isset( $bp->bp_options_nav['testgroup']['foo']['name'] ) );
+        }
+
+        /**
+         * @group groups
+         */
+        public function test_bp_options_nav_unset_group_nav() {
+                $this->set_up_group();
+
+                $bp = buddypress();
+
+                // No support for this - it would create a malformed nav item.
+                /*
+                unset( $bp->bp_options_nav['testgroup']['foo']['user_has_access'] );
+                $this->assertFalse( isset( $bp->bp_options_nav['testgroup']['foo']['user_has_access'] ) );
+                */
+
+                unset( $bp->bp_options_nav['testgroup']['foo'] );
+                $this->assertFalse( isset( $bp->bp_options_nav['testgroup']['foo'] ) );
+
+                unset( $bp->bp_options_nav['testgroup'] );
+                $this->assertFalse( isset( $bp->bp_options_nav['testgroup'] ) );
+        }
+
+        /**
+         * @group groups
+         */
+        public function test_bp_options_nav_get_group_nav() {
+                $this->set_up_group();
+
+                $bp = buddypress();
+
+                $foo = $bp->bp_options_nav['testgroup']['foo'];
+                $this->assertSame( 'Foo', $foo['name'] );
+
+                $this->assertSame( 'Foo', $bp->bp_options_nav['testgroup']['foo']['name'] );
+        }
+
+        /**
+         * @group groups
+         */
+        public function test_bp_options_nav_set_group_nav() {
+                $this->set_up_group();
+
+                $bp = buddypress();
+
+                $bp->bp_options_nav['testgroup']['foo']['name'] = 'Bar';
+                $nav = bp_get_nav_menu_items( 'groups' );
+
+                foreach ( $nav as $_nav ) {
+                        if ( 'foo' === $_nav->css_id ) {
+                                $found = $_nav;
+                                break;
+                        }
+                }
+
+                $this->assertSame( 'Bar', $found->name );
+
+                $subnav = array(
+                        'name' => 'Bar',
+                        'css_id' => 'bar-id',
+                        'link' => 'bar-link',
+                        'slug' => 'bar-slug',
+                        'user_has_access' => true,
+                );
+                $bp->bp_options_nav['testgroup']['foo'] = $subnav;
+                $nav = bp_get_nav_menu_items( 'groups' );
+
+                foreach ( $nav as $_nav ) {
+                        if ( 'bar-id' === $_nav->css_id ) {
+                                $found = $_nav;
+                                break;
+                        }
+                }
+
+                $this->assertSame( 'Bar', $found->name );
+        }
+}