MNT Update utf8 aliases for mysql 8 and mariadb 10.6

This commit is contained in:
Steve Boyd 2022-05-26 12:06:51 +12:00
parent 981f63678e
commit ec5b94facf
3 changed files with 81 additions and 6 deletions

View File

@ -9,6 +9,7 @@ use SilverStripe\Dev\SapphireTest;
use SilverStripe\Dev\TestOnly; use SilverStripe\Dev\TestOnly;
use SilverStripe\ORM\Tests\MySQLPDOConnectorTest\PDOConnector; use SilverStripe\ORM\Tests\MySQLPDOConnectorTest\PDOConnector;
use SilverStripe\ORM\DB; use SilverStripe\ORM\DB;
use SilverStripe\Tests\ORM\Utf8\Utf8TestHelper;
/** /**
* @requires extension PDO * @requires extension PDO
@ -38,8 +39,9 @@ class MySQLPDOConnectorTest extends SapphireTest implements TestOnly
$cset = $connection->query('show variables like "character_set_connection"')->fetch(PDO::FETCH_NUM)[1]; $cset = $connection->query('show variables like "character_set_connection"')->fetch(PDO::FETCH_NUM)[1];
$collation = $connection->query('show variables like "collation_connection"')->fetch(PDO::FETCH_NUM)[1]; $collation = $connection->query('show variables like "collation_connection"')->fetch(PDO::FETCH_NUM)[1];
$this->assertEquals($charset, $cset); $helper = new Utf8TestHelper();
$this->assertEquals($defaultCollation, $collation); $this->assertEquals($helper->getUpdatedUtfCharsetForCurrentDB($charset), $cset);
$this->assertEquals($helper->getUpdatedUtfCollationForCurrentDB($defaultCollation), $collation);
unset($cset, $connection, $connector, $config); unset($cset, $connection, $connector, $config);
} }
@ -66,8 +68,9 @@ class MySQLPDOConnectorTest extends SapphireTest implements TestOnly
$cset = $connection->query('show variables like "character_set_connection"')->fetch(PDO::FETCH_NUM)[1]; $cset = $connection->query('show variables like "character_set_connection"')->fetch(PDO::FETCH_NUM)[1];
$collation = $connection->query('show variables like "collation_connection"')->fetch(PDO::FETCH_NUM)[1]; $collation = $connection->query('show variables like "collation_connection"')->fetch(PDO::FETCH_NUM)[1];
$this->assertEquals($charset, $cset); $helper = new Utf8TestHelper();
$this->assertEquals($customCollation, $collation); $this->assertEquals($helper->getUpdatedUtfCharsetForCurrentDB($charset), $cset);
$this->assertEquals($helper->getUpdatedUtfCollationForCurrentDB($customCollation), $collation);
unset($cset, $connection, $connector, $config); unset($cset, $connection, $connector, $config);
} }

View File

@ -6,6 +6,7 @@ use SilverStripe\Dev\SapphireTest;
use SilverStripe\Dev\TestOnly; use SilverStripe\Dev\TestOnly;
use SilverStripe\ORM\Tests\MySQLiConnectorTest\MySQLiConnector; use SilverStripe\ORM\Tests\MySQLiConnectorTest\MySQLiConnector;
use SilverStripe\ORM\DB; use SilverStripe\ORM\DB;
use SilverStripe\Tests\ORM\Utf8\Utf8TestHelper;
/** /**
* @requires extension mysqli * @requires extension mysqli
@ -31,6 +32,10 @@ class MySQLiConnectorTest extends SapphireTest implements TestOnly
$cset = $connection->get_charset(); $cset = $connection->get_charset();
// Note: we do not need to update the utf charset here because mysqli with newer
// version of mysql/mariadb still self-reports as 'utf8' rather than 'utf8mb3'
// This is unlike self::testConnectionCollationControl()
// And also unlike MySQLPDOConnectorTest::testConnectionCharsetControl()
$this->assertEquals($charset, $cset->charset); $this->assertEquals($charset, $cset->charset);
$this->assertEquals($defaultCollation, $cset->collation); $this->assertEquals($defaultCollation, $cset->collation);
@ -77,8 +82,9 @@ class MySQLiConnectorTest extends SapphireTest implements TestOnly
$cset = $connection->query('show variables like "character_set_connection"')->fetch_array()[1]; $cset = $connection->query('show variables like "character_set_connection"')->fetch_array()[1];
$collation = $connection->query('show variables like "collation_connection"')->fetch_array()[1]; $collation = $connection->query('show variables like "collation_connection"')->fetch_array()[1];
$this->assertEquals($charset, $cset); $helper = new Utf8TestHelper();
$this->assertEquals($customCollation, $collation); $this->assertEquals($helper->getUpdatedUtfCharsetForCurrentDB($charset), $cset);
$this->assertEquals($helper->getUpdatedUtfCollationForCurrentDB($customCollation), $collation);
$connection->close(); $connection->close();
unset($cset, $connection, $connector, $config); unset($cset, $connection, $connector, $config);

View File

@ -0,0 +1,66 @@
<?php
namespace SilverStripe\Tests\ORM\Utf8;
use SilverStripe\Dev\TestOnly;
use SilverStripe\ORM\DB;
class Utf8TestHelper implements TestOnly
{
private ?string $dbVersion = null;
public function getUpdatedUtfCharsetForCurrentDB(string $charset): string
{
if ($charset !== 'utf8') {
return $charset;
}
return $this->isMySqlGte80() || $this->isMariaDBGte106() ? 'utf8mb3' : 'utf8';
}
public function getUpdatedUtfCollationForCurrentDB(string $collation): string
{
if ($collation === 'utf8_general_ci') {
return $this->isMariaDBGte106() ? 'utf8mb3_general_ci' : 'utf8_general_ci';
}
if ($collation === 'utf8_unicode_520_ci') {
return $this->isMariaDBGte106() ? 'utf8mb3_unicode_520_ci' : 'utf8_unicode_520_ci';
}
return $collation;
}
/**
* MySQL has used utf8 as an alias for utf8mb3
* Beginning with MySQL 8.0.28, utf8mb3 is used
* https://dev.mysql.com/doc/refman/8.0/en/charset-unicode-utf8mb3.html
*/
private function isMySqlGte80(): bool
{
// Example MySQL version: 8.0.29
if (preg_match('#^([0-9]+)\.[0-9]+\.[0-9]+$#', $this->getDBVersion(), $m)) {
return (int) $m[1] >= 8;
}
return false;
}
/**
* Until MariaDB 10.5, utf8mb3 was an alias for utf8.
* From MariaDB 10.6, utf8 is by default an alias for utf8mb3
* https://mariadb.com/kb/en/unicode/
*/
private function isMariaDBGte106(): bool
{
// Example mariadb version: 5.5.5-10.6.8-mariadb-1:10.6.8+maria~focal
if (preg_match('#([0-9]+)\.([0-9]+)\.[0-9]+-mariadb#', $this->getDBVersion(), $m)) {
return (int) $m[1] >= 11 || ((int) $m[1] >= 10 && (int) $m[2] >= 6);
}
return false;
}
private function getDBVersion(): string
{
if (is_null($this->dbVersion)) {
$this->dbVersion = strtolower(DB::get_conn()->getVersion());
}
return $this->dbVersion;
}
}