Merge branch '4.4' into 5.4

nicolas-grekas · nicolas-grekas · commit a213cbc80382 · 2022-05-04T16:46:32.000+02:00
* 4.4:
  Remove former core members from code owners
  [Form] fix populating single widget time view data with different timezones
  [DomCrawler][VarDumper] Fix html-encoding emojis
diff --git a/Crawler.php b/Crawler.php
@@ -1186,11 +1186,11 @@ private function convertToHtmlEntities(string $htmlContent, string $charset = 'U
         set_error_handler(function () { throw new \Exception(); });
 
         try {
-            return mb_encode_numericentity($htmlContent, [0x80, 0xFFFF, 0, 0xFFFF], $charset);
+            return mb_encode_numericentity($htmlContent, [0x80, 0x10FFFF, 0, 0x1FFFFF], $charset);
         } catch (\Exception|\ValueError $e) {
             try {
                 $htmlContent = iconv($charset, 'UTF-8', $htmlContent);
-                $htmlContent = mb_encode_numericentity($htmlContent, [0x80, 0xFFFF, 0, 0xFFFF], 'UTF-8');
+                $htmlContent = mb_encode_numericentity($htmlContent, [0x80, 0x10FFFF, 0, 0x1FFFFF], 'UTF-8');
             } catch (\Exception|\ValueError $e) {
             }
 
diff --git a/Tests/AbstractCrawlerTest.php b/Tests/AbstractCrawlerTest.php
@@ -381,6 +381,13 @@ public function testHtml()
         $this->assertSame('my value', $this->createTestCrawler(null)->filterXPath('//ol')->html('my value'));
     }
 
+    public function testEmojis()
+    {
+        $crawler = $this->createCrawler('<body><p>Hey 👋</p></body>');
+
+        $this->assertSame('<body><p>Hey 👋</p></body>', $crawler->html());
+    }
+
     public function testExtract()
     {
         $crawler = $this->createTestCrawler()->filterXPath('//ul[1]/li');

Original file line number	Diff line number	Diff line change
`@@ -381,6 +381,13 @@ public function testHtml()`
`381`	`381`	`$this->assertSame('my value', $this->createTestCrawler(null)->filterXPath('//ol')->html('my value'));`
`382`	`382`	`}`
`383`	`383`
	`384`	`+ public function testEmojis()`
	`385`	`+ {`
	`386`	`+ $crawler = $this->createCrawler('<body><p>Hey 👋</p></body>');`
	`387`	`+`
	`388`	`+ $this->assertSame('<body><p>Hey 👋</p></body>', $crawler->html());`
	`389`	`+ }`
	`390`	`+`
`384`	`391`	`public function testExtract()`
`385`	`392`	`{`
`386`	`393`	`$crawler = $this->createTestCrawler()->filterXPath('//ul[1]/li');`