Changeset 41702 for trunk/tests/phpunit/tests/formatting/Emoji.php

Timestamp:

10/03/2017 09:56:45 AM (7 years ago)

Author:

pento

Message:

Emoji: Fix incorrect emoji encoding in PHP < 5.4.

[41701] included a bug with PHP < 5.4. Prior to then, html_entity_decode() decoded into ISO-8859-1, when we actually need it to use UTF-8.

Fixes #35293.

File:

• trunk/tests/phpunit/tests/formatting/Emoji.php (modified) (1 diff)

trunk/tests/phpunit/tests/formatting/Emoji.php

@@ -123 +123 @@
                 // Simple emoji
                 '🙂',
-                '<img src="' . $this->png_cdn . '1f642.png" alt="" class="wp-smiley" style="height: 1em; max-height: 1em;" />',
+                '<img src="' . $this->png_cdn . '1f642.png" alt="🙂" class="wp-smiley" style="height: 1em; max-height: 1em;" />',
             ),
             array(
                 // Skin tone, gender, ZWJ, emoji selector
                 '👮🏼‍♀️',
-                '<img src="' . $this->png_cdn . '1f46e-1f3fc-200d-2640-fe0f.png" alt="" class="wp-smiley" style="height: 1em; max-height: 1em;" />',
+                '<img src="' . $this->png_cdn . '1f46e-1f3fc-200d-2640-fe0f.png" alt="👮🏼‍♀️" class="wp-smiley" style="height: 1em; max-height: 1em;" />',
             ),
             array(
                 // Unicode 10
                 '🧚',
-                '<img src="' . $this->png_cdn . '1f9da.png" alt="" class="wp-smiley" style="height: 1em; max-height: 1em;" />',
+                '<img src="' . $this->png_cdn . '1f9da.png" alt="🧚" class="wp-smiley" style="height: 1em; max-height: 1em;" />',
             ),
         );
-
-        // Older versions of PHP don't html_entity_decode() emoji, so we need to make sure they're testing in the expected form.
-        foreach ( $data as $key => $datum ) {
-            $emoji = html_entity_decode( wp_encode_emoji( $datum[0] ) );
-            $data[ $key ][1] = str_replace( 'alt=""', 'alt="' . $emoji . '"', $datum[1] );
-        }
 
         return $data;