feat:Adds checks for ithe wrong use of an API

Author: Dasiame

lesson_26/api/java/api_app/src/main/java/com/codedifferently/lesson26/web/MediaItemsController.java

@@ -1,13 +1,10 @@
 package com.codedifferently.lesson26.web;
 
-import com.codedifferently.lesson26.library.Librarian;
-import com.codedifferently.lesson26.library.Library;
-import com.codedifferently.lesson26.library.MediaItem;
-import com.codedifferently.lesson26.library.search.SearchCriteria;
-import jakarta.validation.Valid;
 import java.io.IOException;
 import java.util.List;
 import java.util.Set;
+
+import org.hibernate.validator.constraints.UUID;
 import org.springframework.http.ResponseEntity;
 import org.springframework.web.bind.annotation.CrossOrigin;
 import org.springframework.web.bind.annotation.DeleteMapping;
@@ -17,6 +14,13 @@
 import org.springframework.web.bind.annotation.RequestBody;
 import org.springframework.web.bind.annotation.RestController;
 
+import com.codedifferently.lesson26.library.Librarian;
+import com.codedifferently.lesson26.library.Library;
+import com.codedifferently.lesson26.library.MediaItem;
+import com.codedifferently.lesson26.library.search.SearchCriteria;
+
+import jakarta.validation.Valid;
+
 @RestController
 @CrossOrigin
 public class MediaItemsController {
@@ -37,7 +41,8 @@ public GetMediaItemsResponse getItems() {
   }
 
   @PostMapping("/items")
-  public ResponseEntity<?> postItems(@Valid @RequestBody CreateMediaItemRequest request) {
+  public ResponseEntity<CreateMediaItemResponse> postItems(
+      @Valid @RequestBody CreateMediaItemRequest request) {
     MediaItemRequest itemRequest = request.getItem();
     MediaItem item = MediaItemRequest.asMediaItem(itemRequest);
     library.addMediaItem(item, librarian);
@@ -56,11 +61,11 @@ public ResponseEntity<GetMediaItemsResponse> getItemsId(@PathVariable String id)
   }
 
   @DeleteMapping("/items/{id}")
-  public ResponseEntity<?> deletebById(@PathVariable String id) {
-    Set<MediaItem> items = library.search(SearchCriteria.builder().id(id).build());
+  public ResponseEntity<Void> deletebById(@PathVariable UUID id) {
+    Set<MediaItem> items = library.search(SearchCriteria.builder().id(id.toString()).build());
     MediaItem item = items.iterator().next();
-    if (items.isEmpty() || item == null) {
-      ResponseEntity.notFound().build();
+    if(items.isEmpty()){
+      return ResponseEntity.notFound().build();
     }
     library.removeMediaItem(item, librarian);
     return ResponseEntity.noContent().build();