Nemcio/NodeBB
1
0
Fork 0
mirror of https://github.com/NodeBB/NodeBB.git synced 2025-10-26 16:46:12 +01:00
Code Issues Packages Projects Releases Wiki Activity

feat: move filter:topic.post hook to top of method

Browse Source
This commit is contained in:
Julian Lam
2021-09-21 13:00:45 -04:00
parent 7777812e62
commit f194809f06
1 changed files with 5 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
src/topics/create.js
View File

@@ -76,7 +76,8 @@ module.exports = function (Topics) {
};
Topics.post = async function (data) {
let { uid } = data;
data = await plugins.hooks.fire('filter:topic.post', data);
data.title = String(data.title).trim();
data.tags = data.tags || [];
if (data.content) {
@@ -89,8 +90,8 @@ module.exports = function (Topics) {
const [categoryExists, canCreate, canTag] = await Promise.all([
categories.exists(data.cid),
privileges.categories.can('topics:create', data.cid, data.uid),
privileges.categories.can('topics:tag', data.cid, data.uid),
privileges.categories.can('topics:create', data.cid, uid),
privileges.categories.can('topics:tag', data.cid, uid),
]);
if (!categoryExists) {
@@ -103,11 +104,9 @@ module.exports = function (Topics) {
await guestHandleValid(data);
if (!data.fromQueue) {
await user.isReadyToPost(data.uid, data.cid);
await user.isReadyToPost(uid, data.cid);
}
data = await plugins.hooks.fire('filter:topic.post', data);
({ uid } = data); // uid was explicitly destructured at top, so update if changed
const tid = await Topics.create(data);
let postData = data;