WordPress.org
Get WordPress

Make WordPress Core

Changeset 59766


Ignore:
Timestamp:
02/06/2025 05:02:17 AM (less than one hour ago)
Author:
peterwilsoncc
Message:

Query: Increase WP_Query cache hits for equivalent arguments.

Introduces normalization a number of arguments passed to WP_Query to increase cache hits for equivalent requests. For example author__in => [ 1, 2 ] and author__in => [ 2, 1 ] will now hit the same cache.

Prior to generating the SQL request and cache key, the following are sorted, made unique and type cast as appropriate.

  • post_type when passed as an array
  • post_status when passed as an array
  • term_querys containing terms
  • cat
  • category__in
  • category__not_in
  • category__and
  • tag_slug__in
  • tag__in
  • tag__not_in
  • tag__and
  • tag_slug__in
  • tag_slug__and
  • post_parent__not_in
  • author
  • author__not_in
  • author__in

The following are sorted for the purposes of generating the cache key and SQL WHERE clause but unmodified for use in the ORDER BY SQL clause:

  • post_name__in
  • post__in
  • post_parent__in

This commit includes changes to unrelated tests, assertions in Tests_Query_ParseQuery::test_parse_query_cat_array_mixed() and WP_Test_REST_Posts_Controller::test_get_items_not_sticky_with_exclude() have been modified to account for the sorting of the items above.

Props thekt12, peterwilsoncc, spacedmonkey, joemcgill, flixos90, mukesh27, pbearne, swissspidy.
Fixes #59516.

