WordPress.org

Make WordPress Core

Changeset 29964


Ignore:
Timestamp:
10/19/2014 06:53:55 PM (5 years ago)
Author:
boonebgorges
Message:

Make orderby=meta_value compatible with meta_query when relation=OR.

Passing orderby=meta_value and meta_key=foo to WP_Query should require that
each matched post have a meta value with the key 'foo'. To make this
requirement compatible with meta_query params that have the relation OR, we
nest the meta_query param, and join it using AND to a meta_query clause
generated from the meta_key/meta_compare/meta_type query vars.

Fixes #25538.

Location:
trunk
Files:
3 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/wp-includes/meta.php

    r29953 r29964  
    10581058        $meta_query = array();
    10591059
    1060         // Simple query needs to be first for orderby=meta_value to work correctly.
     1060        /*
     1061         * For orderby=meta_value to work correctly, simple query needs to be
     1062         * first (so that its table join is against an unaliased meta table) and
     1063         * needs to be its own clause (so it doesn't interfere with the logic of
     1064         * the rest of the meta_query).
     1065         */
     1066        $primary_meta_query = array();
    10611067        foreach ( array( 'key', 'compare', 'type' ) as $key ) {
    1062             if ( !empty( $qv[ "meta_$key" ] ) )
    1063                 $meta_query[0][ $key ] = $qv[ "meta_$key" ];
     1068            if ( ! empty( $qv[ "meta_$key" ] ) ) {
     1069                $primary_meta_query[ $key ] = $qv[ "meta_$key" ];
     1070            }
    10641071        }
    10651072
    10661073        // WP_Query sets 'meta_value' = '' by default.
    1067         if ( isset( $qv[ 'meta_value' ] ) && '' !== $qv[ 'meta_value' ] && ( ! is_array( $qv[ 'meta_value' ] ) || $qv[ 'meta_value' ] ) )
    1068             $meta_query[0]['value'] = $qv[ 'meta_value' ];
    1069 
    1070         if ( !empty( $qv['meta_query'] ) && is_array( $qv['meta_query'] ) ) {
    1071             $meta_query = array_merge( $meta_query, $qv['meta_query'] );
     1074        if ( isset( $qv['meta_value'] ) && '' !== $qv['meta_value'] && ( ! is_array( $qv['meta_value'] ) || $qv['meta_value'] ) ) {
     1075            $primary_meta_query['value'] = $qv['meta_value'];
     1076        }
     1077
     1078        $existing_meta_query = isset( $qv['meta_query'] ) && is_array( $qv['meta_query'] ) ? $qv['meta_query'] : array();
     1079
     1080        if ( ! empty( $primary_meta_query ) && ! empty( $existing_meta_query ) ) {
     1081            $meta_query = array(
     1082                'relation' => 'AND',
     1083                $primary_meta_query,
     1084                $existing_meta_query,
     1085            );
     1086        } else if ( ! empty( $primary_meta_query ) ) {
     1087            $meta_query = array(
     1088                $primary_meta_query,
     1089            );
     1090        } else if ( ! empty( $existing_meta_query ) ) {
     1091            $meta_query = $existing_meta_query;
    10721092        }
    10731093
  • trunk/tests/phpunit/tests/meta/query.php

    r29940 r29964  
    105105
    106106        $expected1 = array(
    107             'key' => 'foo1',
    108             'compare' => 'baz1',
    109             'value' => 'bar1',
     107            'relation' => 'OR',
     108            array(
     109                'key' => 'foo1',
     110                'compare' => 'baz1',
     111                'value' => 'bar1',
     112            ),
    110113        );
    111114        $this->assertEquals( $expected1, $query->queries[1] );
  • trunk/tests/phpunit/tests/post/query.php

    r29940 r29964  
    11891189
    11901190    /**
     1191     * @group meta
     1192     * @ticket 29604
     1193     */
     1194    public function test_meta_query_with_orderby_meta_value_relation_or() {
     1195        $posts = $this->factory->post->create_many( 4 );
     1196        update_post_meta( $posts[0], 'foo', 5 );
     1197        update_post_meta( $posts[1], 'foo', 6 );
     1198        update_post_meta( $posts[2], 'foo', 4 );
     1199        update_post_meta( $posts[3], 'foo', 7 );
     1200
     1201        update_post_meta( $posts[0], 'bar1', 'baz' );
     1202        update_post_meta( $posts[1], 'bar1', 'baz' );
     1203        update_post_meta( $posts[2], 'bar2', 'baz' );
     1204
     1205        $query = new WP_Query( array(
     1206            'orderby' => 'meta_value',
     1207            'order' => 'ASC',
     1208            'meta_key' => 'foo',
     1209            'meta_query' => array(
     1210                'relation' => 'OR',
     1211                array(
     1212                    'key' => 'bar1',
     1213                    'value' => 'baz',
     1214                    'compare' => '=',
     1215                ),
     1216                array(
     1217                    'key' => 'bar2',
     1218                    'value' => 'baz',
     1219                    'compare' => '=',
     1220                ),
     1221            ),
     1222            'update_post_meta_cache' => false,
     1223            'update_post_term_cache' => false,
     1224            'fields' => 'ids',
     1225        ) );
     1226
     1227        $this->assertEquals( array( $posts[2], $posts[0], $posts[1] ), $query->posts );
     1228    }
     1229
     1230    /**
     1231     * @group meta
     1232     * @ticket 29604
     1233     */
     1234    public function test_meta_query_with_orderby_meta_value_relation_and() {
     1235        $posts = $this->factory->post->create_many( 4 );
     1236        update_post_meta( $posts[0], 'foo', 5 );
     1237        update_post_meta( $posts[1], 'foo', 6 );
     1238        update_post_meta( $posts[2], 'foo', 4 );
     1239        update_post_meta( $posts[3], 'foo', 7 );
     1240
     1241        update_post_meta( $posts[0], 'bar1', 'baz' );
     1242        update_post_meta( $posts[1], 'bar1', 'baz' );
     1243        update_post_meta( $posts[2], 'bar1', 'baz' );
     1244        update_post_meta( $posts[3], 'bar1', 'baz' );
     1245        update_post_meta( $posts[0], 'bar2', 'baz' );
     1246        update_post_meta( $posts[1], 'bar2', 'baz' );
     1247        update_post_meta( $posts[2], 'bar2', 'baz' );
     1248
     1249        $query = new WP_Query( array(
     1250            'orderby' => 'meta_value',
     1251            'order' => 'ASC',
     1252            'meta_key' => 'foo',
     1253            'meta_query' => array(
     1254                'relation' => 'AND',
     1255                array(
     1256                    'key' => 'bar1',
     1257                    'value' => 'baz',
     1258                    'compare' => '=',
     1259                ),
     1260                array(
     1261                    'key' => 'bar2',
     1262                    'value' => 'baz',
     1263                    'compare' => '=',
     1264                ),
     1265            ),
     1266            'update_post_meta_cache' => false,
     1267            'update_post_term_cache' => false,
     1268            'fields' => 'ids',
     1269        ) );
     1270
     1271        $this->assertEquals( array( $posts[2], $posts[0], $posts[1] ), $query->posts );
     1272    }
     1273
     1274    /**
    11911275     * @ticket 29642
    11921276     * @group meta
Note: See TracChangeset for help on using the changeset viewer.