提交 f4884e93 authored 作者: hzh's avatar hzh

bug修复

上级 4521c6a9
...@@ -22,9 +22,10 @@ public interface RemoteUserService { ...@@ -22,9 +22,10 @@ public interface RemoteUserService {
* *
* @param username 用户名 * @param username 用户名
* @param tenantId 租户id * @param tenantId 租户id
* @param userType 用户类型
* @return 结果 * @return 结果
*/ */
LoginUser getUserInfo(String username, String tenantId) throws UserException; LoginUser getUserInfo(String username, String tenantId, String userType) throws UserException;
/** /**
* 通过用户id查询用户信息 * 通过用户id查询用户信息
......
...@@ -53,7 +53,7 @@ public abstract class AbstractMallStrategy implements IAuthStrategy { ...@@ -53,7 +53,7 @@ public abstract class AbstractMallStrategy implements IAuthStrategy {
thirdUserId = remoteUserService.registerUserInfo(remoteUserBo); thirdUserId = remoteUserService.registerUserInfo(remoteUserBo);
} catch (Exception e) { } catch (Exception e) {
log.info("用户已存在:{},租户id:{}", remoteUserBo.getUserName(), remoteUserBo.getTenantId()); log.info("用户已存在:{},租户id:{}", remoteUserBo.getUserName(), remoteUserBo.getTenantId());
thirdUserId = remoteUserService.getUserInfo(remoteUserBo.getUserName(), remoteUserBo.getTenantId()).getUserId(); thirdUserId = remoteUserService.getUserInfo(remoteUserBo.getUserName(), remoteUserBo.getTenantId(), UserType.SYS_USER.getUserType()).getUserId();
} }
} }
......
...@@ -14,6 +14,7 @@ import org.dromara.auth.service.SysLoginService; ...@@ -14,6 +14,7 @@ import org.dromara.auth.service.SysLoginService;
import org.dromara.common.core.constant.Constants; import org.dromara.common.core.constant.Constants;
import org.dromara.common.core.constant.GlobalConstants; import org.dromara.common.core.constant.GlobalConstants;
import org.dromara.common.core.enums.LoginType; import org.dromara.common.core.enums.LoginType;
import org.dromara.common.core.enums.UserType;
import org.dromara.common.core.exception.user.CaptchaException; import org.dromara.common.core.exception.user.CaptchaException;
import org.dromara.common.core.exception.user.CaptchaExpireException; import org.dromara.common.core.exception.user.CaptchaExpireException;
import org.dromara.common.core.utils.MessageUtils; import org.dromara.common.core.utils.MessageUtils;
...@@ -67,7 +68,7 @@ public class PasswordAuthStrategy implements IAuthStrategy { ...@@ -67,7 +68,7 @@ public class PasswordAuthStrategy implements IAuthStrategy {
validateCaptcha(tenantId, username, code, uuid); validateCaptcha(tenantId, username, code, uuid);
} }
LoginUser loginUser = TenantHelper.dynamic(tenantId, () -> { LoginUser loginUser = TenantHelper.dynamic(tenantId, () -> {
LoginUser user = remoteUserService.getUserInfo(username, tenantId); LoginUser user = remoteUserService.getUserInfo(username, tenantId, UserType.SYS_USER.getUserType());
loginService.checkLogin(LoginType.PASSWORD, tenantId, username, () -> !BCrypt.checkpw(password, user.getPassword())); loginService.checkLogin(LoginType.PASSWORD, tenantId, username, () -> !BCrypt.checkpw(password, user.getPassword()));
return user; return user;
}); });
......
...@@ -71,9 +71,12 @@ public class RemoteUserServiceImpl implements RemoteUserService { ...@@ -71,9 +71,12 @@ public class RemoteUserServiceImpl implements RemoteUserService {
* @return 结果 * @return 结果
*/ */
@Override @Override
public LoginUser getUserInfo(String username, String tenantId) throws UserException { public LoginUser getUserInfo(String username, String tenantId, String userType) throws UserException {
return TenantHelper.dynamic(tenantId, () -> { return TenantHelper.dynamic(tenantId, () -> {
SysUserVo sysUser = userMapper.selectVoOne(new LambdaQueryWrapper<SysUser>().eq(SysUser::getUserName, username)); SysUserVo sysUser = userMapper.selectVoOne(new LambdaQueryWrapper<SysUser>()
.eq(SysUser::getUserName, username)
.eq(StringUtils.isNotBlank(userType), SysUser::getUserType, userType)
);
if (ObjectUtil.isNull(sysUser)) { if (ObjectUtil.isNull(sysUser)) {
throw new UserException("user.not.exists", username); throw new UserException("user.not.exists", username);
} }
......
...@@ -246,7 +246,7 @@ public class SysUserServiceImpl implements ISysUserService { ...@@ -246,7 +246,7 @@ public class SysUserServiceImpl implements ISysUserService {
public boolean checkUserNameUnique(SysUserBo user) { public boolean checkUserNameUnique(SysUserBo user) {
boolean exist = baseMapper.exists(new LambdaQueryWrapper<SysUser>() boolean exist = baseMapper.exists(new LambdaQueryWrapper<SysUser>()
.eq(SysUser::getUserName, user.getUserName()) .eq(SysUser::getUserName, user.getUserName())
.eq(SysUser::getUserType, UserType.SYS_USER) .eq(SysUser::getUserType, UserType.SYS_USER.getUserType())
.ne(ObjectUtil.isNotNull(user.getUserId()), SysUser::getUserId, user.getUserId())); .ne(ObjectUtil.isNotNull(user.getUserId()), SysUser::getUserId, user.getUserId()));
return !exist; return !exist;
} }
...@@ -260,7 +260,7 @@ public class SysUserServiceImpl implements ISysUserService { ...@@ -260,7 +260,7 @@ public class SysUserServiceImpl implements ISysUserService {
public boolean checkPhoneUnique(SysUserBo user) { public boolean checkPhoneUnique(SysUserBo user) {
boolean exist = baseMapper.exists(new LambdaQueryWrapper<SysUser>() boolean exist = baseMapper.exists(new LambdaQueryWrapper<SysUser>()
.eq(SysUser::getPhonenumber, user.getPhonenumber()) .eq(SysUser::getPhonenumber, user.getPhonenumber())
.eq(SysUser::getUserType, UserType.SYS_USER) .eq(SysUser::getUserType, UserType.SYS_USER.getUserType())
.ne(ObjectUtil.isNotNull(user.getUserId()), SysUser::getUserId, user.getUserId())); .ne(ObjectUtil.isNotNull(user.getUserId()), SysUser::getUserId, user.getUserId()));
return !exist; return !exist;
} }
...@@ -274,7 +274,7 @@ public class SysUserServiceImpl implements ISysUserService { ...@@ -274,7 +274,7 @@ public class SysUserServiceImpl implements ISysUserService {
public boolean checkEmailUnique(SysUserBo user) { public boolean checkEmailUnique(SysUserBo user) {
boolean exist = baseMapper.exists(new LambdaQueryWrapper<SysUser>() boolean exist = baseMapper.exists(new LambdaQueryWrapper<SysUser>()
.eq(SysUser::getEmail, user.getEmail()) .eq(SysUser::getEmail, user.getEmail())
.eq(SysUser::getUserType, UserType.SYS_USER) .eq(SysUser::getUserType, UserType.SYS_USER.getUserType())
.ne(ObjectUtil.isNotNull(user.getUserId()), SysUser::getUserId, user.getUserId())); .ne(ObjectUtil.isNotNull(user.getUserId()), SysUser::getUserId, user.getUserId()));
return !exist; return !exist;
} }
......
Markdown 格式
0%
您添加了 0 到此讨论。请谨慎行事。
请先完成此评论的编辑!
注册 或者 后发表评论