-
Notifications
You must be signed in to change notification settings - Fork 0
Open
Description
배경
- PR Feature: menu sorting 구현 #273 (메뉴 정렬 기능)에서 Menu 엔티티의 updateSortOrder 메서드에 입력 검증이 없어 null/음수 값이 저장될 수 있는 가능성이 지적되었습니다.
- 관리 API의 MenuSortUpdateRequest DTO에도 값 검증(Bean Validation) 도입을 함께 고려할 필요가 있습니다.
영향
- 잘못된 정렬 값이 저장되면 메뉴 목록 정렬이 불안정해질 수 있고, 이후 로직에서 예외/에지 케이스가 발생할 수 있습니다.
할 일(고레벨)
- 엔티티 레벨: sortOrder 입력에 대한 방어적 검증 추가.
- API 레벨(선택): MenuSortUpdateRequest에 값 검증 도입(예: null/음수 방지) 및 컨트롤러 유효성 검사 적용 여부 확인.
- 데이터 점검: 기존 데이터에 유효하지 않은 sortOrder가 존재하는지 점검하고 필요 시 초기화 전략 수립(별도 이슈로 분리 가능).
- 테스트: 성공/실패(경계값 포함) 케이스에 대한 단위/통합 테스트 추가.
- 문서화: 에러 메시지 및 제약사항을 운영/개발 문서에 반영.
참고
- 관련 PR: Feature: menu sorting 구현 #273
- 해당 코멘트: Feature: menu sorting 구현 #273 (comment)
- 요청자: @Jjiggu
Metadata
Metadata
Assignees
Labels
No labels