Changeset 51681 for branches/5.8/tests/phpunit/tests/blocks/serialization.php

Timestamp:

08/27/2021 07:08:21 PM (3 years ago)

Author:

desrosj

Message:

Editor: Ensure block attribute serialization in PHP matches the JavaScript equivalent.

The serializeAttributes() function in JavaScript uses JSON.stringify, which does not encode slashes and unicode characters by default. This resulted in the PHP serialization through json_encode() producing different results.

This also switches from json_encode() to wp_json_encode() to prevent failures when any non UTF-8 characters are included.

Props kevinfodness, SergeyBiryukov, timothyblynjacobs.
Merges [51674] to the 5.8 branch.
Fixes #53936.

Location:

branches/5.8

Files:

• . (modified) (1 prop)
• tests/phpunit/tests/blocks/serialization.php (modified) (1 diff)

branches/5.8/tests/phpunit/tests/blocks/serialization.php

@@ -48 +48 @@
             // Block with attribute values that may conflict with HTML comment.
             array( '<!-- wp:attributes {"key":"\\u002d\\u002d\\u003c\\u003e\\u0026\\u0022"} /-->' ),
+
+            // Block with attribute values that should not be escaped.
+            array( '<!-- wp:attributes {"key":"€1.00 / 3 for €2.00"} /-->' ),
         );
     }