Nemcio/NodeBB
1
0
Fork 0
mirror of https://github.com/NodeBB/NodeBB.git synced 2025-10-31 11:05:54 +01:00
Code Issues Packages Projects Releases Wiki Activity

refactor: get rid of exists db call, use promise.parallel

Browse Source
This commit is contained in:
Barış Soner Uşaklı
2023-09-25 11:08:09 -04:00
parent a0ae31d0a7
commit 8895b152c2
1 changed files with 12 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

23
src/posts/create.js
View File

@@ -10,7 +10,6 @@ const topics = require('../topics');
const categories = require('../categories');
const groups = require('../groups');
const privileges = require('../privileges');
const utils = require('../utils');
module.exports = function (Posts) {
Posts.create = async function (data) {
@@ -26,16 +25,7 @@ module.exports = function (Posts) {
}
if (data.toPid) {
const toPidExists = await Posts.exists(data.toPid);
const toPidDeleted = await Posts.getPostField(data.toPid, 'deleted');
const canViewToPid = await privileges.posts.can('posts:view_deleted', data.toPid, uid);
if (
!utils.isNumber(data.toPid) || !toPidExists ||
(toPidDeleted && !canViewToPid)
) {
throw new Error('[[error:invalid-pid]]');
}
await checkToPid(data.toPid);
}
const pid = await db.incrObjectField('global', 'nextPid');
@@ -90,4 +80,15 @@ module.exports = function (Posts) {
db.incrObjectField(`post:${postData.toPid}`, 'replies'),
]);
}
async function checkToPid(toPid, uid) {
const [toPost, canViewToPid] = await Promise.all([
Posts.getPostFields(toPid, ['pid', 'deleted']),
privileges.posts.can('posts:view_deleted', toPid, uid),
]);
const toPidExists = !!toPost.pid;
if (!toPidExists || (toPost.deleted && !canViewToPid)) {
throw new Error('[[error:invalid-pid]]');
}
}
};