Skip to content

Commit 45eebd7

Browse files
committed
[TASK] Explicitly provide all fgetcsv() arguments
With [1] the 5th parameter `$escape` of `fgetcsv()` must be provided either positional or using named arguments or a E_DEPRECATED will be emitted since `PHP 8.4.0 RC1` [2]. This change provide now all five parameter for `fgetcsv()` calls and thus using the positional approach to allow easier backporting to older TYPO3 version where named arguements are not usable. [1] php/php-src#15569 [2] https://github.com/php/php-src/blob/ebee8df27ed/UPGRADING#L617-L622 Releases: main, 8, 7
1 parent 76acb9c commit 45eebd7

File tree

1 file changed

+1
-1
lines changed
  • Classes/Core/Functional/Framework/DataHandling

1 file changed

+1
-1
lines changed

Classes/Core/Functional/Framework/DataHandling/DataSet.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,7 @@ protected static function readData(string $fileName): array
6363
// BOM not found - rewind pointer to start of file.
6464
rewind($fileHandle);
6565
}
66-
while (!feof($fileHandle) && ($values = fgetcsv($fileHandle, 0)) !== false) {
66+
while (!feof($fileHandle) && ($values = fgetcsv($fileHandle, 0, ',', '"', '\\')) !== false) {
6767
$rawData[] = $values;
6868
}
6969
fclose($fileHandle);

0 commit comments

Comments
 (0)