Просмотр исходного кода

Revert "add user upper limit check"

This reverts commit d495efbdc8a91ddb3ab4164684378cf662673efd.
Seiya Tashiro 7 лет назад
Родитель
Сommit
d50132d552
1 измененных файлов с 3 добавлено и 18 удалено
  1. 3 18
      src/server/models/external-account.js

+ 3 - 18
src/server/models/external-account.js

@@ -92,17 +92,9 @@ class ExternalAccount {
               nameToBeRegistered = '';
             }
 
-            User.findAllUsers({status: User.statusActivate})
-            .then(userData => {
-              const userUpperLimit = Number(ExternalAccount.crowi.env['USER_UPPER_LIMIT']);
-              const activeUsers = userData.length;
-              if (userUpperLimit !== 0 && userUpperLimit <= activeUsers) {
-                throw new UserUpperLimitException('ユーザーが上限に達したため登録できません。');
-              }
-              // create a new User with STATUS_ACTIVE
-              debug(`ExternalAccount '${accountId}' is not found, it is going to be registered.`);
-              return User.createUser(nameToBeRegistered, usernameToBeRegistered, mailToBeRegistered, undefined, undefined, User.STATUS_ACTIVE);
-            });
+            // create a new User with STATUS_ACTIVE
+            debug(`ExternalAccount '${accountId}' is not found, it is going to be registered.`);
+            return User.createUser(nameToBeRegistered, usernameToBeRegistered, mailToBeRegistered, undefined, undefined, User.STATUS_ACTIVE);
           })
           .then(newUser => {
             return this.associate(providerType, accountId, newUser);
@@ -163,13 +155,6 @@ class DuplicatedUsernameException {
   }
 }
 
-class UserUpperLimitException {
-  constructor(message) {
-    this.name = this.constructor.name;
-    this.message = message;
-  }
-}
-
 module.exports = function(crowi) {
   ExternalAccount.crowi = crowi;
   schema.loadClass(ExternalAccount);