Skip to content

Commit 37ecbc3

Browse files
committed
chore: made a few changes in the HTTP methods
-MediaItemsController
1 parent de30968 commit 37ecbc3

File tree

1 file changed

+5
-6
lines changed

1 file changed

+5
-6
lines changed

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

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -59,19 +59,18 @@ public MediaItemResponse postItem() {
5959
@GetMapping("/items/:{id}")
6060
public GetMediaItemsResponse getItemById(@PathVariable("id") String id) {
6161
Set<MediaItem> item = library.search(SearchCriteria.builder().id(id).build());
62-
MediaItemResponse responseItemById = item.stream().map(MediaItemResponse::from).toList().findFirst().orElseThrow();
62+
List<MediaItemResponse> responseItemById = item.stream().map(MediaItemResponse::from).toList();
63+
MediaItemResponse findItem = responseItemById.stream().findFirst().orElseThrow();
6364
var response = GetMediaItemsResponse.builder().item(responseItemById).build();
6465
return response;
6566
}
6667

6768
@DeleteMapping("/items/:{id}")
6869
public MediaItemResponse deleteItemById(@PathVariable("id") String id) {
69-
Set<MediaItem> itemToDelete = library.search(SearchCriteria.builder().id(id).build());
70-
//MediaItemRequest deleteItem = CreateMediaItemRequest.builder().build();
71-
CreateMediaItemRequest createDeleteRequest = CreateMediaItemRequest.builder().deleteItem(itemToDelete).build();
70+
MediaItem itemToDelete = library.search(SearchCriteria.builder().id(id).build()).stream().findFirst().orElseThrow();
7271

73-
library.delete(createDeleteRequest.getItem());
72+
library.delete(itemToDelete);
7473

75-
return MediaItemResponse.from(createDeleteRequest.getItem());
74+
return MediaItemResponse.from(itemToDelete);
7675
}
7776
}

0 commit comments

Comments
 (0)