Ticket #39293: 39293.diff

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

@@ -1923 +1923 @@
 
                                 case 'post-formats':
                                         $supports_formats = get_theme_support( 'post-formats' );
+
+                                        // Force to an array. Supports formats can return true even if empty in some cases.
+                                        $supports_formats = is_array( $supports_formats ) ? array_values( $supports_formats[0] ) : array();
+
+                                        $supported_formats = array_merge( array( 'standard' ), $supports_formats );
+
                                         $schema['properties']['format'] = array(
                                                 'description' => __( 'The format for the object.' ),
                                                 'type'        => 'string',
-                                                'enum'        => array_merge( array( 'standard' ), $supports_formats ? array_values( $supports_formats[0] ) : array() ),
+                                                'enum'        => $supported_formats,
                                                 'context'     => array( 'view', 'edit' ),
                                         );
                                         break;

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

@@ -858 +858 @@
                 $this->assertEquals( $post2, $data[0]['id'] );
         }
 
+        /**
+         * Ticket #39293
+         */
+        public function test_get_items_no_supported_post_formats() {
+                // See Ticket #39293. This sets an empty post format which resolves to `true`.
+                add_theme_support( 'post-formats' );
+
+                $request = new WP_REST_Request( 'OPTIONS', '/wp/v2/posts' );
+                $response = $this->server->dispatch( $request );
+                $data = $response->get_data();
+
+                $formats = array( 'standard' );
+
+                $this->assertEquals( $formats, $data['schema']['properties']['format']['enum'] );
+
+                // Set the expected state back for the rest of the tests.
+                global $_wp_theme_features;
+                unset( $_wp_theme_features['post-formats'] );
+                add_theme_support( 'post-formats', array( 'post', 'gallery' ) );
+        }
+
         public function test_get_item() {
                 $request = new WP_REST_Request( 'GET', sprintf( '/wp/v2/posts/%d', self::$post_id ) );
                 $response = $this->server->dispatch( $request );