WordPress.org
Get WordPress

Make WordPress Core

Opened 17 years ago

Closed 17 years ago

#9906 closed defect (bug) (fixed)

get_terms() orderby broken in some cases

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

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 17 years ago.

Download all attachments as: .zip

Change History (2)

@dd32
17 years ago

#1 @azaozz
17 years ago

  • Resolution → fixed
  • Status new → closed

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

Note: See TracTickets for help on using tickets.