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

댓글 불러오기 동작에 트리거 추가 #2344

Merged
merged 1 commit into from Dec 10, 2018

Conversation

2 participants
@kijin
Copy link
Contributor

kijin commented Nov 24, 2018

예전에 document.getDocumentList에 트리거를 추가하여 여러 서드파티 자료에서 유용하게 사용하고 있습니다. 그러나 댓글에는 이와 같은 트리거가 없고, 스킨에서 $oDocument->getComments() 함수로 댓글을 불러오고 있다 보니 서드파티 자료에서 댓글 목록에 접근하려면 이미 렌더링된 HTML 결과물을 정규식으로 파싱해야 하는 불편을 겪습니다. 물론 이렇게 하면 성능과 정확성도 떨어집니다.

이 PR에서는 라이믹스에서 지원하는 것과 동일한 트리거를 추가하여 서드파티 자료들이 댓글 목록에 편리하게 접근할 수 있도록 합니다.

  1. commentModel 클래스의 getCommentList() 메소드에 beforeafter 트리거 추가 (DB에서 불러오는 결과를 직접 조작할 필요가 있을 때 사용)
  2. documentItem 클래스의 getComments() 메소드에 after 트리거 추가 (스킨에서 흔히 사용하는 댓글 목록에 간편하게 접근할 수 있도록 하여 편의성 강화)

@bnu bnu self-assigned this Nov 26, 2018

@bnu bnu added this to the 1.11-next milestone Nov 26, 2018

@bnu bnu closed this Dec 10, 2018

@bnu bnu added this to 이슈 종료 in 이슈 진행 상황 via automation Dec 10, 2018

@bnu bnu reopened this Dec 10, 2018

@bnu bnu merged commit 295bf94 into xpressengine:develop Dec 10, 2018

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

dorami added a commit to daolcms/daolcms that referenced this pull request Dec 23, 2018

dorami added a commit to daolcms/daolcms that referenced this pull request Dec 23, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment