refactor: use topics.post, topics.reply, and posts.edit in Create(Note), Create(Note), and Update(Note), respectively

This commit is contained in:
Julian Lam
2024-03-12 14:24:38 -04:00
parent 52c143aa03
commit f7c8967ee2
5 changed files with 27 additions and 78 deletions

View File

@@ -7,7 +7,6 @@ const utils = require('../utils');
const user = require('../user');
const privileges = require('../privileges');
const plugins = require('../plugins');
const activitypub = require('../activitypub');
const Posts = module.exports;
@@ -46,8 +45,9 @@ Posts.getPostsByPids = async function (pids, uid) {
return [];
}
const remotePids = pids.filter(pid => !utils.isNumber(pid));
await activitypub.notes.assert(uid, remotePids);
// todo: remove if not needed (and if this is still commented out after March 2024, then it wasn't needed)
// const remotePids = pids.filter(pid => !utils.isNumber(pid));
// await activitypub.notes.assert(uid, remotePids);
let posts = await Posts.getPostsData(pids);
posts = await Promise.all(posts.map(Posts.parsePost));
const data = await plugins.hooks.fire('filter:post.getPosts', { posts: posts, uid: uid });

View File

@@ -11,6 +11,7 @@ const groups = require('../groups');
const meta = require('../meta');
const plugins = require('../plugins');
const privileges = require('../privileges');
const utils = require('../utils');
module.exports = function (Posts) {
Posts.getUserInfoForPosts = async function (uids, uid) {
@@ -115,10 +116,10 @@ module.exports = function (Posts) {
}
Posts.isOwner = async function (pids, uid) {
uid = parseInt(uid, 10);
uid = utils.isNumber(uid) ? parseInt(uid, 10) : uid;
const isArray = Array.isArray(pids);
pids = isArray ? pids : [pids];
if (uid <= 0) {
if (utils.isNumber(uid) && uid <= 0) {
return isArray ? pids.map(() => false) : false;
}
const postData = await Posts.getPostsFields(pids, ['uid']);