Добавление валидации #6

Open
muts wants to merge 2 commits from dev into master
Owner

Нужно ревьювить.

Нужно ревьювить.
muts self-assigned this 2025-06-30 22:20:11 +04:00
itc1205 was assigned by muts 2025-06-30 22:20:11 +04:00
muts added 2 commits 2025-06-30 22:20:11 +04:00
Owner

Описание мр'а добавь с тайтлом

Описание мр'а добавь с тайтлом
itc1205 requested changes 2025-07-01 01:13:55 +04:00
@ -40,0 +47,4 @@
@ExceptionHandler(MethodArgumentNotValidException.class)
public ResponseEntity<Map<String, String>> handleValidationExceptions(
MethodArgumentNotValidException ex) {
Map<String, String> errors = new HashMap<>();
Owner

давай сразу введем практику использования var в типе переменных

https://www.baeldung.com/java-10-local-variable-type-inference

давай сразу введем практику использования var в типе переменных https://www.baeldung.com/java-10-local-variable-type-inference
Author
Owner

Принял

Принял
@ -40,0 +53,4 @@
String errorMessage = error.getDefaultMessage();
errors.put(fieldName, errorMessage);
});
return new ResponseEntity<>(errors, HttpStatus.BAD_REQUEST);
Owner

Давай также будем возвращать ошибки в каком то едином формате

Сделай отдельную дтошку для репрезентации ошибок

Давай также будем возвращать ошибки в каком то едином формате Сделай отдельную дтошку для репрезентации ошибок
@ -0,0 +1,4 @@
package ru.team58.profileservice.controller.exceptions;
public class IncorrectEmailException extends BadRequestException {
}
Owner

Где используется?

Где используется?
@ -0,0 +24,4 @@
@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")
Owner

А яну че делать

А яну че делать
Author
Owner

Ну, а янам вход воспрещен🫡

Ну, а янам вход воспрещен🫡
@ -0,0 +10,4 @@
UserResponseMapper INSTANCE = Mappers.getMapper(UserResponseMapper.class);
UserResponse toUserResponse(UserDTO dto);
}
Owner

Нравится, что разбил по разным мапперам

Не нравится, что обращаемся по статике к мапперам

Нравится, что разбил по разным мапперам Не нравится, что обращаемся по статике к мапперам
Author
Owner

А как обращаться?

А как обращаться?
@ -1,6 +1,7 @@
package ru.team58.profileservice.persistence.entity;
import jakarta.persistence.*;
import jakarta.validation.constraints.*;
Owner

Убираем

Убираем
Owner

Фиксим и даю аппрув

Фиксим и даю аппрув
muts changed title from dev to Добавление валидации 2025-07-01 08:43:12 +04:00
This pull request can be merged automatically.
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin dev:dev
git checkout dev

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git checkout master
git merge --no-ff dev
git checkout dev
git rebase master
git checkout master
git merge --ff-only dev
git checkout dev
git rebase master
git checkout master
git merge --no-ff dev
git checkout master
git merge --squash dev
git checkout master
git merge --ff-only dev
git checkout master
git merge dev
git push origin master
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: Shop/UserService#6
No description provided.