fix: generateCarPdf

.gitignore

@@ -3,6 +3,8 @@ target/
 !**/src/main/**/target/
 !**/src/test/**/target/
 
+output-data/
+
 ### STS ###
 .apt_generated
 .classpath

core/src/main/java/cz/muni/pa165/data/model/Car.java

@@ -16,14 +16,12 @@ public class Car extends DomainObject<Long> implements Serializable {
     @OneToOne(fetch = FetchType.LAZY)
     @JoinColumn(name = "driver_id")
     @Nullable
-    @JsonManagedReference
     private Driver driver;
 
     @OneToMany(fetch = FetchType.LAZY,
             mappedBy = "car",
             cascade = {CascadeType.REMOVE, CascadeType.PERSIST, CascadeType.MERGE})
     @Nonnull
-    @JsonManagedReference
     private Set<CarComponent> components;
 
     public Car() {

core/src/main/java/cz/muni/pa165/data/model/CarComponent.java

 package cz.muni.pa165.data.model;
 
-import com.fasterxml.jackson.annotation.JsonBackReference;
+import com.fasterxml.jackson.annotation.JsonIgnore;
 import cz.muni.pa165.data.enums.ComponentTypeEnum;
 import jakarta.annotation.Nonnull;
 import jakarta.annotation.Nullable;
@@ -32,7 +32,7 @@ public class CarComponent extends DomainObject<Long> implements Serializable {
     @ManyToOne(fetch = FetchType.LAZY)
     @JoinColumn(name = "car_id")
     @Nullable
-    @JsonBackReference
+    @JsonIgnore
     private Car car;
 
     public CarComponent() {

core/src/main/java/cz/muni/pa165/data/model/Driver.java

 package cz.muni.pa165.data.model;
 
-import com.fasterxml.jackson.annotation.JsonBackReference;
+import com.fasterxml.jackson.annotation.JsonIgnore;
 import cz.muni.pa165.data.enums.CharacteristicsEnum;
 import jakarta.annotation.Nonnull;
 import jakarta.annotation.Nullable;
@@ -48,7 +48,7 @@ public class Driver extends DomainObject<Long> implements Serializable {
             mappedBy = "driver",
             cascade = {CascadeType.REMOVE, CascadeType.PERSIST, CascadeType.MERGE})
     @Nullable
-    @JsonBackReference
+    @JsonIgnore
     private Car car;
 
     public Driver() {

core/src/main/java/cz/muni/pa165/service/CarService.java

@@ -11,16 +11,17 @@ import cz.muni.pa165.exceptions.ResourceNotFoundException;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 import org.springframework.beans.factory.annotation.Autowired;
-import org.springframework.core.ParameterizedTypeReference;
 import org.springframework.data.domain.Page;
 import org.springframework.data.domain.PageRequest;
 import org.springframework.data.domain.Pageable;
 import org.springframework.http.MediaType;
 import org.springframework.stereotype.Service;
 import org.springframework.transaction.annotation.Transactional;
+import org.springframework.core.io.Resource;
 import org.springframework.web.client.RestClientException;
 import org.springframework.web.reactive.function.client.WebClient;
 
+import java.io.IOException;
 import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
@@ -184,11 +185,7 @@ public class CarService extends DomainService<Car> {
                 .contentType(MediaType.APPLICATION_JSON)
                 .bodyValue(visualization)
                 .retrieve()
-                .bodyToMono(new ParameterizedTypeReference<Map<String, Object>>() {
-                })
-                .doOnError(error -> {
-                    throw new RestClientException("Failed to send POST request", error);
-                })
+                .bodyToMono(Resource.class)
                 .block();
     }
 }

visualization/openapi.yaml

@@ -155,7 +155,12 @@ paths:
         content:
           application/json:
             schema:
-              $ref: '#/components/schemas/CarDto'
+              type: object
+              properties:
+                car:
+                  $ref: '#/components/schemas/CarDto'
+              required:
+                - car
       responses:
         "200":
           description: OK

visualization/src/main/java/cz/muni/pa165/rest/VisualizationController.java

@@ -3,6 +3,7 @@ package cz.muni.pa165.rest;
 import cz.muni.pa165.facade.VisualizationFacade;
 import cz.muni.pa165.generated.api.VisualizationApiDelegate;
 import cz.muni.pa165.generated.model.CarDto;
+import cz.muni.pa165.generated.model.GenerateCarPdfRequest;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 import org.springframework.beans.factory.annotation.Autowired;
@@ -27,12 +28,13 @@ public class VisualizationController implements VisualizationApiDelegate {
     }
 
     @Override
-    public ResponseEntity<Resource> generateCarPdf(CarDto carDto) {
+    public ResponseEntity<Resource> generateCarPdf(GenerateCarPdfRequest generateCarPdfRequest) {
         log.debug("generateCarPdf() called");
         try {
-            var resource = visualizationFacade.generateCarPdf(carDto);
+            var resource = visualizationFacade.generateCarPdf(generateCarPdfRequest.getCar());
             return ResponseEntity.ok()
-                    .header(HttpHeaders.CONTENT_DISPOSITION, "attachment; filename=generated.pdf")
+                    .header(HttpHeaders.CONTENT_DISPOSITION,
+                            String.format("attachment; filename=car-%s.pdf", generateCarPdfRequest.getCar().getId()))
                     .contentType(MediaType.APPLICATION_PDF)
                     .contentLength(resource.contentLength())
                     .body(resource);

visualization/src/main/java/cz/muni/pa165/service/VisualizationService.java

@@ -14,7 +14,6 @@ import java.io.IOException;
 import java.nio.file.Files;
 import java.nio.file.Path;
 import java.nio.file.Paths;
-import java.util.Objects;
 
 @Service
 public class VisualizationService {