Nemcio/NodeBB
1
0
Fork 0
mirror of https://github.com/NodeBB/NodeBB.git synced 2026-01-01 21:30:30 +01:00
Code Issues Packages Projects Releases Wiki Activity

refactor: closes #12629, allow passing arrays to meta.userOrGroupExists

Browse Source
This commit is contained in:
Barış Soner Uşaklı
2024-06-10 16:59:55 -04:00
parent be86d8efc7
commit bad1564301
3 changed files with 32 additions and 28 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
src/meta/index.js
View File

@@ -24,20 +24,26 @@ Meta.templates = require('./templates');
Meta.blacklist = require('./blacklist');
Meta.languages = require('./languages');
const user = require('../user');
const groups = require('../groups');
/* Assorted */
Meta.userOrGroupExists = async function (slug) {
if (!slug) {
const isArray = Array.isArray(slug);
if ((isArray && slug.some(slug => !slug)) || (!isArray && !slug)) {
throw new Error('[[error:invalid-data]]');
}
const user = require('../user');
const groups = require('../groups');
slug = slugify(slug);
slug = isArray ? slug.map(s => slugify(s, false)) : slugify(slug);
const [userExists, groupExists] = await Promise.all([
user.existsBySlug(slug),
groups.existsBySlug(slug),
]);
return userExists || groupExists;
return isArray ?
slug.map((s, i) => userExists[i] || groupExists[i]):
(userExists || groupExists);
};
if (nconf.get('isPrimary')) {

12
src/user/index.js
View File

@@ -50,8 +50,12 @@ User.exists = async function (uids) {
};
User.existsBySlug = async function (userslug) {
const exists = await User.getUidByUserslug(userslug);
return !!exists;
if (Array.isArray(userslug)) {
const uids = await User.getUidsByUserslugs(userslug);
return uids.map(uid => !!uid);
}
const uid = await User.getUidByUserslug(userslug);
return !!uid;
};
User.getUidsFromSet = async function (set, start, stop) {
@@ -112,6 +116,10 @@ User.getUidByUserslug = async function (userslug) {
return await db.sortedSetScore('userslug:uid', userslug);
};
User.getUidsByUserslugs = async function (userslugs) {
return await db.sortedSetScores('userslug:uid', userslugs);
};
User.getUsernamesByUids = async function (uids) {
const users = await User.getUsersFields(uids, ['username']);
return users.map(user => user.username);

32
test/user.js
View File

@@ -1492,28 +1492,18 @@ describe('User', () => {
});
});
it('should return true if user/group exists', (done) => {
meta.userOrGroupExists('registered-users', (err, exists) => {
assert.ifError(err);
assert(exists);
done();
});
});
it('should return true/false if user/group exists or not', async () => {
assert.strictEqual(await meta.userOrGroupExists('registered-users'), true);
assert.strictEqual(await meta.userOrGroupExists('John Smith'), true);
assert.strictEqual(await meta.userOrGroupExists('doesnot exist'), false);
assert.deepStrictEqual(await meta.userOrGroupExists(['doesnot exist', 'nope not here']), [false, false]);
assert.deepStrictEqual(await meta.userOrGroupExists(['doesnot exist', 'John Smith']), [false, true]);
assert.deepStrictEqual(await meta.userOrGroupExists(['administrators', 'John Smith']), [true, true]);
it('should return true if user/group exists', (done) => {
meta.userOrGroupExists('John Smith', (err, exists) => {
assert.ifError(err);
assert(exists);
done();
});
});
it('should return false if user/group does not exists', (done) => {
meta.userOrGroupExists('doesnot exist', (err, exists) => {
assert.ifError(err);
assert(!exists);
done();
});
await assert.rejects(
meta.userOrGroupExists(['', undefined]),
{ message: '[[error:invalid-data]]' },
);
});
it('should delete user', async () => {