Changeset 7465 for trunk/bp-core/bp-core-component.php

Timestamp:

10/23/2013 02:43:31 PM (13 years ago)

Author:

boonebgorges

Message:

Default to the component directory root_slug in BP_Component::setup_globals()

This change will make BP_Component extensions a bit easier to build, as it will
no longer be necessary to fish the component root_slug out of bp_pages
manually (in the majority of standard cases).

Fixes #3578

File:

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

trunk/bp-core/bp-core-component.php

@@ -172 +172 @@
          *           value is generally unused if the component has a root
          *           directory (the slug will be overridden by the post_name of
-         *           the directory page).
+         *           the directory page). Default: the slug of the directory
+         *           page if one is found, otherwise an empty string.
          *     @type bool $has_directory Set to true if the component requires
          *           an associated WordPress page.
@@ -187 +188 @@
                 /** Slugs *************************************************************/
 
+                // If a WP directory page exists for the component, it should
+                // be the default value of 'root_slug'.
+                $default_root_slug = isset( buddypress()->pages->{$this->id}->slug ) ? buddypress()->pages->{$this->id}->slug : '';
+
                 $r = wp_parse_args( $args, array(
                         'slug'                  => $this->id,
-                        'root_slug'             => '',
+                        'root_slug'             => $default_root_slug,
                         'has_directory'         => false,
                         'notification_callback' => '',