Nemcio/NodeBB
1
0
Fork 0
mirror of https://github.com/NodeBB/NodeBB.git synced 2025-11-03 20:45:58 +01:00
Code Issues Packages Projects Releases Wiki Activity

introducing "sort" querystring parameter in topic route, so that sorting can be overridden via query string.

Browse Source
This commit is contained in:
Julian Lam
2014-09-03 12:49:34 -04:00
parent 2073d994fc
commit 063d270f1a
1 changed files with 10 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
src/controllers/topics.js
View File

@@ -16,6 +16,7 @@ var topicsController = {},
topicsController.get = function(req, res, next) {
var tid = req.params.topic_id,
page = req.query.page || 1,
sort = req.query.sort,
uid = req.user ? req.user.uid : 0,
userPrivileges;
@@ -45,7 +46,15 @@ topicsController.get = function(req, res, next) {
var set = 'tid:' + tid + ':posts',
reverse = false;
if (settings.topicPostSort === 'newest_to_oldest') {
// `sort` qs has priority over user setting
if (sort === 'oldest_to_newest') {
reverse = false;
} else if (sort === 'newest_to_oldest') {
reverse = true;
} else if (sort === 'most_votes') {
reverse = true;
set = 'tid:' + tid + ':posts:votes';
} else if (settings.topicPostSort === 'newest_to_oldest') {
reverse = true;
} else if (settings.topicPostSort === 'most_votes') {
reverse = true;