Nemcio/NodeBB
1
0
Fork 0
mirror of https://github.com/NodeBB/NodeBB.git synced 2026-05-07 16:06:10 +02:00
Code Issues Packages Projects Releases Wiki Activity

fix: #10069, don't modify fields array

Browse Source 
use a single setObjectFields for the profile update
This commit is contained in:
Barış Soner Uşaklı
2021-12-08 18:38:02 -05:00
parent c90a89264c
commit a8afdc6019
3 changed files with 18 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
src/user/data.js
View File

@@ -52,11 +52,15 @@ module.exports = function (User) {
uids = uids.map(uid => (isNaN(uid) ? 0 : parseInt(uid, 10)));
const fieldsToRemove = [];
fields = fields.slice();
ensureRequiredFields(fields, fieldsToRemove);
const uniqueUids = _.uniq(uids).filter(uid => uid > 0);
const results = await plugins.hooks.fire('filter:user.whitelistFields', { uids: uids, whitelist: fieldWhitelist.slice() });
const results = await plugins.hooks.fire('filter:user.whitelistFields', {
uids: uids,
whitelist: fieldWhitelist.slice(),
});
if (!fields.length) {
fields = results.whitelist;
} else {