Skip to content

php7 환경에서 발생하는 오류 수정#1704

Merged
2 commits merged into
xpressengine:developfrom
YJSoft-Forth:php7
Sep 22, 2015
Merged

php7 환경에서 발생하는 오류 수정#1704
2 commits merged into
xpressengine:developfrom
YJSoft-Forth:php7

Conversation

@YJSoft

@YJSoft YJSoft commented Aug 12, 2015

Copy link
Copy Markdown
Contributor
Q A
사소한 변경
버그 수정
새로운 기능 아니오
호환성 깨짐 아니오
수정하는 이슈 (없음)

PHP7 이상에서 XE 설치시 발생하는 $this 관련 오류를 수정합니다.

non-static한 함수를 static하게 사용하여 발생하는 오류를 수정합니다.
@ngleader

Copy link
Copy Markdown
Contributor

"Object" 가 예약어라...

@kijin

kijin commented Aug 24, 2015

Copy link
Copy Markdown
Contributor

예약어와 충돌하는 것은 다른 이름으로 바꿔버리고, 코어 내부에서는 새 이름으로 일괄변환하되 PHP 5.x에서는 기존 이름으로 class_alias 걸어주어서 서드파티 호환성 보장하고, PHP 7을 지원하고 싶은 서드파티는 알아서 따라오라고 하죠...

@ghost ghost added this to the next-2015-41 milestone Sep 22, 2015
@ghost ghost self-assigned this Sep 22, 2015
ghost pushed a commit that referenced this pull request Sep 22, 2015
php7 환경에서 발생하는 오류 수정
@ghost ghost merged commit cebe101 into xpressengine:develop Sep 22, 2015
@ghost ghost added the merged-pull-request label Sep 22, 2015
@YJSoft YJSoft deleted the php7 branch September 22, 2015 04:12
@paulryu03

Copy link
Copy Markdown

PHP 7.1 환경에서는 최신 XE 1.8.29 도 설치가 안되네요. :(

This pull request was closed.
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.

4 participants