From fe4d3ff2d618f47f6e6a3c32cf9efacc208bcb03 Mon Sep 17 00:00:00 2001
From: Aysegul <aaydinlik@tudelft.nl>
Date: Fri, 9 Jan 2026 22:16:30 +0100
Subject: [PATCH] added pop up view ingredients + warnings for delete removed
 the star import should also fix the test issue

---
 .../java/server/api/IngredientController.java | 22 +++++++++----------
 1 file changed, 11 insertions(+), 11 deletions(-)

diff --git a/server/src/main/java/server/api/IngredientController.java b/server/src/main/java/server/api/IngredientController.java
index b1f4510..5c36cb1 100644
--- a/server/src/main/java/server/api/IngredientController.java
+++ b/server/src/main/java/server/api/IngredientController.java
@@ -168,27 +168,27 @@ public class IngredientController {
      * @see Ingredient
      */
     @PostMapping("/ingredients")
-    public ResponseEntity<?> createIngredient(@RequestBody Ingredient ingredient) {
+    public ResponseEntity<Ingredient> createIngredient(@RequestBody Ingredient ingredient) {
         if (ingredient == null
                 || ingredient.name == null
                 || ingredient.name.trim().isEmpty()) {
-            return ResponseEntity.badRequest()
-                    .body("Ingredient name cannot be empty.");
+            return ResponseEntity.badRequest().build();
         }
 
         return ingredientService.create(ingredient)
-                .<ResponseEntity<?>>map
-                (saved -> {
-                    messagingTemplate.convertAndSend(Topics.INGREDIENTS,
-                            new CreateIngredientMessage(saved));
+                .map(saved -> {
+                    messagingTemplate.convertAndSend(
+                            Topics.INGREDIENTS,
+                            new CreateIngredientMessage(saved)
+                    );
                     return ResponseEntity.ok(saved);
                 })
-                .orElseGet(() ->
-                        ResponseEntity.status(HttpStatus.CONFLICT)
-                                .body("ingredient name already exists"));
-
+                .orElseGet(() -> ResponseEntity.status
+                        (HttpStatus.CONFLICT).build()
+                );
     }
 
+
     /**
      * Delete an ingredient by its ID.
      * Maps to <code>DELETE /api/ingredients/{id}</code>