|
4 | 4 | import com.codedifferently.lesson26.library.Library; |
5 | 5 | import com.codedifferently.lesson26.library.MediaItem; |
6 | 6 | import com.codedifferently.lesson26.library.search.SearchCriteria; |
| 7 | +import jakarta.validation.Valid; |
7 | 8 | import java.io.IOException; |
8 | 9 | import java.util.List; |
9 | 10 | import java.util.Optional; |
@@ -40,13 +41,11 @@ public ResponseEntity<GetMediaItemsResponse> getItems() { |
40 | 41 | } |
41 | 42 |
|
42 | 43 | @PostMapping("/items") |
43 | | - public ResponseEntity<MediaItemResponse> createItem(@RequestBody MediaItemRequest request) { |
| 44 | + public ResponseEntity<MediaItemResponse> createItem( |
| 45 | + @Valid @RequestBody MediaItemRequest request) { |
44 | 46 | MediaItem newItem = MediaItemRequest.asMediaItem(request); |
45 | | - |
46 | 47 | Librarian librarian = new Librarian( "system", "[email protected]"); |
47 | | - |
48 | 48 | library.addMediaItem(newItem, librarian); |
49 | | - |
50 | 49 | return new ResponseEntity<>(MediaItemResponse.from(newItem), HttpStatus.CREATED); |
51 | 50 | } |
52 | 51 |
|
@@ -77,9 +76,9 @@ public ResponseEntity<Void> deleteItemById(@PathVariable UUID id) { |
77 | 76 | } |
78 | 77 |
|
79 | 78 | MediaItem item = itemOpt.get(); |
80 | | - Librarian librarian = new Librarian( "system", "[email protected]"); // placeholder |
| 79 | + Librarian librarian = new Librarian( "system", "[email protected]"); |
81 | 80 | library.removeMediaItem(item, librarian); |
82 | 81 |
|
83 | | - return ResponseEntity.noContent().build(); // 204 No Content |
| 82 | + return ResponseEntity.noContent().build(); |
84 | 83 | } |
85 | 84 | } |
0 commit comments