WordPress.org
Get WordPress

Make WordPress Core

Changeset 43715


Ignore:
Timestamp:
10/11/2018 07:15:22 AM (7 years ago)
Author:
SergeyBiryukov
Message:

REST API: Revert [43648] from the 4.9 branch.

This change is out of the 4.9.x scope, and will be reintroduced in 5.0.x.

See #40510.

Location:
branches/4.9
Files:
5 edited

Legend:

Unmodified
Added
Removed

  • branches/4.9

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

    r43648 r43715  
    865865             * @param string $value The query_var value.
    866866             */
    867             $query_args[ $key ] = apply_filters( "rest_query_var-{$key}", $value ); // phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
     867            $query_args[ $key ] = apply_filters( "rest_query_var-{$key}", $value );
    868868        }
    869869

  • branches/4.9/src/wp-includes/rest-api/endpoints/class-wp-rest-revisions-controller.php

    r43648 r43715  
    198198        }
    199199
    200         // Ensure a search string is set in case the orderby is set to 'relevance'.
    201         if ( ! empty( $request['orderby'] ) && 'relevance' === $request['orderby'] && empty( $request['search'] ) ) {
    202             return new WP_Error( 'rest_no_search_term_defined', __( 'You need to define a search term to order by relevance.' ), array( 'status' => 400 ) );
    203         }
    204 
    205         // Ensure an include parameter is set in case the orderby is set to 'include'.
    206         if ( ! empty( $request['orderby'] ) && 'include' === $request['orderby'] && empty( $request['include'] ) ) {
    207             return new WP_Error( 'rest_orderby_include_missing_include', __( 'You need to define an include parameter to order by include.' ), array( 'status' => 400 ) );
    208         }
    209 
    210         if ( wp_revisions_enabled( $parent ) ) {
    211             $registered = $this->get_collection_params();
    212             $args       = array(
    213                 'post_parent'      => $parent->ID,
    214                 'post_type'        => 'revision',
    215                 'post_status'      => 'inherit',
    216                 'posts_per_page'   => -1,
    217                 'orderby'          => 'date ID',
    218                 'order'            => 'DESC',
    219                 'suppress_filters' => true,
    220             );
    221 
    222             $parameter_mappings = array(
    223                 'exclude'  => 'post__not_in',
    224                 'include'  => 'post__in',
    225                 'offset'   => 'offset',
    226                 'order'    => 'order',
    227                 'orderby'  => 'orderby',
    228                 'page'     => 'paged',
    229                 'per_page' => 'posts_per_page',
    230                 'search'   => 's',
    231             );
    232 
    233             foreach ( $parameter_mappings as $api_param => $wp_param ) {
    234                 if ( isset( $registered[ $api_param ], $request[ $api_param ] ) ) {
    235                     $args[ $wp_param ] = $request[ $api_param ];
    236                 }
    237             }
    238 
    239             // For backward-compatibility, 'date' needs to resolve to 'date ID'.
    240             if ( isset( $args['orderby'] ) && 'date' === $args['orderby'] ) {
    241                 $args['orderby'] = 'date ID';
    242             }
    243 
    244             /** This filter is documented in wp-includes/rest-api/endpoints/class-wp-rest-posts-controller.php */
    245             $args       = apply_filters( 'rest_revision_query', $args, $request );
    246             $query_args = $this->prepare_items_query( $args, $request );
    247 
    248             $revisions_query = new WP_Query();
    249             $revisions       = $revisions_query->query( $query_args );
    250             $offset          = isset( $query_args['offset'] ) ? (int) $query_args['offset'] : 0;
    251             $page            = (int) $query_args['paged'];
    252             $total_revisions = $revisions_query->found_posts;
    253 
    254             if ( $total_revisions < 1 ) {
    255                 // Out-of-bounds, run the query again without LIMIT for total count.
    256                 unset( $query_args['paged'], $query_args['offset'] );
    257 
    258                 $count_query = new WP_Query();
    259                 $count_query->query( $query_args );
    260 
    261                 $total_revisions = $count_query->found_posts;
    262             }
    263 
    264             if ( $revisions_query->query_vars['posts_per_page'] > 0 ) {
    265                 $max_pages = ceil( $total_revisions / (int) $revisions_query->query_vars['posts_per_page'] );
    266             } else {
    267                 $max_pages = $total_revisions > 0 ? 1 : 0;
    268             }
    269 
    270             if ( $total_revisions > 0 ) {
    271                 if ( $offset >= $total_revisions ) {
    272                     return new WP_Error( 'rest_revision_invalid_offset_number', __( 'The offset number requested is larger than or equal to the number of available revisions.' ), array( 'status' => 400 ) );
    273                 } elseif ( ! $offset && $page > $max_pages ) {
    274                     return new WP_Error( 'rest_revision_invalid_page_number', __( 'The page number requested is larger than the number of pages available.' ), array( 'status' => 400 ) );
    275                 }
    276             }
    277         } else {
    278             $revisions       = array();
    279             $total_revisions = 0;
    280             $max_pages       = 0;
    281             $page            = (int) $request['page'];
    282         }
     200        $revisions = wp_get_post_revisions( $request['parent'] );
    283201
    284202        $response = array();
    … …  
    287205            $response[] = $this->prepare_response_for_collection( $data );
    288206        }
    289 
    290         $response = rest_ensure_response( $response );
    291 
    292         $response->header( 'X-WP-Total', (int) $total_revisions );
    293         $response->header( 'X-WP-TotalPages', (int) $max_pages );
    294 
    295         $request_params = $request->get_query_params();
    296         $base           = add_query_arg( $request_params, rest_url( sprintf( '%s/%s/%d/%s', $this->namespace, $this->parent_base, $request['parent'], $this->rest_base ) ) );
    297 
    298         if ( $page > 1 ) {
    299             $prev_page = $page - 1;
    300 
    301             if ( $prev_page > $max_pages ) {
    302                 $prev_page = $max_pages;
    303             }
    304 
    305             $prev_link = add_query_arg( 'page', $prev_page, $base );
    306             $response->link_header( 'prev', $prev_link );
    307         }
    308         if ( $max_pages > $page ) {
    309             $next_page = $page + 1;
    310             $next_link = add_query_arg( 'page', $next_page, $base );
    311 
    312             $response->link_header( 'next', $next_link );
    313         }
    314 
    315         return $response;
     207        return rest_ensure_response( $response );
    316208    }
    317209
    … …  
    424316        $response->set_data( array( 'deleted' => true, 'previous' => $previous->get_data() ) );
    425317        return $response;
    426     }
    427 
    428     /**
    429      * Determines the allowed query_vars for a get_items() response and prepares
    430      * them for WP_Query.
    431      *
    432      * @since 5.0.0
    433      *
    434      * @param array           $prepared_args Optional. Prepared WP_Query arguments. Default empty array.
    435      * @param WP_REST_Request $request       Optional. Full details about the request.
    436      * @return array Items query arguments.
    437      */
    438     protected function prepare_items_query( $prepared_args = array(), $request = null ) {
    439         $query_args = array();
    440 
    441         foreach ( $prepared_args as $key => $value ) {
    442             /** This filter is documented in wp-includes/rest-api/endpoints/class-wp-rest-posts-controller.php */
    443             $query_args[ $key ] = apply_filters( "rest_query_var-{$key}", $value ); // phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
    444         }
    445 
    446         // Map to proper WP_Query orderby param.
    447         if ( isset( $query_args['orderby'] ) && isset( $request['orderby'] ) ) {
    448             $orderby_mappings = array(
    449                 'id'            => 'ID',
    450                 'include'       => 'post__in',
    451                 'slug'          => 'post_name',
    452                 'include_slugs' => 'post_name__in',
    453             );
    454 
    455             if ( isset( $orderby_mappings[ $request['orderby'] ] ) ) {
    456                 $query_args['orderby'] = $orderby_mappings[ $request['orderby'] ];
    457             }
    458         }
    459 
    460         return $query_args;
    461318    }
    462319
    … …  
    681538     */
    682539    public function get_collection_params() {
    683         $query_params = parent::get_collection_params();
    684 
    685         $query_params['context']['default'] = 'view';
    686 
    687         unset( $query_params['per_page']['default'] );
    688 
    689         $query_params['exclude'] = array(
    690             'description' => __( 'Ensure result set excludes specific IDs.' ),
    691             'type'        => 'array',
    692             'items'       => array(
    693                 'type' => 'integer',
    694             ),
    695             'default'     => array(),
     540        return array(
     541            'context' => $this->get_context_param( array( 'default' => 'view' ) ),
    696542        );
    697 
    698         $query_params['include'] = array(
    699             'description' => __( 'Limit result set to specific IDs.' ),
    700             'type'        => 'array',
    701             'items'       => array(
    702                 'type' => 'integer',
    703             ),
    704             'default'     => array(),
    705         );
    706 
    707         $query_params['offset'] = array(
    708             'description' => __( 'Offset the result set by a specific number of items.' ),
    709             'type'        => 'integer',
    710         );
    711 
    712         $query_params['order'] = array(
    713             'description' => __( 'Order sort attribute ascending or descending.' ),
    714             'type'        => 'string',
    715             'default'     => 'desc',
    716             'enum'        => array( 'asc', 'desc' ),
    717         );
    718 
    719         $query_params['orderby'] = array(
    720             'description' => __( 'Sort collection by object attribute.' ),
    721             'type'        => 'string',
    722             'default'     => 'date',
    723             'enum'        => array(
    724                 'date',
    725                 'id',
    726                 'include',
    727                 'relevance',
    728                 'slug',
    729                 'include_slugs',
    730                 'title',
    731             ),
    732         );
    733 
    734         return $query_params;
    735543    }
    736544

  • branches/4.9/tests/phpunit/tests/rest-api/rest-revisions-controller.php

    r43648 r43715  
    77 */
    88
    9 /**
    10  * @group restapi
    11  */
     9 /**
     10  * @group restapi
     11  */
    1212class WP_Test_REST_Revisions_Controller extends WP_Test_REST_Controller_Testcase {
    1313    protected static $post_id;
    … …  
    3131        wp_update_post( array( 'post_content' => 'This content is better.', 'ID' => self::$post_id ) );
    3232        wp_update_post( array( 'post_content' => 'This content is marvelous.', 'ID' => self::$post_id ) );
    33         wp_update_post(
    34             array(
    35                 'post_content' => 'This content is fantastic.',
    36                 'ID'           => self::$post_id,
    37             )
    38         );
    3933        wp_set_current_user( 0 );
    4034    }
    … …  
    5246        parent::setUp();
    5347
    54         $revisions             = wp_get_post_revisions( self::$post_id );
    55         $this->total_revisions = count( $revisions );
    56         $this->revisions       = $revisions;
    57         $this->revision_1      = array_pop( $revisions );
    58         $this->revision_id1    = $this->revision_1->ID;
    59         $this->revision_2      = array_pop( $revisions );
    60         $this->revision_id2    = $this->revision_2->ID;
    61         $this->revision_3      = array_pop( $revisions );
    62         $this->revision_id3    = $this->revision_3->ID;
     48        $revisions = wp_get_post_revisions( self::$post_id );
     49        $this->revision_1 = array_pop( $revisions );
     50        $this->revision_id1 = $this->revision_1->ID;
     51        $this->revision_2 = array_pop( $revisions );
     52        $this->revision_id2 = $this->revision_2->ID;
    6353    }
    6454
    … …  
    9282        $data = $response->get_data();
    9383        $this->assertEquals( 200, $response->get_status() );
    94         $this->assertCount( $this->total_revisions, $data );
     84        $this->assertCount( 2, $data );
    9585
    9686        // Reverse chron
    97         $this->assertEquals( $this->revision_id3, $data[0]['id'] );
    98         $this->check_get_revision_response( $data[0], $this->revision_3 );
    99 
    100         $this->assertEquals( $this->revision_id2, $data[1]['id'] );
    101         $this->check_get_revision_response( $data[1], $this->revision_2 );
    102 
    103         $this->assertEquals( $this->revision_id1, $data[2]['id'] );
    104         $this->check_get_revision_response( $data[2], $this->revision_1 );
     87        $this->assertEquals( $this->revision_id2, $data[0]['id'] );
     88        $this->check_get_revision_response( $data[0], $this->revision_2 );
     89
     90        $this->assertEquals( $this->revision_id1, $data[1]['id'] );
     91        $this->check_get_revision_response( $data[1], $this->revision_1 );
    10592    }
    10693
    … …  
    375362    }
    376363
    377     /**
    378      * Test the pagination header of the first page.
    379      *
    380      * @ticket 40510
    381      */
    382     public function test_get_items_pagination_header_of_the_first_page() {
    383         wp_set_current_user( self::$editor_id );
    384 
    385         $rest_route  = '/wp/v2/posts/' . self::$post_id . '/revisions';
    386         $per_page    = 2;
    387         $total_pages = (int) ceil( $this->total_revisions / $per_page );
    388         $page        = 1;  // First page.
    389 
    390         $request = new WP_REST_Request( 'GET', $rest_route );
    391         $request->set_query_params(
    392             array(
    393                 'per_page' => $per_page,
    394                 'page'     => $page,
    395             )
    396         );
    397         $response = rest_get_server()->dispatch( $request );
    398         $headers  = $response->get_headers();
    399         $this->assertSame( $this->total_revisions, $headers['X-WP-Total'] );
    400         $this->assertSame( $total_pages, $headers['X-WP-TotalPages'] );
    401         $next_link = add_query_arg(
    402             array(
    403                 'per_page' => $per_page,
    404                 'page'     => $page + 1,
    405             ),
    406             rest_url( $rest_route )
    407         );
    408         $this->assertFalse( stripos( $headers['Link'], 'rel="prev"' ) );
    409         $this->assertContains( '<' . $next_link . '>; rel="next"', $headers['Link'] );
    410     }
    411 
    412     /**
    413      * Test the pagination header of the last page.
    414      *
    415      * @ticket 40510
    416      */
    417     public function test_get_items_pagination_header_of_the_last_page() {
    418         wp_set_current_user( self::$editor_id );
    419 
    420         $rest_route  = '/wp/v2/posts/' . self::$post_id . '/revisions';
    421         $per_page    = 2;
    422         $total_pages = (int) ceil( $this->total_revisions / $per_page );
    423         $page        = 2;  // Last page.
    424 
    425         $request = new WP_REST_Request( 'GET', $rest_route );
    426         $request->set_query_params(
    427             array(
    428                 'per_page' => $per_page,
    429                 'page'     => $page,
    430             )
    431         );
    432         $response = rest_get_server()->dispatch( $request );
    433         $headers  = $response->get_headers();
    434         $this->assertSame( $this->total_revisions, $headers['X-WP-Total'] );
    435         $this->assertSame( $total_pages, $headers['X-WP-TotalPages'] );
    436         $prev_link = add_query_arg(
    437             array(
    438                 'per_page' => $per_page,
    439                 'page'     => $page - 1,
    440             ),
    441             rest_url( $rest_route )
    442         );
    443         $this->assertContains( '<' . $prev_link . '>; rel="prev"', $headers['Link'] );
    444     }
    445 
    446     /**
    447      * Test that invalid 'per_page' query should error.
    448      *
    449      * @ticket 40510
    450      */
    451     public function test_get_items_invalid_per_page_should_error() {
    452         wp_set_current_user( self::$editor_id );
    453 
    454         $per_page        = -1; // Invalid number.
    455         $expected_error  = 'rest_invalid_param';
    456         $expected_status = 400;
    457 
    458         $request = new WP_REST_Request( 'GET', '/wp/v2/posts/' . self::$post_id . '/revisions' );
    459         $request->set_param( 'per_page', $per_page );
    460         $response = rest_get_server()->dispatch( $request );
    461         $this->assertErrorResponse( $expected_error, $response, $expected_status );
    462     }
    463 
    464     /**
    465      * Test that out of bounds 'page' query should error.
    466      *
    467      * @ticket 40510
    468      */
    469     public function test_get_items_out_of_bounds_page_should_error() {
    470         wp_set_current_user( self::$editor_id );
    471 
    472         $per_page        = 2;
    473         $total_pages     = (int) ceil( $this->total_revisions / $per_page );
    474         $page            = $total_pages + 1; // Out of bound page.
    475         $expected_error  = 'rest_revision_invalid_page_number';
    476         $expected_status = 400;
    477 
    478         $request = new WP_REST_Request( 'GET', '/wp/v2/posts/' . self::$post_id . '/revisions' );
    479         $request->set_query_params(
    480             array(
    481                 'per_page' => $per_page,
    482                 'page'     => $page,
    483             )
    484         );
    485         $response = rest_get_server()->dispatch( $request );
    486         $this->assertErrorResponse( $expected_error, $response, $expected_status );
    487     }
    488 
    489     /**
    490      * Test that impossibly high 'page' query should error.
    491      *
    492      * @ticket 40510
    493      */
    494     public function test_get_items_invalid_max_pages_should_error() {
    495         wp_set_current_user( self::$editor_id );
    496 
    497         $per_page        = 2;
    498         $page            = REST_TESTS_IMPOSSIBLY_HIGH_NUMBER; // Invalid number.
    499         $expected_error  = 'rest_revision_invalid_page_number';
    500         $expected_status = 400;
    501 
    502         $request = new WP_REST_Request( 'GET', '/wp/v2/posts/' . self::$post_id . '/revisions' );
    503         $request->set_query_params(
    504             array(
    505                 'per_page' => $per_page,
    506                 'page'     => $page,
    507             )
    508         );
    509         $response = rest_get_server()->dispatch( $request );
    510         $this->assertErrorResponse( $expected_error, $response, $expected_status );
    511     }
    512 
    513     /**
    514      * Test the search query.
    515      *
    516      * @ticket 40510
    517      */
    518     public function test_get_items_search_query() {
    519         wp_set_current_user( self::$editor_id );
    520 
    521         $search_string    = 'better';
    522         $expected_count   = 1;
    523         $expected_content = 'This content is better.';
    524 
    525         $request = new WP_REST_Request( 'GET', '/wp/v2/posts/' . self::$post_id . '/revisions' );
    526         $request->set_param( 'search', $search_string );
    527         $response = rest_get_server()->dispatch( $request );
    528         $data     = $response->get_data();
    529         $this->assertCount( $expected_count, $data );
    530         $this->assertContains( $expected_content, $data[0]['content']['rendered'] );
    531     }
    532 
    533     /**
    534      * Test that the default query should fetch all revisions.
    535      *
    536      * @ticket 40510
    537      */
    538     public function test_get_items_default_query_should_fetch_all_revisons() {
    539         wp_set_current_user( self::$editor_id );
    540 
    541         $expected_count = $this->total_revisions;
    542 
    543         $request  = new WP_REST_Request( 'GET', '/wp/v2/posts/' . self::$post_id . '/revisions' );
    544         $response = rest_get_server()->dispatch( $request );
    545         $this->assertCount( $expected_count, $response->get_data() );
    546     }
    547 
    548     /**
    549      * Test that 'offset' query shouldn't work without 'per_page' (fallback -1).
    550      *
    551      * @ticket 40510
    552      */
    553     public function test_get_items_offset_should_not_work_without_per_page() {
    554         wp_set_current_user( self::$editor_id );
    555 
    556         $offset         = 1;
    557         $expected_count = $this->total_revisions;
    558 
    559         $request = new WP_REST_Request( 'GET', '/wp/v2/posts/' . self::$post_id . '/revisions' );
    560         $request->set_param( 'offset', $offset );
    561         $response = rest_get_server()->dispatch( $request );
    562         $this->assertCount( $expected_count, $response->get_data() );
    563     }
    564 
    565     /**
    566      * Test that 'offset' query should work with 'per_page'.
    567      *
    568      * @ticket 40510
    569      */
    570     public function test_get_items_offset_should_work_with_per_page() {
    571         wp_set_current_user( self::$editor_id );
    572 
    573         $per_page       = 2;
    574         $offset         = 1;
    575         $expected_count = 2;
    576 
    577         $request = new WP_REST_Request( 'GET', '/wp/v2/posts/' . self::$post_id . '/revisions' );
    578         $request->set_query_params(
    579             array(
    580                 'offset'   => $offset,
    581                 'per_page' => $per_page,
    582             )
    583         );
    584         $response = rest_get_server()->dispatch( $request );
    585         $this->assertCount( $expected_count, $response->get_data() );
    586     }
    587 
    588     /**
    589      * Test that 'offset' query should take priority over 'page'.
    590      *
    591      * @ticket 40510
    592      */
    593     public function test_get_items_offset_should_take_priority_over_page() {
    594         wp_set_current_user( self::$editor_id );
    595 
    596         $per_page       = 2;
    597         $offset         = 1;
    598         $page           = 1;
    599         $expected_count = 2;
    600 
    601         $request = new WP_REST_Request( 'GET', '/wp/v2/posts/' . self::$post_id . '/revisions' );
    602         $request->set_query_params(
    603             array(
    604                 'offset'   => $offset,
    605                 'per_page' => $per_page,
    606                 'page'     => $page,
    607             )
    608         );
    609         $response = rest_get_server()->dispatch( $request );
    610         $this->assertCount( $expected_count, $response->get_data() );
    611     }
    612 
    613     /**
    614      * Test that 'offset' query, as the total revisions count, should return empty data.
    615      *
    616      * @ticket 40510
    617      */
    618     public function test_get_items_total_revisions_offset_should_return_empty_data() {
    619         wp_set_current_user( self::$editor_id );
    620 
    621         $per_page        = 2;
    622         $offset          = $this->total_revisions;
    623         $expected_error  = 'rest_revision_invalid_offset_number';
    624         $expected_status = 400;
    625 
    626         $request = new WP_REST_Request( 'GET', '/wp/v2/posts/' . self::$post_id . '/revisions' );
    627         $request->set_query_params(
    628             array(
    629                 'offset'   => $offset,
    630                 'per_page' => $per_page,
    631             )
    632         );
    633         $response = rest_get_server()->dispatch( $request );
    634         $this->assertErrorResponse( $expected_error, $response, $expected_status );
    635     }
    636 
    637     /**
    638      * Test that out of bound 'offset' query should error.
    639      *
    640      * @ticket 40510
    641      */
    642     public function test_get_items_out_of_bound_offset_should_error() {
    643         wp_set_current_user( self::$editor_id );
    644 
    645         $per_page        = 2;
    646         $offset          = $this->total_revisions + 1;
    647         $expected_error  = 'rest_revision_invalid_offset_number';
    648         $expected_status = 400;
    649 
    650         $request = new WP_REST_Request( 'GET', '/wp/v2/posts/' . self::$post_id . '/revisions' );
    651         $request->set_query_params(
    652             array(
    653                 'offset'   => $offset,
    654                 'per_page' => $per_page,
    655             )
    656         );
    657         $response = rest_get_server()->dispatch( $request );
    658         $this->assertErrorResponse( $expected_error, $response, $expected_status );
    659     }
    660 
    661     /**
    662      * Test that impossible high number for 'offset' query should error.
    663      *
    664      * @ticket 40510
    665      */
    666     public function test_get_items_impossible_high_number_offset_should_error() {
    667         wp_set_current_user( self::$editor_id );
    668 
    669         $per_page        = 2;
    670         $offset          = REST_TESTS_IMPOSSIBLY_HIGH_NUMBER;
    671         $expected_error  = 'rest_revision_invalid_offset_number';
    672         $expected_status = 400;
    673 
    674         $request = new WP_REST_Request( 'GET', '/wp/v2/posts/' . self::$post_id . '/revisions' );
    675         $request->set_query_params(
    676             array(
    677                 'offset'   => $offset,
    678                 'per_page' => $per_page,
    679             )
    680         );
    681         $response = rest_get_server()->dispatch( $request );
    682         $this->assertErrorResponse( $expected_error, $response, $expected_status );
    683     }
    684 
    685     /**
    686      * Test that invalid 'offset' query should error.
    687      *
    688      * @ticket 40510
    689      */
    690     public function test_get_items_invalid_offset_should_error() {
    691         wp_set_current_user( self::$editor_id );
    692 
    693         $per_page        = 2;
    694         $offset          = 'moreplease';
    695         $expected_error  = 'rest_invalid_param';
    696         $expected_status = 400;
    697 
    698         $request = new WP_REST_Request( 'GET', '/wp/v2/posts/' . self::$post_id . '/revisions' );
    699         $request->set_query_params(
    700             array(
    701                 'offset'   => $offset,
    702                 'per_page' => $per_page,
    703             )
    704         );
    705         $response = rest_get_server()->dispatch( $request );
    706         $this->assertErrorResponse( $expected_error, $response, $expected_status );
    707     }
    708 
    709     /**
    710      * Test that out of bounds 'page' query should not error when offset is provided,
    711      * because it takes precedence.
    712      *
    713      * @ticket 40510
    714      */
    715     public function test_get_items_out_of_bounds_page_should_not_error_if_offset() {
    716         wp_set_current_user( self::$editor_id );
    717 
    718         $per_page       = 2;
    719         $total_pages    = (int) ceil( $this->total_revisions / $per_page );
    720         $page           = $total_pages + 1; // Out of bound page.
    721         $expected_count = 2;
    722 
    723         $request = new WP_REST_Request( 'GET', '/wp/v2/posts/' . self::$post_id . '/revisions' );
    724         $request->set_query_params(
    725             array(
    726                 'offset'   => 1,
    727                 'per_page' => $per_page,
    728                 'page'     => $page,
    729             )
    730         );
    731         $response = rest_get_server()->dispatch( $request );
    732         $this->assertCount( $expected_count, $response->get_data() );
    733     }
    734364}

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

    r43648 r43715  
    722722                            "description": "Scope under which the request is made; determines fields present in response.",
    723723                            "type": "string"
    724                         },
    725                         "page": {
    726                             "required": false,
    727                             "default": 1,
    728                             "description": "Current page of the collection.",
    729                             "type": "integer"
    730                         },
    731                         "per_page": {
    732                             "required": false,
    733                             "description": "Maximum number of items to be returned in result set.",
    734                             "type": "integer"
    735                         },
    736                         "search": {
    737                             "required": false,
    738                             "description": "Limit results to those matching a string.",
    739                             "type": "string"
    740                         },
    741                         "exclude": {
    742                             "required": false,
    743                             "default": [],
    744                             "description": "Ensure result set excludes specific IDs.",
    745                             "type": "array",
    746                             "items": {
    747                                 "type": "integer"
    748                             }
    749                         },
    750                         "include": {
    751                             "required": false,
    752                             "default": [],
    753                             "description": "Limit result set to specific IDs.",
    754                             "type": "array",
    755                             "items": {
    756                                 "type": "integer"
    757                             }
    758                         },
    759                         "offset": {
    760                             "required": false,
    761                             "description": "Offset the result set by a specific number of items.",
    762                             "type": "integer"
    763                         },
    764                         "order": {
    765                             "required": false,
    766                             "default": "desc",
    767                             "enum": [
    768                                 "asc",
    769                                 "desc"
    770                             ],
    771                             "description": "Order sort attribute ascending or descending.",
    772                             "type": "string"
    773                         },
    774                         "orderby": {
    775                             "required": false,
    776                             "default": "date",
    777                             "enum": [
    778                                 "date",
    779                                 "id",
    780                                 "include",
    781                                 "relevance",
    782                                 "slug",
    783                                 "include_slugs",
    784                                 "title"
    785                             ],
    786                             "description": "Sort collection by object attribute.",
    787                             "type": "string"
    788724                        }
    789725                    }
    … …  
    13271263                            ],
    13281264                            "description": "Scope under which the request is made; determines fields present in response.",
    1329                             "type": "string"
    1330                         },
    1331                         "page": {
    1332                             "required": false,
    1333                             "default": 1,
    1334                             "description": "Current page of the collection.",
    1335                             "type": "integer"
    1336                         },
    1337                         "per_page": {
    1338                             "required": false,
    1339                             "description": "Maximum number of items to be returned in result set.",
    1340                             "type": "integer"
    1341                         },
    1342                         "search": {
    1343                             "required": false,
    1344                             "description": "Limit results to those matching a string.",
    1345                             "type": "string"
    1346                         },
    1347                         "exclude": {
    1348                             "required": false,
    1349                             "default": [],
    1350                             "description": "Ensure result set excludes specific IDs.",
    1351                             "type": "array",
    1352                             "items": {
    1353                                 "type": "integer"
    1354                             }
    1355                         },
    1356                         "include": {
    1357                             "required": false,
    1358                             "default": [],
    1359                             "description": "Limit result set to specific IDs.",
    1360                             "type": "array",
    1361                             "items": {
    1362                                 "type": "integer"
    1363                             }
    1364                         },
    1365                         "offset": {
    1366                             "required": false,
    1367                             "description": "Offset the result set by a specific number of items.",
    1368                             "type": "integer"
    1369                         },
    1370                         "order": {
    1371                             "required": false,
    1372                             "default": "desc",
    1373                             "enum": [
    1374                                 "asc",
    1375                                 "desc"
    1376                             ],
    1377                             "description": "Order sort attribute ascending or descending.",
    1378                             "type": "string"
    1379                         },
    1380                         "orderby": {
    1381                             "required": false,
    1382                             "default": "date",
    1383                             "enum": [
    1384                                 "date",
    1385                                 "id",
    1386                                 "include",
    1387                                 "relevance",
    1388                                 "slug",
    1389                                 "include_slugs",
    1390                                 "title"
    1391                             ],
    1392                             "description": "Sort collection by object attribute.",
    13931265                            "type": "string"
    13941266                        }
Note: See TracChangeset for help on using the changeset viewer.