Changeset 48796

Timestamp:

08/15/2020 04:09:56 PM (5 years ago)

Author:

TimothyBlynJacobs

Message:

REST API: Add uniqueItems as a permitted endpoint argument JSON Schema keyword.

Support for the uniqueItems keyword was added in [48357], but wasn't included in the list of permitted keywords for WP_REST_Controller::get_endpoint_args_for_item_schema(). This prevented the keyword from being automatically supported if specified in a controller's item schema.

Fixes #51021.

Location:

trunk

Files:

• src/wp-includes/rest-api/endpoints/class-wp-rest-controller.php (modified) (1 diff)
• tests/phpunit/tests/rest-api/rest-controller.php (modified) (1 diff)
• tests/phpunit/tests/rest-api/rest-test-controller.php (modified) (1 diff)

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

@@ -646 +646 @@
             'minItems',
             'maxItems',
+            'uniqueItems',
         );
 

trunk/tests/phpunit/tests/rest-api/rest-controller.php

@@ -266 +266 @@
         $this->assertArrayHasKey( 'items', $args['somearray'] );
 
-        foreach ( array( 'minItems', 'maxItems' ) as $property ) {
+        foreach ( array( 'minItems', 'maxItems', 'uniqueItems' ) as $property ) {
             $this->assertArrayHasKey( $property, $args['somearray'] );
         }

trunk/tests/phpunit/tests/rest-api/rest-test-controller.php

@@ -102 +102 @@
                 ),
                 'somearray'      => array(
-                    'type'     => 'array',
-                    'items'    => array(
+                    'type'        => 'array',
+                    'items'       => array(
                         'type' => 'string',
                     ),
-                    'minItems' => 1,
-                    'maxItems' => 10,
-                    'context'  => array( 'view' ),
+                    'minItems'    => 1,
+                    'maxItems'    => 10,
+                    'uniqueItems' => true,
+                    'context'     => array( 'view' ),
                 ),
                 'someobject'     => array(