WordPress.org
Get WordPress

Make WordPress Core

Ticket #40826: 40826.2.diff

File 40826.2.diff, 12.7 KB (added by kadamwhite, 7 years ago)

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

     
    9898         *     @type string       $orderby                Field(s) to order terms by. Accepts term fields ('name',
    9999         *                                                'slug', 'term_group', 'term_id', 'id', 'description', 'parent'),
    100100         *                                                'count' for term taxonomy count, 'include' to match the
    101          *                                                'order' of the $include param, 'meta_value', 'meta_value_num',
     101         *                                                'order' of the $include param, 'slug__in' to match the
     102         *                                                'order' of the $slug param, 'meta_value', 'meta_value_num',
    102103         *                                                the value of `$meta_key`, the array keys of `$meta_query`, or
    103104         *                                                'none' to omit the ORDER BY clause. Defaults to 'name'.
    104105         *     @type string       $order                  Whether to order terms in ascending or descending order.
    … …  
    841842                } elseif ( 'include' == $_orderby && ! empty( $this->query_vars['include'] ) ) {
    842843                        $include = implode( ',', wp_parse_id_list( $this->query_vars['include'] ) );
    843844                        $orderby = "FIELD( t.term_id, $include )";
     845                } elseif ( 'slug__in' == $_orderby && ! empty( $this->query_vars['slug'] ) && is_array( $this->query_vars['slug'] ) ) {
     846                        $slugs = implode( "', '", array_map( 'sanitize_title', $this->query_vars['slug__in'] ) );
     847                        $orderby = "FIELD( t.slug, '" . $slugs . "')";
    844848                } elseif ( 'none' == $_orderby ) {
    845849                        $orderby = '';
    846850                } elseif ( empty( $_orderby ) || 'id' == $_orderby || 'term_id' === $_orderby ) {

  • src/wp-includes/rest-api/endpoints/class-wp-rest-posts-controller.php

     
    874874                // Map to proper WP_Query orderby param.
    875875                if ( isset( $query_args['orderby'] ) && isset( $request['orderby'] ) ) {
    876876                        $orderby_mappings = array(
    877                                 'id'      => 'ID',
    878                                 'include' => 'post__in',
    879                                 'slug'    => 'post_name',
     877                                'id'            => 'ID',
     878                                'include'       => 'post__in',
     879                                'slug'          => 'post_name',
     880                                'include_slugs' => 'post_name__in',
    880881                        );
    881882
    882883                        if ( isset( $orderby_mappings[ $request['orderby'] ] ) ) {
    … …  
    21092110                                'parent',
    21102111                                'relevance',
    21112112                                'slug',
     2113                                'include_slugs',
    21122114                                'title',
    21132115                        ),
    21142116                );

  • src/wp-includes/rest-api/endpoints/class-wp-rest-terms-controller.php

     
    188188                        }
    189189                }
    190190
     191                if ( isset( $prepared_args['orderby'] ) && isset( $request['orderby'] ) ) {
     192                        $orderby_mappings = array(
     193                                'include_slugs' => 'slug__in',
     194                        );
     195
     196                        if ( isset( $orderby_mappings[ $request['orderby'] ] ) ) {
     197                                $prepared_args['orderby'] = $orderby_mappings[ $request['orderby'] ];
     198                        }
     199                }
     200
    191201                if ( isset( $registered['offset'] ) && ! empty( $request['offset'] ) ) {
    192202                        $prepared_args['offset'] = $request['offset'];
    193203                } else {
    … …  
    932942                                'include',
    933943                                'name',
    934944                                'slug',
     945                                'include_slugs',
    935946                                'term_group',
    936947                                'description',
    937948                                'count',

  • src/wp-includes/rest-api/endpoints/class-wp-rest-users-controller.php

     
    243243                                'name'            => 'display_name',
    244244                                'registered_date' => 'registered',
    245245                                'slug'            => 'user_nicename',
     246                                'include_slugs'   => 'nicename__in',
    246247                                'email'           => 'user_email',
    247248                                'url'             => 'user_url',
    248249                        );
    … …  
    13381339                                'name',
    13391340                                'registered_date',
    13401341                                'slug',
     1342                                'include_slugs',
    13411343                                'email',
    13421344                                'url',
    13431345                        ),

  • tests/phpunit/tests/rest-api/rest-categories-controller.php

     
    287287                $this->assertEquals( 'Cantaloupe', $data[2]['name'] );
    288288        }
    289289
     290        public function test_get_items_orderby_slugs() {
     291                $this->factory->category->create( array( 'name' => 'Burrito' ) );
     292                $this->factory->category->create( array( 'name' => 'Taco' ) );
     293                $this->factory->category->create( array( 'name' => 'Chalupa' ) );
     294
     295                $request = new WP_REST_Request( 'GET', '/wp/v2/categories' );
     296                $request->set_param( 'orderby', 'include_slugs' );
     297                $request->set_param( 'slug', array( 'taco', 'burrito', 'chalupa' ) );
     298                $response = $this->server->dispatch( $request );
     299                $data = $response->get_data();
     300                $this->assertEquals( 200, $response->get_status() );
     301                $this->assertEquals( 'taco', $data[0]['slug'] );
     302                $this->assertEquals( 'burrito', $data[1]['slug'] );
     303                $this->assertEquals( 'chalupa', $data[2]['slug'] );
     304        }
     305
    290306        protected function post_with_categories() {
    291307                $post_id = $this->factory->post->create();
    292308                $category1 = $this->factory->category->create( array(

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

     
    598598                $this->assertPostsOrderedBy( '{posts}.post_name DESC' );
    599599        }
    600600
     601        public function test_get_items_with_orderby_slugs() {
     602                $slugs = array( 'burrito', 'taco', 'chalupa' );
     603                foreach ( $slugs as $slug ) {
     604                        $this->factory->post->create( array( 'post_title' => $slug, 'post_name' => $slug, 'post_status' => 'publish' ) );
     605                }
     606
     607                $request = new WP_REST_Request( 'GET', '/wp/v2/posts' );
     608                $request->set_param( 'orderby', 'include_slugs' );
     609                $request->set_param( 'slug', array( 'taco', 'chalupa', 'burrito' ) );
     610
     611                $response = $this->server->dispatch( $request );
     612                $data = $response->get_data();
     613
     614                $this->assertEquals( 'taco', $data[0]['slug'] );
     615                $this->assertEquals( 'chalupa', $data[1]['slug'] );
     616                $this->assertEquals( 'burrito', $data[2]['slug'] );
     617        }
     618
    601619        public function test_get_items_with_orderby_relevance() {
    602620                $id1 = $this->factory->post->create( array( 'post_title' => 'Title is more relevant', 'post_content' => 'Content is', 'post_status' => 'publish' ) );
    603621                $id2 = $this->factory->post->create( array( 'post_title' => 'Title is', 'post_content' => 'Content is less relevant', 'post_status' => 'publish' ) );

  • tests/phpunit/tests/rest-api/rest-tags-controller.php

     
    250250                $this->assertEquals( 'Cantaloupe', $data[2]['name'] );
    251251        }
    252252
     253        public function test_get_items_orderby_slugs() {
     254                $this->factory->tag->create( array( 'name' => 'Burrito' ) );
     255                $this->factory->tag->create( array( 'name' => 'Taco' ) );
     256                $this->factory->tag->create( array( 'name' => 'Chalupa' ) );
     257
     258                $request = new WP_REST_Request( 'GET', '/wp/v2/tags' );
     259                $request->set_param( 'orderby', 'include_slugs' );
     260                $request->set_param( 'slug', array( 'taco', 'burrito', 'chalupa' ) );
     261                $response = $this->server->dispatch( $request );
     262                $data = $response->get_data();
     263                $this->assertEquals( 200, $response->get_status() );
     264                $this->assertEquals( 'taco', $data[0]['slug'] );
     265                $this->assertEquals( 'burrito', $data[1]['slug'] );
     266                $this->assertEquals( 'chalupa', $data[2]['slug'] );
     267        }
     268
    253269        public function test_get_items_post_args() {
    254270                $post_id = $this->factory->post->create();
    255271                $tag1 = $this->factory->tag->create( array( 'name' => 'DC' ) );

  • tests/phpunit/tests/rest-api/rest-users-controller.php

     
    409409                $this->assertEquals( $low_id, $data[0]['id'] );
    410410        }
    411411
     412        public function test_get_items_orderby_slugs() {
     413                wp_set_current_user( self::$user );
     414
     415                $this->factory->user->create( array( 'user_nicename' => 'burrito' ) );
     416                $this->factory->user->create( array( 'user_nicename' => 'taco' ) );
     417                $this->factory->user->create( array( 'user_nicename' => 'chalupa' ) );
     418
     419                $request = new WP_REST_Request( 'GET', '/wp/v2/users' );
     420                $request->set_param( 'orderby', 'include_slugs' );
     421                $request->set_param( 'slug', array( 'taco', 'burrito', 'chalupa' ) );
     422                $response = $this->server->dispatch( $request );
     423                $data = $response->get_data();
     424
     425                $this->assertEquals( 'taco', $data[0]['slug'] );
     426                $this->assertEquals( 'burrito', $data[1]['slug'] );
     427                $this->assertEquals( 'chalupa', $data[2]['slug'] );
     428        }
     429
    412430        public function test_get_items_orderby_email() {
    413431                wp_set_current_user( self::$user );
    414432

  • tests/phpunit/tests/rest-api.php

     
    430430        }
    431431
    432432        /**
     433                * Ensure that nested fields may be whitelisted with request['_fields'].
     434                */
     435        public function test_rest_filter_response_fields_nested_field_filter() {
     436                $response = new WP_REST_Response();
     437
     438                $response->set_data( array(
     439                        'a' => 0,
     440                        'b' => array(
     441                                '1' => 1,
     442                                '2' => 2,
     443                        ),
     444                        'c' => 3,
     445                        'd' => array(
     446                                '4' => 4,
     447                                '5' => 5,
     448                        ),
     449                ) );
     450                $request = array(
     451                        '_fields' => array( 'b.1', 'c', 'd.5' )
     452                );
     453
     454                $response = rest_filter_response_fields( $response, null, $request );
     455                $this->assertEquals( array(
     456                        'b' => array(
     457                                '1' => 1,
     458                        ),
     459                        'c' => 3,
     460                        'd' => array(
     461                                '5' => 5,
     462                        ),
     463                ), $response->get_data() );
     464        }
     465
     466        /**
    433467         * The get_rest_url function should return a URL consistently terminated with a "/",
    434468         * whether the blog is configured with pretty permalink support or not.
    435469         */

  • tests/qunit/fixtures/wp-api-generated.js

     
    284284                                "parent",
    285285                                "relevance",
    286286                                "slug",
     287                                "include_slugs",
    287288                                "title"
    288289                            ],
    289290                            "description": "Sort collection by object attribute.",
    … …  
    905906                                "parent",
    906907                                "relevance",
    907908                                "slug",
     909                                "include_slugs",
    908910                                "title",
    909911                                "menu_order"
    910912                            ],
    … …  
    14431445                                "parent",
    14441446                                "relevance",
    14451447                                "slug",
     1448                                "include_slugs",
    14461449                                "title"
    14471450                            ],
    14481451                            "description": "Sort collection by object attribute.",
    … …  
    20232026                                "include",
    20242027                                "name",
    20252028                                "slug",
     2029                                "include_slugs",
    20262030                                "term_group",
    20272031                                "description",
    20282032                                "count"
    … …  
    22662270                                "include",
    22672271                                "name",
    22682272                                "slug",
     2273                                "include_slugs",
    22692274                                "term_group",
    22702275                                "description",
    22712276                                "count"
    … …  
    24952500                                "name",
    24962501                                "registered_date",
    24972502                                "slug",
     2503                                "include_slugs",
    24982504                                "email",
    24992505                                "url"
    25002506                            ],