Ticket #5245 (closed enhancement: fixed)

Opened 4 years ago

Last modified 4 years ago

Get_terms more general sorting in taxonomy.php

Reported by: michelwp Owned by: ryan
Priority: normal Milestone: 2.5
Component: Administration Version: 2.5
Severity: normal Keywords: terms sorting taxonomy
Cc:

Description

taxonomy is a powerful tool than can be used not only for category, post_tag or link_category. I test it to develop multilingual set (theme + plugin) and create language and dictionary taxonomies... Some small bugs was detected (#5230, #5240). To reuse standard stable scripts, it will be good if sorting can be set not only for count, name or term_id but also for slug and why not for term_group.

current code

if ( 'count' == $orderby )
		$orderby = 'tt.count';
	else if ( 'name' == $orderby )
		$orderby = 't.name';
	else
		$orderby = 't.term_id';

expected code

if ( 'count' == $orderby )
		$orderby = 'tt.count';
	else if ( 'name' == $orderby )
		$orderby = 't.name';
	else if ( 'slug' == $orderby )
		$orderby = 't.slug';
	else if ( 'term_group' == $orderby )
		$orderby = 't.term_group';
	else
		$orderby = 't.term_id';

Hope it will be useful for other developers.

Attachments

taxo_diff071023.diff Download (466 bytes) - added by michelwp 4 years ago.
5245-2.diff Download (559 bytes) - added by andy 4 years ago.

Change History

comment:1   ryan4 years ago

  • Status changed from new to closed
  • Resolution set to fixed

(In [6284]) Add option for ordering retrieved terms by slug or term_group. Props michelwp. fixes #5245

comment:2   andy4 years ago

  • Status changed from closed to reopened
  • Version changed from 2.3 to 2.5
  • Resolution fixed deleted

should also apply to wp_get_object_terms()

patch attached

andy4 years ago

comment:3   andy4 years ago

  • Owner changed from anonymous to ryan
  • Status changed from reopened to new

comment:4   ryan4 years ago

  • Status changed from new to closed
  • Resolution set to fixed

(In [6843]) Add more sort options to wp_get_object_terms(). Props andy. fixes #5245

Note: See TracTickets for help on using tickets.