Skip to content

fix(social): 대댓글 카운트·좋아요 아이콘 노출 정합성 수정#175

Merged
coldsunn merged 1 commit into
developfrom
fix/count-display-consistency
Jun 2, 2026
Merged

fix(social): 대댓글 카운트·좋아요 아이콘 노출 정합성 수정#175
coldsunn merged 1 commit into
developfrom
fix/count-display-consistency

Conversation

@coldsunn
Copy link
Copy Markdown
Collaborator

@coldsunn coldsunn commented Jun 2, 2026

📝 요약(Summary)

"답글 N개 더보기"의 N과 펼친 결과 개수 불일치
좋아요 아이콘 상태(hasLikes)와 좋아요 리스트 결과 불일치 두 케이스 수정

  1. visibleSubCommentCount ↔ 대댓글 목록 불일치
  • 공개 부모 아래 비밀 대댓글이 카운트에서 빠지지만 목록에는 마스킹된 채로 노출되던 문제
  • 카운트 쿼리(countVisibleSubCommentsByParentIds)에서 비밀 필터 절 제거
  • 마스킹되어 노출되는 비밀 대댓글까지 카운트에 포함시켜 "펼친 결과 개수 = 카운트"로 정렬
  • CommentQueryService.buildSubCountMap의 visibleSecretParentIds 계산 로직 제거
  1. hasLikes ↔ 좋아요 리스트 불일치
  • 차단·정지·탈퇴 사용자의 좋아요가 hasLikes에는 카운트되지만 좋아요 리스트에서는 빠져, 아이콘은 채워졌는데 펼치면 빈 리스트가 보이던 문제 (게시글·댓글·대댓글 전부 해당)
  • findReportIdsWithLikes / findCommentIdsWithLikes에 user.id not in :excludedUserIds + user.deletedAt is null 필터 추가
  • FeedQueryService에 getExcludedUserIds 헬퍼 + UserBlockRepository 주입 추가 (CommentQueryService·LikeQueryService와 동일 패턴)

🔗 Related Issue

  • Closes:

💬 공유사항

✅ PR Checklist

PR이 다음 요구 사항을 충족하는지 확인하세요.

  • PR 제목을 커밋 메시지 컨벤션에 맞게 작성했습니다.

visibleSubCommentCount ↔ 대댓글 목록 불일치를 해결하기 위해 (countVisibleSubCommentsByParentIds에서 비밀 필터 절 제거하여
마스킹되어 노출되는 비밀 대댓글까지 카운트에 포함, CommentQueryService.buildSubCountMap의 visibleSecretParentIds 계산 로직 제거,
hasLikes ↔ 좋아요 리스트 불일치를 해결하기 위해 findReportIdsWithLikes / findCommentIdsWithLikes에 user.idnot in
:excludedUserIds + user.deletedAt is null 필터 추가, FeedQueryService에 getExcludedUserIds 헬퍼 +
UserBlockRepository 주입 추가
@1Seob
Copy link
Copy Markdown
Collaborator

1Seob commented Jun 2, 2026

LGTM!

@coldsunn coldsunn merged commit a7fc885 into develop Jun 2, 2026
1 check passed
@coldsunn coldsunn deleted the fix/count-display-consistency branch June 2, 2026 11:59
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.

2 participants