Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GETP-185 fix: 피플 프로필이 미등록 상태임에도 등록할 수 없는 오류 수정 #118

Merged
merged 1 commit into from
Aug 13, 2024

Conversation

scv1702
Copy link
Member

@scv1702 scv1702 commented Aug 13, 2024

✨ 구현한 기능

  • 피플 프로필이 미등록 상태임에도 등록할 수 없는 오류 수정

📢 논의하고 싶은 내용

  • JPA의 @Embeddable에서, JPA의 구현체로 Hibernate를 사용하는 경우 @Embeddable 객체의 모든 필드가 null인 경우 Hibernate는 해당 객체 자체를 null로 설정합니다. 이를 이용해 @EmbeddablePeopleProfilenull인 경우 프로필을 등록하지 않은 것으로 판단했습니다. 그러나 PeopleProfile 객체의 hastags, techStacks, portfolios의 기본값이 null이 아니라 빈 ArrayList이기 때문에 모든 필드가 null이 아님으로 프로필을 등록하지 않아도 PeopleProfilenull이 아니게 되어 버그가 발생했었습니다. 이를 이번 PR과 같이 수정하였습니다.

🎸 기타

Copy link

Test Results

 86 files   - 4   86 suites   - 4   32s ⏱️ +12s
142 tests +3  142 ✅ +3  0 💤 ±0  0 ❌ ±0 
142 runs  +2  142 ✅ +2  0 💤 ±0  0 ❌ ±0 

Results for commit 56e9a8f. ± Comparison against base commit f305f1c.

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