Changeset 40120 for trunk/src/wp-includes/rest-api/endpoints/class-wp-rest-posts-controller.php

Timestamp:

02/24/2017 11:29:04 PM (8 years ago)

Author:

jnylen0

Message:

REST API: Allow setting post formats even if they are not supported by the theme.

A post_format not used by the current theme, but supported by core is not a wrong/broken piece of information. It's just not used at this point in time. Therefore we should allow setting and retrieving any of the standard post formats supported in core, even if the current theme doesn't use them.

After this commit, a post's format value can survive a round trip through the API, which is a good general design principle for an API.

Props JPry, iseulde, davidakennedy, Drivingralle.
Fixes #39232.

File:

• trunk/src/wp-includes/rest-api/endpoints/class-wp-rest-posts-controller.php (modified) (1 diff)

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

@@ -1990 +1990 @@
 
                 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 );
+                    // Get the native post formats and remove the array keys.
+                    $formats = array_values( get_post_format_slugs() );
 
                     $schema['properties']['format'] = array(
                         'description' => __( 'The format for the object.' ),
                         'type'        => 'string',
-                        'enum'        => $supported_formats,
+                        'enum'        => $formats,
                         'context'     => array( 'view', 'edit' ),
                     );