Changeset 7289 for sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-support/functions.php

Timestamp:

06/09/2018 04:12:50 AM (8 years ago)

Author:

obenland

Message:

Support: Continue using wp_title for bbpress

It seems like titles for bbpress views don't work well with wp_get_document_title().

See [7288], #3539.

File:

• sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-support/functions.php (modified) (2 diffs)

sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-support/functions.php

@@ -137 +137 @@
  */
 function wporg_get_global_header() {
-    $GLOBALS['pagetitle'] = wp_get_document_title();
+    $GLOBALS['pagetitle'] = wp_title( '|', false, 'right' ) . ' ' . __( 'WordPress.org', 'wporg-forums' );
     require WPORGPATH . 'header.php';
 }
@@ -150 +150 @@
 }
 
-
 /**
  * Append an optimized site name.
  *
- * @param array $title {
- *     The document title parts.
- *
- *     @type string $title   Title of the viewed page.
- *     @type string $page    Optional. Page number if paginated.
- *     @type string $tagline Optional. Site description when on home page.
- *     @type string $site    Optional. Site title when not on home page.
- * }
+ * @param array $title Parts of the page title.
  * @return array Filtered title parts.
  */
 function wporg_support_document_title( $title ) {
     if ( is_front_page() ) {
-        $title['title']   = _x( 'Support', 'Site title', 'wporg-forums' );
-        $title['tagline'] = __( 'WordPress.org', 'wporg-forums' );
-    } else {
-        $title['site'] = __( 'WordPress.org', 'wporg-forums' );
+        $title[1] = _x( 'Support', 'Site title', 'wporg-forums' );
     }
 
     return $title;
 }
-add_filter( 'document_title_parts', 'wporg_support_document_title' );
-
-/**
- * Set the separator for the document title.
- *
- * @return string Document title separator.
- */
-function wporg_support_document_title_separator() {
-    return '|';
-}
-add_filter( 'document_title_separator', 'wporg_support_document_title_separator' );
+add_filter( 'wp_title_parts', 'wporg_support_document_title' );
 
 /**