Location:
trunk
Files:
4 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/wp-includes/class-wp-query.php

    r59495 r59766  
    474474
    475475    private $compat_methods = array( 'init_query_flags', 'parse_tax_query' );
     476
     477    /**
     478     * The cache key generated by the query.
     479     *
     480     * The cache key is generated by the method ::generate_cache_key() after the
     481     * query has been normalized.
     482     *
     483     * @var string
     484     */
     485    private $query_cache_key = '';
    476486
    477487    /**
    … …  
    11021112        if ( ! empty( $qv['post_type'] ) ) {
    11031113            if ( is_array( $qv['post_type'] ) ) {
    1104                 $qv['post_type'] = array_map( 'sanitize_key', $qv['post_type'] );
     1114                $qv['post_type'] = array_map( 'sanitize_key', array_unique( $qv['post_type'] ) );
     1115                sort( $qv['post_type'] );
    11051116            } else {
    11061117                $qv['post_type'] = sanitize_key( $qv['post_type'] );
    … …  
    11101121        if ( ! empty( $qv['post_status'] ) ) {
    11111122            if ( is_array( $qv['post_status'] ) ) {
    1112                 $qv['post_status'] = array_map( 'sanitize_key', $qv['post_status'] );
     1123                $qv['post_status'] = array_map( 'sanitize_key', array_unique( $qv['post_status'] ) );
     1124                sort( $qv['post_status'] );
    11131125            } else {
    11141126                $qv['post_status'] = preg_replace( '|[^a-z0-9_,-]|', '', $qv['post_status'] );
    … …  
    11831195                $term = $q[ $t->query_var ];
    11841196
    1185                 if ( is_array( $term ) ) {
    1186                     $term = implode( ',', $term );
    1187                 }
     1197                if ( ! is_array( $term ) ) {
     1198                    $term = explode( ',', $term );
     1199                    $term = array_map( 'trim', $term );
     1200                }
     1201                sort( $term );
     1202                $term = implode( ',', $term );
    11881203
    11891204                if ( str_contains( $term, '+' ) ) {
    … …  
    12211236            $cat_array = preg_split( '/[,\s]+/', urldecode( $q['cat'] ) );
    12221237            $cat_array = array_map( 'intval', $cat_array );
    1223             $q['cat']  = implode( ',', $cat_array );
     1238            sort( $cat_array );
     1239            $q['cat'] = implode( ',', $cat_array );
    12241240
    12251241            foreach ( $cat_array as $cat ) {
    … …  
    12631279        if ( ! empty( $q['category__in'] ) ) {
    12641280            $q['category__in'] = array_map( 'absint', array_unique( (array) $q['category__in'] ) );
    1265             $tax_query[]       = array(
     1281            sort( $q['category__in'] );
     1282            $tax_query[] = array(
    12661283                'taxonomy'         => 'category',
    12671284                'terms'            => $q['category__in'],
    … …  
    12731290        if ( ! empty( $q['category__not_in'] ) ) {
    12741291            $q['category__not_in'] = array_map( 'absint', array_unique( (array) $q['category__not_in'] ) );
     1292            sort( $q['category__not_in'] );
    12751293            $tax_query[]           = array(
    12761294                'taxonomy'         => 'category',
    … …  
    12831301        if ( ! empty( $q['category__and'] ) ) {
    12841302            $q['category__and'] = array_map( 'absint', array_unique( (array) $q['category__and'] ) );
    1285             $tax_query[]        = array(
     1303            sort( $q['category__and'] );
     1304            $tax_query[] = array(
    12861305                'taxonomy'         => 'category',
    12871306                'terms'            => $q['category__and'],
    … …  
    13011320        if ( '' !== $q['tag'] && ! $this->is_singular && $this->query_vars_changed ) {
    13021321            if ( str_contains( $q['tag'], ',' ) ) {
     1322                // @todo Handle normalizing `tag` query string.
    13031323                $tags = preg_split( '/[,\r\n\t ]+/', $q['tag'] );
    13041324                foreach ( (array) $tags as $tag ) {
    13051325                    $tag                 = sanitize_term_field( 'slug', $tag, 0, 'post_tag', 'db' );
    13061326                    $q['tag_slug__in'][] = $tag;
     1327                    sort( $q['tag_slug__in'] );
    13071328                }
    13081329            } elseif ( preg_match( '/[+\r\n\t ]+/', $q['tag'] ) || ! empty( $q['cat'] ) ) {
    … …  
    13151336                $q['tag']            = sanitize_term_field( 'slug', $q['tag'], 0, 'post_tag', 'db' );
    13161337                $q['tag_slug__in'][] = $q['tag'];
     1338                sort( $q['tag_slug__in'] );
    13171339            }
    13181340        }
    … …  
    13281350        if ( ! empty( $q['tag__in'] ) ) {
    13291351            $q['tag__in'] = array_map( 'absint', array_unique( (array) $q['tag__in'] ) );
    1330             $tax_query[]  = array(
     1352            sort( $q['tag__in'] );
     1353            $tax_query[] = array(
    13311354                'taxonomy' => 'post_tag',
    13321355                'terms'    => $q['tag__in'],
    … …  
    13361359        if ( ! empty( $q['tag__not_in'] ) ) {
    13371360            $q['tag__not_in'] = array_map( 'absint', array_unique( (array) $q['tag__not_in'] ) );
     1361            sort( $q['tag__not_in'] );
    13381362            $tax_query[]      = array(
    13391363                'taxonomy' => 'post_tag',
    … …  
    13451369        if ( ! empty( $q['tag__and'] ) ) {
    13461370            $q['tag__and'] = array_map( 'absint', array_unique( (array) $q['tag__and'] ) );
    1347             $tax_query[]   = array(
     1371            sort( $q['tag__and'] );
     1372            $tax_query[] = array(
    13481373                'taxonomy' => 'post_tag',
    13491374                'terms'    => $q['tag__and'],
    … …  
    13541379        if ( ! empty( $q['tag_slug__in'] ) ) {
    13551380            $q['tag_slug__in'] = array_map( 'sanitize_title_for_query', array_unique( (array) $q['tag_slug__in'] ) );
    1356             $tax_query[]       = array(
     1381            sort( $q['tag_slug__in'] );
     1382            $tax_query[] = array(
    13571383                'taxonomy' => 'post_tag',
    13581384                'terms'    => $q['tag_slug__in'],
    … …  
    13631389        if ( ! empty( $q['tag_slug__and'] ) ) {
    13641390            $q['tag_slug__and'] = array_map( 'sanitize_title_for_query', array_unique( (array) $q['tag_slug__and'] ) );
    1365             $tax_query[]        = array(
     1391            sort( $q['tag_slug__and'] );
     1392            $tax_query[] = array(
    13661393                'taxonomy' => 'post_tag',
    13671394                'terms'    => $q['tag_slug__and'],
    … …  
    21872214        } elseif ( is_array( $q['post_name__in'] ) && ! empty( $q['post_name__in'] ) ) {
    21882215            $q['post_name__in'] = array_map( 'sanitize_title_for_query', $q['post_name__in'] );
    2189             $post_name__in      = "'" . implode( "','", $q['post_name__in'] ) . "'";
    2190             $where             .= " AND {$wpdb->posts}.post_name IN ($post_name__in)";
     2216            // Duplicate array before sorting to allow for the orderby clause.
     2217            $post_name__in_for_where = array_unique( $q['post_name__in'] );
     2218            sort( $post_name__in_for_where );
     2219            $post_name__in = "'" . implode( "','", $post_name__in_for_where ) . "'";
     2220            $where        .= " AND {$wpdb->posts}.post_name IN ($post_name__in)";
    21912221        }
    21922222
    … …  
    22002230            $where .= " AND {$wpdb->posts}.ID = " . $q['p'];
    22012231        } elseif ( $q['post__in'] ) {
    2202             $post__in = implode( ',', array_map( 'absint', $q['post__in'] ) );
     2232            // Duplicate array before sorting to allow for the orderby clause.
     2233            $post__in_for_where = $q['post__in'];
     2234            $post__in_for_where = array_unique( array_map( 'absint', $post__in_for_where ) );
     2235            sort( $post__in_for_where );
     2236            $post__in = implode( ',', array_map( 'absint', $post__in_for_where ) );
    22032237            $where   .= " AND {$wpdb->posts}.ID IN ($post__in)";
    22042238        } elseif ( $q['post__not_in'] ) {
     2239            sort( $q['post__not_in'] );
    22052240            $post__not_in = implode( ',', array_map( 'absint', $q['post__not_in'] ) );
    22062241            $where       .= " AND {$wpdb->posts}.ID NOT IN ($post__not_in)";
    … …  
    22102245            $where .= $wpdb->prepare( " AND {$wpdb->posts}.post_parent = %d ", $q['post_parent'] );
    22112246        } elseif ( $q['post_parent__in'] ) {
    2212             $post_parent__in = implode( ',', array_map( 'absint', $q['post_parent__in'] ) );
     2247            // Duplicate array before sorting to allow for the orderby clause.
     2248            $post_parent__in_for_where = $q['post_parent__in'];
     2249            $post_parent__in_for_where = array_unique( array_map( 'absint', $post_parent__in_for_where ) );
     2250            sort( $post_parent__in_for_where );
     2251            $post_parent__in = implode( ',', array_map( 'absint', $post_parent__in_for_where ) );
    22132252            $where          .= " AND {$wpdb->posts}.post_parent IN ($post_parent__in)";
    22142253        } elseif ( $q['post_parent__not_in'] ) {
     2254            sort( $q['post_parent__not_in'] );
    22152255            $post_parent__not_in = implode( ',', array_map( 'absint', $q['post_parent__not_in'] ) );
    22162256            $where              .= " AND {$wpdb->posts}.post_parent NOT IN ($post_parent__not_in)";
    … …  
    23422382            $q['author'] = addslashes_gpc( '' . urldecode( $q['author'] ) );
    23432383            $authors     = array_unique( array_map( 'intval', preg_split( '/[,\s]+/', $q['author'] ) ) );
     2384            sort( $authors );
    23442385            foreach ( $authors as $author ) {
    23452386                $key         = $author > 0 ? 'author__in' : 'author__not_in';
    … …  
    23502391
    23512392        if ( ! empty( $q['author__not_in'] ) ) {
    2352             $author__not_in = implode( ',', array_map( 'absint', array_unique( (array) $q['author__not_in'] ) ) );
     2393            if ( is_array( $q['author__not_in'] ) ) {
     2394                $q['author__not_in'] = array_unique( array_map( 'absint', $q['author__not_in'] ) );
     2395                sort( $q['author__not_in'] );
     2396            }
     2397            $author__not_in = implode( ',', (array) $q['author__not_in'] );
    23532398            $where         .= " AND {$wpdb->posts}.post_author NOT IN ($author__not_in) ";
    23542399        } elseif ( ! empty( $q['author__in'] ) ) {
     2400            if ( is_array( $q['author__in'] ) ) {
     2401                $q['author__in'] = array_unique( array_map( 'absint', $q['author__in'] ) );
     2402                sort( $q['author__in'] );
     2403            }
    23552404            $author__in = implode( ',', array_map( 'absint', array_unique( (array) $q['author__in'] ) ) );
    23562405            $where     .= " AND {$wpdb->posts}.post_author IN ($author__in) ";
    … …  
    25892638                $q_status = explode( ',', $q_status );
    25902639            }
     2640            sort( $q_status );
    25912641            $r_status = array();
    25922642            $p_status = array();
    … …  
    49034953        sort( $args['post_type'] );
    49044954
     4955        /*
     4956         * Sort arrays that can be used for ordering prior to cache key generation.
     4957         *
     4958         * These arrays are sorted in the query generator for the purposes of the
     4959         * WHERE clause but the arguments are not modified as they can be used for
     4960         * the orderby clase.
     4961         *
     4962         * Their use in the orderby clause will generate a different SQL query so
     4963         * they can be sorted for the cache key generation.
     4964         */
     4965        $sortable_arrays_with_int_values = array(
     4966            'post__in',
     4967            'post_parent__in',
     4968        );
     4969        foreach ( $sortable_arrays_with_int_values as $key ) {
     4970            if ( isset( $args[ $key ] ) && is_array( $args[ $key ] ) ) {
     4971                $args[ $key ] = array_unique( array_map( 'absint', $args[ $key ] ) );
     4972                sort( $args[ $key ] );
     4973            }
     4974        }
     4975
     4976        // Sort and unique the 'post_name__in' for cache key generation.
     4977        if ( isset( $args['post_name__in'] ) && is_array( $args['post_name__in'] ) ) {
     4978            $args['post_name__in'] = array_unique( $args['post_name__in'] );
     4979            sort( $args['post_name__in'] );
     4980        }
     4981
    49054982        if ( isset( $args['post_status'] ) ) {
    49064983            $args['post_status'] = (array) $args['post_status'];
    … …  
    49435020        }
    49445021
    4945         return "wp_query:$key:$last_changed";
     5022        $this->query_cache_key = "wp_query:$key:$last_changed";
     5023        return $this->query_cache_key;
    49465024    }
    49475025

  • trunk/tests/phpunit/tests/query/cacheResults.php

    r58122 r59766  
    202202
    203203    /**
    204      * @ticket 59442
     204     * @ticket 59516
    205205     *
    206206     * @covers WP_Query::generate_cache_key
    207      *
    208      * @dataProvider data_query_cache_duplicate
    209      */
    210     public function test_generate_cache_key_normalize( $query_vars1, $query_vars2 ) {
     207     */
     208    public function test_post_in_order_by_clauses_are_not_normalized() {
    211209        global $wpdb;
     210
     211        $post_ids = self::$posts;
     212
     213        $query_vars1 = array(
     214            'post__in' => $post_ids,
     215            'orderby'  => 'post__in',
     216        );
     217        $query_vars2 = array(
     218            'post__in' => array_reverse( $post_ids ),
     219            'orderby'  => 'post__in',
     220        );
    212221
    213222        $fields   = "{$wpdb->posts}.ID";
    … …  
    218227        $request2 = str_replace( $fields, "{$wpdb->posts}.*", $query2->request );
    219228
    220         $reflection = new ReflectionMethod( $query1, 'generate_cache_key' );
    221         $reflection->setAccessible( true );
     229        $reflection_q1 = new ReflectionProperty( $query1, 'query_cache_key' );
     230        $reflection_q1->setAccessible( true );
     231
     232        $reflection_q2 = new ReflectionProperty( $query2, 'query_cache_key' );
     233        $reflection_q2->setAccessible( true );
     234
     235        $this->assertNotSame( $request1, $request2, 'Queries should not match' );
     236
     237        $cache_key_1 = $reflection_q1->getValue( $query1 );
     238        $cache_key_2 = $reflection_q2->getValue( $query2 );
     239
     240        $this->assertNotSame( $cache_key_1, $cache_key_2, 'Cache key should differ.' );
     241        $this->assertNotEmpty( $cache_key_1, 'Cache key for query one should not be empty.' );
     242        $this->assertNotEmpty( $cache_key_2, 'Cache key for query two should not be empty.' );
     243
     244        // Test the posts are returned different orders.
     245        $this->assertNotSame( wp_list_pluck( $query1->posts, 'ID' ), wp_list_pluck( $query2->posts, 'ID' ), 'Query one posts should not match the order of query two posts.' );
     246        // Test the posts are the same sets.
     247        $this->assertSameSets( wp_list_pluck( $query1->posts, 'ID' ), wp_list_pluck( $query2->posts, 'ID' ), 'Query one posts should match the set of query two posts.' );
     248    }
     249
     250    /**
     251     * @ticket 59516
     252     *
     253     * @covers WP_Query::generate_cache_key
     254     */
     255    public function test_post_parent_in_order_by_clauses_are_not_normalized() {
     256        global $wpdb;
     257
     258        $parent_pages = self::$pages;
     259        $post_names   = array( 'doctor-dillamond', 'elphaba', 'fiyero', 'glinda', 'the-wizard-of-oz' );
     260        $child_pages  = array();
     261        foreach ( $parent_pages as $key => $parent_page ) {
     262            $child_pages[] = self::factory()->post->create(
     263                array(
     264                    'post_parent' => $parent_page,
     265                    'post_type'   => 'page',
     266                    'post_name'   => $post_names[ $key ],
     267                )
     268            );
     269        }
     270
     271        $query_vars1 = array(
     272            'post_parent__in' => $parent_pages,
     273            'post_type'       => 'page',
     274            'orderby'         => 'post_parent__in',
     275        );
     276
     277        $query_vars2 = array(
     278            'post_parent__in' => array_reverse( $parent_pages ),
     279            'post_type'       => 'page',
     280            'orderby'         => 'post_parent__in',
     281        );
     282
     283        $fields   = "{$wpdb->posts}.ID";
     284        $query1   = new WP_Query( $query_vars1 );
     285        $request1 = str_replace( $fields, "{$wpdb->posts}.*", $query1->request );
     286
     287        $query2   = new WP_Query( $query_vars2 );
     288        $request2 = str_replace( $fields, "{$wpdb->posts}.*", $query2->request );
     289
     290        $reflection_q1 = new ReflectionProperty( $query1, 'query_cache_key' );
     291        $reflection_q1->setAccessible( true );
     292
     293        $reflection_q2 = new ReflectionProperty( $query2, 'query_cache_key' );
     294        $reflection_q2->setAccessible( true );
     295
     296        $this->assertNotSame( $request1, $request2, 'Queries should not match' );
     297
     298        $cache_key_1 = $reflection_q1->getValue( $query1 );
     299        $cache_key_2 = $reflection_q2->getValue( $query2 );
     300
     301        $this->assertNotSame( $cache_key_1, $cache_key_2, 'Cache key should differ.' );
     302        $this->assertNotEmpty( $cache_key_1, 'Cache key for query one should not be empty.' );
     303        $this->assertNotEmpty( $cache_key_2, 'Cache key for query two should not be empty.' );
     304
     305        // Test the posts are returned in the correct order.
     306        $this->assertSame( array( 'doctor-dillamond', 'elphaba', 'fiyero', 'glinda', 'the-wizard-of-oz' ), wp_list_pluck( $query1->posts, 'post_name' ), 'Query one posts should be in alphabetical order' );
     307        $this->assertSame( array( 'the-wizard-of-oz', 'glinda', 'fiyero', 'elphaba', 'doctor-dillamond' ), wp_list_pluck( $query2->posts, 'post_name' ), 'Query two posts should be in reverse alphabetical order.' );
     308        // Test the posts are the same sets.
     309        $this->assertSameSets( wp_list_pluck( $query1->posts, 'ID' ), wp_list_pluck( $query2->posts, 'ID' ), 'Query one posts should match the set of query two posts.' );
     310    }
     311
     312    /**
     313     * @ticket 59516
     314     *
     315     * @covers WP_Query::generate_cache_key
     316     */
     317    public function test_post_name_in_order_by_clauses_are_not_normalized() {
     318        global $wpdb;
     319        $post_names = array( 'doctor-dillamond', 'elphaba', 'glinda', 'the-wizard-of-oz' );
     320        $posts      = array();
     321
     322        foreach ( $post_names as $post_name ) {
     323            $posts[] = self::factory()->post->create(
     324                array(
     325                    'post_name' => $post_name,
     326                )
     327            );
     328        }
     329
     330        $query_vars1 = array(
     331            'post_name__in' => $post_names,
     332            'orderby'       => 'post_name__in',
     333        );
     334
     335        $query_vars2 = array(
     336            'post_name__in' => array_reverse( $post_names ),
     337            'orderby'       => 'post_name__in',
     338        );
     339
     340        $fields   = "{$wpdb->posts}.ID";
     341        $query1   = new WP_Query( $query_vars1 );
     342        $request1 = str_replace( $fields, "{$wpdb->posts}.*", $query1->request );
     343
     344        $query2   = new WP_Query( $query_vars2 );
     345        $request2 = str_replace( $fields, "{$wpdb->posts}.*", $query2->request );
     346
     347        $reflection_q1 = new ReflectionProperty( $query1, 'query_cache_key' );
     348        $reflection_q1->setAccessible( true );
     349
     350        $reflection_q2 = new ReflectionProperty( $query2, 'query_cache_key' );
     351        $reflection_q2->setAccessible( true );
     352
     353        $this->assertNotSame( $request1, $request2, 'Queries should not match' );
     354
     355        $cache_key_1 = $reflection_q1->getValue( $query1 );
     356        $cache_key_2 = $reflection_q2->getValue( $query2 );
     357
     358        $this->assertNotSame( $cache_key_1, $cache_key_2, 'Cache key should differ.' );
     359        $this->assertNotEmpty( $cache_key_1, 'Cache key for query one should not be empty.' );
     360        $this->assertNotEmpty( $cache_key_2, 'Cache key for query two should not be empty.' );
     361
     362        // Test the posts are returned in the correct order.
     363        $this->assertSame( array( 'doctor-dillamond', 'elphaba', 'glinda', 'the-wizard-of-oz' ), wp_list_pluck( $query1->posts, 'post_name' ), 'Query one posts should be in alphabetical order' );
     364        $this->assertSame( array( 'the-wizard-of-oz', 'glinda', 'elphaba', 'doctor-dillamond' ), wp_list_pluck( $query2->posts, 'post_name' ), 'Query two posts should be in reverse alphabetical order.' );
     365        // Test the posts are the same sets.
     366        $this->assertSameSets( wp_list_pluck( $query1->posts, 'ID' ), wp_list_pluck( $query2->posts, 'ID' ), 'Query one posts should match the set of query two posts.' );
     367    }
     368
     369    /**
     370     * @ticket 59442
     371     * @ticket 59516
     372     *
     373     * @covers WP_Query::generate_cache_key
     374     *
     375     * @dataProvider data_query_cache_duplicate
     376     */
     377    public function test_generate_cache_key_normalize( $query_vars1, $query_vars2 ) {
     378        global $wpdb;
     379
     380        $fields   = "{$wpdb->posts}.ID";
     381        $query1   = new WP_Query( $query_vars1 );
     382        $request1 = str_replace( $fields, "{$wpdb->posts}.*", $query1->request );
     383
     384        $query2   = new WP_Query( $query_vars2 );
     385        $request2 = str_replace( $fields, "{$wpdb->posts}.*", $query2->request );
     386
     387        $reflection_q1 = new ReflectionProperty( $query1, 'query_cache_key' );
     388        $reflection_q1->setAccessible( true );
     389
     390        $reflection_q2 = new ReflectionProperty( $query2, 'query_cache_key' );
     391        $reflection_q2->setAccessible( true );
    222392
    223393        $this->assertSame( $request1, $request2, 'Queries should match' );
    224394
    225         $cache_key_1 = $reflection->invoke( $query1, $query_vars1, $request1 );
    226         $cache_key_2 = $reflection->invoke( $query1, $query_vars2, $request2 );
    227 
    228         $this->assertSame( $cache_key_1, $cache_key_2, 'Cache key differs the same paramters.' );
     395        $cache_key_1 = $reflection_q1->getValue( $query1 );
     396        $cache_key_2 = $reflection_q2->getValue( $query2 );
     397
     398        $this->assertSame( $cache_key_1, $cache_key_2, 'Cache key differs the same effective parameters.' );
     399        $this->assertNotEmpty( $cache_key_1, 'Cache key for query one should not be empty.' );
     400        $this->assertNotEmpty( $cache_key_2, 'Cache key for query two should not be empty.' );
    229401    }
    230402
    … …  
    281453    public function data_query_cache_duplicate() {
    282454        return array(
    283             'post type empty'           => array(
     455            'post type empty'                              => array(
    284456                'query_vars1' => array( 'post_type' => '' ),
    285457                'query_vars2' => array( 'post_type' => 'post' ),
    286458            ),
    287             'post type array'           => array(
     459            'post type array'                              => array(
    288460                'query_vars1' => array( 'post_type' => array( 'page' ) ),
    289461                'query_vars2' => array( 'post_type' => 'page' ),
    290462            ),
    291             'orderby empty'             => array(
     463            'orderby empty'                                => array(
    292464                'query_vars1' => array( 'orderby' => null ),
    293465                'query_vars2' => array( 'orderby' => 'date' ),
    294466            ),
    295             'different order parameter' => array(
     467            'different order parameter'                    => array(
    296468                'query_vars1' => array(
    297469                    'post_type'      => 'post',
    … …  
    303475                ),
    304476            ),
    305             'same args'                 => array(
     477            'same args'                                    => array(
    306478                'query_vars1' => array( 'post_type' => 'post' ),
    307479                'query_vars2' => array( 'post_type' => 'post' ),
    308480            ),
    309             'same args any'             => array(
     481            'same args any'                                => array(
    310482                'query_vars1' => array( 'post_type' => 'any' ),
    311483                'query_vars2' => array( 'post_type' => 'any' ),
    312484            ),
    313             'any and post types'        => array(
     485            'any and post types'                           => array(
    314486                'query_vars1' => array( 'post_type' => 'any' ),
    315487                'query_vars2' => array( 'post_type' => array( 'post', 'page', 'attachment' ) ),
    316488            ),
    317             'different order post type' => array(
     489            'different order post type'                    => array(
    318490                'query_vars1' => array( 'post_type' => array( 'post', 'page' ) ),
    319491                'query_vars2' => array( 'post_type' => array( 'page', 'post' ) ),
    320492            ),
    321             'post status array'         => array(
     493            'non-unique post type'                         => array(
     494                'query_vars1' => array( 'post_type' => array( 'post', 'page' ) ),
     495                'query_vars2' => array( 'post_type' => array( 'page', 'post', 'page' ) ),
     496            ),
     497            'post status array'                            => array(
    322498                'query_vars1' => array( 'post_status' => 'publish' ),
    323499                'query_vars2' => array( 'post_status' => array( 'publish' ) ),
    324500            ),
    325             'post status order'         => array(
     501            'post status order'                            => array(
    326502                'query_vars1' => array( 'post_status' => array( 'draft', 'publish' ) ),
    327503                'query_vars2' => array( 'post_status' => array( 'publish', 'draft' ) ),
    328504            ),
    329             'cache parameters'          => array(
     505            'non-unique post status'                       => array(
     506                'query_vars1' => array( 'post_status' => array( 'draft', 'publish' ) ),
     507                'query_vars2' => array( 'post_status' => array( 'draft', 'publish', 'draft' ) ),
     508            ),
     509            'post id int vs string'                        => array(
     510                'query_vars1' => array( 'p' => '1' ),
     511                'query_vars2' => array( 'p' => 1 ),
     512            ),
     513            'page id int vs string'                        => array(
     514                'query_vars1' => array( 'page_id' => '2' ),
     515                'query_vars2' => array( 'page_id' => 2 ),
     516            ),
     517            'attachment id int vs string'                  => array(
     518                'query_vars1' => array( 'attachment_id' => '3' ),
     519                'query_vars2' => array( 'attachment_id' => 3 ),
     520            ),
     521            'date and time values int vs string'           => array(
     522                'query_vars1' => array(
     523                    'year'     => '2013',
     524                    'monthnum' => '12',
     525                    'day'      => '12',
     526                    'hour'     => '12',
     527                    'minute'   => '12',
     528                    'second'   => '12',
     529                ),
     530                'query_vars2' => array(
     531                    'year'     => 2013,
     532                    'monthnum' => 12,
     533                    'day'      => 12,
     534                    'hour'     => 12,
     535                    'minute'   => 12,
     536                    'second'   => 12,
     537                ),
     538            ),
     539            'offset value int vs string'                   => array(
     540                'query_vars1' => array( 'offset' => '5' ),
     541                'query_vars2' => array( 'offset' => 5 ),
     542            ),
     543            'posts per page value int vs string'           => array(
     544                'query_vars1' => array( 'posts_per_page' => '5' ),
     545                'query_vars2' => array( 'posts_per_page' => 5 ),
     546            ),
     547            'paged value int vs string'                    => array(
     548                'query_vars1' => array( 'paged' => '2' ),
     549                'query_vars2' => array( 'paged' => 2 ),
     550            ),
     551            'menu_order value int vs string'               => array(
     552                'query_vars1' => array( 'menu_order' => '2' ),
     553                'query_vars2' => array( 'menu_order' => 2 ),
     554            ),
     555            'post__in different order'                     => array(
     556                'query_vars1' => array( 'post__in' => array( 1, 2, 3, 4, 5 ) ),
     557                'query_vars2' => array( 'post__in' => array( 5, 4, 3, 2, 1 ) ),
     558            ),
     559            'post__in non-unique'                          => array(
     560                'query_vars1' => array( 'post__in' => array( 1, 2, 3, 4, 5 ) ),
     561                'query_vars2' => array( 'post__in' => array( 1, 2, 3, 4, 5, 1, 2, 3 ) ),
     562            ),
     563            'post_parent__in different order'              => array(
     564                'query_vars1' => array( 'post_parent__in' => array( 1, 2, 3, 4, 5 ) ),
     565                'query_vars2' => array( 'post_parent__in' => array( 5, 4, 3, 2, 1 ) ),
     566            ),
     567            'post_parent__in non-unique'                   => array(
     568                'query_vars1' => array( 'post_parent__in' => array( 1, 2, 3, 4, 5 ) ),
     569                'query_vars2' => array( 'post_parent__in' => array( 1, 2, 3, 4, 5, 1, 2, 3 ) ),
     570            ),
     571            'post_name__in different order'                => array(
     572                'query_vars1' => array( 'post_name__in' => array( 'elphaba', 'glinda', 'the-wizard-of-oz', 'doctor-dillamond' ) ),
     573                'query_vars2' => array( 'post_name__in' => array( 'doctor-dillamond', 'elphaba', 'the-wizard-of-oz', 'glinda' ) ),
     574            ),
     575            'post_name__in non-unique'                     => array(
     576                'query_vars1' => array( 'post_name__in' => array( 'elphaba', 'glinda', 'the-wizard-of-oz', 'doctor-dillamond' ) ),
     577                'query_vars2' => array( 'post_name__in' => array( 'elphaba', 'glinda', 'elphaba', 'glinda', 'the-wizard-of-oz', 'doctor-dillamond' ) ),
     578            ),
     579            'cat different order (array)'                  => array(
     580                'query_vars_1' => array( 'cat' => array( '1', '2' ) ),
     581                'query_vars_2' => array( 'cat' => array( '2', '1' ) ),
     582            ),
     583            'cat different order (string)'                 => array(
     584                'query_vars_1' => array( 'cat' => '2,1' ),
     585                'query_vars_2' => array( 'cat' => '1,2' ),
     586            ),
     587            'cat queries int vs string'                    => array(
     588                'query_vars_1' => array( 'cat' => '2' ),
     589                'query_vars_2' => array( 'cat' => 2 ),
     590            ),
     591            'category__in queries different order (array)' => array(
     592                'query_vars_1' => array( 'category__in' => array( '1', '2' ) ),
     593                'query_vars_2' => array( 'category__in' => array( '2', '1' ) ),
     594            ),
     595            'category__in queries with non-unique array'   => array(
     596                'query_vars_1' => array( 'category__in' => array( '1', '1' ) ),
     597                'query_vars_2' => array( 'category__in' => array( '1' ) ),
     598            ),
     599            'category__in queries string vs array (array)' => array(
     600                'query_vars_1' => array( 'category__in' => array( '1' ) ),
     601                'query_vars_2' => array( 'category__in' => array( 1 ) ),
     602            ),
     603            'category__not_in different order (array)'     => array(
     604                'query_vars_1' => array( 'category__not_in' => array( '1', '2' ) ),
     605                'query_vars_2' => array( 'category__not_in' => array( '2', '1' ) ),
     606            ),
     607            'category__not_in with non-unique array'       => array(
     608                'query_vars_1' => array( 'category__not_in' => array( '1', '1' ) ),
     609                'query_vars_2' => array( 'category__not_in' => array( '1' ) ),
     610            ),
     611            'category__not_in queries string vs array (array)' => array(
     612                'query_vars_1' => array( 'category__not_in' => array( '1' ) ),
     613                'query_vars_2' => array( 'category__not_in' => array( 1 ) ),
     614            ),
     615            'category__and queries width different order (array)' => array(
     616                'query_vars_1' => array( 'category__and' => array( '1', '2' ) ),
     617                'query_vars_2' => array( 'category__and' => array( '2', '1' ) ),
     618            ),
     619            'category__and with non-unique array'          => array(
     620                'query_vars_1' => array( 'category__and' => array( '1', '1', '2' ) ),
     621                'query_vars_2' => array( 'category__and' => array( '1', '2' ) ),
     622            ),
     623            'category__and queries string vs array (array)' => array(
     624                'query_vars_1' => array( 'category__and' => array( '1', '2' ) ),
     625                'query_vars_2' => array( 'category__and' => array( 1, 2 ) ),
     626            ),
     627            'author queries different order (string)'      => array(
     628                'query_vars_1' => array( 'author' => '1,2' ),
     629                'query_vars_2' => array( 'author' => '2,1' ),
     630            ),
     631            'author with non-unique string'                => array(
     632                'query_vars_1' => array( 'author' => '1,1' ),
     633                'query_vars_2' => array( 'author' => '1' ),
     634            ),
     635            'author queries int vs string (string)'        => array(
     636                'query_vars_1' => array( 'author' => 1 ),
     637                'query_vars_2' => array( 'author' => '1' ),
     638            ),
     639            'author queries int vs string (array)'         => array(
     640                'query_vars_1' => array( 'author' => array( 1 ) ),
     641                'query_vars_2' => array( 'author' => array( '1' ) ),
     642            ),
     643            'author__in different order'                   => array(
     644                'query_vars_1' => array( 'author__in' => array( 1, 2 ) ),
     645                'query_vars_2' => array( 'author__in' => array( 2, 1 ) ),
     646            ),
     647            'author__in with non-unique array'             => array(
     648                'query_vars_1' => array( 'author__in' => array( 1, 1, 2 ) ),
     649                'query_vars_2' => array( 'author__in' => array( 1, 2 ) ),
     650            ),
     651            'author__in queries int vs string (array)'     => array(
     652                'query_vars_1' => array( 'author__in' => array( 1 ) ),
     653                'query_vars_2' => array( 'author__in' => array( '1' ) ),
     654            ),
     655            'author__not_in different order (array)'       => array(
     656                'query_vars_1' => array( 'author__not_in' => array( 1, 2 ) ),
     657                'query_vars_2' => array( 'author__not_in' => array( 2, 1 ) ),
     658            ),
     659            'author__not_in queries int vs string (array)' => array(
     660                'query_vars_1' => array( 'author__not_in' => array( 1 ) ),
     661                'query_vars_2' => array( 'author__not_in' => array( '1' ) ),
     662            ),
     663            'tag_slug__in order'                           => array(
     664                'query_vars_1' => array( 'tag_slug__in' => array( 'foo', 'bar' ) ),
     665                'query_vars_2' => array( 'tag_slug__in' => array( 'bar', 'foo' ) ),
     666            ),
     667            'tag_slug__in non-unique vs unique'            => array(
     668                'query_vars_1' => array( 'tag_slug__in' => array( 'foo', 'bar', 'bar' ) ),
     669                'query_vars_2' => array( 'tag_slug__in' => array( 'foo', 'bar' ) ),
     670            ),
     671            'tag_slug__and order'                          => array(
     672                'query_vars_1' => array( 'tag_slug__and' => array( 'foo', 'bar' ) ),
     673                'query_vars_2' => array( 'tag_slug__and' => array( 'bar', 'foo' ) ),
     674            ),
     675            'tag_slug__and non-unique'                     => array(
     676                'query_vars_1' => array( 'tag_slug__and' => array( 'foo', 'bar', 'foo' ) ),
     677                'query_vars_2' => array( 'tag_slug__and' => array( 'bar', 'foo' ) ),
     678            ),
     679            'tag__in queries different order (array)'      => array(
     680                'query_vars_1' => array( 'tag__in' => array( 1, 2 ) ),
     681                'query_vars_2' => array( 'tag__in' => array( 2, 1 ) ),
     682            ),
     683            'tag__in queries non-unique array'             => array(
     684                'query_vars_1' => array( 'tag__in' => array( 1, 2, 1 ) ),
     685                'query_vars_2' => array( 'tag__in' => array( 2, 1 ) ),
     686            ),
     687            'tag__in queries int vs string'                => array(
     688                'query_vars_1' => array( 'tag__in' => array( 2, 1 ) ),
     689                'query_vars_2' => array( 'tag__in' => array( '2', '1' ) ),
     690            ),
     691            'tag__and queries different order (array)'     => array(
     692                'query_vars_1' => array( 'tag__and' => array( 1, 2 ) ),
     693                'query_vars_2' => array( 'tag__and' => array( 2, 1 ) ),
     694            ),
     695            'tag__and queries non-unique array'            => array(
     696                'query_vars_1' => array( 'tag__and' => array( 1, 2, 2 ) ),
     697                'query_vars_2' => array( 'tag__and' => array( 2, 1 ) ),
     698            ),
     699            'tag__not_in queries different order (array)'  => array(
     700                'query_vars_1' => array( 'tag__not_in' => array( 1, 2 ) ),
     701                'query_vars_2' => array( 'tag__not_in' => array( 2, 1 ) ),
     702            ),
     703            'tag__not_in queries non-unique array'         => array(
     704                'query_vars_1' => array( 'tag__not_in' => array( 1, 2, 2 ) ),
     705                'query_vars_2' => array( 'tag__not_in' => array( 1, 2 ) ),
     706            ),
     707            'tag__not_in queries int vs string (array)'    => array(
     708                'query_vars_1' => array( 'tag__not_in' => array( '1' ) ),
     709                'query_vars_2' => array( 'tag__not_in' => array( 1 ) ),
     710            ),
     711            'cache parameters'                             => array(
    330712                'query_vars1' => array(
    331713                    'update_post_meta_cache' => true,

  • trunk/tests/phpunit/tests/query/parseQuery.php

    r53891 r59766  
    152152        );
    153153
    154         $this->assertSame( '1,-1', $q->query_vars['cat'] );
     154        $this->assertSame( '-1,1', $q->query_vars['cat'] );
    155155    }
    156156

  • trunk/tests/phpunit/tests/rest-api/rest-posts-controller.php

    r59630 r59766  
    15751575        $this->assertNotContains( $id3, $ids );
    15761576
    1577         $this->assertPostsWhere( " AND {posts}.ID NOT IN ($id3,$id2) AND {posts}.post_type = 'post' AND (({posts}.post_status = 'publish'))" );
     1577        $this->assertPostsWhere( " AND {posts}.ID NOT IN ($id2,$id3) AND {posts}.post_type = 'post' AND (({posts}.post_status = 'publish'))" );
    15781578    }
    15791579
Note: See TracChangeset for help on using the changeset viewer.