Changeset 46206 for trunk/tests/phpunit/tests/rest-api/rest-schema-setup.php

Timestamp:

09/20/2019 08:07:28 PM (5 years ago)

Author:

desrosj

Message:

Code Modernization: Remove JSON extension workarounds for PHP < 5.6.

The PHP native JSON extension has been bundled and compiled with PHP by default since version 5.2.0. Because the minimum version of PHP required by WordPress is now 5.6.20 (see #46594 and [45058]), JSON extension related polyfills and backwards compatibility code can now be removed.

This change removes code that supported JSON related functionality on older versions of PHP. This includes (but is not limited to) checks that json_last_error() exists, checking and setting the JSON_UNESCAPED_SLASHES and JSON_PRETTY_PRINT constants if not previously defined, and deprecating the _wp_json_prepare_data() function (which was 100% workaround code).

Follow up of [46205].

See #47699.
Props jrf, Clorith, pento.

File:

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

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

@@ -489 +489 @@
             $this->assertTrue( ! empty( $data ), $route['name'] . ' route should return data.' );
 
-            if ( version_compare( PHP_VERSION, '5.4', '>=' ) ) {
-                $fixture           = $this->normalize_fixture( $data, $route['name'] );
-                $mocked_responses .= "\nmockedApiResponse." . $route['name'] . ' = '
-                    . json_encode( $fixture, JSON_PRETTY_PRINT | JSON_UNESCAPED_SLASHES )
-                    . ";\n";
-            }
+            $fixture           = $this->normalize_fixture( $data, $route['name'] );
+            $mocked_responses .= "\nmockedApiResponse." . $route['name'] . ' = '
+                . json_encode( $fixture, JSON_PRETTY_PRINT | JSON_UNESCAPED_SLASHES )
+                . ";\n";
         }
 
         // Only generate API client fixtures in single site and when required JSON_* constants are supported.
-        if ( ! is_multisite() && version_compare( PHP_VERSION, '5.4', '>=' ) ) {
+        if ( ! is_multisite() ) {
             // Save the route object for QUnit tests.
             $file = dirname( DIR_TESTROOT ) . '/qunit/fixtures/wp-api-generated.js';