Changeset 7690

Timestamp:

12/17/2013 06:32:41 PM (13 years ago)

Author:

boonebgorges

Message:

Bail out of bp_core_add_page_mappings() when no component names have been passed

This prevents PHP notices in certain cases when the automated test suite is
run incorrectly.

File:

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

trunk/bp-core/bp-core-functions.php

@@ -362 +362 @@
  * @since BuddyPress (1.7.0)
  *
- * @param array $default_components Components to create pages for.
+ * @param array $components Components to create pages for.
  * @param string $existing 'delete' if you want to delete existing page
  *        mappings and replace with new ones. Otherwise existing page mappings
@@ -368 +368 @@
  */
 function bp_core_add_page_mappings( $components, $existing = 'keep' ) {
+
+        // If no value is passed, there's nothing to do.
+        if ( empty( $components ) ) {
+                return;
+        }
 
         // Make sure that the pages are created on the root blog no matter which Dashboard the setup is being run on