fix: 상태 한글로

[vivivim]

📝 Summary

다 바꾸고 필요없는 것들 정리

🙏 Question & PR point

📬 Reference

Summary by CodeRabbit

릴리즈 노트

• API 변경

  • 미사용 REST 엔드포인트 및 관련 컨트롤러 제거: 부품 그룹 조회, 출고 이력, 주문 처리 기능 제거

• 개선

  • 상태 정보가 한글 레이블로 노출되도록 변경되어 UI/응답에서 한국어 상태명이 표시됨
  • 일부 응답 객체의 상태 표현 방식이 한글 문자열로 반환되도록 조정됨

[taemin3]

확인했습니다

[coderabbitai]

요약

REST API 컨트롤러 두 개(OutHistoryController, OrderController)를 제거하고, InventoryController에서 getPartsByGroup 엔드포인트를 삭제했습니다. Status 열거형에 한글 레이블을 추가하여 getKorean()을 노출했고, 여러 DTO에서 문자열 반환용 커스텀 게터를 추가/변경하여 직렬화 출력이 변경되었습니다.

Walkthrough

두 컨트롤러 파일과 InventoryController의 특정 엔드포인트가 삭제되었고, Status enum에 한글 라벨 필드가 추가되었으며 여러 DTO에 한글 라벨을 반환하는 커스텀 게터들이 도입되었습니다. (50단어 이내)

Changes

집단 / 파일(들)	변경 요약
컨트롤러 삭제 src/main/java/com/sampoom/backend/api/inventory/controller/OutHistoryController.java, src/main/java/com/sampoom/backend/api/order/controller/OrderController.java	두 REST 컨트롤러 클래스 전체 삭제 (OutHistoryController: 파일 삭제, OrderController: 주문 처리 PATCH 엔드포인트 삭제)
InventoryController 조정 src/main/java/com/sampoom/backend/api/inventory/controller/InventoryController.java	getPartsByGroup(Long warehouseId, Long groupId) 메서드 삭제
Status enum 확장 src/main/java/com/sampoom/backend/common/entitiy/Status.java	enum 상수에 한글 레이블 추가(ACTIVE("활성"), INACTIVE("비활성")), private final String korean 필드 및 getKorean() 게터 추가
DTO 직렬화/게터 변경 src/main/java/com/sampoom/backend/api/inventory/dto/PartResDto.java, src/main/java/com/sampoom/backend/api/rop/dto/RopResDto.java	PartResDto.getStatus()를 QuantityStatus→String으로 변경(status == null ? null : status.getKorean()), RopResDto에 getAutoOrderStatus() 추가(autoOrderStatus.getKorean() 반환)
Order 관련 DTO/열거형 변경 src/main/java/com/sampoom/backend/api/order/dto/OrderStatus.java, src/main/java/com/sampoom/backend/api/order/dto/POResDto.java	OrderStatus의 @JsonValue 직렬화 헬퍼(toKorean()) 제거; POResDto.getOrderStatus() 호출을 toKorean() → getKorean()으로 변경

Sequence Diagram(s)

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~45분

• 주의 검토 대상:
  • 삭제된 컨트롤러(OutHistoryController, OrderController)의 대체 경로/영향 확인
  • Status enum 생성자 변경에 따른 모든 사용처(직렬화, DB 매핑 등) 영향 검토
  • DTO에서 열거형 → String 반환으로 인한 API 응답 호환성 확인
  • RopResDto의 새 게터가 Lombok 자동 게터와 충돌하지 않는지 확인

Possibly related PRs

• #6 — InventoryController의 getParts 엔드포인트 추가/수정과 직접 충돌 가능 (동일 메서드 영역 변경)
• #16 — OrderController 및 OrderStatus 관련 변경과 중복/상호작용 가능성 있음
• #36 — RopResDto/Status 사용 방식 변경과 직접 연관됨

Suggested reviewers

• Sangyoon98
• Lee-Jong-Jin
• CHOOSLA
• yangjiseonn

Poem

🐇 삭제된 경로를 지나며 나는 깡충,
한글 깃발 든 Status가 반짝이네,
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 제목은 변경 사항의 주요 내용을 명확하게 반영합니다. 코드 전반에 걸쳐 상태(Status) 관련 필드들을 한글로 변환하는 것이 주요 변경 사항이며, 동시에 불필요한 컨트롤러와 메서드를 제거했습니다.

✨ 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-391

---

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

[coderabbitai]

Actionable comments posted: 1

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 1d21b0b and 02df0e1.

📒 Files selected for processing (8)

• src/main/java/com/sampoom/backend/api/inventory/controller/InventoryController.java (0 hunks)
• src/main/java/com/sampoom/backend/api/inventory/controller/OutHistoryController.java (0 hunks)
• src/main/java/com/sampoom/backend/api/inventory/dto/PartResDto.java (1 hunks)
• src/main/java/com/sampoom/backend/api/order/controller/OrderController.java (0 hunks)
• src/main/java/com/sampoom/backend/api/order/dto/OrderStatus.java (0 hunks)
• src/main/java/com/sampoom/backend/api/order/dto/POResDto.java (1 hunks)
• src/main/java/com/sampoom/backend/api/rop/dto/RopResDto.java (1 hunks)
• src/main/java/com/sampoom/backend/common/entitiy/Status.java (1 hunks)

💤 Files with no reviewable changes (4)

• src/main/java/com/sampoom/backend/api/inventory/controller/InventoryController.java
• src/main/java/com/sampoom/backend/api/order/controller/OrderController.java
• src/main/java/com/sampoom/backend/api/inventory/controller/OutHistoryController.java
• src/main/java/com/sampoom/backend/api/order/dto/OrderStatus.java

[coderabbitai]

Actionable comments posted: 0

🧹 Nitpick comments (1)

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

43-48: 이전에 제기된 NPE 이슈가 해결되었습니다.

null 체크가 추가되어 autoOrderStatus가 null일 때 안전하게 null을 반환합니다. 이전 리뷰에서 지적된 NullPointerException 위험이 제거되었습니다.

다만, 필드 타입(Status)과 게터 반환 타입(String)이 다르면 JavaBean 규약에서 벗어나며, 일부 리플렉션 기반 프레임워크에서 혼란을 줄 수 있습니다. 더 명확한 대안으로는 메서드 이름을 getAutoOrderStatusKorean()으로 분리하거나, @JsonGetter("autoOrderStatus")를 사용하여 직렬화 시에만 한글 라벨을 반환하는 방법이 있습니다.

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 02df0e1 and 751f541.

📒 Files selected for processing (1)

• src/main/java/com/sampoom/backend/api/rop/dto/RopResDto.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)