[Translation] Fix handling of empty lines in CsvFileLoader

fnogatz · nicolas-grekas · commit 61a1414a530d · 2026-01-12T20:15:33.000+01:00
diff --git a/src/Symfony/Component/Translation/Loader/CsvFileLoader.php b/src/Symfony/Component/Translation/Loader/CsvFileLoader.php
@@ -34,7 +34,7 @@ protected function loadResource(string $resource): array
             throw new NotFoundResourceException(\sprintf('Error opening file "%s".', $resource), 0, $e);
         }
 
-        $file->setFlags(\SplFileObject::READ_CSV | \SplFileObject::SKIP_EMPTY);
+        $file->setFlags(\SplFileObject::READ_CSV | \SplFileObject::SKIP_EMPTY | \SplFileObject::DROP_NEW_LINE);
         $file->setCsvControl($this->delimiter, $this->enclosure, $this->escape);
 
         foreach ($file as $data) {
diff --git a/src/Symfony/Component/Translation/Tests/Fixtures/resources.csv b/src/Symfony/Component/Translation/Tests/Fixtures/resources.csv
@@ -1,4 +1,6 @@
 "foo"; "bar"
 #"bar"; "foo"
+
+# all incorrect examples:
 "incorrect"; "number"; "columns"; "will"; "be"; "ignored"
-"incorrect"
+"incorrect"

Original file line number	Diff line number	Diff line change
`@@ -34,7 +34,7 @@ protected function loadResource(string $resource): array`
`34`	`34`	`throw new NotFoundResourceException(\sprintf('Error opening file "%s".', $resource), 0, $e);`
`35`	`35`	`}`
`36`	`36`
`37`		`- $file->setFlags(\SplFileObject::READ_CSV \| \SplFileObject::SKIP_EMPTY);`
	`37`	`+ $file->setFlags(\SplFileObject::READ_CSV \| \SplFileObject::SKIP_EMPTY \| \SplFileObject::DROP_NEW_LINE);`
`38`	`38`	`$file->setCsvControl($this->delimiter, $this->enclosure, $this->escape);`
`39`	`39`
`40`	`40`	`foreach ($file as $data) {`