mirror of
https://github.com/WordPress/WordPress.git
synced 2024-12-22 17:18:32 +01:00
Use the primary meta_query
clause when parsing orderby
in WP_Query
.
When using legacy `meta_key`, `meta_value`, etc. arguments in `WP_Query`, they're converted into the first clause of a `meta_query`. By using that clause instead of the original arguments, we make sure that behavior is consistent between the two available formats. props boonebgorges. fixes #16814. Built from https://develop.svn.wordpress.org/trunk@29855 git-svn-id: http://core.svn.wordpress.org/trunk@29618 1a063a9b-81f0-0310-95a4-ce76da25c4cd
This commit is contained in:
parent
2b88252b38
commit
5f17e0952d
@ -2223,9 +2223,16 @@ class WP_Query {
|
||||
'parent', 'type', 'ID', 'menu_order', 'comment_count', 'rand',
|
||||
);
|
||||
|
||||
$meta_key = $this->get( 'meta_key' );
|
||||
if ( ! empty( $meta_key ) ) {
|
||||
$allowed_keys[] = $meta_key;
|
||||
$primary_meta_key = '';
|
||||
$primary_meta_query = false;
|
||||
if ( ! empty( $this->meta_query->queries ) ) {
|
||||
$primary_meta_query = reset( $this->meta_query->queries );
|
||||
|
||||
if ( ! empty( $primary_meta_query['key'] ) ) {
|
||||
$primary_meta_key = $primary_meta_query['key'];
|
||||
$allowed_keys[] = $primary_meta_key;
|
||||
}
|
||||
|
||||
$allowed_keys[] = 'meta_value';
|
||||
$allowed_keys[] = 'meta_value_num';
|
||||
}
|
||||
@ -2250,12 +2257,11 @@ class WP_Query {
|
||||
case 'rand':
|
||||
$orderby = 'RAND()';
|
||||
break;
|
||||
case $meta_key:
|
||||
case $primary_meta_key:
|
||||
case 'meta_value':
|
||||
$type = $this->get( 'meta_type' );
|
||||
if ( ! empty( $type ) ) {
|
||||
$meta_type = $this->meta_query->get_cast_for_type( $type );
|
||||
$orderby = "CAST($wpdb->postmeta.meta_value AS {$meta_type})";
|
||||
if ( ! empty( $primary_meta_query['type'] ) ) {
|
||||
$sql_type = $this->meta_query->get_cast_for_type( $primary_meta_query['type'] );
|
||||
$orderby = "CAST($wpdb->postmeta.meta_value AS {$sql_type})";
|
||||
} else {
|
||||
$orderby = "$wpdb->postmeta.meta_value";
|
||||
}
|
||||
|
Loading…
Reference in New Issue
Block a user