Browse Source

quick-fix-endpoint

Makumbacz 3 years ago
parent
commit
dad81c0a4b

+ 3 - 2
src/main/java/com/example/plantsforyou/order/OrderController.java

@@ -49,8 +49,9 @@ public class OrderController {
     }
 
     @PatchMapping("/{orderId}")
-    public ResponseEntity<Order>editOrderById(@RequestBody UpdateOrderDto updateOrderDto, @PathVariable("orderId") Long id) throws RejectedRequestException {
-        return new ResponseEntity<>(orderService.editOrderById(updateOrderDto,id), HttpStatus.OK);
+    public ResponseEntity<Object>editOrderById(@RequestBody UpdateOrderDto updateOrderDto, @PathVariable("orderId") Long id) throws RejectedRequestException {
+        orderService.editOrderById(updateOrderDto,id);
+        return new ResponseEntity<>(HttpStatus.OK);
     }
 
     @GetMapping("/all")

+ 1 - 2
src/main/java/com/example/plantsforyou/order/OrderService.java

@@ -70,7 +70,7 @@ public class OrderService {
         return orderRepository.findAll();
     }
 
-    public Order editOrderById(UpdateOrderDto updateOrderDto, Long id) throws RejectedRequestException {
+    public void editOrderById(UpdateOrderDto updateOrderDto, Long id) throws RejectedRequestException {
         if(!orderRepository.existsById(id)){
             throw new RejectedRequestException("Order does not exist", HttpStatus.BAD_REQUEST);
         }
@@ -78,7 +78,6 @@ public class OrderService {
 
         order.setStatus(updateOrderDto.getStatus());
         orderRepository.save(order);
-        return order;
     }
 
     public Order getOrderById(Long id) {