Nemcio/NodeBB
1
0
Fork 0
mirror of https://github.com/NodeBB/NodeBB.git synced 2026-01-01 21:30:30 +01:00
Code Issues Packages Projects Releases Wiki Activity

feat: add methods for adding/removing tags from topics

Browse Source
This commit is contained in:
Barış Soner Uşaklı
2020-05-12 23:00:45 -04:00
parent e20ad5c56e
commit bfad457275
2 changed files with 43 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

32
src/topics/tags.js
View File

@@ -152,12 +152,15 @@ module.exports = function (Topics) {
};
Topics.getTopicTags = async function (tid) {
return await db.getSetMembers('topic:' + tid + ':tags');
const tags = await db.getSetMembers('topic:' + tid + ':tags');
return tags.sort();
};
Topics.getTopicsTags = async function (tids) {
const keys = tids.map(tid => 'topic:' + tid + ':tags');
return await db.getSetsMembers(keys);
const tags = await db.getSetsMembers(keys);
tags.forEach(tags => tags.sort());
return tags;
};
Topics.getTopicTagsObjects = async function (tid) {
@@ -192,6 +195,31 @@ module.exports = function (Topics) {
return topicTags;
};
Topics.addTags = async function (tags, tids) {
const topicData = await Topics.getTopicsFields(tids, ['timestamp']);
const sets = tids.map(tid => 'topic:' + tid + ':tags');
for (let i = 0; i < tags.length; i++) {
/* eslint-disable no-await-in-loop */
await Promise.all([
db.setsAdd(sets, tags[i]),
db.sortedSetAdd('tag:' + tags[i] + ':topics', topicData.map(t => t.timestamp), tids),
]);
await updateTagCount(tags[i]);
}
};
Topics.removeTags = async function (tags, tids) {
const sets = tids.map(tid => 'topic:' + tid + ':tags');
for (let i = 0; i < tags.length; i++) {
/* eslint-disable no-await-in-loop */
await Promise.all([
db.setsRemove(sets, tags[i]),
db.sortedSetRemove('tag:' + tags[i] + ':topics', tids),
]);
await updateTagCount(tags[i]);
}
};
Topics.updateTopicTags = async function (tid, tags) {
await Topics.deleteTopicTags(tid);
const timestamp = await Topics.getTopicField(tid, 'timestamp');

13
test/topics.js
View File

@@ -1753,6 +1753,19 @@ describe('Topic\'s', function () {
});
});
});
it('should add and remove tags from topics properly', async () => {
const result = await topics.post({ uid: adminUid, tags: ['tag4', 'tag2', 'tag1', 'tag3'], title: 'tag topic', content: 'topic 1 content', cid: topic.categoryId });
const tid = result.topicData.tid;
let tags = await topics.getTopicTags(tid);
assert.deepStrictEqual(tags, ['tag1', 'tag2', 'tag3', 'tag4']);
await topics.addTags(['tag7', 'tag6', 'tag5'], [tid]);
tags = await topics.getTopicTags(tid);
assert.deepStrictEqual(tags, ['tag1', 'tag2', 'tag3', 'tag4', 'tag5', 'tag6', 'tag7']);
await topics.removeTags(['tag1', 'tag3', 'tag5', 'tag7'], [tid]);
tags = await topics.getTopicTags(tid);
assert.deepStrictEqual(tags, ['tag2', 'tag4', 'tag6']);
});
});
describe('follow/unfollow', function () {