Nemcio/NodeBB
1
0
Fork 0
mirror of https://github.com/NodeBB/NodeBB.git synced 2026-02-18 04:27:56 +01:00
Code Issues Packages Projects Releases Wiki Activity

fix: improper handling of single vs. multiple inputs for hidePrivateData

Browse Source
This commit is contained in:
Julian Lam
2022-06-29 16:19:52 -04:00
parent ecf0e8ce22
commit f38b2a737a
1 changed files with 3 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
src/user/data.js
View File

@@ -146,8 +146,10 @@ module.exports = function (User) {
}; };
User.hidePrivateData = async function (users, callerUID) { User.hidePrivateData = async function (users, callerUID) {
let single = false;
if (!Array.isArray(users)) { if (!Array.isArray(users)) {
users = [users]; users = [users];
single = true;
} }
const [userSettings, isAdmin, isGlobalModerator] = await Promise.all([ const [userSettings, isAdmin, isGlobalModerator] = await Promise.all([
@@ -171,7 +173,7 @@ module.exports = function (User) {
return _userData; return _userData;
})); }));
return users.length === 1 ? users.pop() : users; return single ? users.pop() : users;
}; };
async function modifyUserData(users, requestedFields, fieldsToRemove) { async function modifyUserData(users, requestedFields, fieldsToRemove) {