Changeset 2042

Timestamp:

11/05/2015 06:50:05 PM (9 years ago)

Author:

coffee2code

Message:

apps.wordpress.org: Filter 'document_title_parts' instead of the deprecated 'wp_title' to customize document title. See #1353.

File:

• sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-apps/inc/extras.php (modified) (1 diff)

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

@@ -37 +37 @@
 
 /**
- * Filters wp_title to print a neat <title> tag based on what is being viewed.
+ * Filters document title to add context 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 wpmobileapps_wp_title( $title, $sep ) {
+function wpmobileapps_document_title( $parts ) {
     if ( is_feed() ) {
-        return $title;
+        return $parts;
     }
 
     global $page, $paged;
 
-    // Add the blog name
-    $title .= get_bloginfo( 'name', 'display' );
+    $title = $parts['title'];
+    $sep = '|';
 
     // Add the blog description for the home/front page.
     $site_description = get_bloginfo( 'description', 'display' );
     if ( $site_description && ( is_home() || is_front_page() ) ) {
-        $title .= " $sep $site_description";
+        $title = $site_description;
     }
 
     // Add a page number if necessary:
-    if ( $paged >= 2 || $page >= 2 ) {
-        $title .= " $sep " . sprintf( __( 'Page %s', 'wpmobileapps' ), max( $paged, $page ) );
+    if ( isset( $parts['page'] ) && $parts['page'] >= 2 ) {
+        $title .= " $sep " . sprintf( __( 'Page %s', 'wpmobileapps' ), $parts['page'] );
     }
 
-    return $title;
+    $parts['title'] = $title;
+    return $parts;
 }
-add_filter( 'wp_title', 'wpmobileapps_wp_title', 10, 2 );
+add_filter( 'document_title_parts', 'wpmobileapps_document_title' );
 
 /**