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

임시저장글의 대한 오작동(?) 제보... #827

Closed
bjrambo opened this issue Jul 4, 2014 · 7 comments
Closed

임시저장글의 대한 오작동(?) 제보... #827

bjrambo opened this issue Jul 4, 2014 · 7 comments

Comments

@bjrambo
Copy link
Contributor

bjrambo commented Jul 4, 2014

일단 제가 생각하는 임시저장글의 경우 XE팀이 생각하는 임시저장글의 대한 개념을 다르게 이해할 수도 있음을 미리 말씀드립니다.

제가 생각하는 임시저장의 경우 회원들에게 노출되지 않으며 회원들이 보기 직전에 글들을 수정하고 최종 결과물을 비로서 발행에 옮김으로 하여 최종 글쓰기를 종료하는 작업중의 한가지라고 봐야 한다고 생각합니다.

그런데 XE코어에서 임시저장을 이용하여 그 글을 저장하였을 경우 문제가 있습니다.

insert.ducment 이라는 트리거가 갑자기 작동하여 insertdocument 관련 트리거를 사용하는 모듈에서 작동을 하는 것으로 보여집니다.

임시저장 -> insertdocument 트리거 실행명령-> 기타 모듈에서 작업 진행..

예를 들겠습니다. 사이트에 설치된 모듈은 XE알림센터Lite 와, 메뉴에 새 글 표시 모둘을 사용중이라 볼때 둘다 insertdocument 트리거의 대한 작동을 하도록 되어있습니다.
만일

@BJ람보

라는 글을 적어놓고 임시저장을 할 경우 알림센터는 BJ람보에게 너님 얘가 언급함 이렇게 알려주게 되며, 메뉴의 새글 표시에 N이 뜨게 되지요.
정작 N이뜬 메뉴를 접속하게 될 경우 그글은 없으며 다른회원의 경우 어떤지 확인은 안됬지만, 글쓴이의 경우 링크를 통해서 열람을 할 수 있는 것으로 확인까지 되었습니다./document_srl 으로 접근/
이러한 여러가지 종합적으로 따졌을때 임시저장의 작동이 조금 잘못된 것이 아닌가 생각되어 이슈 남깁니다

@YJSoft
Copy link
Contributor

YJSoft commented Jul 7, 2014

이건 알림센터 쪽에서 해당 문서가 임시문서인지 아닌지 판단해서 처리해야 할 부분이라고 생각합니다

@bjrambo
Copy link
Contributor Author

bjrambo commented Jul 7, 2014

@YJSoft 임시 저장글일때 트리거를 아에 작동시킨다는게 조금 의아합니다.
실제적으로 insertdocument 라는 트리거자체가 게시글이 올라왓을때 작동이 되는 트리거가 아닌가 싶은데요.. 만일 알림센터를 제외한 다른 insertdocument 트리거자체를 실행시키는 동작이 있을까 하지 않는 우려가 나타납니다.

물론, 이러한 결점이 발견된다면 알림센터에서 처리 해주는 것도 나쁘지 않지만, 추후 나오는 모듈의 경우에서도 동일하게 사람들이 모르고 만들었다가 오류로 고쳐지는것이 아닌 코어에서 직접 처리 되도록 해주는 것이 맞을 것 같습니다.

일딴 제가 생각하는 생각으로는 이렇습니다.

@smaker
Copy link
Contributor

smaker commented Jul 10, 2014

해당 모듈에서 예외처리하는 게 좋을 것 같습니다.

임시 저장글도 보통 글과 마찬가지로 xe_documents 테이블에 입력되는데, 트리거가 작동하지 않는다면 서드파티 모듈의 개발을 제한할 수 있을 것 같습니다.

@izuzero
Copy link
Contributor

izuzero commented Jul 12, 2014

이 문제는 서드파티 모듈에서 해결하도록 하는 것이 좋고, 임시저장글의 document_srl 값을 알고 있으면 접근할 수 있는 문제만 해결되면 좋을 것 같습니다.

@playsite
Copy link

이건 코어에서 처리해야하지 않나 생각합니다. 알림센터 말고도 각종 위젯에서도 임시글이 다 노출됩니다. 그래서 저는 임시글을 아예 사용하지 못하도록 해뒀습니다.

@playsite
Copy link

좋은 기능인데 쓰질 못한다는 건 아쉬운 듯 해요.

@akasima
Copy link
Member

akasima commented Jul 31, 2014

임시 저장 글이 실제로는 documents table에 데이터를 입력하는 것이어서 트리거를 제거 할 수 없습니다.

서드파티에서 사용 시 사용할 수 있는 값이 있으니 그 값을 사용하면 될것으로 보입니다.
임시저장글의 경우 'status => TEMP' 면 처리 안하면 될것으로 같습니다.
본 글에서 에서 말씀 하신 경우는 status 가 PUBLISH 일 때만 동작하도록 처리하면 될것 같습니다.

코어 이슈는 아닐것 같네요. 이슈 닫습니다.

@akasima akasima closed this as completed Jul 31, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants