Skip to content

[#455][FIX] 개인 회고가 책 상세의 회고 기록 목록에 안 보이던 문제 수정 - #456

Merged
onuyyy merged 1 commit into
devfrom
fix/retrospective-records-personal-book-id
Jun 17, 2026
Merged

[#455][FIX] 개인 회고가 책 상세의 회고 기록 목록에 안 보이던 문제 수정#456
onuyyy merged 1 commit into
devfrom
fix/retrospective-records-personal-book-id

Conversation

@onuyyy

@onuyyy onuyyy commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

PR 요약

책 상세의 "개인 회고 목록"(GET /api/book/{personalBookId}/records/retrospectives)이 항상 비어 있던 버그를 수정합니다. 회고 저장은 정상이었고, 조회 시 personalBookId를 실제 Book.id로 오인해 매칭에 실패한 것이 원인입니다.

  • 버그 수정

이슈 번호

Closes #455


주요 변경 사항

  • PersonalRetrospectiveService.getRetrospectiveRecords: path의 personalBookId(책장 항목 PK)를 실제 Book.id로 그대로 사용하던 것을, validatePersonalBook으로 PersonalBook을 조회해 실제 bookId로 변환하도록 수정. 회고 조회 쿼리는 WHERE b.id = :bookId(실제 도서 PK 기대)라 실데이터에서 personalBookId != bookId이면 결과가 항상 비어 있었음. 타임라인 서비스(getTimeline)와 동일한 방식으로 정렬하고, 소유자 검증도 함께 수행.
  • PersonalRetrospectiveServiceTest: 기존 테스트가 입력값을 bookId라 부르며 동일 값으로 버그를 가리고 있던 것을, personalBookId(7L) != bookId(1L)로 분리해 "변환된 실제 bookId로 조회"하는지 검증하도록 갱신(회귀 방지). 없는 책 케이스는 BOOK_NOT_IN_SHELF로 변경.

참고 사항

  • 영향 엔드포인트: GET /api/book/{personalBookId}/records/retrospectives
  • 동작 변경: 책장에 없는 personalBookId 요청 시 에러코드가 BOOK_NOT_FOUNDBOOK_NOT_IN_SHELF로 바뀜(타임라인 엔드포인트와 일관).
  • 회귀 테스트: ./gradlew test --tests "com.dokdok.retrospective.service.PersonalRetrospectiveServiceTest" 통과(BUILD SUCCESSFUL).
  • 참고: 같은 WHERE b.id = :bookId 쿼리를 쓰는 findByBookAndUser의 다른 호출부도 실제 bookId를 넘기는지 후속 점검 권장(이번 PR 범위 밖).

🤖 Generated with Claude Code

getRetrospectiveRecords가 path의 personalBookId(책장 항목 PK)를 실제
Book.id로 취급해 WHERE b.id = :bookId 쿼리에 그대로 전달, 실데이터에서
personalBookId != bookId라 회고 목록이 항상 비어 있었음.

타임라인 서비스와 동일하게 PersonalBook을 거쳐 실제 bookId를 해석하도록
수정하고, 소유자 검증(validatePersonalBook)도 함께 수행.

테스트는 personalBookId != bookId로 분리해 변환된 bookId로 조회하는지
검증(회귀 방지).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@onuyyy
onuyyy merged commit 8be71c5 into dev Jun 17, 2026
2 checks passed
@onuyyy
onuyyy deleted the fix/retrospective-records-personal-book-id branch June 17, 2026 06:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] 개인 회고가 책 상세의 회고 기록 목록에 안 보임

1 participant