diff options
author | Dries Buytaert <dries@buytaert.net> | 2010-10-12 02:50:03 +0000 |
---|---|---|
committer | Dries Buytaert <dries@buytaert.net> | 2010-10-12 02:50:03 +0000 |
commit | 738367593558ba170cb6d65dda35ec9230654bc3 (patch) | |
tree | 4a8025e4e16aba7be4aeee0ae179e6866cd12d94 /includes/install.core.inc | |
parent | 563c673ea3b8977e9f739f7979acb62abcd78310 (diff) | |
download | brdo-738367593558ba170cb6d65dda35ec9230654bc3.tar.gz brdo-738367593558ba170cb6d65dda35ec9230654bc3.tar.bz2 |
- Patch #818374 by Damien Tournoud, ksenzee, Dave Reid: add a requirements check error if PECL PDO is being used.
Diffstat (limited to 'includes/install.core.inc')
-rw-r--r-- | includes/install.core.inc | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/includes/install.core.inc b/includes/install.core.inc index f162bebce..db0951338 100644 --- a/includes/install.core.inc +++ b/includes/install.core.inc @@ -819,7 +819,12 @@ function install_verify_settings() { * Verify PDO library. */ function install_verify_pdo() { - return extension_loaded('pdo'); + // PDO was moved to PHP core in 5.2.0, but the old extension (targeting 5.0 + // and 5.1) is still available from PECL, and can still be built without + // errors. To verify that the correct version is in use, we check the + // PDO::ATTR_DEFAULT_FETCH_MODE constant, which is not available in the + // PECL extension. + return extension_loaded('pdo') && defined('PDO::ATTR_DEFAULT_FETCH_MODE'); } /** |