Changeset 28659 for trunk/src/wp-includes/meta.php

Timestamp:

06/04/2014 05:47:19 AM (10 years ago)

Author:

wonderboymusic

Message:

Fix SQL generation when meta_query has an 'relation' => 'OR' for its queries and wants to 'orderby' => 'meta_value'.

Adds unit test.

Props jackreichert.
Fixes #25538.

File:

• trunk/src/wp-includes/meta.php (modified) (3 diffs)

trunk/src/wp-includes/meta.php

@@ -973 +973 @@
         }
 
+        $where_meta_key = array();
         foreach ( $queries as $k => $q ) {
             $meta_key = isset( $q['key'] ) ? trim( $q['key'] ) : '';
@@ -1015 +1016 @@
 
             $where[$k] = '';
-            if ( !empty( $meta_key ) )
-                $where[$k] = $wpdb->prepare( "$alias.meta_key = %s", $meta_key );
+            if ( ! empty( $meta_key ) ) {
+                if ( isset( $q['compare'] ) ) {
+                    $where_meta_key[$k] = $wpdb->prepare( "$alias.meta_key = %s", $meta_key );
+                } else {
+                    $where[$k] = $wpdb->prepare( "$alias.meta_key = %s", $meta_key );
+                }
+            }
 
             if ( is_null( $meta_value ) ) {
-                if ( empty( $where[$k] ) )
+                if ( empty( $where[$k] ) && empty( $where_meta_key ) ) {
                     unset( $join[$i] );
+                }
                 continue;
             }
@@ -1060 +1067 @@
         else
             $where = ' AND (' . implode( "\n{$this->relation} ", $where ) . ' )';
+
+        if ( ! empty( $where_meta_key ) ) {
+            $where .= "\nAND " . implode( "\nAND ", $where_meta_key );
+        }
 
         $join = implode( "\n", $join );