Skip to content

lindaquinoa/lesson_23/Implemented MediaItemsController with complete CRUD operations #818

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

lindaquinoa/lesson_23/Implemented MediaItemsController with complete CRUD operations #818

wants to merge 4 commits into from

Conversation

@lquinoa252
Copy link
Contributor

@lquinoa252 lquinoa252 commented Oct 22, 2025

  • GET /items - Retrieve all media items (returns 200 with array)
  • GET /items/{id} - Retrieve specific media item by ID (returns 200 or 404)
  • POST /items - Create new media item (returns 200 with created item)
  • DELETE /items/{id} - Remove media item by ID (returns 204 or 404)

Implementation Details:

  • No try/catch blocks, let Spring handle validation
  • Used TypeScript pattern as reference for clean controller structure
  • Class-level @RequestMapping("/api/v1/media") annotation
  • Proper HTTP status codes and error handling
  • All endpoints use Librarian authorization for library operations

Testing:

  • All 7 unit tests passing
  • Manual API testing with curl verified all endpoints working correctly
  • Changes isolated to single file: MediaItemsController.java

@lquinoa252
feat: implement GET /items/{id} endpoint
fb4d4cf 
- Add UUID parsing with error handling for malformed IDs
- Search library using SearchCriteria with ID filter
- Return 404 for non-existent items, 200 for found items
- Return 400 for invalid UUID format
@lquinoa252
feat: implement POST /items endpoint for creating media items
2a9190f 
- Add validation with @Valid for automatic error handling
- Convert MediaItemRequest to MediaItem using existing helper
- Add items to library via librarian authorization
- Return created item in response with proper status codes
@lquinoa252
feat: implement DELETE /items/{id} endpoint
ff7ec8c 
- Remove all try/catch blocks
- Add @RequestMapping class-level mapping
- Let Spring handle validation and error responses automatically
@lquinoa252
Fix code formatting with spotlessApply
2a618ba 
- Reorder imports (jakarta.validation.Valid before java.* imports)
- Remove unused import (org.springframework.http.HttpStatus)
- Clean up trailing whitespace on empty lines
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@lquinoa252