Showing
1 changed file
with
11 additions
and
7 deletions
@@ -92,7 +92,7 @@ public class YtUserServiceImpl extends AbstractBaseService<UserMapper, User> | @@ -92,7 +92,7 @@ public class YtUserServiceImpl extends AbstractBaseService<UserMapper, User> | ||
92 | boolean userExist; | 92 | boolean userExist; |
93 | if (StringUtils.isNotBlank(userDTO.getPassword())) { | 93 | if (StringUtils.isNotBlank(userDTO.getPassword())) { |
94 | user.setPassword(passwordEncoder.encode(userDTO.getPassword())); | 94 | user.setPassword(passwordEncoder.encode(userDTO.getPassword())); |
95 | - }else { | 95 | + } else { |
96 | user.setPassword(passwordEncoder.encode(FastIotConstants.DEFAULT_PWD)); | 96 | user.setPassword(passwordEncoder.encode(FastIotConstants.DEFAULT_PWD)); |
97 | } | 97 | } |
98 | userExist = | 98 | userExist = |
@@ -162,17 +162,21 @@ public class YtUserServiceImpl extends AbstractBaseService<UserMapper, User> | @@ -162,17 +162,21 @@ public class YtUserServiceImpl extends AbstractBaseService<UserMapper, User> | ||
162 | LambdaQueryWrapper<User> queryWrapper = new LambdaQueryWrapper<>(); | 162 | LambdaQueryWrapper<User> queryWrapper = new LambdaQueryWrapper<>(); |
163 | if (null == userDTO.getId()) { | 163 | if (null == userDTO.getId()) { |
164 | queryWrapper.eq(User::getPhoneNumber, userDTO.getPhoneNumber()); | 164 | queryWrapper.eq(User::getPhoneNumber, userDTO.getPhoneNumber()); |
165 | - queryWrapper.or().eq(User::getEmail, userDTO.getEmail()); | ||
166 | - }else{ | 165 | + queryWrapper |
166 | + .or(StringUtils.isNotEmpty(userDTO.getEmail())) | ||
167 | + .eq(User::getEmail, userDTO.getEmail()); | ||
168 | + } else { | ||
167 | queryWrapper.ne(User::getId, userDTO.getId()); | 169 | queryWrapper.ne(User::getId, userDTO.getId()); |
168 | queryWrapper.and( | 170 | queryWrapper.and( |
169 | (userQueryWrapper -> { | 171 | (userQueryWrapper -> { |
170 | userQueryWrapper.eq(User::getPhoneNumber, userDTO.getPhoneNumber()); | 172 | userQueryWrapper.eq(User::getPhoneNumber, userDTO.getPhoneNumber()); |
171 | - userQueryWrapper.or().eq(User::getEmail, userDTO.getEmail()); | 173 | + userQueryWrapper |
174 | + .or(StringUtils.isNotEmpty(userDTO.getEmail())) | ||
175 | + .eq(User::getEmail, userDTO.getEmail()); | ||
172 | })); | 176 | })); |
173 | } | 177 | } |
174 | List<User> existPhoneOrEmail = baseMapper.selectList(queryWrapper); | 178 | List<User> existPhoneOrEmail = baseMapper.selectList(queryWrapper); |
175 | - if (null != existPhoneOrEmail && existPhoneOrEmail.size() >0) { | 179 | + if (null != existPhoneOrEmail && existPhoneOrEmail.size() > 0) { |
176 | throw new YtDataValidationException(ErrorMessage.PHONE_OR_EMAIL_HAS_REGISTER.getMessage()); | 180 | throw new YtDataValidationException(ErrorMessage.PHONE_OR_EMAIL_HAS_REGISTER.getMessage()); |
177 | } | 181 | } |
178 | } | 182 | } |
@@ -314,7 +318,7 @@ public class YtUserServiceImpl extends AbstractBaseService<UserMapper, User> | @@ -314,7 +318,7 @@ public class YtUserServiceImpl extends AbstractBaseService<UserMapper, User> | ||
314 | if (null != organizationId && (userIds.size() == FastIotConstants.MagicNumber.ZERO)) { | 318 | if (null != organizationId && (userIds.size() == FastIotConstants.MagicNumber.ZERO)) { |
315 | userPage = new Page<>(); | 319 | userPage = new Page<>(); |
316 | } else { | 320 | } else { |
317 | - queryMap.put("isTenantAdmin",true); | 321 | + queryMap.put("isTenantAdmin", true); |
318 | userPage = baseMapper.getUserPage(userIPage, queryMap); | 322 | userPage = baseMapper.getUserPage(userIPage, queryMap); |
319 | } | 323 | } |
320 | } | 324 | } |
@@ -576,7 +580,7 @@ public class YtUserServiceImpl extends AbstractBaseService<UserMapper, User> | @@ -576,7 +580,7 @@ public class YtUserServiceImpl extends AbstractBaseService<UserMapper, User> | ||
576 | UserDTO userDTO = new UserDTO(); | 580 | UserDTO userDTO = new UserDTO(); |
577 | userDTO.setPhoneNumber(phoneNumber); | 581 | userDTO.setPhoneNumber(phoneNumber); |
578 | List<UserDTO> list = baseMapper.findUserInfoByPhoneNumber(userDTO); | 582 | List<UserDTO> list = baseMapper.findUserInfoByPhoneNumber(userDTO); |
579 | - return null !=list && list.size()>0 ? list.get(0):null; | 583 | + return null != list && list.size() > 0 ? list.get(0) : null; |
580 | } | 584 | } |
581 | 585 | ||
582 | @Override | 586 | @Override |