Alternative Topic Sorting

[baris]

Just committed https://github.com/NodeBB/NodeBB/commit/7610c11cd1b9de53ce185227ab80f86d55d3bd69, this is a pretty big change even though it's just ~300 lines of code. It changes how we store the main post so that alternative sorting works.

If anyone decides to upgrade please make a backup of your database just incase something goes wrong, I have tested it on this forums dataset and it seemed OK but you never know :). Will do more testing as we get closer to 0.5.0.

There are 3 options to sort oldest to newest, newest to oldest and most votes. The sorting can be changed from a dropdown in the bar below the main post and it is saved as a user setting. The admin can also select a default setting from the ACP.

Let us know if you encouter any bugs.

5 barisusakli committed to NodeBB/NodeBB

closes #450

backup database before upgrade!
upgrade script will take the first post of each topic and set the
`mainPid` property on the topic. then it will remove that pid from the
sorted sets for that topic, this was done to make alternative sorting
work.

added a new sorted set called `tid:<id>:posts:votes` that is used to
sort topic posts by vote count, the original sorted set `tid:<id>:posts`
is used to sort by oldest first or newest first.

the main post is added to the returned posts array on topic load and is
always at the top.
theme changes are minimal just a few new data properties on the posts
and the sorting dropdown.
hopefully didn't miss anything too critical.

[A Former User]

Awesome! Is this on mobile too? Can't find it.

[a_5mith]

@zenkamal it is when logged in. You can set the default in admin if you like a specific one.

[julian]

@a_5mith Relevant xkcd

[a_5mith]

@julian I was tempted to charge it for 20 minutes and taking the screenshot again for that exact reason.

Edit: https://i.imgur.com/GmD1WUT.png