diff options
| author | Oleg Pudeyev <oleg@bsdpower.com> | 2012-12-13 19:07:49 -0500 |
|---|---|---|
| committer | Oleg Pudeyev <oleg@bsdpower.com> | 2012-12-13 19:15:24 -0500 |
| commit | 9e3fd3bf8e8ff08b159d9151aef6f9bb6b9244ee (patch) | |
| tree | ac564d54f2d397abe53ae46f22906677ddeda1a5 /phpBB/includes/functions.php | |
| parent | 1a1ae1b663079792c775c4a51d5aaaf0259fb05c (diff) | |
| download | forums-9e3fd3bf8e8ff08b159d9151aef6f9bb6b9244ee.tar forums-9e3fd3bf8e8ff08b159d9151aef6f9bb6b9244ee.tar.gz forums-9e3fd3bf8e8ff08b159d9151aef6f9bb6b9244ee.tar.bz2 forums-9e3fd3bf8e8ff08b159d9151aef6f9bb6b9244ee.tar.xz forums-9e3fd3bf8e8ff08b159d9151aef6f9bb6b9244ee.zip | |
[ticket/11015] Fix 3.0 to 3.1 dbms conversion for mysqli.
PHPBB3-11015
Diffstat (limited to 'phpBB/includes/functions.php')
| -rw-r--r-- | phpBB/includes/functions.php | 26 |
1 files changed, 21 insertions, 5 deletions
diff --git a/phpBB/includes/functions.php b/phpBB/includes/functions.php index 4bf991ca9e..5c08d457b6 100644 --- a/phpBB/includes/functions.php +++ b/phpBB/includes/functions.php @@ -5550,23 +5550,39 @@ function phpbb_to_numeric($input) } /** -* Convert 3.0 dbms to 3.1 db driver class name +* Convert either 3.0 dbms or 3.1 db driver class name to 3.1 db driver class name. +* +* If $dbms is a valid 3.1 db driver class name, returns it unchanged. +* Otherwise prepends phpbb_db_driver_ to the dbms to convert a 3.0 dbms +* to 3.1 db driver class name. * * @param string $dbms dbms parameter * @return db driver class */ function phpbb_convert_30_dbms_to_31($dbms) { - if (class_exists($dbms)) + // Note: this check is done first because mysqli extension + // supplies a mysqli class, and class_exists($dbms) would return + // true for mysqli class. + // However, per the docblock any valid 3.1 driver name should be + // recognized by this function, and have priority over 3.0 dbms. + if (class_exists('phpbb_db_driver_' . $dbms)) { - return $dbms; + return 'phpbb_db_driver_' . $dbms; } - if (class_exists('phpbb_db_driver_' . $dbms)) + if (class_exists($dbms)) { - return 'phpbb_db_driver_' . $dbms; + return $dbms; } + // Additionally we could check that $dbms extends phpbb_db_driver. + // http://php.net/manual/en/class.reflectionclass.php + // Beware of possible performance issues: + // http://stackoverflow.com/questions/294582/php-5-reflection-api-performance + // We could check for interface implementation in all paths or + // only when we do not prepend phpbb_db_driver_. + throw new \RuntimeException("You have specified an invalid dbms driver: $dbms"); } |