Ticket #8609 (closed defect (bug): fixed)
Tag cloud order priority is broken
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Priority: | normal | Milestone: | 2.7.1 |
| Component: | General | Version: | 2.7 |
| Severity: | normal | Keywords: | has-patch |
| Cc: |
Description
If you specify the order parameter (ASC or DESC) for wp_tag_cloud, you get the opposite ordering you asked for.
On line 654 of category-template.php, there's some code that looks like this:
if ( 'DESC' == $order )
$tags = array_reverse( $tags, true );
But up at line 560, the query that gets the tags always uses DESC. So reversing it gets an ascending order. Line 654 should be changed to "if ( 'ASC' == $order )". Beyond that, having ASC as a default for ordering something on frequency is kind of odd, don't you think?
Attachments
Change History
Try it with these options:
<?php wp_tag_cloud('format=list&orderby=count&order=DESC&number=10'); ?>
You'd expect the most common tags on top, right? You can just look at the code, too - it does a "DESC" ordering in the select, then reverses it if you have the "DESC" order option. Thus it selects the top 10 most popular tags, and then displays them from least popular to most popular.
-
attachment
category-template.diff
added
Category template fixes count order by for trunk
-
attachment
category-template27.diff
added
Category template diff fixes count orderby for 2.7
- Keywords has-patch, needs-testing added
- Milestone changed from 2.8 to 2.7.1
I see what you mean now, I was looking at the default ordering, which is by tag name, and works as expected. Attached fixes orderby COUNT. May need testing.
- Status changed from new to closed
- Resolution set to fixed

Tag cloud ordering works for me in 2.7