Skip to content

Commit 0fc7ca2

Browse files
committed
feat: enhances MediaItemsController with CRUD operations and validation error handling
1 parent e98e18e commit 0fc7ca2

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

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

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
package com.codedifferently.lesson26.web;
22

3-
import com.codedifferently.lesson26.library.Librarian; // Ensure this import is present
3+
import com.codedifferently.lesson26.library.Librarian;
44
import com.codedifferently.lesson26.library.Library;
55
import com.codedifferently.lesson26.library.MediaItem;
66
import com.codedifferently.lesson26.library.search.SearchCriteria;
@@ -21,7 +21,7 @@
2121
import org.springframework.web.bind.annotation.RequestMapping;
2222
import org.springframework.web.bind.annotation.RestController;
2323

24-
// DTO imports
24+
2525

2626
@RestController
2727
@CrossOrigin
@@ -63,7 +63,7 @@ public ResponseEntity<CreateMediaItemResponse> createItem(
6363
@Valid @RequestBody CreateMediaItemRequest request) {
6464
try {
6565
MediaItem item = MediaItemRequest.asMediaItem(request.getItem());
66-
library.addMediaItem(item, librarian); // Ensure you're passing a Librarian object
66+
library.addMediaItem(item, librarian);
6767
var response = CreateMediaItemResponse.builder().item(MediaItemResponse.from(item)).build();
6868
return ResponseEntity.ok(response);
6969
} catch (IllegalArgumentException e) {
@@ -79,7 +79,7 @@ public ResponseEntity<Void> deleteItemById(@PathVariable String id) {
7979
if (items.isEmpty()) {
8080
return ResponseEntity.notFound().build();
8181
}
82-
library.removeMediaItem(uuid, librarian); // Ensure you're passing a Librarian object
82+
library.removeMediaItem(uuid, librarian);
8383
return ResponseEntity.noContent().build();
8484
} catch (IllegalArgumentException e) {
8585
return ResponseEntity.notFound().build();
@@ -98,7 +98,7 @@ public ResponseEntity<Map<String, List<Map<String, String>>>> handleValidationEr
9898
"message", fieldError.getDefaultMessage()))
9999
.toList();
100100

101-
// Optional: Log the validation errors
101+
102102
System.out.println("Validation errors: " + errors);
103103

104104
return ResponseEntity.badRequest().body(Map.of("errors", errors));

0 commit comments

Comments
 (0)