Nemcio/NodeBB
1
0
Fork 0
mirror of https://github.com/NodeBB/NodeBB.git synced 2026-03-18 02:20:49 +01:00
Code Issues Packages Projects Releases Wiki Activity

fix: restored popular calculation behaviour that was broken by e2131d1d2e, removed followingOnly arg passing for popular

Browse Source
This commit is contained in:
Julian Lam
2026-03-13 13:19:29 -04:00
parent 702f7c6251
commit 1af835641e
1 changed files with 10 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

22
src/topics/sorted.js
View File

@@ -46,8 +46,8 @@ module.exports = function (Topics) {
let tids;
if (params.term !== 'alltime') {
if (params.sort === 'posts') {
const { cids, uid, term, includeRemote, followingOnly } = params;
tids = await getTidsWithMostPostsInTerm({ cids, uid, term, includeRemote, followingOnly });
const { cids, uid, term, includeRemote } = params;
tids = await getTidsWithMostPostsInTerm({ cids, uid, term, includeRemote });
} else {
const cids = await getCids(params.cids, params.uid);
tids = await Topics.getLatestTidsFromSet(
@@ -132,26 +132,24 @@ module.exports = function (Topics) {
return cids;
}
async function getTidsWithMostPostsInTerm({ cids, uid, term, includeRemote, followingOnly }) {
async function getTidsWithMostPostsInTerm({ cids, uid, term, includeRemote }) {
cids = await getCids(cids, uid, includeRemote);
const sets = cids.map(cid => `cid:${cid}:tids`);
if (followingOnly && sets.includes('cid:-1:tids')) {
sets.splice(sets.indexOf('cid:-1:tids'), 1, `uid:${uid}:inbox`);
}
const tids = await db.getSortedSetRevRangeByScore(
const sets = cids.map(cid => `cid:${cid}:pids`);
const pids = await db.getSortedSetRevRangeByScore(
sets,
0,
1000,
'+inf',
Date.now() - Topics.getSinceFromTerm(term)
);
const postObjs = await db.getObjectsFields(pids.map(pid => `post:${pid}`), ['tid']);
const tidToCount = {};
tids.forEach((tid) => {
tidToCount[tid] = tidToCount[tid] || 0;
tidToCount[tid] += 1;
postObjs.forEach((post) => {
tidToCount[post.tid] = tidToCount[post.tid] || 0;
tidToCount[post.tid] += 1;
});
return _.uniq(tids)
return _.uniq(postObjs.map(post => String(post.tid)))
.sort((t1, t2) => tidToCount[t2] - tidToCount[t1]);
}