Develop fix

src/main/java/ru/yandex/practicum/filmorate/model/StartDate.java → src/main/java/ru/yandex/practicum/filmorate/annotation/StartDate.java

@@ -1,4 +1,4 @@
-package ru.yandex.practicum.filmorate.model;
+package ru.yandex.practicum.filmorate.annotation;
 
 import jakarta.validation.Constraint;
 import jakarta.validation.Payload;

src/main/java/ru/yandex/practicum/filmorate/model/StartDateValidator.java → src/main/java/ru/yandex/practicum/filmorate/annotation/StartDateValidator.java

@@ -1,4 +1,4 @@
-package ru.yandex.practicum.filmorate.model;
+package ru.yandex.practicum.filmorate.annotation;
 
 import jakarta.validation.ConstraintValidator;
 import jakarta.validation.ConstraintValidatorContext;

src/main/java/ru/yandex/practicum/filmorate/dal/JdbcFilmRepository.java

@@ -8,7 +8,6 @@
 import org.springframework.jdbc.support.GeneratedKeyHolder;
 import org.springframework.jdbc.support.rowset.SqlRowSet;
 import org.springframework.stereotype.Repository;
-import org.springframework.transaction.annotation.Transactional;
 import ru.yandex.practicum.filmorate.Enum.EventType;
 import ru.yandex.practicum.filmorate.Enum.OperationType;
 import ru.yandex.practicum.filmorate.dal.mappers.DirectorRowMapper;
@@ -283,7 +282,6 @@ public void connectDirectors(Collection<Film> films) {
         }
     }
 
-    @Transactional
     @Override
     public void addLike(long filmId, long userId) {
 
@@ -302,7 +300,6 @@ public void addLike(long filmId, long userId) {
         }
     }
 
-    @Transactional
     @Override
     public void deleteLike(long filmId, long userId) {
         GeneratedKeyHolder keyHolder = new GeneratedKeyHolder();

src/main/java/ru/yandex/practicum/filmorate/dal/JdbcReviewRepository.java

@@ -5,7 +5,6 @@
 import org.springframework.jdbc.core.namedparam.NamedParameterJdbcOperations;
 import org.springframework.jdbc.support.GeneratedKeyHolder;
 import org.springframework.stereotype.Repository;
-import org.springframework.transaction.annotation.Transactional;
 import ru.yandex.practicum.filmorate.Enum.EventType;
 import ru.yandex.practicum.filmorate.Enum.OperationType;
 import ru.yandex.practicum.filmorate.dal.mappers.ReviewRowMapper;
@@ -134,7 +133,6 @@ public List<Review> getAllReviewsByFilmId(long filmId, long count) {
                 .collect(Collectors.toList());
     }
 
-    @Transactional
     @Override
     public Review create(Review review) {
         GeneratedKeyHolder keyHolder = new GeneratedKeyHolder();
@@ -158,7 +156,6 @@ public Review create(Review review) {
         return review;
     }
 
-    @Transactional
     @Override
     public Review update(Review review) {
         GeneratedKeyHolder keyHolder = new GeneratedKeyHolder();
@@ -180,7 +177,6 @@ public Review update(Review review) {
         return reviewUpdated;
     }
 
-    @Transactional
     @Override
     public void deleteReview(long reviewId) {
         MapSqlParameterSource params = new MapSqlParameterSource();

src/main/java/ru/yandex/practicum/filmorate/exception/ErrorHandler.java

@@ -1,10 +1,13 @@
 package ru.yandex.practicum.filmorate.exception;
 
+import jakarta.validation.ConstraintViolationException;
 import lombok.extern.slf4j.Slf4j;
 import org.springframework.http.HttpStatus;
+import org.springframework.web.bind.MethodArgumentNotValidException;
+import org.springframework.web.bind.annotation.ExceptionHandler;
+import org.springframework.web.bind.annotation.ResponseBody;
 import org.springframework.web.bind.annotation.ResponseStatus;
 import org.springframework.web.bind.annotation.RestControllerAdvice;
-import org.springframework.web.bind.annotation.ExceptionHandler;
 
 @RestControllerAdvice
 @Slf4j
@@ -38,4 +41,28 @@ public ErrorResponse handleServerError(final InternalServerException e) {
                 e.getMessage()
         );
     }
+
+    @ResponseBody
+    @ExceptionHandler(ConstraintViolationException.class)
+    @ResponseStatus(HttpStatus.BAD_REQUEST)
+    public ErrorResponse onConstraintValidationException(ConstraintViolationException e) {
+        log.error("Параметры не прошли валидацию Spring  {}", e.getMessage());
+        return new ErrorResponse(
+                "Объект не прошёл валидацию Spring ",
+                e.getMessage()
+        );
+    }
+
+    @ExceptionHandler(MethodArgumentNotValidException.class)
+    @ResponseStatus(HttpStatus.BAD_REQUEST)
+    @ResponseBody
+    public ErrorResponse onMethodArgumentNotValidException(MethodArgumentNotValidException e) {
+        log.error("Объект не прошёл валидацию Spring  {}", e.getMessage());
+        return new ErrorResponse(
+                "Объект не прошёл валидацию Spring ",
+                e.getMessage()
+        );
+    }
+
+
 }

src/main/java/ru/yandex/practicum/filmorate/model/Film.java

@@ -2,6 +2,7 @@
 
 import jakarta.validation.constraints.*;
 import lombok.*;
+import ru.yandex.practicum.filmorate.annotation.StartDate;
 
 import java.time.LocalDate;
 import java.util.LinkedHashSet;

src/main/resources/application.properties

@@ -1,6 +1,6 @@
 logging.level.org.zalando.logbook=TRACE
 spring.sql.init.mode=always
-spring.datasource.url=jdbc:h2:file:./db/filmorate
+spring.datasource.url=jdbc:h2:file:./db/filmorate-test
 spring.datasource.driverClassName=org.h2.Driver
 spring.datasource.username=sa
 spring.datasource.password=password