Merge pull request #36 from dacharyc/java-mismatches

Java final cleanup

Author: dacharyc

mflix/client/package.json

@@ -1,5 +1,8 @@
 {
   "name": "sample-mflix-front-end",
+  "description": "Next.js frontend for MongoDB sample mflix application demonstrating CRUD operations, aggregations, and search",
+  "license": "Apache-2.0",
+  "author": "MongoDB Documentation Team",
   "version": "0.1.0",
   "private": true,
   "scripts": {

mflix/server/java-spring/src/main/java/com/mongodb/samplemflix/controller/MovieControllerImpl.java

@@ -40,9 +40,9 @@
  * - DELETE /api/movies/{id} - Delete a movie
  * - DELETE /api/movies - Delete multiple movies
  * - DELETE /api/movies/{id}/find-and-delete - Find and delete a movie
- * - GET /api/movies/aggregations/comments - Aggregate movies with most comments
- * - GET /api/movies/aggregations/years - Aggregate movies by year with statistics
- * - GET /api/movies/aggregations/directors - Aggregate directors with most movies
+ * - GET /api/movies/aggregations/reportingByComments - Aggregate movies with most comments
+ * - GET /api/movies/aggregations/reportingByYear - Aggregate movies by year with statistics
+ * - GET /api/movies/aggregations/reportingByDirectors - Aggregate directors with most movies
  * - GET /api/movies/search - Text search using MongoDB Search Index across multiple fields (plot, fullplot, directors, writers, cast)
  * - GET /api/movies/vector-search - Vector search using Voyage AI embeddings to find movies with similar plots
  * - GET /api/movies/find-similar-movies - Vector search to find similar movies based on plot embeddings
@@ -296,7 +296,7 @@ public ResponseEntity<SuccessResponse<DeleteResponse>> deleteMoviesBatch(
         description = "Aggregates movies with their most recent comments using MongoDB $lookup (join) operation. " +
                      "Demonstrates how to combine data from the movies and comments collections."
     )
-    @GetMapping("/aggregations/comments")
+    @GetMapping("/aggregations/reportingByComments")
     public ResponseEntity<SuccessResponse<List<MovieWithCommentsResult>>> getMoviesWithMostRecentComments(
             @Parameter(description = "Maximum number of movies to return (default: 10, max: 50)")
             @RequestParam(defaultValue = "10") Integer limit,
@@ -331,7 +331,7 @@ public ResponseEntity<SuccessResponse<List<MovieWithCommentsResult>>> getMoviesW
         description = "Aggregates movies by year with statistics including movie count and average rating. " +
                      "Demonstrates MongoDB $group operation for statistical aggregation."
     )
-    @GetMapping("/aggregations/years")
+    @GetMapping("/aggregations/reportingByYear")
     public ResponseEntity<SuccessResponse<List<MoviesByYearResult>>> getMoviesByYearWithStats() {
 
         List<MoviesByYearResult> results = movieService.getMoviesByYearWithStats();
@@ -352,7 +352,7 @@ public ResponseEntity<SuccessResponse<List<MoviesByYearResult>>> getMoviesByYear
         description = "Aggregates directors with the most movies and their statistics. " +
                      "Demonstrates MongoDB $unwind operation for array flattening and aggregation."
     )
-    @GetMapping("/aggregations/directors")
+    @GetMapping("/aggregations/reportingByDirectors")
     public ResponseEntity<SuccessResponse<List<DirectorStatisticsResult>>> getDirectorsWithMostMovies(
             @Parameter(description = "Maximum number of directors to return (default: 20, max: 100)")
             @RequestParam(defaultValue = "20") Integer limit) {

mflix/server/java-spring/src/main/java/com/mongodb/samplemflix/exception/DatabaseOperationException.java

@@ -12,9 +12,5 @@ public class DatabaseOperationException extends RuntimeException {
     public DatabaseOperationException(String message) {
         super(message);
     }
-    
-    public DatabaseOperationException(String message, Throwable cause) {
-        super(message, cause);
-    }
 }
 

mflix/server/java-spring/src/main/java/com/mongodb/samplemflix/exception/ResourceNotFoundException.java

@@ -4,17 +4,12 @@
  * Exception thrown when a requested resource is not found.
  * 
  * This exception results in a 404 Not Found response.
- * 
- * TODO: Phase 7 - Implement custom exception
+ *
  */
 public class ResourceNotFoundException extends RuntimeException {
 
     public ResourceNotFoundException(String message) {
         super(message);
     }
-    
-    public ResourceNotFoundException(String message, Throwable cause) {
-        super(message, cause);
-    }
 }
 

mflix/server/java-spring/src/main/java/com/mongodb/samplemflix/exception/ValidationException.java

@@ -4,17 +4,12 @@
  * Exception thrown when request validation fails.
  * 
  * This exception results in a 400 Bad Request response.
- * 
- * TODO: Phase 7 - Implement custom exception
+ *
  */
 public class ValidationException extends RuntimeException {
 
     public ValidationException(String message) {
         super(message);
     }
-    
-    public ValidationException(String message, Throwable cause) {
-        super(message, cause);
-    }
 }
 

mflix/server/java-spring/src/main/java/com/mongodb/samplemflix/model/Comment.java

@@ -24,7 +24,7 @@ public class MovieWithCommentsResult {
     /**
      * Movie ID as string.
      */
-    private String id;
+    private String _id;
 
     /**
      * Movie title.
@@ -52,9 +52,9 @@ public class MovieWithCommentsResult {
     private List<String> genres;
 
     /**
-     * IMDB rating information.
+     * IMDB rating (0.0 to 10.0).
      */
-    private ImdbInfo imdb;
+    private Double imdbRating;
 
     /**
      * Most recent comments for this movie.
@@ -71,25 +71,6 @@ public class MovieWithCommentsResult {
      */
     private Date mostRecentCommentDate;
 
-    /**
-     * Nested class for IMDB information.
-     */
-    @Data
-    @Builder
-    @NoArgsConstructor
-    @AllArgsConstructor
-    public static class ImdbInfo {
-        /**
-         * IMDB rating (0.0 to 10.0).
-         */
-        private Double rating;
-
-        /**
-         * Number of votes.
-         */
-        private Integer votes;
-    }
-
     /**
      * Nested class for comment information.
      */