fix: rop 조회

[vivivim]

📝 Summary

rop 조회 시 아이디 함께 드려요

🙏 Question & PR point

📬 Reference

Summary by CodeRabbit

변경 사항

• 개선 사항
  • 재주문점(ROP) 데이터 응답에 고유 식별자 필드가 추가되었습니다. API 응답 시 각 재주문점 레코드가 개별 식별자와 함께 제공됩니다.

[coderabbitai]

워크스루

RopResDto DTO 클래스에 새로운 ropId 필드를 추가하고 생성자 시그니처를 업데이트했습니다. RopQueryRepositoryImpl은 쿼리 실행 시 새 필드를 조회에 포함시키도록 프로젝션을 수정했습니다.

변경사항

내용 / 파일(들)	변경 요약
DTO 필드 추가 src/main/java/com/sampoom/backend/api/rop/dto/RopResDto.java	ropId 필드(Long 타입) 추가, 생성자 파라미터에 ropId 포함 (quantity 다음), 생성자 내부에서 필드 할당
쿼리 프로젝션 업데이트 src/main/java/com/sampoom/backend/api/rop/repository/RopQueryRepositoryImpl.java	QRopResDto 생성자 호출에 rop.id 인자 추가, select 절 업데이트

예상 코드 리뷰 소요 시간

🎯 2 (단순) | ⏱️ ~10분

• 주의 영역:
  • 다른 곳에서 RopResDto 생성자를 호출하는 모든 위치 확인 필요
  • 새로 추가된 ropId 필드가 API 응답에서 올바르게 직렬화되는지 검증

관련 PR

• fix: rop조회 #41: RopResDto 생성자 및 RopQueryRepositoryImpl의 QRopResDto/QRop 프로젝션 사용 방식을 수정하는 관련 변경사항
• feat: rop 조회 #36: 동일한 RopResDto DTO 및 RopQueryRepositoryImpl의 QRopResDto 생성자 호출을 업데이트하는 직접적 관련 변경사항

제안 레이블

ready-to-merge

제안 리뷰어

• Lee-Jong-Jin
• CHOOSLA
• yangjiseonn

시

🐰 새로운 필드 ropId가 태어났네,
DTO에 자리 잡고 쿼리에 반영되네,
생성자도 춤을 추며 매끄럽게 흐르고,
데이터는 한 발 더 가까워졌어! 🌟

Pre-merge checks and finishing touches

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	PR 제목은 'fix: rop 조회'로 변경사항의 핵심인 ROP 조회 API에 ropId 필드를 추가하는 내용을 명확하게 반영합니다.

✨ 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 SPM-417

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between e686fcb and f9ebd53.

📒 Files selected for processing (2)

• src/main/java/com/sampoom/backend/api/rop/dto/RopResDto.java (2 hunks)
• src/main/java/com/sampoom/backend/api/rop/repository/RopQueryRepositoryImpl.java (1 hunks)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: Analyze (java-kotlin)

🔇 Additional comments (2)

src/main/java/com/sampoom/backend/api/rop/repository/RopQueryRepositoryImpl.java (1)

59-59: LGTM! ROP ID가 올바르게 추가되었습니다.

QueryDSL 프로젝션에 rop.id가 추가되어 PR 목표를 달성합니다. 생성자 파라미터 순서도 RopResDto의 업데이트된 생성자 시그니처와 일치합니다.

src/main/java/com/sampoom/backend/api/rop/dto/RopResDto.java (1)

21-21: LGTM! ropId 필드가 올바르게 추가되었습니다.

새로운 ropId 필드가 일관되게 추가되었습니다:

• 필드 선언 (line 21)
• @QueryProjection 생성자 파라미터 (line 30)
• 필드 할당 (line 37)

파라미터 순서가 RopQueryRepositoryImpl의 쿼리 프로젝션과 정확히 일치합니다.

Also applies to: 29-31, 37-37

---

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.}

[taemin3]

확인했습니다