차량 정보 필드 null-safety 검증 추가 필요 (PurchaseOrder 관련)

[coderabbitai]

배경

PR #30에서 수리 접수 없이 부품만 요청할 수 있도록 vehicleNumber와 vehicleModel 컬럼의 nullable = false 제약을 제거했습니다. 하지만 null 값 처리에 대한 검증 로직이 누락되어 있어, 본사 발주 API 수정 시 함께 보완이 필요합니다.

수정이 필요한 부분

1. PurchaseOrderRequest.java

   • vehicleNumber, vehicleModel 필드에 검증 어노테이션 누락
   • null 값이 그대로 서비스 레이어로 전달될 수 있음

2. PurchaseOrderServiceImpl.java (Line 38)

   • request.getVehicleNumber() 사용 시 null 체크 없음
   • null 값이 엔티티에 그대로 설정됨

3. PurchaseOrder.java - createShipmentCommand() 메서드 (Line 145)

   • this.vehicleNumber를 null 체크 없이 ShipmentCommand 생성자에 전달
   • 다운스트림에서 오류 발생 가능

4. 비즈니스 규칙 검증 로직 부재

   • receiptNum 또는 vehicleNumber 중 하나는 반드시 존재해야 하는 규칙 강제 불가
   • 서비스 또는 컨트롤러 레벨에서 검증 필요

참고 링크

• PR: [Main] 수리접수 ID(Long) → 접수번호(String)으로 변경 #30
• 관련 코멘트: [Main] 수리접수 ID(Long) → 접수번호(String)으로 변경 #30 (comment)
• 요청자: @ksoheee

작업 시점

본사 발주 관련 API 수정 시 함께 처리 예정