Changeset 2077

Timestamp:

11/11/2015 08:18:51 PM (9 years ago)

Author:

coffee2code

Message:

W.org Showcase: Handle deprecation of wp_title().

• Get title via wp_get_document_title() instead of wp_title().
• Hook 'document_title_parts' to customize document title rather than doing so in header.php.
• Adjust breadcrumb to use wp_get_document_title() instead of wp_title().
• Filter 'document_title_separator' to customize the title separator.
• Omit front page's title ('Home') from front page document title.

See #1353.

Location:

sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-showcase

Files:

• functions.php (modified) (2 diffs)
• header.php (modified) (1 diff)

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

@@ -152 +152 @@
             <?php endif; // is_category ?>
 
-        <?php echo wp_title(); ?>
+            &raquo; <?php echo wp_get_document_title(); ?>
         <?php endif; // is_search ?>
 
@@ -231 +231 @@
 add_filter( 'excerpt_more', create_function( '$more', 'return "...";' ) );
 
-
+/**
+ * Filters document title to add context based on what is being viewed.
+ *
+ * @param array  $parts The document title parts.
+ * @return array The document title parts.
+ */
+function wporg_showcase_document_title( $parts ) {
+    // wp_get_document_title() is used by the theme in breadcrumb(), whereby it
+    // only really needs the title.
+    if ( did_action( 'wp_head' ) ) {
+        return array( 'title' => $parts['title'] );
+    }
+
+    if ( is_front_page() ) {
+        // Omit page name from the home page.
+        $parts['title'] = '';
+    } elseif ( is_category() ) {
+        // Prepend 'Flavor: ' to category document titles.
+        $parts['title'] = 'Flavor: ' . $parts['title'];
+    } elseif ( is_tag() ) {
+        // Prepend 'Tag: ' to tag document titles.
+        $parts['title'] = 'Tag: ' . $parts['title'];
+    }
+
+    return $parts;
+}
+add_filter( 'document_title_parts', 'wporg_showcase_document_title' );
+
+// Change the document title separator.
+add_filter( 'document_title_separator', create_function( '$separator', 'return "&#8250;";' ) );

sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-showcase/header.php

@@ -1 +1 @@
 <?php
-global $pagetitle;
-
-if ( is_single() )
-    $pagetitle = 'Showcase &raquo; ' . wp_title( '', false );
-elseif ( is_category() )
-    $pagetitle = 'Showcase &raquo; Flavor &raquo; ' . wp_title( '', false );
-elseif ( is_tag() )
-    $pagetitle = 'Showcase &raquo; Tag &raquo; ' . wp_title( '', false );
+$GLOBALS['pagetitle'] = wp_get_document_title();
 
 $prefix = is_ssl() ? 'https://' : 'http://s.';