chore: update MediaItemsController so that it can compile

Author: VicenteVigueras

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

@@ -1,16 +1,11 @@
 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 com.codedifferently.lesson26.web.GetMediaItemsResponse.GetMediaItemsResponseBuilder;
 
-import jakarta.validation.Valid;
 import java.io.IOException;
 import java.util.List;
 import java.util.Map;
 import java.util.Set;
 import java.util.UUID;
+
 import org.springframework.http.ResponseEntity;
 import org.springframework.validation.FieldError;
 import org.springframework.web.bind.MethodArgumentNotValidException;
@@ -23,8 +18,13 @@
 import org.springframework.web.bind.annotation.RequestBody;
 import org.springframework.web.bind.annotation.RequestMapping;
 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
@@ -33,10 +33,12 @@ public class MediaItemsController {
 
   private final Library library;
   private final Librarian librarian;
+
   public MediaItemsController(Library library) throws IOException {
     this.library = library;
     this.librarian = library.getLibrarians().stream().findFirst().orElseThrow();
   }
+
   @GetMapping("/items")
   public ResponseEntity<GetMediaItemsResponse> getItems() {
     Set<MediaItem> items = library.search(SearchCriteria.builder().build());
@@ -48,7 +50,7 @@ public ResponseEntity<GetMediaItemsResponse> getItems() {
   @GetMapping("/items/{id}")
   public ResponseEntity<MediaItemResponse> getItemById(@PathVariable String id) {
     try {
-      Set<MediaItem> items = library.search(((Object) SearchCriteria.builder()).id(id).build());
+      Set<MediaItem> items = library.search((SearchCriteria.builder().id(id).build()));
       if (items.isEmpty()) {
         return ResponseEntity.notFound().build();
       }
@@ -60,20 +62,19 @@ public ResponseEntity<MediaItemResponse> getItemById(@PathVariable String id) {
   }
 
   @PostMapping("/items")
-  public ResponseEntity<Object> createItem(
-      @Valid @RequestBody CreateMediaItemRequest request) {
+  public ResponseEntity<Object> createItem(@Valid @RequestBody CreateMediaItemRequest request) {
     try {
       MediaItem item = MediaItemRequest.asMediaItem(request.getItem());
       library.addMediaItem(item, librarian);
-      var response = ((GetMediaItemsResponseBuilder) CreateMediaItemResponse.builder()).item(MediaItemResponse.from(item)).build();
+      var response = (CreateMediaItemResponse.builder()).item(MediaItemResponse.from(item)).build();
       return ResponseEntity.ok(response);
     } catch (IllegalArgumentException e) {
       return ResponseEntity.badRequest().build();
     }
   }
 
   @DeleteMapping("/items/{id}")
-  public ResponseEntity<Void> deleteItemById(@PathVariable("id")  String id) {
+  public ResponseEntity<Void> deleteItemById(@PathVariable("id") String id) {
     try {
       UUID uuid = UUID.fromString(id);
       Set<MediaItem> items = library.search(SearchCriteria.builder().id(id).build());
@@ -94,4 +95,4 @@ public ResponseEntity<Map<String, List<String>>> handleValidationErrors(
         ex.getBindingResult().getFieldErrors().stream().map(FieldError::getDefaultMessage).toList();
     return ResponseEntity.badRequest().body(Map.of("errors", errors));
   }
-}
+}