Changeset 11080 for trunk/src/bp-blogs/classes/class-bp-blogs-component.php

Timestamp:

09/10/2016 02:25:50 PM (8 years ago)

Author:

hnla

Message:

Use WP page titles for BP directory pages headings

The activity directory heading historically used a hardcoded string for it's WP page title (post_title), with other component directories taking the name from the actual WP page title.

This commit brings all page headings for BP directories into parity, screens will look to the WP page(post_title) for the string to use, & BP sets a new series of initial default strings for new installs page creation.

An update routine is provided to ensure that no existing sites will suddenly see a new title string, if a previous install then update the post_title to older hardcoded strings (unless the page title changed by user), thereafter changes to the WP page title determine what BP will display.

Props boonebgorges, imath, dcavins, hnla
Fixes #6765

File:

• trunk/src/bp-blogs/classes/class-bp-blogs-component.php (modified) (5 diffs)

trunk/src/bp-blogs/classes/class-bp-blogs-component.php

@@ -7 +7 @@
  * information from those blogs to make querying blogs in bulk more performant.
  *
- * @package BuddyPress
+    * @package BuddyPress
  * @subpackage BlogsCore
  * @since 1.5.0
@@ -45 +45 @@
      * @since 1.5.0
      *
-     * @see BP_Component::setup_globals() for description of parameters.
+        * @see BP_Component::setup_globals() for description of parameters.
      *
      * @param array $args See {@link BP_Component::setup_globals()}.
@@ -65 +65 @@
             'blog' => $bp->table_prefix . 'bp_user_blogs_blogmeta',
         );
+
+        // Fetch the default directory title.
+        $default_directory_titles = bp_core_get_directory_page_default_titles();
+        $default_directory_title  = $default_directory_titles[$this->id];
 
         // All globals for blogs component.
@@ -71 +75 @@
             'root_slug'             => isset( $bp->pages->blogs->slug ) ? $bp->pages->blogs->slug : BP_BLOGS_SLUG,
             'has_directory'         => is_multisite(), // Non-multisite installs don't need a top-level Sites directory, since there's only one site.
-            'directory_title'       => _x( 'Sites', 'component directory title', 'buddypress' ),
+            'directory_title'       => isset( $bp->pages->blogs->title ) ? $bp->pages->blogs->title : $default_directory_title,
             'notification_callback' => 'bp_blogs_format_notifications',
             'search_string'         => __( 'Search sites...', 'buddypress' ),
@@ -151 +155 @@
      * @param array $main_nav Optional. See BP_Component::setup_nav() for
      *                        description.
-     * @param array $sub_nav  Optional. See BP_Component::setup_nav() for
+        * @param array $sub_nav  Optional. See BP_Component::setup_nav() for
      *                        description.
      */