Changeset 30375 for trunk/src/wp-includes/wp-db.php

Timestamp:

11/18/2014 03:37:23 AM (11 years ago)

Author:

pento

Message:

WPDB: When a db.php drop-in is being used, and it doesn't explicitly define itself as connecting to MySQL, skip the character set checks. This ensures that existing drop-ins won't accidentally run checks that they don't support.

See #21212.

File:

• trunk/src/wp-includes/wp-db.php (modified) (4 diffs)

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

@@ -2212 +2212 @@
 
         foreach ( $columns as $column ) {
-            if ( $column->Collation ) {
+            if ( ! empty( $column->Collation ) ) {
                 list( $charset ) = explode( '_', $column->Collation );
                 $charsets[ strtolower( $charset ) ] = true;
@@ -2238 +2238 @@
         } elseif ( 0 === $count ) {
             // No charsets, assume this table can store whatever.
-            $charset = 'latin1';
+            $charset = false;
         } else {
             // More than one charset. Remove latin1 if present and recalculate.
@@ -2292 +2292 @@
         }
 
+        // Skip this entirely if this isn't a MySQL database.
+        if ( false === $this->is_mysql ) {
+            return false;
+        }
+
         if ( empty( $this->table_charset[ $table ] ) ) {
             // This primes column information for us.
@@ -2379 +2384 @@
             $charset = $value['charset'];
 
-            // latin1 will happily store anything.
-            if ( 'latin1' === $charset ) {
+            // Column isn't a string, or is latin1, which will will happily store anything.
+            if ( false === $charset || 'latin1' === $charset ) {
                 continue;
             }
 
-            // Column or value isn't a string.
-            if ( false === $charset || ! is_string( $value['value'] ) ) {
+            if ( ! is_string( $value['value'] ) ) {
                 continue;
             }