Content server: Fix regression that broke browsing by rating

This commit is contained in:
Kovid Goyal 2010-12-18 10:03:56 -07:00
parent d45398fe47
commit 3d03f5c4a4

View File

@ -373,13 +373,18 @@ class BrowseServer(object):
script='toplevel();', main=main)
def browse_sort_categories(self, items, sort):
if sort not in ('rating', 'name', 'popularity'):
sort = 'name'
items.sort(key=lambda x: sort_key(getattr(x, 'sort', x.name)))
def keyg(x):
x = getattr(x, 'sort', x.name)
if isinstance(x, unicode):
return sort_key(x)
return x
if sort == 'popularity':
items.sort(key=operator.attrgetter('count'), reverse=True)
elif sort == 'rating':
items.sort(key=operator.attrgetter('avg_rating'), reverse=True)
else:
items.sort(key=keyg)
return sort
def browse_category(self, category, sort):