Changeset 2a552fe for src/main/java/com/example/autopartz/controller/UserController.java

Timestamp:

07/26/23 19:26:35 (16 months ago)

Author:

andrejtodorovski <82031894+andrejtodorovski@…>

Branches:

main

Parents:

e02787e

Message:

Added transactional methods to secure the database integrity and refactoring

File:

• src/main/java/com/example/autopartz/controller/UserController.java (modified) (4 diffs)

src/main/java/com/example/autopartz/controller/UserController.java

@@ -25 +25 @@
     private final UserService userService;
     private final CarService carService;
-    private final CarSampleRepository carSampleRepository;
-    private final ServiceBookRepository serviceBookRepository;
     private final RepairService repairService;
     private final ReviewRepository reviewRepository;
-    private final RepairRepository repairRepository;
     private final OrderRepository orderRepository;
 
-    public UserController(OrdersForUserRepository ordersForUserRepository, RepairsForUserRepository repairsForUserRepository, ReviewsForUserRepository reviewsForUserRepository, UserService userService, CarService carService, CarSampleRepository carSampleRepository, ServiceBookRepository serviceBookRepository, RepairService repairService, ReviewRepository reviewRepository,
-                          RepairRepository repairRepository,
+    public UserController(OrdersForUserRepository ordersForUserRepository, RepairsForUserRepository repairsForUserRepository, ReviewsForUserRepository reviewsForUserRepository, UserService userService, CarService carService, RepairService repairService, ReviewRepository reviewRepository,
                           OrderRepository orderRepository) {
         this.ordersForUserRepository = ordersForUserRepository;
@@ -40 +36 @@
         this.userService = userService;
         this.carService = carService;
-        this.carSampleRepository = carSampleRepository;
-        this.serviceBookRepository = serviceBookRepository;
         this.repairService = repairService;
         this.reviewRepository = reviewRepository;
-        this.repairRepository = repairRepository;
         this.orderRepository = orderRepository;
     }
@@ -92 +85 @@
                                     @RequestParam Integer km, @RequestParam Integer cartype,
                                     HttpServletRequest request, HttpServletResponse response){
-        CarSample cs = new CarSample(vin,year,power,displacement,fuel,km, (Client) userService.findByUsername(request.getRemoteUser()),carService.findById(cartype));
-        carSampleRepository.save(cs);
-        serviceBookRepository.save(new ServiceBook(cs));
+        userService.addCarSampleForUser(vin,year,power,displacement,fuel,km,cartype,request.getRemoteUser());
         try {
             response.sendRedirect("/");
@@ -109 +100 @@
     @PostMapping("/repairs/addReview/{id}")
     public void addReview(@PathVariable Integer id, @RequestParam Integer rating, @RequestParam(required = false) String comment,
-                          Model model, HttpServletResponse response, HttpServletRequest request){
+                          HttpServletResponse response, HttpServletRequest request){
         Client client = (Client) userService.findByUsername(request.getRemoteUser());
         Repair repair = repairService.findById(id);