Changeset 33055

Timestamp:

07/03/2015 03:25:07 AM (11 years ago)

Author:

dd32

Message:

Enable utf8mb4 for MySQL extension users. Previously utf8mb4 was limited to MySQLi users only unintentionally.
This change does the following things

• Allows utf8mb4 for the MySQL extension
• Re-runs the utf8->utf8mb4 conversion for single sites, this will do nothing for tables already converted
• Re-runs the utf8->utf8mb4 conversion for global tables in multisite when the environment supports utf8mb4
• Removes upgrade_420() calling as upgrade_430() will perform those changes now instead

The index shortenings should have still taken place on utf8 sites previously, so there's no need to run those again.

Props kovshenin, pento, dd32
Fixes #32127 for trunk.

Location:

trunk/src

Files:

• wp-admin/includes/upgrade.php (modified) (5 diffs)
• wp-includes/version.php (modified) (1 diff)
• wp-includes/wp-db.php (modified) (1 diff)

trunk/src/wp-admin/includes/upgrade.php

@@ -532 +532 @@
                 upgrade_400();
 
-        // Don't harsh my mellow. upgrade_430() must be called before
-        // upgrade_420() to catch bad comments prior to any auto-expansion of
-        // MySQL column widths.
-        if ( $wp_current_db_version < 32814 )
+        if ( $wp_current_db_version < 33055 )
                 upgrade_430();
-
-        if ( $wp_current_db_version < 31351 )
-                upgrade_420();
 
         maybe_disable_link_manager();
@@ -1495 +1489 @@
  * @global wpdb  $wpdb
  */
-function upgrade_420() {
+function upgrade_420() {}
+
+/**
+ * Execute changes made in WordPress 4.3.0.
+ *
+ * @since 4.3.0
+ *
+ * @global int   $wp_current_db_version
+ * @global wpdb  $wpdb
+ */
+function upgrade_430() {
         global $wp_current_db_version, $wpdb;
 
-        if ( $wp_current_db_version < 31351 && $wpdb->charset === 'utf8mb4' ) {
+        if ( $wp_current_db_version < 32364 ) {
+                upgrade_430_fix_comments();
+        }
+
+        if ( $wp_current_db_version < 32814 ) {
+                split_all_shared_terms();
+        }
+
+        if ( $wp_current_db_version < 33055 && 'utf8mb4' === $wpdb->charset ) {
                 if ( is_multisite() ) {
                         $tables = $wpdb->tables( 'blog' );
@@ -1504 +1516 @@
                         $tables = $wpdb->tables( 'all' );
                 }
-
+        
                 foreach ( $tables as $table ) {
                         maybe_convert_table_to_utf8mb4( $table );
                 }
-        }
-}
-
-/**
- * Execute changes made in WordPress 4.3.0.
- *
- * @since 4.3.0
- *
- * @global int   $wp_current_db_version
- * @global wpdb  $wpdb
- */
-function upgrade_430() {
-        global $wp_current_db_version, $wpdb;
-
-        if ( $wp_current_db_version < 32364 ) {
-                upgrade_430_fix_comments();
-        }
-
-        if ( $wp_current_db_version < 32814 ) {
-                split_all_shared_terms();
         }
 }
@@ -1697 +1689 @@
 
         // 4.3
-        if ( $wp_current_db_version < 32378 && 'utf8mb4' === $wpdb->charset ) {
+        if ( $wp_current_db_version < 33055 && 'utf8mb4' === $wpdb->charset ) {
                 if ( ! ( defined( 'DO_NOT_UPGRADE_GLOBAL_TABLES' ) && DO_NOT_UPGRADE_GLOBAL_TABLES ) ) {
                         $upgrade = false;
@@ -1710 +1702 @@
                         if ( $upgrade ) {
                                 $wpdb->query( "ALTER TABLE $wpdb->signups DROP INDEX domain_path, ADD INDEX domain_path(domain(140),path(51))" );
+                        }
+
+                        $tables = $wpdb->tables( 'global' );
+
+                        foreach ( $tables as $table ) {
+                                maybe_convert_table_to_utf8mb4( $table );
                         }
                 }

trunk/src/wp-includes/version.php

@@ -12 +12 @@
  * @global int $wp_db_version
  */
-$wp_db_version = 32814;
+$wp_db_version = 33055;
 
 /**

trunk/src/wp-includes/wp-db.php

@@ -741 +741 @@
                 }
 
-                if ( ( $this->use_mysqli && ! ( $this->dbh instanceof mysqli ) )
-                  || ( empty( $this->dbh ) || ! ( $this->dbh instanceof mysqli ) ) ) {
+                if ( ( $this->use_mysqli && ! ( $this->dbh instanceof mysqli ) ) || empty( $this->dbh ) ) {
                         return;
                 }