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.

Sign up for GitHub

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

Jump to bottom

[리팩토링] AuditindFields의 잘못 표현된 부분 개선 #52

Closed
bokyoung89 opened this issue Sep 8, 2023 · 0 comments · Fixed by #53
Closed

[리팩토링] AuditindFields의 잘못 표현된 부분 개선 #52

bokyoung89 opened this issue Sep 8, 2023 · 0 comments · Fixed by #53
Assignees
@bokyoung89
Labels
enhancement New feature or request
Milestone
work-board-project

Comments

@bokyoung89
Copy link
Owner

bokyoung89 commented Sep 8, 2023 •
edited
Loading

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

위 내용이 당장은 비즈니스 요구사항이 없어 문제가 되지 않았으나,
이제 #51 를 작업하면서, 회원 도메인에서 인증이 없는 상태의 회원 정보를 저장하기 위해서
작성자(createdBy), 수정자(modifiedBy)를 엔티티가 직접 작성해야 하는 요구사항이 생길 것이므로
이것이 가능하게끔 접근 제어자를 제대로 수정해주도록 한다.
@bokyoung89 bokyoung89 converted this from a draft issue Sep 8, 2023
@bokyoung89 bokyoung89 self-assigned this Sep 8, 2023
@bokyoung89 bokyoung89 added the enhancement New feature or request label Sep 8, 2023
@bokyoung89 bokyoung89 added this to the work-board-project milestone Sep 8, 2023
bokyoung89 added a commit that referenced this issue Sep 8, 2023
@bokyoung89
refactor #52 - AuditingFields: 필드 접근제어자를 protected로 수정
2e00e78 
원래 이 부분은 abstract class에 맞게 `protected`여야 했다.
그러나 초기 설계에서 지나치게 닫힌 형태로 작업함.

이제 회원 엔티티가 이 부분을 직접 참조해야 하므로
제대로 수정해줌.
이것으로 생성자에서 인증 없이 회원 정보를 저장할 수 있음.
필요한 시나리오는 회원 가입, 회원 생성.
@bokyoung89 bokyoung89 mentioned this issue Sep 8, 2023
Merged
bokyoung89 added a commit that referenced this issue Sep 8, 2023
@bokyoung89
Merge pull request #53 from bokyoung89/feature/#52-refact
0224429 
[리팩토링] AuditingFields 의 잘못 표현된 부분 개선
@github-project-automation github-project-automation bot moved this from 🏗 In progress to ✅ Done in work-board-project Sep 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bokyoung89 bokyoung89

Labels
enhancement New feature or request
Projects
work-board-project
Status: ✅ Done
Milestone
work-board-project
Development

Successfully merging a pull request may close this issue.

[리팩토링] AuditingFields 의 잘못 표현된 부분 개선 bokyoung89/work-board-project
1 participant
@bokyoung89