Add the ability to get data on a single location.

This commit is contained in:
Justin Shreve 2016-03-07 12:44:07 -08:00
parent eb9e5edd22
commit 9369d55ff4
3 changed files with 147 additions and 19 deletions

View File

@ -33,9 +33,17 @@ class WC_Rest_Settings_Controller extends WP_Rest_Controller {
'permission_callback' => array( $this, 'permissions_check' ),
'args' => $this->get_locations_params(),
),
'schema' => array( $this, 'get_locations_schema' ),
'schema' => array( $this, 'get_location_schema' ),
) );
register_rest_route( WC_API::REST_API_NAMESPACE, '/' . $this->rest_base . '/locations/(?P<location>[\w-]+)', array(
array(
'methods' => WP_REST_Server::READABLE,
'callback' => array( $this, 'get_location' ),
'permission_callback' => array( $this, 'permissions_check' ),
),
'schema' => array( $this, 'get_location_schema' ),
) );
}
/**
@ -93,6 +101,36 @@ class WC_Rest_Settings_Controller extends WP_Rest_Controller {
return $response;
}
/**
* Return a single setting location.
* @since 2.7.0
* @param WP_REST_Request $request Full details about the request.
* @return WP_Error|WP_REST_Response
*/
public function get_location( $request ) {
$locations = apply_filters( 'woocommerce_settings_locations', array() );
if ( empty( $locations ) ) {
return new WP_Error( 'rest_setting_location_invalid_id', __( 'Invalid location id.' ), array( 'status' => 404 ) );
}
$index_key = $this->get_array_key_from_location_id( $locations, $request['location'] );
if ( is_null( $index_key ) || empty( $locations[ $index_key ] ) ) {
return new WP_Error( 'rest_setting_location_invalid_id', __( 'Invalid location id.' ), array( 'status' => 404 ) );
}
$location = $locations[ $index_key ];
if ( is_null( $location['id'] ) || is_null( $location['label'] ) || is_null( $location['type'] ) ) {
return new WP_Error( 'rest_setting_location_invalid_id', __( 'Invalid location id.' ), array( 'status' => 404 ) );
}
$filtered_location = array_intersect_key(
$location,
array_flip( array_filter( array_keys( $location ), array( $this, 'filter_location_keys' ) ) )
);
return $filtered_location;
}
/**
* Callback for Allowed keys for each location response.
* @since 2.7.0
@ -124,17 +162,49 @@ class WC_Rest_Settings_Controller extends WP_Rest_Controller {
* @since 2.7.0
* @return array
*/
public function get_locations_schema() {
/*$schema = array(
public function get_location_schema() {
$schema = array(
'$schema' => 'http://json-schema.org/draft-04/schema#',
'title' => 'locations',
'title' => 'settings-locations',
'type' => 'object',
'properties' =>
);*/
'properties' => array(
'id' => array(
'description' => __( 'A unique identifier that can be used to link settings together.' ),
'type' => 'string',
'arg_options' => array(
'sanitize_callback' => 'sanitize_title',
),
),
'type' => array(
'description' => __( 'Context for where the settings in this location are going to be displayed.' ),
'type' => 'string',
'arg_options' => array(
'sanitize_callback' => 'sanitize_title',
),
),
'label' => array(
'description' => __( 'A human readable label. This is a translated string that can be used in interfaces.' ),
'type' => 'string',
'arg_options' => array(
'sanitize_callback' => 'sanitize_text_field',
),
),
'description' => array(
'description' => __( 'A human readable description. This is a translated string that can be used in interfaces.' ),
'type' => 'string',
'arg_options' => array(
'sanitize_callback' => 'sanitize_text_field',
),
),
),
);
return $this->add_additional_fields_schema( $schema );
}
/**
* Returns a list of allowed setting location types.
* @todo move this?
* @since 2.7.0
* @return array
*/
@ -142,4 +212,20 @@ class WC_Rest_Settings_Controller extends WP_Rest_Controller {
return apply_filters( 'woocommerce_settings_location_types', array( 'page', 'metabox', 'shipping-zone' ) );
}
/**
* Returns the array key for a specific location ID so it can be pulled out of the 'locations' array.
* @todo move this?
* @param array $locations woocommerce_settings_locations
* @param string $id Location ID to get an array key index for
* @return integer|null
*/
protected function get_array_key_from_location_id( $locations, $id ) {
foreach ( $locations as $key => $location ) {
if ( $id === $location['id'] ) {
return $key;
}
}
return null;
}
}

View File

@ -1,3 +1,7 @@
# Locations
## Basic Info
Settings can be grouped together by location.
A location is just a grouping of settings that all share a common 'location' key.
@ -18,13 +22,15 @@ The coupon data box is considered location and would be represented like so:
There are 4 fields that make up a location:
* _id_: A unique identifier that can be used to link settings together. id is an alphanumeric string that should contain no spaces. Required.
* _type_: Context for where the settings in this location are going to be displayed. Right now core accepts 'page' for settings pages (pages currently under WooCommerce > Settings), 'metabox' (for metabox grouped settings like Coupon Data - this name is subject to change as this API develops), and 'shipping-zone' for settings associated with shipping zone settings. Required, defaults to 'page'.
* _id_: A unique identifier that can be used to link settings together. Alphanumeric string that contains no spaces. Required.
* _type_: Context for where the settings in this location are going to be displayed. Right now core accepts 'page' for settings pages (pages currently under WooCommerce > Settings), 'metabox' (for metabox grouped settings like Coupon Data - this name is subject to change as this API develops), and 'shipping-zone' for settings associated with shipping zone settings. Alphanumeric string that contains no spaces. Required, defaults to 'page'.
* _label_: A human readable label. This is a translated string that can be used in the UI. Optional.
* _description_: A human readable description. This is a translated string that can be used in the UI. Optional.
Any other fields passed will be stripped out before the JSON response is sent back to the client.
## Registering a Location
Locations can be registered with the `woocommerce_settings_locations` filter:
add_filter( 'woocommerce_settings_locations', function( $locations ) {
@ -36,3 +42,15 @@ Locations can be registered with the `woocommerce_settings_locations` filter:
);
return $locations;
} );
## Endpoints
### GET /settings/locations
Returns a list of all locations supported by WC.
There is an optional ?type parameter that allows you to return only locations for a specific context (like all page locations).
### GET /settings/locations/$id
Returns information on a single location.

View File

@ -23,10 +23,10 @@ class Settings extends \WP_Test_REST_Controller_Testcase {
}
/**
* Test normal /settings/locations response.
* Test getting all locations.
* @since 2.7.0
*/
public function test_get_items() {
public function test_get_locations() {
}
@ -55,12 +55,36 @@ class Settings extends \WP_Test_REST_Controller_Testcase {
}
public function test_context_param() { }
/**
* Test /settings/locations schema.
* @since 2.7.0
*/
public function test_get_item_schema() {
}
/**
* Test getting a single location item.
* @since 2.7.0
*/
public function test_get_location() {
}
/**
* Test getting a single location item.
* @since 2.7.0
*/
public function test_get_location_without_permission() {
}
public function test_get_items() { }
public function test_get_item() { }
public function test_context_param() { }
public function test_create_item() { }
public function test_update_item() { }
public function test_delete_item() { }
public function test_prepare_item() { }
public function test_get_item_schema() { }
}