Changeset 39087 for trunk/src/wp-includes/rest-api/class-wp-rest-request.php

Timestamp:

11/02/2016 05:52:12 AM (9 years ago)

Author:

rmccue

Message:

REST API: Change method of merging parameters.

array_merge() incorrectly reindexes numeric parameters, causing things like {"123": true} to be "dropped".

Props sswells, joehoyle.
Fixes #38306.

File:

• trunk/src/wp-includes/rest-api/class-wp-rest-request.php (modified) (1 diff)

trunk/src/wp-includes/rest-api/class-wp-rest-request.php

@@ -452 +452 @@
         $params = array();
         foreach ( $order as $type ) {
-            $params = array_merge( $params, (array) $this->params[ $type ] );
+            // array_merge / the "+" operator will mess up
+            // numeric keys, so instead do a manual foreach.
+            foreach ( (array) $this->params[ $type ] as $key => $value ) {
+                $params[ $key ] = $value;
+            }
         }