Changeset 54069 for trunk/src/wp-admin/includes/class-wp-site-health.php

Timestamp:

09/05/2022 03:18:13 PM (2 years ago)

Author:

SergeyBiryukov

Message:

Site Health: Bump the recommended MySQL and MariaDB versions.

• MySQL 5.6 has reached EOL (“End of Life”) in February 2021. The recommended minimum is bumped to 5.7 for now.
• MariaDB 10.2 has reached EOL in May 2022. The recommended minimum is bumped to 10.3 for now.

This commit brings the Site Health recommendations in line with readme.html.

Includes:

• Adding two unit tests to ensure the SQL server versions recommended by Site Health match readme.html.
• Consistently declaring the recommended and required versions as the WP_Site_Health class properties.
• Renaming some pre-existing private properties for clarity.

Follow-up to [44986], [52319], [52358], [52420], [52424], [53431], [53433], [53435], ​[meta11407], ​[meta11866].

See #55791, ​#meta5999, ​#meta6322.

File:

• trunk/src/wp-admin/includes/class-wp-site-health.php (modified) (5 diffs)

trunk/src/wp-admin/includes/class-wp-site-health.php

@@ -11 +11 @@
     private static $instance = null;
 
-    private $mysql_min_version_check;
-    private $mysql_rec_version_check;
-
-    public $is_mariadb                           = false;
-    private $mysql_server_version                = '';
-    private $health_check_mysql_required_version = '5.5';
-    private $health_check_mysql_rec_version      = '';
+    private $is_acceptable_mysql_version;
+    private $is_recommended_mysql_version;
+
+    public $is_mariadb                   = false;
+    private $mysql_server_version        = '';
+    private $mysql_required_version      = '5.5';
+    private $mysql_recommended_version   = '5.7';
+    private $mariadb_recommended_version = '10.3';
 
     public $php_memory_limit;
@@ -210 +211 @@
         $this->mysql_server_version = $wpdb->get_var( 'SELECT VERSION()' );
 
-        $this->health_check_mysql_rec_version = '5.6';
-
         if ( stristr( $mysql_server_type, 'mariadb' ) ) {
-            $this->is_mariadb                     = true;
-            $this->health_check_mysql_rec_version = '10.0';
-        }
-
-        $this->mysql_min_version_check = version_compare( '5.5', $this->mysql_server_version, '<=' );
-        $this->mysql_rec_version_check = version_compare( $this->health_check_mysql_rec_version, $this->mysql_server_version, '<=' );
+            $this->is_mariadb                = true;
+            $this->mysql_recommended_version = $this->mariadb_recommended_version;
+        }
+
+        $this->is_acceptable_mysql_version  = version_compare( $this->mysql_required_version, $this->mysql_server_version, '<=' );
+        $this->is_recommended_mysql_version = version_compare( $this->mysql_recommended_version, $this->mysql_server_version, '<=' );
     }
 
@@ -1198 +1197 @@
         $db_dropin = file_exists( WP_CONTENT_DIR . '/db.php' );
 
-        if ( ! $this->mysql_rec_version_check ) {
+        if ( ! $this->is_recommended_mysql_version ) {
             $result['status'] = 'recommended';
 
@@ -1209 +1208 @@
                     __( 'For optimal performance and security reasons, you should consider running %1$s version %2$s or higher. Contact your web hosting company to correct this.' ),
                     ( $this->is_mariadb ? 'MariaDB' : 'MySQL' ),
-                    $this->health_check_mysql_rec_version
+                    $this->mysql_recommended_version
                 )
             );
         }
 
-        if ( ! $this->mysql_min_version_check ) {
+        if ( ! $this->is_acceptable_mysql_version ) {
             $result['status'] = 'critical';
 
@@ -1226 +1225 @@
                     __( 'WordPress requires %1$s version %2$s or higher. Contact your web hosting company to correct this.' ),
                     ( $this->is_mariadb ? 'MariaDB' : 'MySQL' ),
-                    $this->health_check_mysql_required_version
+                    $this->mysql_required_version
                 )
             );