Compare commits
No commits in common. "dev" and "master" have entirely different histories.
18 changed files with 67 additions and 188 deletions
|
|
@ -41,7 +41,6 @@ dependencies {
|
|||
implementation("org.springframework.boot:spring-boot-starter-data-jpa")
|
||||
implementation("org.springframework.boot:spring-boot-starter-security")
|
||||
implementation("org.springframework.boot:spring-boot-starter-web")
|
||||
implementation("org.springframework.boot:spring-boot-starter-validation")
|
||||
implementation("org.flywaydb:flyway-core")
|
||||
implementation("org.flywaydb:flyway-database-postgresql:11.10.0")
|
||||
implementation("org.postgresql:postgresql")
|
||||
|
|
|
|||
|
|
@ -1,8 +1,6 @@
|
|||
package ru.team58.profileservice.controller;
|
||||
|
||||
import jakarta.validation.Valid;
|
||||
import org.springframework.beans.factory.annotation.Autowired;
|
||||
import org.springframework.http.MediaType;
|
||||
import org.springframework.web.bind.annotation.*;
|
||||
import ru.team58.profileservice.controller.exceptions.BadRequestException;
|
||||
import ru.team58.profileservice.controller.exceptions.UnauthorizedException;
|
||||
|
|
@ -11,7 +9,7 @@ import ru.team58.profileservice.controller.schemes.CreateUserRequest;
|
|||
import ru.team58.profileservice.controller.schemes.GetUserResponse;
|
||||
import ru.team58.profileservice.controller.schemes.UpdateUserRequest;
|
||||
import ru.team58.profileservice.controller.schemes.UserResponse;
|
||||
import ru.team58.profileservice.mapper.*;
|
||||
import ru.team58.profileservice.mapper.UserMapper;
|
||||
import ru.team58.profileservice.service.UserDTO;
|
||||
import ru.team58.profileservice.service.UserService;
|
||||
import ru.team58.profileservice.service.exceptions.UserAlreadyExistsException;
|
||||
|
|
@ -36,11 +34,11 @@ public class UserController {
|
|||
throw new UserNotFoundException();
|
||||
}
|
||||
|
||||
return UserResponseMapper.INSTANCE.toUserResponse(userService.getByUsername(principal.getName()));
|
||||
return UserMapper.INSTANCE.toUserResponse(userService.getByUsername(principal.getName()));
|
||||
}
|
||||
|
||||
@PutMapping(value = "/", consumes = {MediaType.APPLICATION_JSON_VALUE})
|
||||
public void updateMe(Principal principal, @Valid @RequestBody UpdateUserRequest request) {
|
||||
@PutMapping("/")
|
||||
public void updateMe(Principal principal, @RequestBody UpdateUserRequest request) {
|
||||
if (principal == null) {
|
||||
throw new UnauthorizedException();
|
||||
}
|
||||
|
|
@ -51,7 +49,7 @@ public class UserController {
|
|||
throw new UserNotFoundException();
|
||||
}
|
||||
|
||||
userService.updateUser(UpdateUserMapper.INSTANCE.toUserDTO(request));
|
||||
userService.updateUser(UserMapper.INSTANCE.toUserDTO(request));
|
||||
}
|
||||
|
||||
@DeleteMapping("/")
|
||||
|
|
@ -71,9 +69,9 @@ public class UserController {
|
|||
userService.deleteUser(user.getId());
|
||||
}
|
||||
|
||||
@PostMapping(value = "/", consumes = MediaType.APPLICATION_JSON_VALUE)
|
||||
public void createMe(@Valid @RequestBody CreateUserRequest request) {
|
||||
UserDTO user = CreateUserMapper.INSTANCE.toUserDTO(request);
|
||||
@PostMapping("/")
|
||||
public void createMe(@RequestBody CreateUserRequest request) {
|
||||
UserDTO user = UserMapper.INSTANCE.toUserDTO(request);
|
||||
|
||||
try {
|
||||
userService.createUser(user);
|
||||
|
|
@ -94,7 +92,7 @@ public class UserController {
|
|||
throw new UserNotFoundException();
|
||||
}
|
||||
|
||||
return GetUserMapper.INSTANCE.toGetUserResponse(user);
|
||||
return UserMapper.INSTANCE.toGetUserResponse(user);
|
||||
}
|
||||
|
||||
@DeleteMapping("/{id}")
|
||||
|
|
|
|||
|
|
@ -2,16 +2,10 @@ package ru.team58.profileservice.controller.advice;
|
|||
|
||||
import org.springframework.http.HttpStatus;
|
||||
import org.springframework.http.ResponseEntity;
|
||||
import org.springframework.validation.FieldError;
|
||||
import org.springframework.web.bind.MethodArgumentNotValidException;
|
||||
import org.springframework.web.bind.annotation.ControllerAdvice;
|
||||
import org.springframework.web.bind.annotation.ExceptionHandler;
|
||||
import org.springframework.web.bind.annotation.ResponseStatus;
|
||||
import ru.team58.profileservice.controller.exceptions.*;
|
||||
|
||||
import java.util.HashMap;
|
||||
import java.util.Map;
|
||||
|
||||
@ControllerAdvice
|
||||
public class UserControllerExceptionHandler {
|
||||
@ExceptionHandler(UserNotFoundException.class)
|
||||
|
|
@ -43,16 +37,4 @@ public class UserControllerExceptionHandler {
|
|||
ErrorDetails errorDetails = new ErrorDetails("Unauthorized", HttpStatus.UNAUTHORIZED.value());
|
||||
return new ResponseEntity<>(errorDetails, HttpStatus.UNAUTHORIZED);
|
||||
}
|
||||
|
||||
@ExceptionHandler(MethodArgumentNotValidException.class)
|
||||
public ResponseEntity<Map<String, String>> handleValidationExceptions(
|
||||
MethodArgumentNotValidException ex) {
|
||||
Map<String, String> errors = new HashMap<>();
|
||||
ex.getBindingResult().getAllErrors().forEach((error) -> {
|
||||
String fieldName = ((FieldError) error).getField();
|
||||
String errorMessage = error.getDefaultMessage();
|
||||
errors.put(fieldName, errorMessage);
|
||||
});
|
||||
return new ResponseEntity<>(errors, HttpStatus.BAD_REQUEST);
|
||||
}
|
||||
}
|
||||
|
|
|
|||
|
|
@ -1,4 +0,0 @@
|
|||
package ru.team58.profileservice.controller.exceptions;
|
||||
|
||||
public class IncorrectEmailException extends BadRequestException {
|
||||
}
|
||||
|
|
@ -1,10 +1,15 @@
|
|||
package ru.team58.profileservice.controller.schemes;
|
||||
|
||||
import lombok.*;
|
||||
import lombok.experimental.SuperBuilder;
|
||||
import lombok.Builder;
|
||||
import lombok.Getter;
|
||||
import lombok.Setter;
|
||||
|
||||
@Getter
|
||||
@Setter
|
||||
@SuperBuilder
|
||||
@NoArgsConstructor
|
||||
public class CreateUserRequest extends UserScheme {}
|
||||
@Builder
|
||||
public class CreateUserRequest {
|
||||
String username;
|
||||
String firstName;
|
||||
String lastName;
|
||||
String email;
|
||||
}
|
||||
|
|
|
|||
|
|
@ -1,13 +1,18 @@
|
|||
package ru.team58.profileservice.controller.schemes;
|
||||
|
||||
import lombok.*;
|
||||
import lombok.experimental.SuperBuilder;
|
||||
import lombok.Builder;
|
||||
import lombok.Getter;
|
||||
import lombok.Setter;
|
||||
|
||||
import java.util.UUID;
|
||||
|
||||
@Getter
|
||||
@Setter
|
||||
@SuperBuilder
|
||||
@NoArgsConstructor
|
||||
public class UpdateUserRequest extends UserScheme {
|
||||
@Builder
|
||||
public class UpdateUserRequest {
|
||||
UUID id;
|
||||
String username;
|
||||
String firstName;
|
||||
String lastName;
|
||||
String email;
|
||||
}
|
||||
|
|
|
|||
|
|
@ -1,40 +0,0 @@
|
|||
package ru.team58.profileservice.controller.schemes;
|
||||
|
||||
import jakarta.validation.constraints.*;
|
||||
import lombok.AllArgsConstructor;
|
||||
import lombok.Getter;
|
||||
import lombok.NoArgsConstructor;
|
||||
import lombok.Setter;
|
||||
import lombok.experimental.SuperBuilder;
|
||||
|
||||
import java.util.UUID;
|
||||
|
||||
@SuperBuilder
|
||||
@Getter
|
||||
@Setter
|
||||
@NoArgsConstructor
|
||||
@AllArgsConstructor
|
||||
public class UserScheme {
|
||||
private UUID id;
|
||||
|
||||
@NotBlank(message = "Username cannot be empty")
|
||||
@Pattern(regexp = "^[a-z0-9.]+$", message = "Username can have only characters: a-z and 0-9")
|
||||
@Size(min = 3, max = 14, message = "Username must be bigger than 3 and less than 14")
|
||||
private String username;
|
||||
|
||||
@NotBlank(message = "First name cannot be empty")
|
||||
@Pattern(regexp = "^[a-zA-Zа-яА-Я]*$", message = "First name can contain only letters")
|
||||
@Size(min = 3, max = 255, message = "First name must be bigger than 3 and less than 255")
|
||||
private String firstName;
|
||||
|
||||
@NotBlank(message = "Last name cannot be empty")
|
||||
@Pattern(regexp = "^[a-zA-Zа-яА-Я]*$", message = "Last name can contain only letters")
|
||||
@Size(min = 3, max = 255, message = "Last name must be bigger than 3 and less than 255")
|
||||
private String lastName;
|
||||
|
||||
@Email(regexp = "[a-z0-9._%+-]+@[a-z0-9.-]+\\.[a-z]{2,3}", message = """
|
||||
Email can contain: a-z, 0-9, '.', '_', '%', '+', '-'.
|
||||
And have format like: "example@domain.com".""")
|
||||
@Size(min = 5, max = 255, message = "Email must be bigger than 5 and less than 255")
|
||||
private String email;
|
||||
}
|
||||
|
|
@ -1,16 +0,0 @@
|
|||
package ru.team58.profileservice.mapper;
|
||||
|
||||
import org.mapstruct.Mapper;
|
||||
import org.mapstruct.Mapping;
|
||||
import org.mapstruct.factory.Mappers;
|
||||
import ru.team58.profileservice.controller.schemes.CreateUserRequest;
|
||||
import ru.team58.profileservice.service.UserDTO;
|
||||
|
||||
@Mapper
|
||||
public interface CreateUserMapper {
|
||||
CreateUserMapper INSTANCE = Mappers.getMapper(CreateUserMapper.class);
|
||||
|
||||
@Mapping(target = "id", ignore = true)
|
||||
UserDTO toUserDTO(CreateUserRequest request);
|
||||
|
||||
}
|
||||
|
|
@ -1,13 +0,0 @@
|
|||
package ru.team58.profileservice.mapper;
|
||||
|
||||
import org.mapstruct.Mapper;
|
||||
import org.mapstruct.factory.Mappers;
|
||||
import ru.team58.profileservice.controller.schemes.GetUserResponse;
|
||||
import ru.team58.profileservice.service.UserDTO;
|
||||
|
||||
@Mapper
|
||||
public interface GetUserMapper {
|
||||
GetUserMapper INSTANCE = Mappers.getMapper(GetUserMapper.class);
|
||||
|
||||
GetUserResponse toGetUserResponse(UserDTO dto);
|
||||
}
|
||||
|
|
@ -1,13 +0,0 @@
|
|||
package ru.team58.profileservice.mapper;
|
||||
|
||||
import org.mapstruct.Mapper;
|
||||
import org.mapstruct.factory.Mappers;
|
||||
import ru.team58.profileservice.controller.schemes.UpdateUserRequest;
|
||||
import ru.team58.profileservice.service.UserDTO;
|
||||
|
||||
@Mapper
|
||||
public interface UpdateUserMapper {
|
||||
UpdateUserMapper INSTANCE = Mappers.getMapper(UpdateUserMapper.class);
|
||||
|
||||
UserDTO toUserDTO(UpdateUserRequest request);
|
||||
}
|
||||
|
|
@ -1,13 +0,0 @@
|
|||
package ru.team58.profileservice.mapper;
|
||||
|
||||
import org.mapstruct.Mapper;
|
||||
import org.mapstruct.factory.Mappers;
|
||||
import ru.team58.profileservice.persistence.entity.UserEntity;
|
||||
import ru.team58.profileservice.service.UserDTO;
|
||||
|
||||
@Mapper
|
||||
public interface UserDTOMapper {
|
||||
UserDTOMapper INSTANCE = Mappers.getMapper(UserDTOMapper.class);
|
||||
|
||||
UserDTO toUserDTO(UserEntity entity);
|
||||
}
|
||||
|
|
@ -1,13 +0,0 @@
|
|||
package ru.team58.profileservice.mapper;
|
||||
|
||||
import org.mapstruct.Mapper;
|
||||
import org.mapstruct.factory.Mappers;
|
||||
import ru.team58.profileservice.persistence.entity.UserEntity;
|
||||
import ru.team58.profileservice.service.UserDTO;
|
||||
|
||||
@Mapper
|
||||
public interface UserEntityMapper {
|
||||
UserEntityMapper INSTANCE = Mappers.getMapper(UserEntityMapper.class);
|
||||
|
||||
UserEntity toUserEntity(UserDTO dto);
|
||||
}
|
||||
|
|
@ -0,0 +1,27 @@
|
|||
package ru.team58.profileservice.mapper;
|
||||
|
||||
import org.mapstruct.Mapper;
|
||||
import org.mapstruct.Mapping;
|
||||
import org.mapstruct.factory.Mappers;
|
||||
import ru.team58.profileservice.controller.schemes.CreateUserRequest;
|
||||
import ru.team58.profileservice.controller.schemes.GetUserResponse;
|
||||
import ru.team58.profileservice.controller.schemes.UpdateUserRequest;
|
||||
import ru.team58.profileservice.controller.schemes.UserResponse;
|
||||
import ru.team58.profileservice.persistence.entity.UserEntity;
|
||||
import ru.team58.profileservice.service.UserDTO;
|
||||
|
||||
@Mapper(componentModel = "spring")
|
||||
public interface UserMapper {
|
||||
UserMapper INSTANCE = Mappers.getMapper(UserMapper.class);
|
||||
|
||||
UserDTO toUserDTO(UserEntity entity);
|
||||
UserDTO toUserDTO(UpdateUserRequest request);
|
||||
@Mapping(target = "id", ignore = true)
|
||||
UserDTO toUserDTO(CreateUserRequest request);
|
||||
|
||||
UserEntity toUserEntity(UserDTO dto);
|
||||
|
||||
UserResponse toUserResponse(UserDTO dto);
|
||||
|
||||
GetUserResponse toGetUserResponse(UserDTO dto);
|
||||
}
|
||||
|
|
@ -1,13 +0,0 @@
|
|||
package ru.team58.profileservice.mapper;
|
||||
|
||||
import org.mapstruct.Mapper;
|
||||
import org.mapstruct.factory.Mappers;
|
||||
import ru.team58.profileservice.controller.schemes.UserResponse;
|
||||
import ru.team58.profileservice.service.UserDTO;
|
||||
|
||||
@Mapper
|
||||
public interface UserResponseMapper {
|
||||
UserResponseMapper INSTANCE = Mappers.getMapper(UserResponseMapper.class);
|
||||
|
||||
UserResponse toUserResponse(UserDTO dto);
|
||||
}
|
||||
|
|
@ -1,15 +0,0 @@
|
|||
package ru.team58.profileservice.mapper;
|
||||
|
||||
import org.mapstruct.Mapper;
|
||||
import org.mapstruct.Mapping;
|
||||
import org.mapstruct.factory.Mappers;
|
||||
import ru.team58.profileservice.controller.schemes.UserScheme;
|
||||
import ru.team58.profileservice.service.UserDTO;
|
||||
|
||||
@Mapper(uses = {CreateUserMapper.class, UpdateUserMapper.class})
|
||||
public interface UserSchemeMapper {
|
||||
UserSchemeMapper INSTANCE = Mappers.getMapper(UserSchemeMapper.class);
|
||||
|
||||
@Mapping(target = "id", ignore = true)
|
||||
UserDTO toUserDTO(UserScheme user);
|
||||
}
|
||||
|
|
@ -1,7 +1,6 @@
|
|||
package ru.team58.profileservice.persistence.entity;
|
||||
|
||||
import jakarta.persistence.*;
|
||||
import jakarta.validation.constraints.*;
|
||||
import lombok.*;
|
||||
|
||||
import java.util.UUID;
|
||||
|
|
|
|||
|
|
@ -1,7 +1,12 @@
|
|||
package ru.team58.profileservice.service;
|
||||
|
||||
import jakarta.validation.Valid;
|
||||
import lombok.*;
|
||||
import ru.team58.profileservice.persistence.entity.UserEntity;
|
||||
|
||||
import java.nio.charset.StandardCharsets;
|
||||
import java.security.MessageDigest;
|
||||
import java.security.NoSuchAlgorithmException;
|
||||
import java.util.Objects;
|
||||
import java.util.UUID;
|
||||
|
||||
@Getter
|
||||
|
|
|
|||
|
|
@ -2,8 +2,7 @@ package ru.team58.profileservice.service;
|
|||
|
||||
import org.springframework.beans.factory.annotation.Autowired;
|
||||
import org.springframework.stereotype.Service;
|
||||
import ru.team58.profileservice.mapper.UserDTOMapper;
|
||||
import ru.team58.profileservice.mapper.UserEntityMapper;
|
||||
import ru.team58.profileservice.mapper.UserMapper;
|
||||
import ru.team58.profileservice.persistence.entity.UserEntity;
|
||||
import ru.team58.profileservice.persistence.repo.UserRepository;
|
||||
import ru.team58.profileservice.service.exceptions.UserAlreadyExistsException;
|
||||
|
|
@ -19,13 +18,13 @@ public class UserServiceImpl implements UserService {
|
|||
@Override
|
||||
public UserDTO getById(UUID id) {
|
||||
UserEntity user = userRepository.findById(id).orElseThrow(UserNotFoundException::new);
|
||||
return UserDTOMapper.INSTANCE.toUserDTO(user);
|
||||
return UserMapper.INSTANCE.toUserDTO(user);
|
||||
}
|
||||
|
||||
@Override
|
||||
public UserDTO getByUsername(String username) {
|
||||
UserEntity user = userRepository.findByUsername(username).orElseThrow(UserNotFoundException::new);
|
||||
return UserDTOMapper.INSTANCE.toUserDTO(user);
|
||||
return UserMapper.INSTANCE.toUserDTO(user);
|
||||
}
|
||||
|
||||
@Override
|
||||
|
|
@ -33,7 +32,7 @@ public class UserServiceImpl implements UserService {
|
|||
userRepository.findByUsername(user.username).ifPresent(u -> {
|
||||
throw new UserAlreadyExistsException();
|
||||
});
|
||||
return userRepository.save(UserEntityMapper.INSTANCE.toUserEntity(user)).getId();
|
||||
return userRepository.save(UserMapper.INSTANCE.toUserEntity(user)).getId();
|
||||
}
|
||||
|
||||
@Override
|
||||
|
|
@ -45,6 +44,6 @@ public class UserServiceImpl implements UserService {
|
|||
@Override
|
||||
public void updateUser(UserDTO user) {
|
||||
userRepository.findById(user.getId()).orElseThrow(UserNotFoundException::new);
|
||||
userRepository.save(UserEntityMapper.INSTANCE.toUserEntity(user));
|
||||
userRepository.save(UserMapper.INSTANCE.toUserEntity(user));
|
||||
}
|
||||
}
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue