diff --git a/src/wp-includes/general-template.php b/src/wp-includes/general-template.php index 91ae19db43..20855ae2c5 100644 --- a/src/wp-includes/general-template.php +++ b/src/wp-includes/general-template.php @@ -1173,7 +1173,7 @@ function get_archives_link($url, $text, $format = 'html', $before = '', $after = * @param string|array $args Optional. Override defaults. * @return string|null String when retrieving, null when displaying. */ -function wp_get_archives($args = '') { +function wp_get_archives( $args = '' ) { global $wpdb, $wp_locale; $defaults = array( @@ -1184,19 +1184,20 @@ function wp_get_archives($args = '') { ); $r = wp_parse_args( $args, $defaults ); - extract( $r, EXTR_SKIP ); - if ( '' == $type ) - $type = 'monthly'; - - if ( '' != $limit ) { - $limit = absint($limit); - $limit = ' LIMIT '.$limit; + if ( '' == $r['type'] ) { + $r['type'] = 'monthly'; } - $order = strtoupper( $order ); - if ( $order !== 'ASC' ) + if ( '' != $r['limit'] ) { + $r['limit'] = absint( $r['limit'] ); + $r['limit'] = ' LIMIT ' . $r['limit']; + } + + $order = strtoupper( $r['order'] ); + if ( $order !== 'ASC' ) { $order = 'DESC'; + } // this is what will separate dates on weekly archive links $archive_week_separator = '–'; @@ -1211,10 +1212,10 @@ function wp_get_archives($args = '') { $archive_week_start_date_format = 'Y/m/d'; $archive_week_end_date_format = 'Y/m/d'; - if ( !$archive_date_format_over_ride ) { - $archive_day_date_format = get_option('date_format'); - $archive_week_start_date_format = get_option('date_format'); - $archive_week_end_date_format = get_option('date_format'); + if ( ! $archive_date_format_over_ride ) { + $archive_day_date_format = get_option( 'date_format' ); + $archive_week_start_date_format = get_option( 'date_format' ); + $archive_week_end_date_format = get_option( 'date_format' ); } /** @@ -1245,7 +1246,9 @@ function wp_get_archives($args = '') { wp_cache_set( 'last_changed', $last_changed, 'posts' ); } - if ( 'monthly' == $type ) { + $limit = $r['limit']; + + if ( 'monthly' == $r['type'] ) { $query = "SELECT YEAR(post_date) AS `year`, MONTH(post_date) AS `month`, count(ID) as posts FROM $wpdb->posts $join $where GROUP BY YEAR(post_date), MONTH(post_date) ORDER BY post_date $order $limit"; $key = md5( $query ); $key = "wp_get_archives:$key:$last_changed"; @@ -1254,17 +1257,18 @@ function wp_get_archives($args = '') { wp_cache_set( $key, $results, 'posts' ); } if ( $results ) { - $afterafter = $after; + $after = $r['after']; foreach ( (array) $results as $result ) { $url = get_month_link( $result->year, $result->month ); /* translators: 1: month name, 2: 4-digit year */ - $text = sprintf(__('%1$s %2$d'), $wp_locale->get_month($result->month), $result->year); - if ( $show_post_count ) - $after = ' ('.$result->posts.')' . $afterafter; - $output .= get_archives_link($url, $text, $format, $before, $after); + $text = sprintf( __( '%1$s %2$d' ), $wp_locale->get_month( $result->month ), $result->year ); + if ( $r['show_post_count'] ) { + $after = ' ('.$result->posts.')' . $after; + } + $output .= get_archives_link( $url, $text, $r['format'], $r['before'], $after ); } } - } elseif ('yearly' == $type) { + } elseif ( 'yearly' == $r['type'] ) { $query = "SELECT YEAR(post_date) AS `year`, count(ID) as posts FROM $wpdb->posts $join $where GROUP BY YEAR(post_date) ORDER BY post_date $order $limit"; $key = md5( $query ); $key = "wp_get_archives:$key:$last_changed"; @@ -1273,16 +1277,17 @@ function wp_get_archives($args = '') { wp_cache_set( $key, $results, 'posts' ); } if ( $results ) { - $afterafter = $after; + $after = $r['after']; foreach ( (array) $results as $result) { - $url = get_year_link($result->year); - $text = sprintf('%d', $result->year); - if ($show_post_count) - $after = ' ('.$result->posts.')' . $afterafter; - $output .= get_archives_link($url, $text, $format, $before, $after); + $url = get_year_link( $result->year ); + $text = sprintf( '%d', $result->year ); + if ( $r['show_post_count'] ) { + $r['after'] = ' ('.$result->posts.')' . $after; + } + $output .= get_archives_link( $url, $text, $r['format'], $r['before'], $r['after'] ); } } - } elseif ( 'daily' == $type ) { + } elseif ( 'daily' == $r['type'] ) { $query = "SELECT YEAR(post_date) AS `year`, MONTH(post_date) AS `month`, DAYOFMONTH(post_date) AS `dayofmonth`, count(ID) as posts FROM $wpdb->posts $join $where GROUP BY YEAR(post_date), MONTH(post_date), DAYOFMONTH(post_date) ORDER BY post_date $order $limit"; $key = md5( $query ); $key = "wp_get_archives:$key:$last_changed"; @@ -1292,17 +1297,18 @@ function wp_get_archives($args = '') { wp_cache_set( $key, $results, 'posts' ); } if ( $results ) { - $afterafter = $after; + $after = $r['after']; foreach ( (array) $results as $result ) { - $url = get_day_link($result->year, $result->month, $result->dayofmonth); - $date = sprintf('%1$d-%2$02d-%3$02d 00:00:00', $result->year, $result->month, $result->dayofmonth); - $text = mysql2date($archive_day_date_format, $date); - if ($show_post_count) - $after = ' ('.$result->posts.')'.$afterafter; - $output .= get_archives_link($url, $text, $format, $before, $after); + $url = get_day_link( $result->year, $result->month, $result->dayofmonth ); + $date = sprintf( '%1$d-%2$02d-%3$02d 00:00:00', $result->year, $result->month, $result->dayofmonth ); + $text = mysql2date( $archive_day_date_format, $date ); + if ( $r['show_post_count'] ) { + $after = ' (' . $result->posts . ')' . $after; + } + $output .= get_archives_link( $url, $text, $r['format'], $r['before'], $after ); } } - } elseif ( 'weekly' == $type ) { + } elseif ( 'weekly' == $r['type'] ) { $week = _wp_mysql_week( '`post_date`' ); $query = "SELECT DISTINCT $week AS `week`, YEAR( `post_date` ) AS `yr`, DATE_FORMAT( `post_date`, '%Y-%m-%d' ) AS `yyyymmdd`, count( `ID` ) AS `posts` FROM `$wpdb->posts` $join $where GROUP BY $week, YEAR( `post_date` ) ORDER BY `post_date` $order $limit"; $key = md5( $query ); @@ -1312,25 +1318,26 @@ function wp_get_archives($args = '') { wp_cache_set( $key, $results, 'posts' ); } $arc_w_last = ''; - $afterafter = $after; + $after = $r['after']; if ( $results ) { foreach ( (array) $results as $result ) { if ( $result->week != $arc_w_last ) { $arc_year = $result->yr; $arc_w_last = $result->week; - $arc_week = get_weekstartend($result->yyyymmdd, get_option('start_of_week')); - $arc_week_start = date_i18n($archive_week_start_date_format, $arc_week['start']); - $arc_week_end = date_i18n($archive_week_end_date_format, $arc_week['end']); - $url = sprintf('%1$s/%2$s%3$sm%4$s%5$s%6$sw%7$s%8$d', home_url(), '', '?', '=', $arc_year, '&', '=', $result->week); + $arc_week = get_weekstartend( $result->yyyymmdd, get_option( 'start_of_week' ) ); + $arc_week_start = date_i18n( $archive_week_start_date_format, $arc_week['start'] ); + $arc_week_end = date_i18n( $archive_week_end_date_format, $arc_week['end'] ); + $url = sprintf( '%1$s/%2$s%3$sm%4$s%5$s%6$sw%7$s%8$d', home_url(), '', '?', '=', $arc_year, '&', '=', $result->week ); $text = $arc_week_start . $archive_week_separator . $arc_week_end; - if ($show_post_count) - $after = ' ('.$result->posts.')'.$afterafter; - $output .= get_archives_link($url, $text, $format, $before, $after); + if ( $r['show_post_count'] ) { + $after = ' (' . $result->posts . ')' . $after; + } + $output .= get_archives_link( $url, $text, $r['format'], $r['before'], $after ); } } } - } elseif ( ( 'postbypost' == $type ) || ('alpha' == $type) ) { - $orderby = ('alpha' == $type) ? 'post_title ASC ' : 'post_date DESC '; + } elseif ( ( 'postbypost' == $r['type'] ) || ('alpha' == $r['type'] ) ) { + $orderby = ( 'alpha' == $r['type'] ) ? 'post_title ASC ' : 'post_date DESC '; $query = "SELECT * FROM $wpdb->posts $join $where ORDER BY $orderby $limit"; $key = md5( $query ); $key = "wp_get_archives:$key:$last_changed"; @@ -1348,15 +1355,16 @@ function wp_get_archives($args = '') { } else { $text = $result->ID; } - $output .= get_archives_link($url, $text, $format, $before, $after); + $output .= get_archives_link( $url, $text, $r['format'], $r['before'], $r['after'] ); } } } } - if ( $echo ) + if ( $r['echo'] ) { echo $output; - else + } else { return $output; + } } /** @@ -2509,7 +2517,7 @@ function paginate_links( $args = '' ) { $r .= join("\n\t
  • ", $page_links); $r .= "
  • \n\n"; break; - + default : $r = join("\n", $page_links); break; diff --git a/tests/phpunit/tests/functions/getArchives.php b/tests/phpunit/tests/functions/getArchives.php new file mode 100644 index 0000000000..5c3491905c --- /dev/null +++ b/tests/phpunit/tests/functions/getArchives.php @@ -0,0 +1,85 @@ + 'monthly', 'limit' => '', + 'format' => 'html', 'before' => '', + 'after' => '', 'show_post_count' => false, + 'echo' => 1, 'order' => 'DESC', +); +*/ +class Tests_Get_Archives extends WP_UnitTestCase { + protected $post_ids; + protected $month_url; + + function setUp() { + parent::setUp(); + + $this->month_url = get_month_link( date( 'Y' ), date( 'm' ) ); + $this->year_url = get_year_link( date( 'Y' ) ); + $this->post_ids = $this->factory->post->create_many( 8, array( 'post_type' => 'post', 'post_author' => '1' ) ); + } + + function test_wp_get_archives_default() { + $expected['default'] = "
  • " . date( 'F Y' ) . "
  • "; + $this->assertEquals( $expected['default'], trim( wp_get_archives( array( 'echo' => false ) ) ) ); + } + + function test_wp_get_archives_type() { + $expected['type'] = "
  • " . date( 'Y' ) . "
  • "; + $this->assertEquals( $expected['type'], trim( wp_get_archives( array( 'echo' => false, 'type' => 'yearly' ) ) ) ); + } + + function test_wp_get_archives_limit() { + $ids = array_slice( array_reverse( $this->post_ids ), 0, 5 ); + + $expected['limit'] = <<Post title 8 +
  • Post title 7
  • +
  • Post title 6
  • +
  • Post title 5
  • +
  • Post title 4
  • +EOF; + $this->assertEquals( $expected['limit'], trim( wp_get_archives( array( 'echo' => false, 'type' => 'postbypost', 'limit' => 5 ) ) ) ); + } + + function test_wp_get_archives_format() { + $expected['format'] = "'; + $this->assertEquals( $expected['format'], trim( wp_get_archives( array( 'echo' => false, 'format' => 'option' ) ) ) ); + } + + function test_wp_get_archives_before_and_after() { + $expected['before_and_after'] = "
    " . date( 'F Y' ) . '
    '; + $this->assertEquals( $expected['before_and_after'], trim( wp_get_archives( array( 'echo' => false, 'format' => 'custom', 'before' => '
    ', 'after' => '
    ' ) ) ) ); + } + + function test_wp_get_archives_show_post_count() { + $expected['show_post_count'] = "
  • " . date( 'F Y' ) . " (8)
  • "; + $this->assertEquals( $expected['show_post_count'], trim( wp_get_archives( array( 'echo' => false, 'show_post_count' => 1 ) ) ) ); + } + + function test_wp_get_archives_echo() { + $expected['echo'] = "
  • " . date( 'F Y' ) . '
  • '; + ob_start(); + wp_get_archives( array( 'echo' => true ) ); + $actual = ob_get_clean(); + $this->assertEquals( $expected['echo'], trim( $actual ) ); + } + + function test_wp_get_archives_order() { + $this->factory->post->create( array( 'post_type' => 'post', 'post_author' => '1', 'post_date' => '2012-10-23 19:34:42' ) ); + + $date_full = date( 'F Y' ); + $oct_url = get_month_link( 2012, 10 ); + $expected['order_asc'] = <<October 2012 +
  • $date_full
  • +EOF; + $this->assertEquals( $expected['order_asc'], trim( wp_get_archives( array( 'echo' => false, 'order' => 'ASC' ) ) ) ); + + $expected['order_desc'] = <<$date_full +
  • October 2012
  • +EOF; + $this->assertEquals( $expected['order_desc'], trim( wp_get_archives( array( 'echo' => false, 'order' => 'DESC' ) ) ) ); + } +} \ No newline at end of file