WordPress.org

Make WordPress Core

Opened 5 years ago

Closed 5 years ago

#9906 closed defect (bug) (fixed)

get_terms() orderby broken in some cases

Reported by: dd32 Owned by: 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 5 years ago.

Download all attachments as: .zip

Change History (2)

dd325 years ago

comment:1 azaozz5 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.