feat: 스포일러 글일 경우 이미지 보이지 않도록 구현

[s9hn]

📌𝘐𝘴𝘴𝘶𝘦𝘴

• closed feat: 스포일러 글일 경우 이미지 보이지 않도록 구현 #763

📎𝘞𝘰𝘳𝘬 𝘋𝘦𝘴𝘤𝘳𝘪𝘱𝘵𝘪𝘰𝘯

• 스포일러 커스텀 게터 프로퍼티 하나 추가했습니다.

Summary by CodeRabbit

릴리스 노트

• Refactor
  • 피드 항목의 이미지 표시 논리를 개선하여 일관된 가시성 처리를 구현했습니다.

[coderabbitai]

Walkthrough

FeedModel에 스포일러 여부와 이미지 존재 여부를 검사하는 isVisible 계산 속성을 추가하고, item_feed.xml에서 이미지 UI 요소의 가시성 바인딩을 새 속성으로 통일했습니다.

Changes

Cohort / File(s)	Summary
모델 계산 속성 추가 app/src/main/java/com/into/websoso/ui/main/feed/model/FeedModel.kt	isVisible 계산 속성 추가: 스포일러가 아니고 이미지가 존재할 때만 참을 반환
레이아웃 바인딩 통일 app/src/main/res/layout/item_feed.xml	iv_feed_image, tv_feed_image_count의 가시성 바인딩을 feed.visible로 변경

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

• isVisible 속성의 논리 검증 (!isSpoiler && imageUrls.isNotEmpty())
• 데이터 바인딩 표현식이 올바르게 적용되었는지 확인
• 스포일러 로직과 이미지 표시 로직의 일관성 검토

Possibly related PRs

• feat: 이미지 첨부 기능 구현 #709: 동일하게 피드 모델의 이미지 데이터 기반 가시성 로직을 추가하고 item_feed.xml의 UI 바인딩을 수정하는 변경사항

Suggested labels

♻️ [REFACTOR]

Suggested reviewers

• yeonjeen
• m6z1

Poem

🐰 스포일러 가려 숨겨주고,
이미지들을 현명하게 골라,
계산 속성으로 깔끔하게 묶어,
XML에 바인딩 맞춰두니,
사용자의 눈은 행복해져라! ✨

Pre-merge checks and finishing touches

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	PR 제목은 '스포일러 글일 경우 이미지 보이지 않도록 구현'이며, 실제 변경 내용과 정확하게 일치합니다. 코드 변경 사항은 스포일러 여부를 확인하는 isVisible 프로퍼티를 추가하고, UI 레이아웃에서 이를 통해 이미지 표시 여부를 제어하는 것입니다. 제목이 명확하고 구체적이며 주요 변경사항을 정확하게 반영하고 있습니다.
Description check	✅ Passed	PR 설명은 템플릿 구조를 따르고 있으며, '📌이슈' 섹션에서 #763을 명시하고 '📎작업 설명' 섹션에서 스포일러 커스텀 게터 프로퍼티 추가 내용을 기술했습니다. 다만 '📷스크린샷'과 '💬리뷰어들께' 섹션은 미작성 상태이지만, 필수 정보(이슈 번호와 작업 설명)는 충분히 포함되어 있습니다.
Linked Issues check	✅ Passed	Issue #763의 목적은 '스포일러 글인 경우 피드의 이미지를 보이지 않도록 기능 구현'입니다. PR의 코드 변경 사항은 FeedModel에 isVisible 프로퍼티를 추가하고 (!isSpoiler && imageUrls.isNotEmpty()), 레이아웃에서 이를 적용하여 스포일러 글일 때 이미지를 숨기는 기능을 완벽하게 구현했습니다. 링크된 이슈의 요구사항을 충족합니다.
Out of Scope Changes check	✅ Passed	PR의 모든 변경사항은 Issue #763의 목표인 '스포일러 글일 경우 이미지 보이지 않도록 구현'과 직접적으로 관련이 있습니다. FeedModel의 isVisible 프로퍼티 추가와 레이아웃의 가시성 바인딩 변경은 모두 이 기능 구현을 위한 필수 변경사항이며, 범위를 벗어난 변경사항은 없습니다.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch feat/763

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 0

🧹 Nitpick comments (1)

app/src/main/java/com/into/websoso/ui/main/feed/model/FeedModel.kt (1)

23-23: 구현이 올바르게 되었습니다!

스포일러 여부와 이미지 존재 여부를 모두 확인하는 로직이 정확하며, 계산 속성(computed property)을 사용한 것이 적절합니다. 가시성 로직을 한 곳에서 관리할 수 있어 유지보수성이 향상됩니다.

선택적 제안: 속성 이름을 isImageVisible 또는 shouldShowImages처럼 더 구체적으로 변경하면 이 속성이 이미지 가시성을 위한 것임이 더 명확해집니다. 하지만 현재 이름도 컨텍스트상 충분히 명확하므로 변경은 선택사항입니다.

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between a9f5eaf and a6386d8.

📒 Files selected for processing (2)

• app/src/main/java/com/into/websoso/ui/main/feed/model/FeedModel.kt (1 hunks)
• app/src/main/res/layout/item_feed.xml (2 hunks)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: build

🔇 Additional comments (1)

app/src/main/res/layout/item_feed.xml (1)

133-133: 통합된 가시성 로직을 올바르게 적용했습니다!

이미지 뷰와 이미지 개수 배지 모두 새로운 feed.visible 속성을 사용하여 가시성을 제어하도록 변경되었습니다. 이를 통해 스포일러 게시물의 이미지가 적절하게 숨겨지며, 두 UI 요소의 가시성이 일관되게 동기화됩니다.

Also applies to: 153-153