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

mysqli extension만 on 되어 있을 경우 버그 #1832

Closed
Joungkyun opened this issue Oct 6, 2015 · 1 comment
Closed

mysqli extension만 on 되어 있을 경우 버그 #1832

Joungkyun opened this issue Oct 6, 2015 · 1 comment
Assignees
Milestone

Comments

@Joungkyun
Copy link

시스템에 mysql extension이 비활성 되어 있고, mysqli만 활성 되어 있을 경우 동작 버그

mysqli adapter에서 MYSQL constant를 사용하고 있어서 발생하는 문제. 1.10.7 까지 존재함.. 업데이트 마다 귀찮아서 결국 보고함 --;

diff -urNp my.orig/framework/data/MySQLi/Adapter.php my/framework/data/MySQLi/Adapter.php
--- my.orig/framework/data/MySQLi/Adapter.php   2015-10-07 00:13:14.492965878 +0900
+++ my/framework/data/MySQLi/Adapter.php    2015-10-07 00:10:53.307841000 +0900
@@ -130,11 +130,11 @@ class DBAdapter implements IAdapter {
    }

    public static function queryCell($query, $field = 0, $useCache=true) {
-       $type = MYSQL_BOTH;
+       $type = MYSQLI_BOTH;
        if (is_numeric($field)) {
-           $type = MYSQL_NUM;
+           $type = MYSQLI_NUM;
        } else {
-           $type = MYSQL_ASSOC;
+           $type = MYSQLI_ASSOC;
        }

        if( $useCache ) {
@@ -331,12 +331,12 @@ class DBAdapter implements IAdapter {
    public static function __queryType($type) {
        switch(strtolower($type)) {
            case 'num':
-               return MYSQL_NUM;
+               return MYSQLI_NUM;
            case 'assoc':
-               return MYSQL_ASSOC;
+               return MYSQLI_ASSOC;
            case 'both':
            default:
-               return MYSQL_BOTH;
+               return MYSQLI_BOTH;
        }
    }
@inureyes inureyes added this to the 1.10.8 milestone Oct 13, 2015
@inureyes inureyes self-assigned this Oct 13, 2015
@inureyes
Copy link
Member

감사합니다!

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

2 participants