Changeset 40081 for branches/4.7

Timestamp:

02/19/2017 04:35:28 AM (8 years ago)

Author:

rachelbaker

Message:

REST API: Include the status property in view context responses from the Posts endpoints.

Previously the status for a Post (or other post_types) was only exposed under the edit context, which doesn't really make much sense considering we support querying by post status without authentication. Originally introduced in v2.0 beta 1: ​https://github.com/WP-API/WP-API/commit/69f617d749e4f165b9e31c1c042eec45ade3e41f without any explanation in the commit message.

Props dhanendran, jnylen0, rachelbaker.
Merges [40080] to the 4.7 branch.
Fixes #39466.

Location:

branches/4.7

Files:

• . (modified) (1 prop)
• src/wp-includes/rest-api/endpoints/class-wp-rest-posts-controller.php (modified) (1 diff)
• tests/phpunit/includes/testcase-rest-post-type-controller.php (modified) (1 diff)

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

@@ -1759 +1759 @@
                     'type'        => 'string',
                     'enum'        => array_keys( get_post_stati( array( 'internal' => false ) ) ),
-                    'context'     => array( 'edit' ),
+                    'context'     => array( 'view', 'edit' ),
                 ),
                 'type'            => array(

branches/4.7/tests/phpunit/includes/testcase-rest-post-type-controller.php

@@ -135 +135 @@
         }
 
+        $this->assertEquals( $post->post_status, $data['status'] );
         $this->assertEquals( $post->guid, $data['guid']['rendered'] );
 
         if ( 'edit' === $context ) {
             $this->assertEquals( $post->guid, $data['guid']['raw'] );
-            $this->assertEquals( $post->post_status, $data['status'] );
 
             if ( '0000-00-00 00:00:00' === $post->post_date_gmt ) {