Пофиксил мелкие баги
This commit is contained in:
parent
2365d240cb
commit
1eb0514cc9
@ -91,10 +91,6 @@ public class AdminUserController {
|
||||
BindingResult bindingResult,
|
||||
Model model,
|
||||
RedirectAttributes redirectAttributes) {
|
||||
if (bindingResult.hasErrors()) {
|
||||
model.addAttribute(PAGE_ATTRIBUTE, page);
|
||||
return USER_EDIT_VIEW;
|
||||
}
|
||||
|
||||
UserEntity userEntity;
|
||||
|
||||
@ -118,24 +114,18 @@ public class AdminUserController {
|
||||
|
||||
if (userService.checkUserWithSameUsernameExist(userEntity)) {
|
||||
bindingResult.rejectValue("username", "userEdit:username", "Пользователь с таким логином уже есть");
|
||||
model.addAttribute(PAGE_ATTRIBUTE, page);
|
||||
userEditDTO.setPasswordConfirm(null);
|
||||
model.addAttribute(USER_ATTRIBUTE, userEditDTO);
|
||||
return USER_EDIT_VIEW;
|
||||
}
|
||||
|
||||
if (userEditDTO.getId() == null && (userEditDTO.getPassword() == null || userEditDTO.getPassword() == "")) {
|
||||
bindingResult.rejectValue("password", "userEdit:passwords", "Укажите пароль");
|
||||
model.addAttribute(PAGE_ATTRIBUTE, page);
|
||||
userEditDTO.setPasswordConfirm(null);
|
||||
model.addAttribute(USER_ATTRIBUTE, userEditDTO);
|
||||
return USER_EDIT_VIEW;
|
||||
}
|
||||
|
||||
if (!Objects.equals(userEditDTO.getPassword(), userEditDTO.getPasswordConfirm())) {
|
||||
bindingResult.rejectValue("password", "userEdit:passwords", "Пароли не совпадают");
|
||||
bindingResult.rejectValue("passwordConfirm", "userEdit:passwords", "Пароли не совпадают");
|
||||
}
|
||||
|
||||
if (bindingResult.hasErrors()) {
|
||||
model.addAttribute(PAGE_ATTRIBUTE, page);
|
||||
userEditDTO.setPasswordConfirm(null);
|
||||
model.addAttribute(USER_ATTRIBUTE, userEditDTO);
|
||||
return USER_EDIT_VIEW;
|
||||
}
|
||||
|
@ -31,13 +31,6 @@ public class UserSignupController {
|
||||
this.userService = userService;
|
||||
}
|
||||
|
||||
private UserEntity toEntity(UserSignupDto dto) {
|
||||
UserEntity userEntity = new UserEntity();
|
||||
userEntity.setUsername(dto.getUsername());
|
||||
userEntity.setPassword(dto.getPassword());
|
||||
return userEntity;
|
||||
}
|
||||
|
||||
@GetMapping
|
||||
public String getSignup(Model model) {
|
||||
model.addAttribute(USER_ATTRIBUTE, new UserSignupDto());
|
||||
@ -49,15 +42,29 @@ public class UserSignupController {
|
||||
@ModelAttribute(name = USER_ATTRIBUTE) @Valid UserSignupDto user,
|
||||
BindingResult bindingResult,
|
||||
Model model) {
|
||||
if (bindingResult.hasErrors()) {
|
||||
return SIGNUP_VIEW;
|
||||
|
||||
UserEntity userEntity = new UserEntity();
|
||||
userEntity.setUsername(user.getUsername());
|
||||
userEntity.setPassword(user.getPassword());
|
||||
|
||||
if (userService.checkUserWithSameUsernameExist(userEntity)) {
|
||||
bindingResult.rejectValue("username", "signup:username", "Пользователь с таким логином уже есть");
|
||||
}
|
||||
|
||||
if (user.getPassword() == null || user.getPassword() == "") {
|
||||
bindingResult.rejectValue("password", "signup:passwords", "Укажите пароль");
|
||||
}
|
||||
|
||||
if (!Objects.equals(user.getPassword(), user.getPasswordConfirm())) {
|
||||
bindingResult.rejectValue("password", "signup:passwords", "Пароли не совпадают.");
|
||||
bindingResult.rejectValue("passwordConfirm", "signup:passwords", "Пароли не совпадают");
|
||||
}
|
||||
|
||||
if (bindingResult.hasErrors()) {
|
||||
model.addAttribute(USER_ATTRIBUTE, user);
|
||||
return SIGNUP_VIEW;
|
||||
}
|
||||
userService.create(toEntity(user));
|
||||
|
||||
userService.create(userEntity);
|
||||
return Constants.REDIRECT_VIEW + Constants.LOGIN_URL + "?signup";
|
||||
}
|
||||
}
|
||||
|
Loading…
x
Reference in New Issue
Block a user