REST API: Provide better method for generating CURIEs

In [36533] CURIEs were added to the API responses for the link relation URIs, this makes
it a lot easier for clients to look up links by relation. That patch was functional, but
broke on edge cases such as embedded responses and collection items with links in the items.

This patch instead takes a less obtrusive approach by creating a new `get_compact_response_links`
to compliment `get_response_links` making both old and new functionality available.

Also the regex for curie relations has been relaxed to `.+` as rel names can have any uri-valid charector in it.

Fixes #34729.


Built from https://develop.svn.wordpress.org/trunk@37041


git-svn-id: http://core.svn.wordpress.org/trunk@37008 1a063a9b-81f0-0310-95a4-ce76da25c4cd
This commit is contained in:
Joe Hoyle 2016-03-22 00:16:27 +00:00
parent f3f84d2f21
commit a07988c1c5
2 changed files with 43 additions and 17 deletions

View File

@ -421,7 +421,7 @@ class WP_REST_Server {
*/ */
public function response_to_data( $response, $embed ) { public function response_to_data( $response, $embed ) {
$data = $response->get_data(); $data = $response->get_data();
$links = $this->get_response_links( $response ); $links = $this->get_compact_response_links( $response );
if ( ! empty( $links ) ) { if ( ! empty( $links ) ) {
// Convert links to part of the data. // Convert links to part of the data.
@ -454,13 +454,45 @@ class WP_REST_Server {
*/ */
public static function get_response_links( $response ) { public static function get_response_links( $response ) {
$links = $response->get_links(); $links = $response->get_links();
if ( empty( $links ) ) { if ( empty( $links ) ) {
return array(); return array();
} }
// Convert links to part of the data. // Convert links to part of the data.
$data = array(); $data = array();
foreach ( $links as $rel => $items ) {
$data[ $rel ] = array();
foreach ( $items as $item ) {
$attributes = $item['attributes'];
$attributes['href'] = $item['href'];
$data[ $rel ][] = $attributes;
}
}
return $data;
}
/**
* Retrieves the CURIEs (compact URIs) used for relations.
*
* Extracts the links from a response into a structured hash, suitable for
* direct output.
*
* @since 4.5.0
* @access public
* @static
*
* @param WP_REST_Response $response Response to extract links from.
* @return array Map of link relation to list of link hashes.
*/
public static function get_compact_response_links( $response ) {
$links = self::get_response_links( $response );
if ( empty( $links ) ) {
return array();
}
$curies = $response->get_curies(); $curies = $response->get_curies();
$used_curies = array(); $used_curies = array();
@ -472,32 +504,26 @@ class WP_REST_Server {
if ( strpos( $rel, $href_prefix ) !== 0 ) { if ( strpos( $rel, $href_prefix ) !== 0 ) {
continue; continue;
} }
$used_curies[ $curie['name'] ] = $curie;
// Relation now changes from '$uri' to '$curie:$relation' // Relation now changes from '$uri' to '$curie:$relation'
$rel_regex = str_replace( '\{rel\}', '([\w]+)', preg_quote( $curie['href'], '!' ) ); $rel_regex = str_replace( '\{rel\}', '(.+)', preg_quote( $curie['href'], '!' ) );
preg_match( '!' . $rel_regex . '!', $rel, $matches ); preg_match( '!' . $rel_regex . '!', $rel, $matches );
if ( $matches ) { if ( $matches ) {
$rel = $curie['name'] . ':' . $matches[1]; $new_rel = $curie['name'] . ':' . $matches[1];
$used_curies[ $curie['name'] ] = $curie;
$links[ $new_rel ] = $items;
unset( $links[ $rel ] );
break;
} }
break;
}
$data[ $rel ] = array();
foreach ( $items as $item ) {
$attributes = $item['attributes'];
$attributes['href'] = $item['href'];
$data[ $rel ][] = $attributes;
} }
} }
// Push the curies onto the start of the links array. // Push the curies onto the start of the links array.
if ( $used_curies ) { if ( $used_curies ) {
$data = array_merge( array( 'curies' => array_values( $used_curies ) ), $data ); $links['curies'] = array_values( $used_curies );
} }
return $data; return $links;
} }
/** /**

View File

@ -4,7 +4,7 @@
* *
* @global string $wp_version * @global string $wp_version
*/ */
$wp_version = '4.5-beta4-37040'; $wp_version = '4.5-beta4-37041';
/** /**
* Holds the WordPress DB revision, increments when changes are made to the WordPress DB schema. * Holds the WordPress DB revision, increments when changes are made to the WordPress DB schema.