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

AuditingFields Filed Protected로 변경 #86

Closed
Tracked by #85
rhakdnj opened this issue Dec 27, 2022 · 0 comments
Closed
Tracked by #85

AuditingFields Filed Protected로 변경 #86

rhakdnj opened this issue Dec 27, 2022 · 0 comments

Comments

@rhakdnj
Copy link
Owner

rhakdnj commented Dec 27, 2022

AuditingFields 클래스는 추상 클래스이고, 각 필드는 상속 받는 자식 엔티티에서 접근 및 수정이 가능해야 한다.
따라서 접근 제어자를 protected로 했어야 했는데, 이 접근 제어를 초기 설계에서 지나치게 폐쇄적으로 작성했다.

위 내용이 당장은 비즈니스 요구사항이 없어 문제가 되지 않았으나,
카카오 로그인을 작업하면서, 회원 도메인에서 인증이 없는 상태의 회원 정보를 저장하기 위해서
작성자(createdBy), 수정자(modifiedBy)를 엔티티가 직접 작성해야 하는 요구사항이 생길 것이므로
이것이 가능하게끔 접근 제어자를 제대로 수정해주도록 한다.

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

No branches or pull requests

1 participant