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

[#18] [#5] 연관관계로 인한 삭제 버그 해결 #19

Merged
merged 2 commits into from
Jul 14, 2021
Merged

Conversation

mangdo
Copy link
Contributor

@mangdo mangdo commented Jul 14, 2021

연관된 Reply를 삭제한 후에, Plan을 삭제합니다.
@manytoone의 fetch 기본값은 EAGER이기 때문에 LAZY로 수정하였습니다.

Copy link
Contributor

@cwangg897 cwangg897 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

확인

Copy link
Contributor

@SOULIKK SOULIKK left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

게시글에 연관된 댓글들 먼저 삭제 후 해당 글을 삭제한다
replyRepository.deleteAllByPlan(plan);
planRepository.deleteById(id);

@mangdo mangdo merged commit c9ef508 into master Jul 14, 2021
@mangdo mangdo deleted the dto_refac branch July 14, 2021 00:54
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.

None yet

3 participants