[CEEZ-73] fix: 앨범 사진 조회 오류 해결

[zyovn]

🔗 연관된 이슈

• CEEZ-73

🚀 변경 유형

• ✨ 기능 추가 (feature)
• 🐛 버그 수정 (fix)
• 📝 문서 변경 (docs)
• ♻️ 리팩토링 (refactor)
• 🧪 테스트 추가 / 수정 (test)
• ⚙️ 설정 변경 (chore)

📝 작업 내용

• 앨범 사진 조회 오류 해결
• 조회 관련 메서에 앨범 입장자인지 확인하는 로직 추가

📸 스크린샷

사진 목록 조회

사진 상세 조회

띱한 사용자 목록 조회

내가 띱한 사진 목록 조회

💬 리뷰 요구사항

📜 리뷰 규칙

Reviewer는 아래 P5 Rule을 참고하여 리뷰를 진행합니다.
P5 Rule을 통해 Reviewer는 Reviewee에게 리뷰의 의도를 보다 정확히 전달할 수 있습니다.

• P1: 꼭 반영해주세요 (Comment)
• P2: 적극적으로 고려해주세요 (Comment)
• P3: 웬만하면 반영해 주세요 (Comment)
• P4: 반영해도 좋고 넘어가도 좋습니다 (Approve)
• P5: 그냥 사소한 의견입니다 (Approve)

• 저번에 급하게 짜다 보니 입장자 확인 로직을................................. 까먹엇습니다....

Summary by CodeRabbit

릴리스 노트

• 개선 사항

  • 사진 조회 기능에 앨범 및 참여자 검증이 추가되었습니다.

• 스타일

  • 불필요한 코드 라인 정리

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

요약

PhotoQueryService에 AlbumValidator 의존성을 주입하고, getPhotoPage, getPhotoLiked, getPhotoDetail 메서드에 앨범 및 참여자 유효성 검사 로직을 추가했습니다. 또한 사용되지 않는 임포트를 제거하고 불필요한 공백을 삭제했습니다.

변경사항

응집단 / 파일	요약
PhotoQueryService 유효성 검사 추가 src/main/java/com/cheeeese/photo/application/PhotoQueryService.java	AlbumValidator 의존성을 주입받도록 생성자를 수정하고, 세 개의 쿼리 메서드(getPhotoPage, getPhotoLiked, getPhotoDetail)에 앨범/참여자 유효성 검사 호출을 추가했습니다. 불필요한 임포트도 제거했습니다.
PhotoInfoService 포맷팅 src/main/java/com/cheeeese/photo/application/PhotoInfoService.java	할당문 이후의 불필요한 공백 줄을 제거했습니다.

코드 검토 노력

🎯 2 (Simple) | ⏱️ ~10분

관련 가능성 있는 PR

• feat: 앨범 입장 로직 구현 #13: 메인 PR의 AlbumValidator 필드 추가 및 유효성 검사 호출이 검색된 PR에서 도입한 AlbumValidator 및 앨범 진입 유효성 검사 로직에 직접 의존합니다.
• fix: 업로드 로직 수정 #76: 두 PR 모두 앨범/참여자 확인을 위해 AlbumValidator를 사용하도록 사진 관련 서비스를 수정합니다.
• fix: QA에서 발견한 오류 수정 #75: 메인 PR이 앨범 접근 검증을 위해 AlbumValidator를 추가하고 사용하며, 검색된 PR은 AlbumValidator의 동작 방식을 수정합니다.

추천 검토자

• dahyun24

시

🐰 앨범을 지키는 검증자 추가되고,
접근 권한 확인하는 벽이 세워졌네.
안전한 사진들을 위해,
검사 통과한 이들만 들어가도록,
한 줄의 공백도 정리하며 변화는 계속된다네! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	PR 제목은 버그 수정의 핵심 변경사항을 명확하게 반영하고 있습니다. 앨범 사진 조회 오류 해결이라는 주요 목적을 간결하고 구체적으로 표현했습니다.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix/CEEZ-73

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[dahyun24]

구웃~ 수고하셨습니다!!!!