diff options
author | Yorhel <git@yorhel.nl> | 2009-10-17 14:20:06 +0200 |
---|---|---|
committer | Yorhel <git@yorhel.nl> | 2009-10-17 14:32:24 +0200 |
commit | e7e53553cb42824b7360ebbc15bb86aaa015c677 (patch) | |
tree | 1bd797552e6d1fe8d165e568dc3e1fad5cbf53e3 | |
parent | c816a8a462e925f8ef8133241c29d5d53afdce64 (diff) |
Handler::ULists: Add secundary order by title when ordered by vote
This fixes a strange issue that the ordering changes when browsing to
the next page, resulting in some VNs not showing up at all.
-rw-r--r-- | ChangeLog | 1 | ||||
-rw-r--r-- | lib/VNDB/Handler/ULists.pm | 2 |
2 files changed, 2 insertions, 1 deletions
@@ -21,6 +21,7 @@ git - ? - Optimized dbUserGet (mostly for the user list) - All languages are listed on /r and /v/all instead of only those in use - Copy over search query when switching search type (htmlSearchBox) + - Fixed obscure sorting bug on user VN list 2.7 - 2009-09-24 - Improved styling of the threeboxes layout diff --git a/lib/VNDB/Handler/ULists.pm b/lib/VNDB/Handler/ULists.pm index 19db9f1e..6a0d6c9e 100644 --- a/lib/VNDB/Handler/ULists.pm +++ b/lib/VNDB/Handler/ULists.pm @@ -229,7 +229,7 @@ sub vnlist { uid => $uid, results => 50, page => $f->{p}, - order => $f->{s}.' '.($f->{o} eq 'd' ? 'DESC' : 'ASC'), + order => $f->{s}.' '.($f->{o} eq 'd' ? 'DESC' : 'ASC').($f->{s} eq 'vote' ? ', title ASC' : ''), voted => $f->{v}, $f->{c} ne 'all' ? (char => $f->{c}) : (), ); |