WordPress.org
Get WordPress

Make WordPress Core

Opened 15 years ago

Closed 15 years ago

#9906 closed defect (bug) (fixed)

get_terms() orderby broken in some cases

Reported by: dd32's profile dd32 Owned by: filosofo's profile filosofo
Milestone: 2.8 Priority: normal
Severity: normal Version: 2.8
Component: Taxonomy Keywords: has-patch
Focuses: Cc:

Description

[11250] introduced a bit of a bug.. $orderby in the past has been set to an upper case 'ID', which was fine, since it droped through to the else case..

Now however.. It doesnt, It drops through and adds a 'ORDER BY ID' remark..

attached patch works off a lower case'd string, as well as fixing one instance where orderby=ID is defaulted to.

Attachments (1)

9906.diff​ (1.4 KB) - added by dd32 15 years ago.

Download all attachments as: .zip

Change History (2)

@dd32
15 years ago

#1 @azaozz
15 years ago

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

(In [11428]) Make get_terms() orderby case insensitive, props DD32, fixes #9906

Note: See TracTickets for help on using tickets.