Fix: Memo를 생성 순으로 반환 & 독서기록 응답의 updatedAt은 Memo 걸로 반환

Author: dainnida

src/main/java/com/cmc/mercury/domain/memo/repository/MemoRepository.java

@@ -13,8 +13,8 @@
 @Repository
 public interface MemoRepository extends JpaRepository<Memo, Long> {
 
-    // RecordDetail의 모든 Memo들을 찾아서 업데이트 순으로 정렬 후 최상단 반환
-    Optional<Memo> findTopByRecordDetailOrderByUpdatedAtDesc(RecordDetail recordDetail);
+    // RecordDetail의 모든 Memo들을 찾아서 생성순으로 정렬 후 최상단 반환
+    Optional<Memo> findTopByRecordDetailOrderByCreatedAtDesc(RecordDetail recordDetail);
 
     // RecordDetail의 모든 Memo들을 찾아서 생성 순으로 정렬 후 반환
     List<Memo> findAllByRecordDetailOrderByCreatedAtDesc(RecordDetail recordDetail);

src/main/java/com/cmc/mercury/domain/record/dto/RecordDetailResponse.java

@@ -2,6 +2,7 @@
 
 import com.cmc.mercury.domain.book.dto.BookResponse;
 import com.cmc.mercury.domain.memo.dto.MemoResponse;
+import com.cmc.mercury.domain.memo.entity.Memo;
 import com.cmc.mercury.domain.record.entity.Record;
 import com.cmc.mercury.domain.record.entity.RecordDetail;
 import io.swagger.v3.oas.annotations.media.Schema;
@@ -25,13 +26,13 @@ public record RecordDetailResponse(
         @Schema(description = "메모 객체 목록")
         List<MemoResponse> memos
 ) {
-        public static RecordDetailResponse of(Record record, RecordDetail detail, List<MemoResponse> memos) {
+        public static RecordDetailResponse of(Record record, RecordDetail recordDetail, Memo latestMemo, List<MemoResponse> memos) {
                 return new RecordDetailResponse(
                         record.getId(),
-                        detail.getUpdatedGauge(),
+                        recordDetail.getUpdatedGauge(),
                         BookResponse.from(record.getBook()),
                         record.getCreatedAt(),
-                        record.getUpdatedAt(),
+                        latestMemo != null ? latestMemo.getUpdatedAt() : record.getUpdatedAt(),
                         memos
                 );
         }

src/main/java/com/cmc/mercury/domain/record/dto/RecordResponse.java

@@ -1,11 +1,13 @@
 package com.cmc.mercury.domain.record.dto;
 
 import com.cmc.mercury.domain.book.dto.BookResponse;
+import com.cmc.mercury.domain.memo.entity.Memo;
 import com.cmc.mercury.domain.record.entity.Record;
 import com.cmc.mercury.domain.record.entity.RecordDetail;
 import io.swagger.v3.oas.annotations.media.Schema;
 
 import java.time.LocalDateTime;
+import java.util.Optional;
 
 @Schema(title = "기록 객체 응답 형식")
 public record RecordResponse (
@@ -25,14 +27,14 @@ public record RecordResponse (
         @Schema(description = "얻은 경험치")
         int acquiredExp
 ) {
-        public static RecordResponse of(Record record, RecordDetail detail, String content) {
+        public static RecordResponse of(Record record, RecordDetail recordDetail, Memo memo, String content) {
 
                 return new RecordResponse(
                         record.getId(),
-                        detail.getUpdatedGauge(),
+                        recordDetail.getUpdatedGauge(),
                         content,
                         record.getCreatedAt(),
-                        record.getUpdatedAt(),
+                        memo != null ? memo.getUpdatedAt() : record.getUpdatedAt(),
                         BookResponse.from(record.getBook()),
                         record.getAcquiredExp()
                 );

src/main/java/com/cmc/mercury/domain/record/service/RecordService.java

@@ -25,6 +25,7 @@
 import java.time.LocalDate;
 import java.time.LocalDateTime;
 import java.util.List;
+import java.util.Optional;
 
 @Service
 @RequiredArgsConstructor
@@ -99,7 +100,7 @@ public RecordResponse createRecord(User user, RecordRequest request) {
         habitHistoryRepository.save(history);
 
         //* return RecordResponse.of(record, recordDetail, savedMemo.getContent());
-        return RecordResponse.of(savedRecord, savedRecord.getRecordDetail(), memo.getContent());
+        return RecordResponse.of(savedRecord, savedRecord.getRecordDetail(), memo, memo.getContent());
     }
 
     private int calculateRecordExp(Long userId, LocalDateTime deviceTime) {
@@ -160,6 +161,7 @@ public void deleteRecord(User user, Long recordId) {
 
         user.updateExp(user.getExp() - reduceExp);*/
 
+        // CascadeType.ALL로 인해 recordDetail도 같이 삭제
         recordRepository.delete(record);
     }
 
@@ -181,22 +183,22 @@ public RecordDetailResponse getRecordDetail(User user, Long recordId) {
                 .map(memo -> MemoResponse.from(memo, record.getId()))
                 .toList();
 
-        return RecordDetailResponse.of(record, recordDetail, memoResponses);
+        Memo latestMemo = memoRepository.findTopByRecordDetailOrderByCreatedAtDesc(recordDetail).orElse(null);
+
+        return RecordDetailResponse.of(record, recordDetail, latestMemo, memoResponses);
     }
 
     public List<RecordResponse> toRecordResponses(List<Record> records) {
 
         // RecordResponse 리스트로 변환
         return records.stream()
                 .map(record -> {
-                    RecordDetail detail = record.getRecordDetail();
+                    RecordDetail recordDetail = record.getRecordDetail();
+                    Memo latestMemo = memoRepository.findTopByRecordDetailOrderByCreatedAtDesc(recordDetail).orElse(null);
 
-                    String latestMemoContent = memoRepository
-                            .findTopByRecordDetailOrderByUpdatedAtDesc(detail)
-                            .map(Memo::getContent)
-                            .orElse("");
+                    String latestMemoContent = (latestMemo != null) ? latestMemo.getContent() : "";
 
-                    return RecordResponse.of(record, detail, latestMemoContent);
+                    return RecordResponse.of(record, recordDetail, latestMemo, latestMemoContent);
                 })
                 .toList();
     }