From b9be17360f5fc7799cc720d40e2e52ee07d3cd38 Mon Sep 17 00:00:00 2001 From: soohwa Date: Sat, 7 Oct 2017 08:33:28 +0900 Subject: Add comment --- models/users.js | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/models/users.js b/models/users.js index 459462e3..6b835d41 100644 --- a/models/users.js +++ b/models/users.js @@ -535,6 +535,12 @@ if (Meteor.isServer) { Users.after.insert((userId, doc) => { if (doc.createdThroughApi) { + // The admin user should be able to create a user despite disabling registration because + // it is two different things (registration and creation). + // So, when a new user is created via the api (only admin user can do that) one must avoid + // the disableRegistration check. + // Issue : https://github.com/wekan/wekan/issues/1232 + // PR : https://github.com/wekan/wekan/pull/1251 Users.update(doc._id, { $set: { createdThroughApi: '' } }); return; } -- cgit v1.2.3-1-g7c22