fix: regress. rescheduling shouldn't add to sets that pinning removed… (

#9477) * fix: regress. rescheduling shouldn't add to sets that pinning removed from * test: remove the also from tests
NodeBB · Apr 21, 2021 · 8b79c7f · 8b79c7f
1 parent 427e4f4
commit 8b79c7f
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 2 deletions.
diff --git a/src/topics/scheduled.js b/src/topics/scheduled.js
@@ -64,7 +64,6 @@ Scheduled.reschedule = async function ({ cid, tid, timestamp, uid }) {
 			'topics:scheduled',
 			`uid:${uid}:topics`,
 			'topics:tid',
-			`cid:${cid}:tids`,
 			`cid:${cid}:uid:${uid}:tids`,
 		], timestamp, tid),
 		shiftPostTimes(tid, timestamp),

diff --git a/test/topics.js b/test/topics.js
@@ -2830,7 +2830,6 @@ describe('Topic\'s', () => {
 				'topics:scheduled',
 				`uid:${adminUid}:topics`,
 				'topics:tid',
-				`cid:${topicData.cid}:tids`,
 				`cid:${topicData.cid}:uid:${adminUid}:tids`,
 			], topicData.tid);
 			assert(scores.every(publishTime => publishTime === editedTopic.timestamp));