database is created.
* (bug 47191) Fixed "Column 'si_title' cannot be part of FULLTEXT index"
MySQL error when installing using the binary character set option.
-* (bug 45288) Support mysqli PHP extension
=== API changes in 1.22 ===
* (bug 25553) The JSON output formatter now leaves forward slashes unescaped
'DatabaseMssql' => 'includes/db/DatabaseMssql.php',
'DatabaseMysql' => 'includes/db/DatabaseMysql.php',
'DatabaseMysqlBase' => 'includes/db/DatabaseMysqlBase.php',
- 'DatabaseMysqli' => 'includes/db/DatabaseMysqli.php',
'DatabaseOracle' => 'includes/db/DatabaseOracle.php',
'DatabasePostgres' => 'includes/db/DatabasePostgres.php',
'DatabaseSqlite' => 'includes/db/DatabaseSqlite.php',
/**
* Given a DB type, construct the name of the appropriate child class of
* DatabaseBase. This is designed to replace all of the manual stuff like:
- * $class = 'Database' . ucfirst( strtolower( $dbType ) );
+ * $class = 'Database' . ucfirst( strtolower( $type ) );
* as well as validate against the canonical list of DB types we have
*
* This factory function is mostly useful for when you need to connect to a
*
* @param string $dbType A possible DB type
* @param array $p An array of options to pass to the constructor.
- * Valid options are: host, user, password, dbname, flags, tablePrefix, driver
+ * Valid options are: host, user, password, dbname, flags, tablePrefix
* @return DatabaseBase subclass or null
*/
final public static function factory( $dbType, $p = array() ) {
$canonicalDBTypes = array(
- 'mysql' => array( 'mysqli', 'mysql' ),
- 'postgres' => array(),
- 'sqlite' => array(),
- 'oracle' => array(),
- 'mssql' => array(),
+ 'mysql', 'postgres', 'sqlite', 'oracle', 'mssql'
);
-
$dbType = strtolower( $dbType );
- if ( isset( $canonicalDBTypes[$dbType] ) && $canonicalDBTypes[$dbType] ) {
- $driver = isset( $p['driver'] ) ? $p['driver'] : null;
- $possibleDrivers = $canonicalDBTypes[$dbType];
- if ( $driver ) {
- if ( in_array( $driver, $possibleDrivers ) ) {
- $dbType = $driver;
- } else {
- throw new MWException( __METHOD__ .
- " cannot construct Database with type '$dbType' and driver '$driver'" );
- }
- } else {
- foreach ( $possibleDrivers as $posDriver ) {
- if ( extension_loaded( $posDriver ) ) {
- $dbType = $posDriver;
- break;
- }
- }
- throw new MWException( __METHOD__ .
- " no viable database extension found for type '$dbType'" );
- }
- }
-
$class = 'Database' . ucfirst( $dbType );
- if ( class_exists( $class ) && is_subclass_of( $class, 'DatabaseBase' ) ) {
+
+ if ( in_array( $dbType, $canonicalDBTypes ) || ( class_exists( $class ) && is_subclass_of( $class, 'DatabaseBase' ) ) ) {
return new $class(
isset( $p['host'] ) ? $p['host'] : false,
isset( $p['user'] ) ? $p['user'] : false,
*/
class MySQLField implements Field {
private $name, $tablename, $default, $max_length, $nullable,
- $is_pk, $is_unique, $is_multiple, $is_key, $type, $binary;
+ $is_pk, $is_unique, $is_multiple, $is_key, $type;
function __construct( $info ) {
$this->name = $info->name;
$this->is_multiple = $info->multiple_key;
$this->is_key = ( $this->is_pk || $this->is_unique || $this->is_multiple );
$this->type = $info->type;
- $this->binary = isset( $info->binary ) ? $info->binary : false;
}
/**
function isMultipleKey() {
return $this->is_multiple;
}
-
- function isBinary() {
- return $this->binary;
- }
}
class MySQLMasterPos implements DBMasterPos {
+++ /dev/null
-<?php
-/**
- * This is the MySQLi database abstraction layer.
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License along
- * with this program; if not, write to the Free Software Foundation, Inc.,
- * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
- * http://www.gnu.org/copyleft/gpl.html
- *
- * @file
- * @ingroup Database
- */
-
-/**
- * Database abstraction object for PHP extension mysqli.
- *
- * @ingroup Database
- * @since 1.22
- * @see Database
- */
-class DatabaseMysqli extends DatabaseMysqlBase {
-
- /**
- * @param $sql string
- * @return resource
- */
- protected function doQuery( $sql ) {
- if ( $this->bufferResults() ) {
- $ret = $this->mConn->query( $sql );
- } else {
- $ret = $this->mConn->query( $sql, MYSQLI_USE_RESULT );
- }
- return $ret;
- }
-
- protected function mysqlConnect( $realServer ) {
- # Fail now
- # Otherwise we get a suppressed fatal error, which is very hard to track down
- if ( !function_exists( 'mysqli_init' ) ) {
- throw new DBConnectionError( $this, "MySQLi functions missing,"
- . " have you compiled PHP with the --with-mysqli option?\n" );
- }
-
- $connFlags = 0;
- if ( $this->mFlags & DBO_SSL ) {
- $connFlags |= MYSQLI_CLIENT_SSL;
- }
- if ( $this->mFlags & DBO_COMPRESS ) {
- $connFlags |= MYSQLI_CLIENT_COMPRESS;
- }
- if ( $this->mFlags & DBO_PERSISTENT ) {
- $realServer = 'p:' . $realServer;
- }
-
- $mysqli = mysqli_init();
- $numAttempts = 2;
-
- for ( $i = 0; $i < $numAttempts; $i++ ) {
- if ( $i > 1 ) {
- usleep( 1000 );
- }
- if ( $mysqli->real_connect( $realServer, $this->mUser,
- $this->mPassword, $this->mDBname, null, null, $connFlags ) )
- {
- return $mysqli;
- }
- }
-
- return false;
- }
-
- /**
- * @return bool
- */
- protected function closeConnection() {
- return $this->mConn->close();
- }
-
- /**
- * @return int
- */
- function insertId() {
- return $this->mConn->insert_id;
- }
-
- /**
- * @return int
- */
- function lastErrno() {
- if ( $this->mConn ) {
- return $this->mConn->errno;
- } else {
- return mysqli_connect_errno();
- }
- }
-
- /**
- * @return int
- */
- function affectedRows() {
- return $this->mConn->affected_rows;
- }
-
- /**
- * @param $db
- * @return bool
- */
- function selectDB( $db ) {
- $this->mDBname = $db;
- return $this->mConn->select_db( $db );
- }
-
- /**
- * @return string
- */
- function getServerVersion() {
- return $this->mConn->server_info;
- }
-
- protected function mysqlFreeResult( $res ) {
- $res->free_result();
- return true;
- }
-
- protected function mysqlFetchObject( $res ) {
- $object = $res->fetch_object();
- if ( $object === null ) {
- return false;
- }
- return $object;
- }
-
- protected function mysqlFetchArray( $res ) {
- $array = $res->fetch_array();
- if ( $array === null ) {
- return false;
- }
- return $array;
- }
-
- protected function mysqlNumRows( $res ) {
- return $res->num_rows;
- }
-
- protected function mysqlNumFields( $res ) {
- return $res->field_count;
- }
-
- protected function mysqlFetchField( $res, $n ) {
- $field = $res->fetch_field_direct( $n );
- $field->not_null = $field->flags & MYSQLI_NOT_NULL_FLAG;
- $field->primary_key = $field->flags & MYSQLI_PRI_KEY_FLAG;
- $field->unique_key = $field->flags & MYSQLI_UNIQUE_KEY_FLAG;
- $field->multiple_key = $field->flags & MYSQLI_MULTIPLE_KEY_FLAG;
- $field->binary = $field->flags & MYSQLI_BINARY_FLAG;
- return $field;
- }
-
- protected function mysqlFieldName( $res, $n ) {
- $field = $res->fetch_field_direct( $n );
- return $field->name;
- }
-
- protected function mysqlDataSeek( $res, $row ) {
- return $res->data_seek( $row );
- }
-
- protected function mysqlError( $conn = null ) {
- if ($conn === null) {
- return mysqli_connect_error();
- } else {
- return $conn->error;
- }
- }
-
- protected function mysqlRealEscapeString( $s ) {
- return $this->mConn->real_escape_string( $s );
- }
-
- protected function mysqlPing() {
- return $this->mConn->ping();
- }
-
-}
return true;
}
- $fieldInfo = $this->db->fieldInfo( $table, $field );
- if ( $fieldInfo->isBinary() ) {
+ $tableName = $this->db->tableName( $table );
+ $res = $this->db->query( "SELECT $field FROM $tableName LIMIT 0", __METHOD__ );
+ $flags = explode( ' ', mysql_field_flags( $res->result, 0 ) );
+
+ if ( in_array( 'binary', $flags ) ) {
$this->output( "...$table table has correct $field encoding.\n" );
} else {
$this->applyPatch( $patchFile, false, "Fixing $field encoding on $table table" );