REST API: Correctly parse body parameters for DELETE requests.

DELETE was inadvertently omitted from the list of non-POST HTTP methods that should be able to accept body parameters.  Parameters passed to DELETE requests as JSON are already parsed correctly; this commit fixes `application/x-www-form-urlencoded` parameters as well.

Props mnelson4.
Fixes #39933.


git-svn-id: https://develop.svn.wordpress.org/trunk@40105 602fd350-edb4-49c9-b593-d223f7449a82
This commit is contained in:
James Nylen 2017-02-23 20:09:11 +00:00
parent 6a23c57ad5
commit f366757b55
2 changed files with 16 additions and 7 deletions

View File

@ -364,7 +364,7 @@ class WP_REST_Request implements ArrayAccess {
$this->parse_body_params();
}
$accepts_body_data = array( 'POST', 'PUT', 'PATCH' );
$accepts_body_data = array( 'POST', 'PUT', 'PATCH', 'DELETE' );
if ( in_array( $this->method, $accepts_body_data ) ) {
$order[] = 'POST';
}

View File

@ -203,11 +203,22 @@ class Tests_REST_Request extends WP_UnitTestCase {
$this->assertEmpty( $this->request->get_param( 'has_json_params' ) );
}
public function non_post_http_methods_with_request_body_provider() {
return array(
array( 'PUT' ),
array( 'PATCH' ),
array( 'DELETE' ),
);
}
/**
* PUT requests don't get $_POST automatically parsed, so ensure that
* WP_REST_Request does it for us.
* Tests that methods supporting request bodies have access to the
* request's body. For POST this is straightforward via `$_POST`; for
* other methods `WP_REST_Request` needs to parse the body for us.
*
* @dataProvider non_post_http_methods_with_request_body_provider
*/
public function test_parameters_for_put() {
public function test_non_post_body_parameters( $request_method ) {
$data = array(
'foo' => 'bar',
'alot' => array(
@ -219,11 +230,9 @@ class Tests_REST_Request extends WP_UnitTestCase {
'stuff',
),
);
$this->request->set_method( 'PUT' );
$this->request->set_method( $request_method );
$this->request->set_body_params( array() );
$this->request->set_body( http_build_query( $data ) );
foreach ( $data as $key => $expected_value ) {
$this->assertEquals( $expected_value, $this->request->get_param( $key ) );
}