REST API: Add basic fields to /themes endpoint

[Bernhard Reiter]

The /wp/v2/themes ​endpoint currently only exposes each theme's theme_supports field, but nothing else -- not even theme name or title.

I need a few more theme properties for ​full-site editing, for which I'll file a PR against the wordpress-develop git repo. I'm filing this ticket to have something to link to :)

The /wp/v2/themes [endpoint](​https://developer.wordpress.org/rest-api/reference/themes/) currently only exposes each theme's theme_supports field, but nothing else -- not even theme name or title.

See https://core.trac.wordpress.org/ticket/49906.

Needed for ​https://github.com/WordPress/gutenberg/pull/21578 -- you can use that PR to test this one :slightly_smiling_face:

[prbot]

​ockham commented on ​PR #222:

/cc @kadamwhite

[prbot]

​TimothyBJacobs commented on ​PR #222:

Looks great! Are you interested in adding tests for this?

[prbot]

​ockham commented on ​PR #222:

Looks great! Are you interested in adding tests for this?

Thanks! Sure, I'll add some tomorrow :smile:

[prbot]

​ockham commented on ​PR #222:

Update, I've fixed the failing unit tests and lint.

I haven't yet added new tests, since I'm still struggling with setting up the tests environment on my Linux box. I asked on Core Slack: ​https://wordpress.slack.com/archives/C02RQBWTW/p1586985925240400

[prbot]

​ockham commented on ​PR #222:

I managed to get unit tests to run locally, and added a few more fields and corresponding tests (see updated PR description). I'd say this is ready for review :slightly_smiling_face:

[prbot]

​ockham commented on ​PR #222:

cc/ @mcsf for review. Because it's a themes endpoint! :tada:

[prbot]

​mcsf commented on ​PR #222:

Because it's a themes endpoint! 🎉

Good times!

Looks great, as far as I'm concerned. I'd defer to Timothy or K Adam for the final green light, though.

[prbot]

​ockham commented on ​PR #222:

Looks great, as far as I'm concerned. I'd defer to Timothy or K Adam for the final green light, though.

@kadamwhite @TimothyBJacobs Pretty pleaaase? :blush:

[prbot]

​ockham commented on ​PR #222:

Addressed all feedback. Should be ready for another look!

[prbot]

​TimothyBJacobs commented on ​PR #222:

Thanks, looking good!

One small thing, we should figure out some better language for as it exists in the database. These values aren't stored in the database, they are pulled from the theme's files.

The values are also sanitized and translated, so I'm not sure we could say as it exists in the theme's style.css file either. Not sure ¯\_(ツ)_/¯.

Also tagging @spacedmonkey for a general review as he's expressed interest in expanding the themes endpoint.

[prbot]

​ockham commented on ​PR #222:

Thanks, looking good!

One small thing, we should figure out some better language for as it exists in the database. These values aren't stored in the database, they are pulled from the theme's files.

Ah yeah, good point. Lazy copy-pasta from another endpoint :grimacing:

The values are also sanitized and translated, so I'm not sure we could say as it exists in the theme's style.css file either. Not sure ¯_(ツ)_/¯.

I'll think of something :+1:

Also tagging @spacedmonkey for a general review as he's expressed interest in expanding the themes endpoint.

:+1:

[prbot]

​TimothyBJacobs commented on ​PR #222:

That wording sounds good to me!

[prbot]

​ockham commented on ​PR #222:

:wave: @TimothyBJacobs @spacedmonkey

Just checking in if there's anything else I should do or if mayyybe I could get y'all's approval :blush:

[prbot]

​TimothyBJacobs commented on ​PR #222:

This still looks good to me @ockham! Will commit later today/tomorrow unless I hear different from @spacedmonkey.

[prbot]

​spacedmonkey commented on ​PR #222:

At the moment, these fields can only display data on the currently active theme.

With a very small tweak, to get_items, we could display this data for all themes.
`

public function get_items( $request ) {

Retrieve the list of registered collection query parameters.
$registered = $this->get_collection_params();
$themes = array();

if ( isset( $registeredstatus?, $requeststatus? ) && in_array( 'active', $requeststatus?, true ) ) {

$active_theme = wp_get_theme();
$active_theme = $this->prepare_item_for_response( $active_theme, $request );
$themes[] = $this->prepare_response_for_collection( $active_theme );

} else {

$active_themes = wp_get_themes();
foreach( $active_themes as $theme ) {

$active_theme = $this->prepare_item_for_response( $theme, $request );
$themes[] = $this->prepare_response_for_collection( $active_theme );

}

}

$response = rest_ensure_response( $themes );

$response->header( 'X-WP-Total', count( $themes ) );
$response->header( 'X-WP-TotalPages', count( $themes ) );

return $response;

}

`

The only issue with this, is that theme supports can not be displayed on themes that are not active.

[prbot]

​TimothyBJacobs commented on ​PR #222:

At the moment, these fields can only display data on the currently active theme.

With a very small tweak, to get_items, we could display this data for all themes.

I think it makes sense to expand that in 5.5, but let's make it a separate ticket.

[prbot]

​spacedmonkey commented on ​PR #222:

I have created another PR for showing all themes in the API - ​https://github.com/WordPress/wordpress-develop/pull/264

[prbot]

​TimothyBJacobs commented on ​PR #222:

Hey @ockham just checking in on this!

[prbot]

​ockham commented on ​PR #222:

Hey @ockham just checking in on this!

Thanks! I'm back today from 2 weeks of maintenance rotation, picking this up again :+1:

[prbot]

​TimothyBJacobs commented on ​PR #222:

@ockham To clarify, the type should still be string but have a format => uri.

[prbot]

​ockham commented on ​PR #222:

@ockham To clarify, the type should still be string but have a format => uri.

Fixed by 7f5db79 :slightly_smiling_face:

[TimothyBlynJacobs]

In 47921:

REST API: Add additional fields to the themes controller.

When the themes controller was introduced it only returned a theme's supported features. This adds the majority of a theme's header information to the response.

Props ockham, spacedmonkey.
Fixes #49906.

[prbot]

​TimothyBJacobs commented on ​PR #222:

Merged in [r47921](https://core.trac.wordpress.org/changeset/47921).

[afragen]

@TimothyBlynJacobs in a quick glance at r47921, it seems that Requires_WP should only be Requires. I think that’s how it’s listed in the theme headers.

[TimothyBlynJacobs]

Do you mean how we should be fetching that data or returning that data?

If fetching, WP_Theme indicates it as ​RequiresWP and we have a ​test for it here.

[afragen]

Just ignore me. I’m at work and was going by memory. 🤦🏻‍♂️

[TimothyBlynJacobs]

@afragen no worries, thanks for taking a look!

[justinahinon]

Devnote for this: ​https://make.wordpress.org/core/2020/07/15/register-theme-feature-api/