소스 검색

clean code

Yuki Takei 8 년 전
부모
커밋
4f4a8d3044
1개의 변경된 파일2개의 추가작업 그리고 2개의 파일을 삭제
  1. 2 2
      lib/routes/login-passport.js

+ 2 - 2
lib/routes/login-passport.js

@@ -113,7 +113,7 @@ module.exports = function(crowi, app) {
       // find or register(create) user
       ExternalAccount.findOrRegister('ldap', ldapAccountId, usernameToBeRegistered)
         .catch((err) => {
-          if (err.name != null && err.name === 'DuplicatedUsernameException') {
+          if (err.name === 'DuplicatedUsernameException') {
             // get option
             const isSameUsernameTreatedAsIdenticalUser = Config.isSameUsernameTreatedAsIdenticalUser(config, 'ldap');
             if (isSameUsernameTreatedAsIdenticalUser) {
@@ -137,7 +137,7 @@ module.exports = function(crowi, app) {
           });
         })
         .catch((err) => {
-          if (err.name != null && err.name === 'DuplicatedUsernameException') {
+          if (err.name === 'DuplicatedUsernameException') {
             req.flash('isDuplicatedUsernameExceptionOccured', true);
             return next();
           }