Dynamodb #4
Dynamodb #4
Conversation
…h in UserMovieStatus. Deleting Watchlist.
…t to make them by myself.
…indAll, countMovies, findByPlatform.
…ieStatusRepositoryImpl. Removing AWS credentials from application.properties.
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
This is the final PR Bugbot will review for you during this billing cycle
Your free Bugbot reviews will reset on November 14
Details
You are on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle.
To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.
| if (!auth.isAuthenticated) { | ||
| auth.signinRedirect(); | ||
| return null; | ||
| } |
There was a problem hiding this comment.
Bug: Authentication Blockage in Main Layout
The MainLayout component unconditionally returns a "Cargando autenticación..." message. This prevents the main application UI from rendering and makes all subsequent authentication checks unreachable, effectively blocking the app from loading.
|
|
||
| @DynamoDbAttribute("ageRating") | ||
| public Rating getAgeRating() {return rating; } | ||
| public void setRating(Rating rating) { this.rating = rating; } |
There was a problem hiding this comment.
Bug: JavaBean Naming Inconsistency
The Movie class has a naming inconsistency for the rating field. The getter getAgeRating() is mapped to ageRating, but the corresponding setter is setRating(). This deviates from JavaBean conventions and may affect DynamoDB mapping and serialization.
|
|
||
| public String generateMovieId() { | ||
| return this.year + "#" + this.title.toLowerCase().replace(" ", "-"); | ||
| } |
There was a problem hiding this comment.
Bug: Movie ID Format Mismatch Causes Query Failures
The generateMovieId() method generates movieId in a year#title format. This conflicts with the Movie class's sort key definition and repository queries, which expect a genre#title format. This inconsistency leads to incorrect key generation and prevents movies from being found.
| ))} | ||
| <div className="mt-2"> | ||
| <select | ||
| value={selectedGenre} |
There was a problem hiding this comment.
Bug: Undefined Genre Causes Select Element Issues
The <select> element in GenrePanel uses selectedGenre for its value prop. Since selectedGenre can be undefined, this might cause issues as HTML <select> elements expect a string value.
No description provided.