Skip to:

Opened 9 years ago

Closed 9 years ago

#6118 closed defect (bug) (fixed)

Sorting Blogs alphabetically fails

Reported by: imath's profile imath Owned by: imath's profile imath
Milestone: 2.2 Priority: normal
Severity: normal Version:
Component: Blogs Keywords: has-patch commit


Since r9328 as the table aliases changed and the alphabetical $order_sql clause is still using an old table alias ('bm2' instead of 'bm_name'), the query doesn't get any blog anymore.

Attachments (2)

6118.patch (495 bytes) - added by imath 9 years ago.
6118.unittests.patch (2.3 KB) - added by imath 9 years ago.
BP_Blogs_Blog::get() orderby unit tests

Download all attachments as: .zip

Change History (4)

9 years ago

9 years ago

BP_Blogs_Blog::get() orderby unit tests

#1 @r-a-y
9 years ago

  • Keywords commit added

Nice catch, imath. And thanks for adding a unit test!

#2 @imath
9 years ago

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

In 9354:

Make sure sorting blogs alphabetically is fetching items

Since r9328 we have changed the table aliases used in the sql query to get the blogs, but we forgot to edit the order clause so that the alphabetical type uses the new table alias (bm_name).

Fixes #6118

Note: See TracTickets for help on using tickets.