MDL-65896 admin: Fix checking for full unicode support
authorJun Pataleta <jun@moodle.com>
Wed, 23 Oct 2019 06:08:13 +0000 (14:08 +0800)
committerJun Pataleta <jun@moodle.com>
Wed, 23 Oct 2019 07:25:27 +0000 (15:25 +0800)
Instead of checking whether the charset is set to 'utf8' when
determining full unicode support, it should be checking whether the
charset is set to 'utf8mb4'.

admin/settings/subsystems.php

index e31b043..01de61c 100644 (file)
@@ -52,16 +52,12 @@ if ($hassiteconfig) { // speedup for non-admins, add all caps used on this page
     $optionalsubsystems->add(new admin_setting_configcheckbox('enableanalytics', new lang_string('enableanalytics', 'admin'),
         new lang_string('configenableanalytics', 'admin'), 1, 1, 0));
 
-
     $fullunicodesupport = true;
     if ($DB->get_dbfamily() == 'mysql') {
         $collation = $DB->get_dbcollation();
         $collationinfo = explode('_', $collation);
         $charset = reset($collationinfo);
-
-        if ($charset !== 'utf8') {
-            $fullunicodesupport = false;
-        }
+        $fullunicodesupport = $charset === 'utf8mb4';
     }
 
     if ($fullunicodesupport) {