WordPress.org
Get WordPress

Make WordPress Core

Ticket #38583: 38583.3.diff

File 38583.3.diff, 8.7 KB (added by joehoyle, 7 years ago)

  • src/wp-includes/rest-api.php 

    diff --git a/src/wp-includes/rest-api.php b/src/wp-includes/rest-api.php
index 06bb9f9..bc775f2 100644
    a b function rest_validate_value_from_schema( $value, $args, $param = '' ) { 
    10511051                        }
    10521052                }
    10531053        }
     1054
     1055        if ( 'object' === $args['type'] ) {
     1056                if ( ! is_array( $value ) ) {
     1057                        /* translators: 1: parameter, 2: type name */
     1058                        return new WP_Error( 'rest_invalid_param', sprintf( __( '%1$s is not of type %2$s.' ), $param, 'object' ) );
     1059                }
     1060
     1061                foreach ( $value as $property => $v ) {
     1062                        if ( ! isset( $args['properties'][ $property ] ) ) {
     1063                                continue;
     1064                        }
     1065                        $is_valid = rest_validate_value_from_schema( $v, $args['properties'][ $property ], $param . '[' . $property . ']' );
     1066
     1067                        if ( is_wp_error( $is_valid ) ) {
     1068                                return $is_valid;
     1069                        }
     1070                }
     1071        }
     1072
    10541073        if ( ! empty( $args['enum'] ) ) {
    10551074                if ( ! in_array( $value, $args['enum'], true ) ) {
    10561075                        /* translators: 1: parameter, 2: list of valid values */
    … … function rest_sanitize_value_from_schema( $value, $args ) { 
    11701189                $value = array_values( $value );
    11711190                return $value;
    11721191        }
     1192
     1193        if ( 'object' === $args['type'] ) {
     1194                if ( ! is_array( $value ) ) {
     1195                        return array();
     1196                }
     1197
     1198                foreach ( $value as $property => $v ) {
     1199                        if ( ! isset( $args['properties'][ $property ] ) ) {
     1200                                unset( $value[ $property ] );
     1201                                continue;
     1202                        }
     1203                        $value[ $property ] = rest_sanitize_value_from_schema( $v, $args['properties'][ $property ] );
     1204                }
     1205
     1206                return $value;
     1207        }
     1208
    11731209        if ( 'integer' === $args['type'] ) {
    11741210                return (int) $value;
    11751211        }

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

    diff --git a/src/wp-includes/rest-api/endpoints/class-wp-rest-attachments-controller.php b/src/wp-includes/rest-api/endpoints/class-wp-rest-attachments-controller.php
index 76802fe..ed10181 100644
    a b class WP_REST_Attachments_Controller extends WP_REST_Posts_Controller { 
    391391                        'context'     => array( 'view', 'edit', 'embed' ),
    392392                        'arg_options' => array(
    393393                                'sanitize_callback' => null, // Note: sanitization implemented in self::prepare_item_for_database()
     394                                'validate_callback' => null, // Note: validation implemented in self::prepare_item_for_database()
    394395                        ),
    395396                        'properties'  => array(
    396397                                'raw' => array(
    … … class WP_REST_Attachments_Controller extends WP_REST_Posts_Controller { 
    413414                        'context'     => array( 'view', 'edit' ),
    414415                        'arg_options' => array(
    415416                                'sanitize_callback' => null, // Note: sanitization implemented in self::prepare_item_for_database()
     417                                'validate_callback' => null, // Note: validation implemented in self::prepare_item_for_database()
    416418                        ),
    417419                        'properties'  => array(
    418420                                'raw' => array(

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

    diff --git a/src/wp-includes/rest-api/endpoints/class-wp-rest-comments-controller.php b/src/wp-includes/rest-api/endpoints/class-wp-rest-comments-controller.php
index 810a3fb..415ba8e 100644
    a b class WP_REST_Comments_Controller extends WP_REST_Controller { 
    12001200                                        'context'         => array( 'view', 'edit', 'embed' ),
    12011201                                        'arg_options'     => array(
    12021202                                                'sanitize_callback' => null, // Note: sanitization implemented in self::prepare_item_for_database()
     1203                                                'validate_callback' => null, // Note: validation implemented in self::prepare_item_for_database()
    12031204                                        ),
    12041205                                        'properties'      => array(
    12051206                                                'raw'         => array(

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

    diff --git a/src/wp-includes/rest-api/endpoints/class-wp-rest-posts-controller.php b/src/wp-includes/rest-api/endpoints/class-wp-rest-posts-controller.php
index 018b66e..80027cf 100644
    a b class WP_REST_Posts_Controller extends WP_REST_Controller { 
    18461846                                                'context'     => array( 'view', 'edit', 'embed' ),
    18471847                                                'arg_options' => array(
    18481848                                                        'sanitize_callback' => null, // Note: sanitization implemented in self::prepare_item_for_database()
     1849                                                        'validate_callback' => null, // Note: validation implemented in self::prepare_item_for_database()
    18491850                                                ),
    18501851                                                'properties'  => array(
    18511852                                                        'raw' => array(
    … … class WP_REST_Posts_Controller extends WP_REST_Controller { 
    18701871                                                'context'     => array( 'view', 'edit' ),
    18711872                                                'arg_options' => array(
    18721873                                                        'sanitize_callback' => null, // Note: sanitization implemented in self::prepare_item_for_database()
     1874                                                        'validate_callback' => null, // Note: validation implemented in self::prepare_item_for_database()
    18731875                                                ),
    18741876                                                'properties'  => array(
    18751877                                                        'raw' => array(
    … … class WP_REST_Posts_Controller extends WP_REST_Controller { 
    19081910                                                'context'     => array( 'view', 'edit', 'embed' ),
    19091911                                                'arg_options' => array(
    19101912                                                        'sanitize_callback' => null, // Note: sanitization implemented in self::prepare_item_for_database()
     1913                                                        'validate_callback' => null, // Note: validation implemented in self::prepare_item_for_database()
    19111914                                                ),
    19121915                                                'properties'  => array(
    19131916                                                        'raw' => array(

  • tests/phpunit/tests/rest-api/rest-schema-sanitization.php 

    diff --git a/tests/phpunit/tests/rest-api/rest-schema-sanitization.php b/tests/phpunit/tests/rest-api/rest-schema-sanitization.php
index e1ae29d..a365d9c 100644
    a b class WP_Test_REST_Schema_Sanitization extends WP_UnitTestCase { 
    146146                $this->assertEquals( array( '1', '2' ), rest_sanitize_value_from_schema( array( 'first' => '1', 'second' => '2' ), $schema ) );
    147147        }
    148148
     149        public function test_type_object() {
     150                $schema = array(
     151                        'type'       => 'object',
     152                        'properties' => array(
     153                                'a' => array(
     154                                        'type' => 'number'
     155                                ),
     156                        ),
     157                );
     158                $this->assertEquals( array( 'a' => 1 ), rest_sanitize_value_from_schema( array( 'a' => 1 ), $schema ) );
     159                $this->assertEquals( array( 'a' => 1 ), rest_sanitize_value_from_schema( array( 'a' => '1' ), $schema ) );
     160        }
     161
     162        public function test_type_object_nested() {
     163                $schema = array(
     164                        'type' => 'object',
     165                        'properties' => array(
     166                                'a' => array(
     167                                        'type'  => 'object',
     168                                        'properties' => array(
     169                                                'b' => array( 'type' => 'number' ),
     170                                                'c' => array( 'type' => 'number' ),
     171                                        )
     172                                )
     173                        ),
     174                );
     175
     176                $this->assertEquals(
     177                        array(
     178                                'a' => array(
     179                                        'b' => 1,
     180                                        'c' => 3,
     181                                ),
     182                        ),
     183                        rest_sanitize_value_from_schema(
     184                                array(
     185                                        'a' => array(
     186                                                'b' => '1',
     187                                                'c' => '3',
     188                                        ),
     189                                ),
     190                                $schema
     191                        )
     192                );
     193                $this->assertEquals(
     194                        array(
     195                                'a' => array(
     196                                        'b' => 1,
     197                                        'c' => 3,
     198                                ),
     199                        ),
     200                        rest_sanitize_value_from_schema(
     201                                array(
     202                                        'a' => array(
     203                                                'b' => '1',
     204                                                'c' => '3',
     205                                                'd' => '1',
     206                                        ),
     207                                        'b' => 1,
     208                                ),
     209                                $schema
     210                        )
     211                );
     212                $this->assertEquals( array( 'a' => array() ), rest_sanitize_value_from_schema( array( 'a' => null ), $schema ) );
     213        }
     214
    149215        public function test_type_unknown() {
    150216                $schema = array(
    151217                        'type' => 'lalala',

  • tests/phpunit/tests/rest-api/rest-schema-validation.php 

    diff --git a/tests/phpunit/tests/rest-api/rest-schema-validation.php b/tests/phpunit/tests/rest-api/rest-schema-validation.php
index 82f58e5..c47ed10 100644
    a b class WP_Test_REST_Schema_Validation extends WP_UnitTestCase { 
    181181                $this->assertWPError( rest_validate_value_from_schema( array( 'first' => '1', 'second' => '2' ), $schema ) );
    182182        }
    183183
     184        public function test_type_object() {
     185                $schema = array(
     186                        'type'       => 'object',
     187                        'properties' => array(
     188                                'a' => array(
     189                                        'type' => 'number'
     190                                ),
     191                        ),
     192                );
     193                $this->assertTrue( rest_validate_value_from_schema( array( 'a' => 1 ), $schema ) );
     194                $this->assertWPError( rest_validate_value_from_schema( array( 'a' => 'invalid' ), $schema ) );
     195        }
     196
     197        public function test_type_object_nested() {
     198                $schema = array(
     199                        'type' => 'object',
     200                        'properties' => array(
     201                                'a' => array(
     202                                        'type'  => 'object',
     203                                        'properties' => array(
     204                                                'b' => array( 'type' => 'number' ),
     205                                                'c' => array( 'type' => 'number' ),
     206                                        )
     207                                )
     208                        ),
     209                );
     210                $this->assertTrue(
     211                        rest_validate_value_from_schema(
     212                                array(
     213                                        'a' => array(
     214                                                'b' => '1',
     215                                                'c' => 3,
     216                                        ),
     217                                ),
     218                                $schema
     219                        )
     220                );
     221                $this->assertWPError( rest_validate_value_from_schema( array( 'a' => array( 'b' => 1, 'c' => 'invalid' ) ), $schema ) );
     222                $this->assertWPError( rest_validate_value_from_schema( array( 'a' => 1 ), $schema ) );
     223        }
     224
    184225        public function test_type_unknown() {
    185226                $schema = array(
    186227                        'type'  => 'lalala',