search

jongpak / prob-board

A simple web board based on prob-framework
MIT License
1 stars 0 forks source link

ViewModelFilter 필터에서 엔티티 값 수정이 적절한지..? #25

Open jongpak opened 7 years ago

jongpak commented 7 years ago

https://github.com/jongpak/prob-board/commit/6fbe67d1cd8f7f8a3fc14f182bb828b00da748f3

현재 ViewModelFilter의 위치대로라면 뷰모델필터에서 엔티티를 수정하고 commit/flush 하는 경우 그대로 저장이 된다. (즉, 원본 데이터가 변경됨) 이 동작이 적절한 것인지 판단해볼 필요가 있음.

https://github.com/jongpak/prob-board/blob/6fbe67d1cd8f7f8a3fc14f182bb828b00da748f3/core/ControllerDispatcher/ViewRenderer.php#L45

뷰모델필터의 권한을 어디까지 부여하는가의 문제임.

jongpak commented 7 years ago

VO객체로 불변하게 만드는 것이 올바른 듯 하다

jongpak commented 7 years ago

불변객체로 한다면, https://github.com/jongpak/prob-board/blob/6fbe67d1cd8f7f8a3fc14f182bb828b00da748f3/core/ControllerDispatcher/ViewRenderer.php#L43

이 부분에서 clone 을 주어야함