Changeset 33311

Timestamp:

07/17/2015 07:09:32 AM (9 years ago)

Author:

pento

Message:

WPDB: ::strip_text_from_query() doesn't pass a length to ::strip_invalid_text(), which was causing queries to fail when they contained characters that needed to be sanity checked by MySQL.

Props dd32, mdawaffe, pento.

Merges [33310] to the 4.2 branch.

Fixes #32279.

Location:

branches/4.2

Files:

• src/wp-includes/wp-db.php (modified) (3 diffs)
• tests/phpunit/tests/db/charset.php (modified) (2 diffs)

branches/4.2/src/wp-includes/wp-db.php

@@ -2595 +2595 @@
             if ( is_array( $value['length'] ) ) {
                 $length = $value['length']['length'];
+
             } else {
                 $length = false;
+
+
+
+
             }
 
@@ -2608 +2613 @@
                 continue;
             }
-
-            $truncate_by_byte_length = 'byte' === $value['length']['type'];
 
             $needs_validation = true;
@@ -2684 +2687 @@
                     }
 
-                    $queries[ $col ] = $this->prepare( "CONVERT( LEFT( CONVERT( %s USING $charset ), %.0f ) USING {$this->charset} )", $value['value'], $value['length']['length'] );
+                    if ( is_array( $value['length'] ) ) {
+                        $queries[ $col ] = $this->prepare( "CONVERT( LEFT( CONVERT( %s USING $charset ), %.0f ) USING {$this->charset} )", $value['value'], $value['length']['length'] );
+                    } else if ( 'binary' !== $charset ) {
+                        // If we don't have a length, there's no need to convert binary - it will always return the same result.
+                        $queries[ $col ] = $this->prepare( "CONVERT( CONVERT( %s USING $charset ) USING {$this->charset} )", $value['value'] );
+                    }
 
                     unset( $data[ $col ]['db'] );

branches/4.2/tests/phpunit/tests/db/charset.php

@@ -214 +214 @@
                 'expected' => "\xd8ord\xd0ress",
                 'length'   => array( 'type' => 'char', 'length' => 100 ),
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
             ),
             'cp1251_char_length' => array(
@@ -808 +823 @@
         $this->assertEquals( 255, strlen( $stripped ) );
     }
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
 }