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

#588 템플릿 파일 오류시 오류 정보와 해당 오류가 발생한 템플릿 파일을 출력 #670

Merged
merged 5 commits into from
Aug 7, 2014

Conversation

YJSoft
Copy link
Contributor

@YJSoft YJSoft commented Apr 26, 2014

eval에서 발생하는 오류 대신 실제 오류가 발생한 템플릿 파일을 출력함으로 디버깅시 참고 가능하게함
file 캐시 사용시에 캐시파일을 include하는 대신 readFile후 eval하도록 수정

개선 코드(by @lansi951 ) 반영
오타 수정
#659 는 닫습니다.

eval에서 발생하는 오류 대신 실제 오류가 발생한 템플릿 파일을 출력함으로 디버깅시 참고 가능하게함

개선 코드(by @lansi951 ) 반영
include문 사용시 parse error 예외처리가 불가능함으로 FileHandler::readFile로 읽어 들인뒤 eval하는 방식으로 수정하였습니다.

추가:pharse가 아니라 parse네요;;;
@ngleader
Copy link
Contributor

오류를 echo 대신 exception 처리로 해주세요

@YJSoft
Copy link
Contributor Author

YJSoft commented May 11, 2014

@ngleader exception으로 수정 완료하였습니다

@ngleader
Copy link
Contributor

아무래도 eval 되는 것이 조금 부담이 될 것 같습니다.

@akasima akasima self-assigned this May 12, 2014
__DEBUG__가 0일경우 기존 include 방식을 사용
@YJSoft
Copy link
Contributor Author

YJSoft commented May 29, 2014

@ngleader __DEBUG__ 1 선언시에만 eval로 실행하도록 변경하였습니다(file 캐시 사용시)

@akasima akasima added this to the 1.8.0 milestone Aug 7, 2014
akasima added a commit that referenced this pull request Aug 7, 2014
#588 템플릿 파일 오류시 오류 정보와 해당 오류가 발생한 템플릿 파일을 출력
@akasima akasima merged commit 23d9612 into xpressengine:develop Aug 7, 2014
@ghost ghost modified the milestones: 1.7.6, 1.8.0 Sep 3, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants