Skip to content

Commit

Permalink
Merge pull request #14129 from eileenmcnaughton/512
Browse files Browse the repository at this point in the history
Fix typo in DROP table query.
  • Loading branch information
seamuslee001 authored Apr 25, 2019
2 parents 8badc3f + e6c39e9 commit b41de82
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion Civi/Install/Requirements.php
Original file line number Diff line number Diff line change
Expand Up @@ -609,14 +609,15 @@ public function checkMysqlUtf8mb4($db_config) {
return $results;
}

mysqli_query($conn, 'DROP TABLE IF EXISTS civicrm_utf8mb4_test');
$r = mysqli_query($conn, 'CREATE TABLE civicrm_utf8mb4_test (id VARCHAR(255), PRIMARY KEY(id(255))) CHARACTER SET utf8mb4 COLLATE utf8mb4_unicode_ci ROW_FORMAT=DYNAMIC ENGINE=INNODB');
if (!$r) {
$results['severity'] = $this::REQUIREMENT_WARNING;
$results['details'] = 'It is recommended, though not yet required, to configure your MySQL server for utf8mb4 support. You will need the following MySQL server configuration: innodb_large_prefix=true innodb_file_format=barracuda innodb_file_per_table=true';
mysqli_close($conn);
return $results;
}
mysqli_query('DROP TABLE civicrm_utf8mb4_test');
mysqli_query($conn, 'DROP TABLE civicrm_utf8mb4_test');

// Ensure that the MySQL driver supports utf8mb4 encoding.
$version = mysqli_get_client_info($conn);
Expand Down

0 comments on commit b41de82

Please sign in to comment.