Ticket #8609 (closed defect (bug): fixed)

Opened 3 years ago

Last modified 3 years ago

Tag cloud order priority is broken

Reported by: brh Owned by: anonymous
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

category-template.diff Download (561 bytes) - added by mrmist 3 years ago.
Category template fixes count order by for trunk
category-template27.diff Download (504 bytes) - added by mrmist 3 years ago.
Category template diff fixes count orderby for 2.7

Change History

Tag cloud ordering works for me in 2.7

comment:2   brh3 years ago

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.

mrmist3 years ago

Category template fixes count order by for trunk

mrmist3 years ago

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.

  • Keywords has-patch added; has-patch, needs-testing removed
  • Status changed from new to closed
  • Resolution set to fixed

(In [10316]) Fix ASC/DESC tag ordering by count, props mrmist, fixes #8609 for trunk

(In [10317]) Fix ASC/DESC tag ordering by count, props mrmist, fixes #8609 for 2.7

Note: See TracTickets for help on using tickets.