Skip to content

Commit 39cbdb3

Browse files
authored
Merge pull request #85 from Nexters/dev
2 parents f25d14a + 5422183 commit 39cbdb3

File tree

2 files changed

+6
-4
lines changed

2 files changed

+6
-4
lines changed

src/main/java/donmani/donmani_server/reward/repository/UserEquippedItemRepository.java

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,11 +12,13 @@ public interface UserEquippedItemRepository extends JpaRepository<UserEquippedIt
1212
@Query("SELECT e FROM UserEquippedItem e " +
1313
"WHERE e.user = :user " +
1414
"AND FUNCTION('YEAR', e.savedAt) = :year " +
15-
"AND FUNCTION('MONTH', e.savedAt) = :month")
16-
Optional<UserEquippedItem> findByUserAndSavedAtInCurrentMonth(
15+
"AND FUNCTION('MONTH', e.savedAt) = :month " +
16+
"ORDER BY e.savedAt DESC")
17+
Optional<UserEquippedItem> findTopByUserAndSavedAtInCurrentMonth(
1718
@Param("user") User user,
1819
@Param("year") int year,
1920
@Param("month") int month
2021
);
2122

23+
2224
}

src/main/java/donmani/donmani_server/reward/service/RewardService.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -230,7 +230,7 @@ public void saveItem(RewardItemSaveRequestDTO request) {
230230
throw new IllegalArgumentException("유효하지 않은 요청입니다.");
231231
}
232232

233-
Optional<UserEquippedItem> equippedItem = userEquippedItemRepository.findByUserAndSavedAtInCurrentMonth(user, year, month);
233+
Optional<UserEquippedItem> equippedItem = userEquippedItemRepository.findTopByUserAndSavedAtInCurrentMonth(user, year, month);
234234

235235
RewardItem updateBackground = rewardItemRepository.findById(request.getBackgroundId()).orElseThrow();
236236
RewardItem updateEffect = rewardItemRepository.findById(request.getEffectId()).orElseThrow();
@@ -270,7 +270,7 @@ public List<RewardItemResponseDTO> getSavedItem(String userKey, int year, int mo
270270

271271
User user = userRepository.findByUserKey(userKey).orElseThrow(() -> new RuntimeException("USER NOT FOUND"));
272272

273-
Optional<UserEquippedItem> savedItem = userEquippedItemRepository.findByUserAndSavedAtInCurrentMonth(user, year, month);
273+
Optional<UserEquippedItem> savedItem = userEquippedItemRepository.findTopByUserAndSavedAtInCurrentMonth(user, year, month);
274274

275275
if(savedItem.isPresent()) {
276276
UserEquippedItem presentSavedItem = savedItem.get();

0 commit comments

Comments
 (0)