Ticket #6107: 6107.04.patch

src/bp-core/bp-core-filters.php

@@ -509 +509 @@
  * @param string $seplocation Direction to display title.
  * @return string New page title.
  */
-function bp_modify_page_title( $title, $sep = '', $seplocation = '' ) {
+function bp_modify_page_title( $title, $sep = '»', $seplocation = 'right' ) {
 
         // If this is not a BP page, just return the title produced by WP
         if ( bp_is_blog_page() ) {
@@ -526 +526 @@
                 return $title;
         }
 
-        $bp    = buddypress();
-        $title = '';
+        $bp          = buddypress();
+        $title_parts = array();
 
         // Displayed user
         if ( bp_get_displayed_user_fullname() && ! is_404() ) {
@@ -566 +566 @@
 
                 // 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() ) {
-                        $title = bp_get_displayed_user_fullname();
+                        $title_parts[] = bp_get_displayed_user_fullname();
 
                 // Use component name on member pages
                 } else {
+                        $title_parts = array_merge( $title_parts, array_map( 'strip_tags', array(
+                                bp_get_displayed_user_fullname(),
+                                $component_name,
+                        ) ) );
+
                         // If we have a subnav name, add it separately for localization
                         if ( ! empty( $component_subnav_name ) ) {
-                                // translators: construct the page title. 1 = user name, 2 = component name, 3 = separator, 4 = component subnav name
-                                $title = strip_tags( sprintf( __( '%1$s %3$s %2$s %3$s %4$s', 'buddypress' ), bp_get_displayed_user_fullname(), $component_name, $sep, $component_subnav_name ) );
-
-                        } else {
-                                // translators: construct the page title. 1 = user name, 2 = component name, 3 = separator
-                                $title = strip_tags( sprintf( __( '%1$s %3$s %2$s', 'buddypress' ), bp_get_displayed_user_fullname(), $component_name, $sep ) );
+                                $title_parts[] = strip_tags( $component_subnav_name );
                         }
                 }
 
         // 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'] : '';
-                // translators: 1 = group name, 2 = group nav section name, 3 = separator
-                $title = sprintf( __( '%1$s %3$s %2$s', 'buddypress' ), $bp->bp_options_title, $subnav, $sep );
+                $title_parts = array( $bp->bp_options_title, $subnav );
 
         // A single item from a component other than groups
         } elseif ( bp_is_single_item() ) {
-                // translators: 1 = component item name, 2 = component nav section name, 3 = separator
-                $title = sprintf( __( '%1$s %3$s %2$s', 'buddypress' ), $bp->bp_options_title, $bp->bp_options_nav[ bp_current_item() ][ bp_current_action() ]['name'], $sep );
+                $title_parts = array( $bp->bp_options_title, $bp->bp_options_nav[ bp_current_item() ][ bp_current_action() ]['name'] );
 
         // An index or directory
         } elseif ( bp_is_directory() ) {
                 $current_component = bp_current_component();
 
                 // No current component (when does this happen?)
-                if ( empty( $current_component ) ) {
-                        $title = _x( 'Directory', 'component directory title', 'buddypress' );
-                } else {
-                        $title = bp_get_directory_title( $current_component );
+                $title_parts = array( _x( 'Directory', 'component directory title', 'buddypress' ) );
+
+                if ( ! empty( $current_component ) ) {
+                        $title_parts = array( bp_get_directory_title( $current_component ) );
                 }
 
         // Sign up page
         } elseif ( bp_is_register_page() ) {
-                $title = __( 'Create an Account', 'buddypress' );
+                $title_parts = array( __( 'Create an Account', 'buddypress' ) );
 
         // Activation page
         } elseif ( bp_is_activation_page() ) {
-                $title = __( 'Activate your Account', 'buddypress' );
+                $title_parts = array( __( 'Activate your Account', 'buddypress' ) );
 
         // Group creation page
         } elseif ( bp_is_group_create() ) {
-                $title = __( 'Create a Group', 'buddypress' );
+                $title_parts = array( __( 'Create a Group', 'buddypress' ) );
 
         // Blog creation page
         } elseif ( bp_is_create_blog() ) {
-                $title = __( 'Create a Site', 'buddypress' );
+                $title_parts = array( __( 'Create a Site', 'buddypress' ) );
         }
 
         // Some BP nav items contain item counts. Remove them
-        $title = preg_replace( '|<span>[0-9]+</span>|', '', $title );
-
-        return apply_filters( 'bp_modify_page_title', $title . ' ' . $sep . ' ', $title, $sep, $seplocation );
+        $title_parts = array_map( function( $t ) {
+            return preg_replace( '|<span>[0-9]+</span>|' , '', $t );
+        }, $title_parts);
+
+        // Append the site titl to title parts
+        $title_parts = array_merge( $title_parts, array( get_bloginfo( 'name', 'display' ) ) );
+
+        /**
+         * @param  string the BuddyPress page title
+         * @param  string $title the original WordPress page title
+         * @param  string $sep the title parts separator
+         * @param  string $seplocation Location of the separator (left or right).
+         */
+        return apply_filters( 'bp_modify_page_title', join( ' ' . $sep . ' ', $title_parts ), $title, $sep, $seplocation );
 }
-add_filter( 'wp_title', 'bp_modify_page_title', 10, 3 );
+add_filter( 'wp_title', 'bp_modify_page_title', 20, 3 );
 add_filter( 'bp_modify_page_title', 'wptexturize'     );
 add_filter( 'bp_modify_page_title', 'convert_chars'   );
 add_filter( 'bp_modify_page_title', 'esc_html'        );