🐛 [Fix] findPChangeByGameId optional 버그 수정 (#438)

42organization · Dec 22, 2023 · 51377ba · 51377ba
1 parent 65ca710
commit 51377ba
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/main/java/com/gg/server/domain/game/service/GameService.java b/src/main/java/com/gg/server/domain/game/service/GameService.java
@@ -212,7 +212,7 @@ private void updatePchangeIsChecked(Game game, Long loginUserId) {
     }
 
     public void savePChange(Game game, List<TeamUser> teamUsers, Long loginUserId) {
-        if (pChangeRepository.findPChangeByGameId(game.getId()).isPresent()){
+        if (!pChangeRepository.findPChangesByGameId(game.getId()).isEmpty()){
             return ;
         }
         Long team1UserId = teamUsers.get(0).getUser().getId();

diff --git a/src/main/java/com/gg/server/domain/pchange/data/PChangeRepository.java b/src/main/java/com/gg/server/domain/pchange/data/PChangeRepository.java
@@ -17,10 +17,10 @@ public interface PChangeRepository extends JpaRepository<PChange, Long> , PChang
     List<PChange> findAllByUserId(@Param("userId") Long userId);
 
     @Query(value = "SELECT pc FROM PChange pc join fetch pc.user join fetch pc.game WHERE pc.user.id = :userId and pc.game.mode in :modes order by pc.id desc")
-    List<PChange> findAllByUserIdGameModeIn(@Param("userId") Long userId, List<Mode> modes);
+    List<PChange> findAllByUserIdGameModeIn(@Param("userId") Long userId, @Param("modes") List<Mode> modes);
     Optional<PChange> findByUserIdAndGameId(Long userId, Long gameId);
 
     Optional<PChange> findPChangeByUserIdAndGameId(Long userId, Long gameId);
 
-    Optional<PChange> findPChangeByGameId(Long gameId);
+    List<PChange> findPChangesByGameId(Long gameId);
 }