Skip to content

fix: 순서를 생략한 사진 추가를 기존 사진 뒤에 붙이기 - #137

Merged
redjungi09 merged 2 commits into
developfrom
fix/photo-append-sort-order/#133
Sep 3, 2026
Merged

fix: 순서를 생략한 사진 추가를 기존 사진 뒤에 붙이기#137
redjungi09 merged 2 commits into
developfrom
fix/photo-append-sort-order/#133

Conversation

@redjungi09

Copy link
Copy Markdown
Collaborator

관련 이슈

변경 내용

  • TreatmentRecord.nextSortOrder() 추가 — 현재 최대 순번 다음, 사진이 없으면 0
  • sort_order 를 생략한 사진 추가는 기존 사진들 뒤에 붙도록 변경
  • AddCommand.sortOrder 를 nullable 로 바꿔 "생략"과 "0 지정"을 구분

작업 목적

기본값 0 이 이미 0 인 사진과 동점이 되고, 조회 정렬이 동점을 created_at 오름차순으로 풀어 새로 올린 사진이 뒤로 밀렸다. 목록 썸네일은 첫 사진을 쓰므로 사진을 바꿔도 화면에는 옛 사진이 계속 보였다.

테스트 방법

  • 단위 테스트 추가/수정
    • 통합 테스트 완료
    • 수동 테스트 완료

체크리스트

  • 코드가 프로젝트의 코딩 스타일을 따릅니다
    • 자체 코드 리뷰를 완료했습니다
    • 변경 사항에 대한 테스트를 추가했습니다
    • Breaking Changes가 없습니다

기타 사항

  • 정렬 규칙이 아니라 추가 시점의 기본값을 고쳤다. 정렬을 바꾸면 이미 저장된 기록의 표시 순서까지 흔들린다
  • 순번은 기록 행을 쓰기 잠금으로 읽은 뒤 계산해 동시 추가가 같은 값을 집지 않는다
  • 등록(POST /treatment-records)은 이미 배열 인덱스를 기본값으로 써서 이 문제가 없었다. 변경하지 않았다

기본값 0 이 이미 0 인 사진과 동점이 되고, 조회 정렬이 동점을 created_at
오름차순으로 풀어 새로 올린 사진이 뒤로 밀렸다. 목록 썸네일은 첫 사진을
쓰므로 사진을 바꿔도 화면에는 옛 사진이 계속 보였다.

기존 행의 순서를 건드리면 이미 저장된 기록의 표시 순서가 흔들리므로
정렬 규칙이 아니라 추가 시점의 기본값을 고친다.
@redjungi09
redjungi09 merged commit 39c7129 into develop Sep 3, 2026
1 check passed
@redjungi09
redjungi09 deleted the fix/photo-append-sort-order/#133 branch September 3, 2026 01:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant