Merge pull request #21 from bouanani-soufiane/PIG-64-Lofts

PIG-64-Lofts
bouanani-soufiane · Nov 11, 2024 · b265d8e · b265d8e
2 parents 4d2aa9d + 5c083fc
commit b265d8e
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 5 deletions.
diff --git a/src/main/java/ma/yc/PigeonSkyRace/piegon/api/LoftController.java b/src/main/java/ma/yc/PigeonSkyRace/piegon/api/LoftController.java
@@ -17,13 +17,12 @@
 @RestController
 @RequestMapping("/api/v1/lofts")
 @RequiredArgsConstructor
-@Validated
 class LoftController {
 
     private final DefaultLoftDomainService service;
 
     @PostMapping()
-    public ResponseEntity<ResponseApi<LoftResponseDTO>> createLoft ( @Valid @RequestBody LoftRequestDTO loftRequest ) {
+    public ResponseEntity<ResponseApi<LoftResponseDTO>> create ( @Valid @RequestBody LoftRequestDTO loftRequest ) {
         LoftResponseDTO response = service.create(loftRequest);
         return ResponseEntity
                 .status(HttpStatus.CREATED)

diff --git a/src/main/java/ma/yc/PigeonSkyRace/piegon/application/dto/request/CoordinateRequestDTO.java b/src/main/java/ma/yc/PigeonSkyRace/piegon/application/dto/request/CoordinateRequestDTO.java
@@ -3,6 +3,7 @@
 import jakarta.validation.constraints.NotNull;
 
 public record CoordinateRequestDTO(
-        @NotNull Double latitude,
-        @NotNull Double longitude) {
+
+        @NotNull(message = "latitude cannot be null" ) Double latitude,
+        @NotNull(message = "longitude cannot be null" ) Double longitude) {
 }
diff --git a/src/main/java/ma/yc/PigeonSkyRace/piegon/application/dto/request/LoftRequestDTO.java b/src/main/java/ma/yc/PigeonSkyRace/piegon/application/dto/request/LoftRequestDTO.java
@@ -1,11 +1,14 @@
 package ma.yc.PigeonSkyRace.piegon.application.dto.request;
 
+import jakarta.validation.Valid;
 import jakarta.validation.constraints.NotNull;
 import ma.yc.PigeonSkyRace.common.application.validation.EntityExists;
 import ma.yc.PigeonSkyRace.user.domain.model.aggregate.User;
 
 public record LoftRequestDTO(
-        @NotNull CoordinateRequestDTO coordinate,
+        @Valid
+        @NotNull(message = "coordinate cannot be null" )
+        CoordinateRequestDTO coordinate,
 
         @EntityExists(entity = User.class, message = "The specified breeder does not exist")
         @NotNull String userId) {

diff --git a/src/main/java/ma/yc/PigeonSkyRace/piegon/domain/service/impl/DefaultLoftDomainService.java b/src/main/java/ma/yc/PigeonSkyRace/piegon/domain/service/impl/DefaultLoftDomainService.java
@@ -14,12 +14,14 @@
 import ma.yc.PigeonSkyRace.piegon.domain.service.LoftNameGenerator;
 import ma.yc.PigeonSkyRace.piegon.infrastructure.repository.LoftRepository;
 import org.springframework.stereotype.Service;
+import org.springframework.validation.annotation.Validated;
 
 import java.util.List;
 
 @Service
 @RequiredArgsConstructor
 @Slf4j
+@Validated
 public class DefaultLoftDomainService implements LoftDomainService, LoftApplicationService {
 
     private final LoftRepository repository;