fix: import

Angular2Guy · Oct 21, 2023 · 7b98811 · 7b98811
1 parent 4e42e28
commit 7b98811
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/backend/src/main/java/ch/xxx/moviemanager/usecase/service/MovieService.java b/backend/src/main/java/ch/xxx/moviemanager/usecase/service/MovieService.java
@@ -179,7 +179,7 @@ public boolean importMovie(int movieDbId, String bearerStr) throws InterruptedEx
 		List<Genere> generes = new ArrayList<>(this.genereRep.findAll());
 		for (GenereDto g : result.getGenres()) {
 			Genere genereEntity = generes.stream()
-					.filter(myGenere -> myGenere.getGenereId() != null && myGenere.getGenereId().equals(g.getId()))
+					.filter(myGenere -> Optional.ofNullable(myGenere.getGenereId()).stream().anyMatch(myGenereId -> myGenereId.equals(g.getId())))
 					.findFirst().orElse(this.mapper.convert(g));
 			if (genereEntity.getId() == null) {
 				genereEntity = genereRep.save(genereEntity);
@@ -190,8 +190,9 @@ public boolean importMovie(int movieDbId, String bearerStr) throws InterruptedEx
 		LOG.info("Start import Movie with Id: {movieDbId}", movieDbId);
 		MovieDto movieDto = this.movieDbRestClient.fetchMovie(this.decrypt(user.getMoviedbkey(), user.getUuid()),
 				movieDbId);
-		Movie movieEntity = this.movieRep.findByMovieId(movieDto.getMovieId(), user.getId()).orElse(null);
-		if (movieEntity == null) {
+		Optional<Movie> movieOpt = this.movieRep.findByMovieId(movieDto.getMovieId(), user.getId());
+		Movie movieEntity = movieOpt.isPresent() ? movieOpt.get() : null; 
+		if (movieOpt.isEmpty()) {
 			LOG.info("Movie not found by id");
 			List<Movie> movies = this.movieRep.findByTitleAndRelDate(movieDto.getTitle(), movieDto.getReleaseDate(),
 					this.userDetailService.getCurrentUser(bearerStr).getId());
@@ -255,6 +256,7 @@ public boolean importMovie(int movieDbId, String bearerStr) throws InterruptedEx
 				}
 			}
 		}
+		LOG.info("Finished import");
 		return true;
 	}