Changeset 37594 for trunk/tests/phpunit/tests/meta/query.php

Timestamp:

05/30/2016 04:35:16 AM (9 years ago)

Author:

boonebgorges

Message:

In WP_Meta_Query, don't cast meta_value to CHAR.

CHAR is redundant, since the meta_value column is LONGTEXT. Meanwhile,
use of CAST() causes MySQL to ignore any index that the administrator may
have added to the column.

A number of automated tests were doing searches for CAST in the SQL strings
generated by WP_Meta_Query (for reasons unrelated to the CAST() behavior).
These tests have been updated to expect the new query format.

Props ericlewis.
Fixes #36625.

File:

• trunk/tests/phpunit/tests/meta/query.php (modified) (6 diffs)

trunk/tests/phpunit/tests/meta/query.php

@@ -494 +494 @@
         $sql = $query->get_sql( 'post', $wpdb->posts, 'ID', $this );
 
-        $this->assertEquals( 1, substr_count( $sql['where'], "CAST($wpdb->postmeta.meta_value AS CHAR) = ''" ) );
+        $this->assertEquals( 1, substr_count( $sql['where'], "$wpdb->postmeta.meta_value = ''" ) );
     }
 
@@ -602 +602 @@
         $sql = $query->get_sql( 'post', $wpdb->posts, 'ID', $this );
 
-        $this->assertSame( 1, substr_count( $sql['where'], "CAST($wpdb->postmeta.meta_value AS CHAR) = ''" ) );
+        $this->assertSame( 1, substr_count( $sql['where'], "$wpdb->postmeta.meta_value = ''" ) );
     }
 
@@ -633 +633 @@
         $sql = $query->get_sql( 'post', $wpdb->posts, 'ID', $this );
 
-        $this->assertSame( 1, substr_count( $sql['where'], "CAST($wpdb->postmeta.meta_value AS CHAR) IN" ) );
+        $this->assertSame( 1, substr_count( $sql['where'], "$wpdb->postmeta.meta_value IN" ) );
     }
 
@@ -648 +648 @@
         $sql = $query->get_sql( 'post', $wpdb->posts, 'ID', $this );
 
-        $this->assertSame( 1, substr_count( $sql['where'], "CAST($wpdb->postmeta.meta_value AS CHAR) =" ) );
+        $this->assertSame( 1, substr_count( $sql['where'], "$wpdb->postmeta.meta_value =" ) );
     }
 
@@ -664 +664 @@
         $sql = $query->get_sql( 'post', $wpdb->posts, 'ID', $this );
 
-        $this->assertSame( 1, substr_count( $sql['where'], "CAST($wpdb->postmeta.meta_value AS CHAR) =" ) );
+        $this->assertSame( 1, substr_count( $sql['where'], "$wpdb->postmeta.meta_value =" ) );
     }
 
@@ -761 +761 @@
         $sql = $query->get_sql( 'post', $wpdb->posts, 'ID', $this );
 
-        $this->assertSame( 1, substr_count( $sql['where'], "CAST($wpdb->postmeta.meta_value AS CHAR) = 'bar'" ) );
+        $this->assertSame( 1, substr_count( $sql['where'], "$wpdb->postmeta.meta_value = 'bar'" ) );
     }