From 10341cb8258b7100b0fd24ff95624baa95a6d442 Mon Sep 17 00:00:00 2001 From: mridul-20 Date: Wed, 18 Jun 2025 18:26:02 +0530 Subject: [PATCH] refactor: standardize delete endpoint response with ResponseEntity --- .../java/com/example/demo/controller/EmployeeController.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/employee-manager/src/main/java/com/example/demo/controller/EmployeeController.java b/employee-manager/src/main/java/com/example/demo/controller/EmployeeController.java index b18fd89f..e63bdbb2 100644 --- a/employee-manager/src/main/java/com/example/demo/controller/EmployeeController.java +++ b/employee-manager/src/main/java/com/example/demo/controller/EmployeeController.java @@ -54,12 +54,12 @@ public ResponseEntity updateEmployee(@PathVariable(value = "id") Long // //delete employee @DeleteMapping("employees/{id}") - public Map deleteEmployee(@PathVariable(value = "id") Long employeeId) throws ResourceNotFoundException { + public ResponseEntity> deleteEmployee(@PathVariable(value = "id") Long employeeId) throws ResourceNotFoundException { Employee employee = employeeRepository.findById(employeeId).orElseThrow(() -> new ResourceNotFoundException("Employee not found for this id :: " + employeeId)); this.employeeRepository.delete(employee); Map response = new HashMap<>(); response.put("deleted", Boolean.TRUE); - return response; + return ResponseEntity.ok(response); } } \ No newline at end of file