ORDER BY needs commas between clauses when ordering by multiple fields.
Test included.

CommentFileSizeAuthor
database.multiple-orderby.patch2.45 KBbdragon

Comments

Crell’s picture

Crell commented
Status: Needs review » Reviewed & tested by the community

Bug confirmed. Code looks good. Includes unit test. :-) All DB unit tests pass.

damien tournoud’s picture

damien tournoud commented

Looks great, for those wondering at home: no need to test this for PostgreSQL, as this is really standard SQL.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Reviewed & tested by the community » Fixed

Looks like this was accidentally committed along with the ereg -> preg stuff. Anyway, fixed now. ;)

bdragon’s picture

bdragon commented

For reference:
http://drupal.org/cvs?commit=138143

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.