golem
/
twentyGOLEM
9
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Make wp_title() sort titles based on separator direction. Reverse the titles in bundled theme. Better SEO. fixes #7338 

git-svn-id: http://svn.automattic.com/wordpress/trunk@9376 1a063a9b-81f0-0310-95a4-ce76da25c4cd
This commit is contained in:
markjaquith 2008-10-28 04:53:52 +00:00
parent 7699e2d46b
commit acfecce577
3 changed files with 17 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
wp-content/themes/classic/header.php
View File

@ -10,7 +10,7 @@
<head profile="http://gmpg.org/xfn/11">
<meta http-equiv="Content-Type" content="<?php bloginfo('html_type'); ?>; charset=<?php bloginfo('charset'); ?>" />
<title><?php bloginfo('name'); ?><?php wp_title(); ?></title>
<title><?php wp_title('&laquo;', true, 'right'); ?> <?php bloginfo('name'); ?></title>
<style type="text/css" media="screen">
@import url( <?php bloginfo('stylesheet_url'); ?> );

2
wp-content/themes/default/header.php
View File

@ -10,7 +10,7 @@
<head profile="http://gmpg.org/xfn/11">
<meta http-equiv="Content-Type" content="<?php bloginfo('html_type'); ?>; charset=<?php bloginfo('charset'); ?>" />
<title><?php bloginfo('name'); ?> <?php if ( is_single() ) { ?> &raquo; Blog Archive <?php } ?> <?php wp_title(); ?></title>
<title><?php wp_title('&laquo;', true, 'right'); ?> <?php bloginfo('name'); ?></title>
<link rel="stylesheet" href="<?php bloginfo('stylesheet_url'); ?>" type="text/css" media="screen" />
<link rel="alternate" type="application/rss+xml" title="<?php bloginfo('name'); ?> RSS Feed" href="<?php bloginfo('rss2_url'); ?>" />

27
wp-includes/general-template.php
View File

@ -377,6 +377,8 @@ function wp_title($sep = '&raquo;', $display = true, $seplocation = '') {
$day = get_query_var('day');
$title = '';
$t_sep = '%WP_TITILE_SEP%'; // Temporary separator, for accurate flipping, if necessary
// If there's a category
if ( !empty($cat) ) {
// category exclusion
@ -418,15 +420,15 @@ function wp_title($sep = '&raquo;', $display = true, $seplocation = '') {
$my_year = substr($m, 0, 4);
$my_month = $wp_locale->get_month(substr($m, 4, 2));
$my_day = intval(substr($m, 6, 2));
$title = "$my_year" . ($my_month ? "$sep $my_month" : "") . ($my_day ? "$sep $my_day" : "");
$title = "$my_year" . ($my_month ? "$t_sep$my_month" : "") . ($my_day ? "$t_sep$my_day" : "");
}
if ( !empty($year) ) {
$title = $year;
if ( !empty($monthnum) )
$title .= " $sep " . $wp_locale->get_month($monthnum);
$title .= "$t_sep" . $wp_locale->get_month($monthnum);
if ( !empty($day) )
$title .= " $sep " . zeroise($day, 2);
$title .= "$t_sep" . zeroise($day, 2);
}
// If there is a post
@ -442,10 +444,7 @@ function wp_title($sep = '&raquo;', $display = true, $seplocation = '') {
$tax = $tax->label;
$term = $wp_query->get_queried_object();
$term = $term->name;
if ( 'right' == $seplocation )
$title = "$term $sep $tax";
else
$title = "$tax $sep $term";
$title = "$tax$t_sep$term";
}
if ( is_404() ) {
@ -456,11 +455,15 @@ function wp_title($sep = '&raquo;', $display = true, $seplocation = '') {
if ( !empty($title) )
$prefix = " $sep ";
// Determines position of the separator
if ( 'right' == $seplocation )
$title = $title . $prefix;
else
$title = $prefix . $title;
// Determines position of the separator and direction of the breadcrumb
if ( 'right' == $seplocation ) { // sep on right, so reverse the order
$title_array = explode( $t_sep, $title );
$title_array = array_reverse( $title_array );
$title = implode( " $sep ", $title_array ) . $prefix;
} else {
$title_array = explode( $t_sep, $title );
$title = $prefix . implode( " $sep ", $title_array );
}
$title = apply_filters('wp_title', $title, $sep, $seplocation);