Browse Source

shaped before hand over

itizawa 6 years ago
parent
commit
a6d0e58654
2 changed files with 2 additions and 4 deletions
  1. 0 3
      src/server/models/user.js
  2. 2 1
      src/server/routes/admin.js

+ 0 - 3
src/server/models/user.js

@@ -627,9 +627,6 @@ module.exports = function(crowi) {
     const createUser = async(email) => {
     const createUser = async(email) => {
       const newUser = new User();
       const newUser = new User();
 
 
-      // eslint-disable-next-line no-param-reassign
-      email = email.trim();
-
       // email check
       // email check
       const userData = await User.findOne({ email, userStatus: !STATUS_DELETED });
       const userData = await User.findOne({ email, userStatus: !STATUS_DELETED });
       // The user is exists
       // The user is exists

+ 2 - 1
src/server/routes/admin.js

@@ -456,9 +456,10 @@ module.exports = function(crowi, app) {
 
 
     const array = req.body.emailInputValue.split('\n');
     const array = req.body.emailInputValue.split('\n');
     const emailList = array.filter((element) => { return element.match(/.+@.+\..+/) });
     const emailList = array.filter((element) => { return element.match(/.+@.+\..+/) });
+    const shapedEmailList = emailList.map((email)=>email.trim())
 
 
     try {
     try {
-      await User.createUsersByInvitation(emailList, req.body.sendEmail);
+      await User.createUsersByInvitation(shapedEmailList, req.body.sendEmail);
       return res.json(ApiResponse.success());
       return res.json(ApiResponse.success());
     }
     }
     catch (err) {
     catch (err) {