Changeset 39105 for trunk/src/wp-includes/rest-api/endpoints/class-wp-rest-terms-controller.php

Timestamp:

11/03/2016 02:17:39 AM (8 years ago)

Author:

rachelbaker

Message:

REST API: Clean-up our validation callbacks and add missing array items properties in our endpoint schemas.

Props joehoyle, jnylen0.
Fixes #38617.

File:

• trunk/src/wp-includes/rest-api/endpoints/class-wp-rest-terms-controller.php (modified) (11 diffs)

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

@@ -888 +888 @@
             'description'       => __( 'Ensure result set excludes specific ids.' ),
             'type'              => 'array',
+            'items'             => array(
+                'type'          => 'integer',
+            ),
             'default'           => array(),
-            'sanitize_callback' => 'wp_parse_id_list',
         );
 
@@ -895 +897 @@
             'description'       => __( 'Limit result set to specific ids.' ),
             'type'              => 'array',
+            'items'             => array(
+                'type'          => 'integer',
+            ),
             'default'           => array(),
-            'sanitize_callback' => 'wp_parse_id_list',
         );
 
@@ -903 +907 @@
                 'description'       => __( 'Offset the result set by a specific number of items.' ),
                 'type'              => 'integer',
-                'sanitize_callback' => 'absint',
-                'validate_callback' => 'rest_validate_request_arg',
             );
         }
@@ -911 +913 @@
             'description'       => __( 'Order sort attribute ascending or descending.' ),
             'type'              => 'string',
-            'sanitize_callback' => 'sanitize_key',
             'default'           => 'asc',
             'enum'              => array(
@@ -917 +918 @@
                 'desc',
             ),
-            'validate_callback' => 'rest_validate_request_arg',
         );
 
@@ -923 +923 @@
             'description'       => __( 'Sort collection by resource attribute.' ),
             'type'              => 'string',
-            'sanitize_callback' => 'sanitize_key',
             'default'           => 'name',
             'enum'              => array(
@@ -934 +933 @@
                 'count',
             ),
-            'validate_callback' => 'rest_validate_request_arg',
         );
 
@@ -941 +939 @@
             'type'              => 'boolean',
             'default'           => false,
-            'sanitize_callback' => 'rest_sanitize_request_arg',
-            'validate_callback' => 'rest_validate_request_arg',
         );
 
@@ -949 +945 @@
                 'description'       => __( 'Limit result set to resources assigned to a specific parent.' ),
                 'type'              => 'integer',
-                'sanitize_callback' => 'absint',
-                'validate_callback' => 'rest_validate_request_arg',
             );
         }
@@ -958 +952 @@
             'type'              => 'integer',
             'default'           => null,
-            'validate_callback' => 'rest_validate_request_arg',
         );
 
@@ -964 +957 @@
             'description'       => __( 'Limit result set to resources with a specific slug.' ),
             'type'              => 'string',
-            'validate_callback' => 'rest_validate_request_arg',
         );