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

PHP 7에서는 자동으로 mysqli 사용 #2345

Merged
merged 1 commit into from Dec 10, 2018

Conversation

2 participants
@kijin
Copy link
Contributor

kijin commented Nov 24, 2018

기존에 mysql로 설치된 사이트(사실상 현존하는 대부분의 사이트)를 PHP 7.x로 업그레이드하면 db.config.php를 직접 열어서 db_type을 mysql 또는 mysql_innodb에서 mysqli 또는 mysqli_innodb로 변경해 주어야 합니다. 그러나 이 작업이 필요하다는 사실에 대한 어떤 안내도 없어서 에러가 나더라도 해결 방법을 찾기가 어렵고, FTP에서 해당 파일을 수정할 권한이 없는 경우도 많습니다.

mysql 또는 mysql_innodb로 설치된 사이트를 PHP 7.0 이상 버전에서 구동하면 기존에 설정된 db_type 대신 mysqli 또는 mysqli_innodb에 해당하는 DB 클래스를 로딩하도록 하여 PHP 버전 변경에 따른 불편을 줄여 보았습니다.

큐브리드나 MS SQL을 사용하는 사이트에는 영향이 없습니다.

@bnu bnu self-assigned this Nov 26, 2018

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

@bnu bnu merged commit 093dc2c into xpressengine:develop Dec 10, 2018

1 check passed

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

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

@kijin

This comment has been minimized.

Copy link
Contributor

kijin commented Dec 10, 2018

이제 보니 DB type에 _innodb가 붙지 않는 경우를 감안하지 못했네요.
정규식을 /^mysql(_.+)?$/로 바꿔야겠습니다 ㅜㅜ

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