Changeset 48883 for branches/5.5/src/wp-includes/rest-api.php

Timestamp:

08/27/2020 02:43:40 PM (6 years ago)

Author:

SergeyBiryukov

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, TimothyBlynJacobs.
Merges [48881] to the 5.5 branch.
Fixes #51146.

Location:

branches/5.5

Files:

• . (modified) (1 prop)
• src/wp-includes/rest-api.php (modified) (1 diff)

branches/5.5/src/wp-includes/rest-api.php

@@ -1316 +1316 @@
  */
 function rest_is_integer( $maybe_integer ) {
-        return round( floatval( $maybe_integer ) ) === floatval( $maybe_integer );
+        return is_numeric( $maybe_integer ) && round( floatval( $maybe_integer ) ) === floatval( $maybe_integer );
 }