Merge pull request #176 from Team-HMH/fix/#174-signup-npe

fix - Optional 안에 null이 들어갈 수 있도록 코드를 변경하여 NPE 방지
Team-HMH · Jul 15, 2024 · e552391 · e552391
2 parents a489c9a + 31ec77e
commit e552391
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/src/main/java/sopt/org/hmh/domain/challenge/service/ChallengeFacade.java b/src/main/java/sopt/org/hmh/domain/challenge/service/ChallengeFacade.java
@@ -33,7 +33,8 @@ public class ChallengeFacade {
     @Transactional
     public Challenge addChallenge(Long userId, ChallengeRequest challengeRequest, String os) {
         User user = userService.findByIdOrThrowException(userId);
-        Optional<Long> previousChallengeId = Optional.of(user.getCurrentChallengeId());
+
+        Optional<Long> previousChallengeId = Optional.ofNullable(user.getCurrentChallengeId());
 
         Challenge challenge = challengeService.save(challengeRequest.toEntity(userId));
         user.changeCurrentChallengeId(challenge.getId());