Changeset 6412 for trunk/bp-core/bp-core-buddybar.php

Timestamp:

10/15/2012 07:39:54 PM (12 years ago)

Author:

boonebgorges

Message:

Fixes PHP warning when attempting to remove nav item that doesn't exist

Fixes #4533

Props wpdennis, jkudish

File:

• trunk/bp-core/bp-core-buddybar.php (modified) (2 diffs)

trunk/bp-core/bp-core-buddybar.php

@@ -374 +374 @@
  *
  * @package BuddyPress Core
- * @param $parent_id The id of the parent navigation item.
- * @param $slug The slug of the sub navigation item.
+ * @param int $parent_id The id of the parent navigation item.
+ * @param bool|string false if the parent item doesn't exist or $slug the slug of the sub navigation item.
  */
 function bp_core_remove_nav_item( $parent_id ) {
@@ -386 +386 @@
         }
     }
+
+    if ( empty( $bp->bp_nav[ $parent_id ] ) )
+        return false;
 
     if ( $function = $bp->bp_nav[$parent_id]['screen_function'] ) {