From b03cf04dbd60219dc4f758030f3492bb0539a70f Mon Sep 17 00:00:00 2001
From: Sean Harvey <halkyon@gmail.com>
Date: Sat, 28 Apr 2012 12:13:58 +1200
Subject: [PATCH] BUGFIX Updating MySQLDatabaseConfigurationHelper to check for
 MySQLi instead of mysql_connect and other old function usage, as we've moved
 to MySQLi in MySQLDatabase.php now

---
 _register_database.php                        |  2 +-
 .../MySQLDatabaseConfigurationHelper.php      | 28 +++++++++----------
 2 files changed, 15 insertions(+), 15 deletions(-)

diff --git a/_register_database.php b/_register_database.php
index ea68b83ab..a99a52b7e 100644
--- a/_register_database.php
+++ b/_register_database.php
@@ -8,7 +8,7 @@ DatabaseAdapterRegistry::register(
 		'class' => 'MySQLDatabase',
 		'title' => 'MySQL 5.0+',
 		'helperPath' => $frameworkPath . '/dev/install/MySQLDatabaseConfigurationHelper.php',
-		'supported' => function_exists('mysql_connect'),
+		'supported' => class_exists('MySQLi'),
 	)
 );
 
diff --git a/dev/install/MySQLDatabaseConfigurationHelper.php b/dev/install/MySQLDatabaseConfigurationHelper.php
index 1001db4de..766b0693b 100644
--- a/dev/install/MySQLDatabaseConfigurationHelper.php
+++ b/dev/install/MySQLDatabaseConfigurationHelper.php
@@ -18,7 +18,7 @@ class MySQLDatabaseConfigurationHelper implements DatabaseConfigurationHelper {
 	 * @return boolean
 	 */
 	public function requireDatabaseFunctions($databaseConfig) {
-		return (function_exists('mysql_connect')) ? true : false;
+		return class_exists('MySQLi');
 	}
 
 	/**
@@ -29,12 +29,12 @@ class MySQLDatabaseConfigurationHelper implements DatabaseConfigurationHelper {
 	public function requireDatabaseServer($databaseConfig) {
 		$success = false;
 		$error = '';
-		$conn = @mysql_connect($databaseConfig['server'], $databaseConfig['username'], $databaseConfig['password']);
-		if($conn || mysql_errno() < 2000) {
+		$conn = new MySQLi($databaseConfig['server'], $databaseConfig['username'], $databaseConfig['password']);
+		if($conn && $conn->connect_errno < 2000) {
 			$success = true;
 		} else {
 			$success = false;
-			$error = mysql_error();
+			$error = ($conn) ? $conn->connect_error : '';
 		}
 		return array(
 			'success' => $success,
@@ -48,13 +48,13 @@ class MySQLDatabaseConfigurationHelper implements DatabaseConfigurationHelper {
 	 * @return mixed string Version number as string | boolean FALSE on failure
 	 */
 	public function getDatabaseVersion($databaseConfig) {
-		$conn = @mysql_connect($databaseConfig['server'], $databaseConfig['username'], $databaseConfig['password']);
+		$conn = new MySQLi($databaseConfig['server'], $databaseConfig['username'], $databaseConfig['password']);
 		if(!$conn) return false;
-		$version = @mysql_get_server_info($conn);
+		$version = $conn->server_info;
 		if(!$version) {
 			// fallback to trying a query
-			$result = @mysql_query("SELECT VERSION()");
-			$row = @mysql_fetch_array($result);
+			$result = $conn->query('SELECT VERSION()');
+			$row = $result->fetch_array();
 			if($row && isset($row[0])) {
 				$version = trim($row[0]);
 			}
@@ -93,12 +93,12 @@ class MySQLDatabaseConfigurationHelper implements DatabaseConfigurationHelper {
 	public function requireDatabaseConnection($databaseConfig) {
 		$success = false;
 		$error = '';
-		$conn = @mysql_connect($databaseConfig['server'], $databaseConfig['username'], $databaseConfig['password']);
+		$conn = new MySQLi($databaseConfig['server'], $databaseConfig['username'], $databaseConfig['password']);
 		if($conn) {
 			$success = true;
 		} else {
 			$success = false;
-			$error = mysql_error();
+			$error = ($conn) ? $conn->connect_error : '';
 		}
 		return array(
 			'success' => $success,
@@ -116,13 +116,13 @@ class MySQLDatabaseConfigurationHelper implements DatabaseConfigurationHelper {
 	public function requireDatabaseOrCreatePermissions($databaseConfig) {
 		$success = false;
 		$alreadyExists = false;
-		$conn = @mysql_connect($databaseConfig['server'], $databaseConfig['username'], $databaseConfig['password']);
-		if(@mysql_select_db($databaseConfig['database'], $conn)) {
+		$conn = new MySQLi($databaseConfig['server'], $databaseConfig['username'], $databaseConfig['password']);
+		if($conn && $conn->select_db($databaseConfig['database'])) {
 			$success = true;
 			$alreadyExists = true;
 		} else {
-			if(@mysql_query("CREATE DATABASE testing123", $conn)) {
-				mysql_query("DROP DATABASE testing123", $conn);
+			if($conn && $conn->query('CREATE DATABASE testing123')) {
+				mysql_query('DROP DATABASE testing123', $conn);
 				$success = true;
 				$alreadyExists = false;
 			}