Repository navigation
refactor(afternote): 미디어 URL 빈 값 정규화를 입구로 모은다 (#2207) - #2208
Merged
Merged
Conversation
상세 data 매퍼와 썸네일 결과 반영에서 빈 문자열을 null 로 맞추고, 폼 매퍼의 미디어 URL ifBlank 를 걷어낸다.
Contributor
|
이 PR 은
|
…o refactor/2207-media-blank-entry # Conflicts: # feature/afternote/data/src/main/kotlin/com/afternote/feature/afternote/data/mapper/AfternoteDetailMapper.kt
Sadturtleman
approved these changes
Oct 4, 2026
Sadturtleman
left a comment
Collaborator
There was a problem hiding this comment.
서버 상세 매퍼와 썸네일 결과 입구에서 빈 값이 null 로 맞춰지고, 사진·영상의 다른 입력 경로도 MediaInput 판정과 ofOrNull 로 이미 걸러지는 것을 확인했습니다. 승인합니다.
github-merge-queue
Bot
removed this pull request from the merge queue due to no response for status checks
Oct 5, 2026
Contributor
merge queue 방출 — 재투입했다사유 |
This was referenced Oct 5, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #2207
Refs #2163
추억 노트의 사진·영상·썸네일 URL은 쓰는 자리마다
ifBlank { null }로 빈 문자열을 걸러 왔습니다. 이 PR은 값이 들어오는 입구에서 한 번만 빈 값을null로 맞추고, 폼 매퍼에서는 미디어 URL 정규화를 걷어냅니다.입구
AfternoteMemorialDto.toMemorialMedia()에서 빈 값을null로 맞춥니다. BEmain은 빈 미디어를null로 저장하므로(PlaylistRelationStrategy.createMediaSlot·applyMediaSlot) 옛 데이터 방어입니다.photoMediaInput이 이미 빈 기존 URL을MediaInput.None으로 보내null이 됩니다. 변경 없음.MemorialVideoAttachment.ofOrNull이 빈 영상 URL을 첨부 없음으로 만듭니다. 변경 없음.withSelectionThumbnail에서 빈 값을null로 맞춥니다. 이 경로만ofOrNull을 거치지 않았습니다.계정 id·비밀번호의
ifBlank는 사용자가 입력하는 텍스트 칸이라 빈 문자열이 정상 입력이므로 그대로 둡니다.리뷰어에게
fileUrl을 돌려주는 경우는 이제null로 바뀌지 않고 빈 문자열로 서버에 가서 저장이 거절됩니다. 예전에는 조용히 미디어 없음으로 저장됐습니다. 서버 버그를 드러내는 쪽이 맞다고 보고 업로드 결과는 입구로 삼지 않았습니다.refactor/2166-media-mapping입니다. #2167이 모은memorialVideoPayload에서 정규화를 걷어내므로 그 위에 얹었습니다. refactor(afternote): 에디터 곡·영상 변환 규칙을 공통화한다 (#2166) #2167 위의 에디터 스택(refactor(afternote): 합본 UserRepository 대신 좁은 계약 UserReceiverRepository·MyProfileRepository 를 주입한다 (#1743) #1837 이후)은 이 변경에 의존하지 않습니다.검증
HEAD
fff76a315(부모 브랜치의 develop 병합을 받은 뒤)에서 다음을 통과했습니다.:feature:afternote:data:testDebugUnitTest,:feature:afternote:presentation:testDebugUnitTest, 두 모듈 ktlintCheck:app:compileDebugUnitTestKotlin,:konsist:test --rerun-tasksAfternoteDetailMapperTest의 빈 미디어 URL 정규화,EditableMemorialVideoTest의 빈 썸네일 결과. data 매퍼의 사진 정규화를 빼면 앞의 테스트가 실패하는 것을 확인했습니다.🧪 CI Test Plan
{ "androidTest": { "mode": "none", "reason": "data 매퍼와 폼 상태의 문자열 정규화 위치만 옮기며 UI·Android API 변경이 없어 기존 생성·부분 PATCH 단위 테스트와 새 회귀 테스트로 검증합니다.", "tests": [] } }