Changeset 2010 for sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-developer/inc/extras.php

Timestamp:

10/26/2015 07:41:51 PM (9 years ago)

Author:

coffee2code

Message:

developer.wordpress.org: Accommodate deprecation of wp_title().

Note: Because the global W.org header requires the title to be passed via $GLOBALS['pagetitle'], add_theme_support( 'title-tag') can't be used.

• Get title via wp_get_document_title() instead of wp_title().
• Filter 'document_title_separator' to customize the title separator.
• Hook 'wporg_developer_wp_title' to filter 'document_title_parts' instead of 'wp_title'.
• Update wporg_developer_wp_title() to account for new args being passed to it.
• Omit 'Home' from title of home page.

Props coffee2code, BandonRandon.
See #1353.

File:

• sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-developer/inc/extras.php (modified) (3 diffs)

sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-developer/inc/extras.php

@@ -39 +39 @@
  * Filters wp_title to print a neat <title> tag based on what is being viewed.
  *
- * @param string $title Default title text for current view.
- * @param string $sep Optional separator.
- * @return string The filtered title.
+ * @param array $parts The document title parts.
+ * @return array The document title parts.
  */
-function wporg_developer_wp_title( $title, $sep ) {
+function wporg_developer_wp_title( $parts ) {
     global $page, $paged;
 
     if ( is_feed() ) {
-        return $title;
+        return $parts;
     }
+
+    $title = $parts['title'];
+    $sep = '|';
 
     $post_type = get_query_var( 'post_type' );
 
+    // Omit 'Home' from the home page.
+    if ( 'Home' === $title ) {
+        $title = '';
+    }
     // Add post type to title if it's a parsed item.
-    if ( is_singular() && \DevHub\is_parsed_post_type( $post_type ) ) {
+    elseif ( is_singular() && \DevHub\is_parsed_post_type( $post_type ) ) {
         if ( $post_type_object = get_post_type_object( $post_type ) ) {
-            $title .= get_post_type_object( $post_type )->labels->singular_name . " $sep ";
+            $title .= " $sep " . get_post_type_object( $post_type )->labels->singular_name;
         }
     }
@@ -61 +67 @@
     elseif ( is_singular() && false !== strpos( $post_type, 'handbook' ) ) {
         if ( $post_type_object = get_post_type_object( $post_type ) ) {
-            $handbook_label = get_post_type_object( $post_type )->labels->name . " $sep ";
-            $handbook_name  = \WPorg_Handbook::get_name( $post_type ) . " Handbook $sep ";
+            $handbook_label = " $sep " . get_post_type_object( $post_type )->labels->name;
+            $handbook_name  = " $sep " . \WPorg_Handbook::get_name( $post_type ) . " Handbook";
 
             // Replace title with handbook name if this is landing page for the handbook
@@ -75 +81 @@
 
     // Add a page number if necessary:
-    if ( $paged >= 2 || $page >= 2 ) {
-        $title .= sprintf( __( 'Page %s', 'wporg' ), max( $paged, $page ) ) . " $sep ";
+    if ( isset( $parts['page'] ) && $parts['page'] >= 2 ) {
+        $title .= " $sep " . sprintf( __( 'Page %s', 'wporg' ), $parts['page'] );
     }
 
-    // Add the blog name
-    $title .= get_bloginfo( 'name' );
-
-    return $title;
+    $parts['title'] = $title;
+    return $parts;
 }
-add_filter( 'wp_title', 'wporg_developer_wp_title', 10, 2 );
+add_filter( 'document_title_parts', 'wporg_developer_wp_title' );
 
 /**