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

ERROR 워링메세지도 debugMessage파일에 추가. #1872

Closed
wants to merge 5 commits into from
Closed

ERROR 워링메세지도 debugMessage파일에 추가. #1872

wants to merge 5 commits into from

Conversation

bjrambo
Copy link
Contributor

@bjrambo bjrambo commented Mar 6, 2016

현재 XE는 E_WARNING메세지에 대해서 전부 무시하고 넘어가도록 되어있습니다.

그래서 간단한 워링 에러의 대해서 개발자들이 모르고 넘어가는 경우가 많습니다.
ex : http://xepushapp.com/board_vxJz62/18675

보통은 5.3 웹호스팅에서 아직도 개발하는 서드개발자들이 있어서, object 들이 선언되지않아 일으키는 문제점을 잘 인지 못하는 경우가 많습니다.

그래서 이 부분을 개발자들에게 debugPrint에 사용되는 files에 넣어 함께 기록하도록 개선했습니다.

@ghost ghost self-assigned this Mar 8, 2016
@kijin
Copy link
Contributor

kijin commented Jun 18, 2016

  • ModuleHandler::init() 이전에 발생하는 에러도 잡을 수 있으면 좋겠습니다.
  • 에러 기록 모듈처럼 치명적인 오류도 기록할 수 있으면 좋겠습니다.
  • 위의 기능을 개발하기 위해 에러 기록 모듈을 참고하실 경우 해당 코드에는 LGPL 라이선스를 적용해 드리겠습니다.
  • xeErrorLog() 메소드 내에서 restore_error_handler()를 해버리면 다음에 발생하는 에러는 기록되지 않을 것 같습니다.

@bjrambo
Copy link
Contributor Author

bjrambo commented Jun 18, 2016

@kijin 한번 검토 해보도록 할께요! 항상 도움주셔서 감사합니닷.

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

Successfully merging this pull request may close these issues.

None yet

2 participants