Changeset 48881 for trunk/tests/phpunit/tests/rest-api/rest-schema-validation.php

Timestamp:

08/27/2020 02:55:39 AM (5 years ago)

Author:

TimothyBlynJacobs

Message:

REST API: Fix multi-type schemas with integer fields.

In [48306] support for multi-typed schemas was improved to first detect the data type of the value before applying further validation. The integer data type was detected using the new rest_is_integer function. This function used logic, however, that assumed that the value had already passed an is_numeric check. This meant that if integer and string were both acceptable types, the value would always be considered an integer causing the later accurate type validation to fail.

This commit fixes the rest_is_integer logic to include an is_numeric check.

Props rtagliento.
Fixes #51146.

File:

• trunk/tests/phpunit/tests/rest-api/rest-schema-validation.php (modified) (1 diff)

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

@@ -1041 +1041 @@
         $this->assertTrue( rest_validate_value_from_schema( $data, $schema ) );
     }
+
+    /**
+     * @ticket 50300
+     */
+    public function test_string_or_integer() {
+        $schema = array(
+            'type' => array( 'integer', 'string' ),
+        );
+
+        $this->assertTrue( rest_validate_value_from_schema( 'garbage', $schema ) );
+        $this->assertTrue( rest_validate_value_from_schema( 15, $schema ) );
+        $this->assertTrue( rest_validate_value_from_schema( '15', $schema ) );
+        $this->assertTrue( rest_validate_value_from_schema( '15.5', $schema ) );
+        $this->assertWPError( rest_validate_value_from_schema( 15.5, $schema ) );
+    }
+
 }