[Translation] Fix CSV escape char in CsvFileLoader on PHP >= 7.4

alexandre-daubois · alexandre-daubois · commit a83b39cf7eb9 · 2024-07-26T13:55:51.000+02:00
diff --git a/src/Symfony/Component/Translation/Loader/CsvFileLoader.php b/src/Symfony/Component/Translation/Loader/CsvFileLoader.php
@@ -22,7 +22,7 @@ class CsvFileLoader extends FileLoader
 {
     private $delimiter = ';';
     private $enclosure = '"';
-    private $escape = '\\';
+    private $escape = '';
 
     /**
      * {@inheritdoc}
@@ -56,10 +56,10 @@ protected function loadResource(string $resource)
     /**
      * Sets the delimiter, enclosure, and escape character for CSV.
      */
-    public function setCsvControl(string $delimiter = ';', string $enclosure = '"', string $escape = '\\')
+    public function setCsvControl(string $delimiter = ';', string $enclosure = '"', string $escape = '')
     {
         $this->delimiter = $delimiter;
         $this->enclosure = $enclosure;
-        $this->escape = $escape;
+        $this->escape = '' === $escape && \PHP_VERSION_ID < 70400 ? '\\' : $escape;
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -22,7 +22,7 @@ class CsvFileLoader extends FileLoader`
`22`	`22`	`{`
`23`	`23`	`private $delimiter = ';';`
`24`	`24`	`private $enclosure = '"';`
`25`		`- private $escape = '\\';`
	`25`	`+ private $escape = '';`
`26`	`26`
`27`	`27`	`/**`
`28`	`28`	`* {@inheritdoc}`
`@@ -56,10 +56,10 @@ protected function loadResource(string $resource)`
`56`	`56`	`/**`
`57`	`57`	`* Sets the delimiter, enclosure, and escape character for CSV.`
`58`	`58`	`*/`
`59`		`- public function setCsvControl(string $delimiter = ';', string $enclosure = '"', string $escape = '\\')`
	`59`	`+ public function setCsvControl(string $delimiter = ';', string $enclosure = '"', string $escape = '')`
`60`	`60`	`{`
`61`	`61`	`$this->delimiter = $delimiter;`
`62`	`62`	`$this->enclosure = $enclosure;`
`63`		`- $this->escape = $escape;`
	`63`	`+ $this->escape = '' === $escape && \PHP_VERSION_ID < 70400 ? '\\' : $escape;`
`64`	`64`	`}`
`65`	`65`	`}`