Project

General

Profile

Actions
Copy link

Task #105157

closed

Explicitly provide all `fgetcsv()` arguments

Added by Stefan Bürk about 2 months ago. Updated about 2 months ago.

Status:
Closed
Priority:
Should have
Assignee:
Stefan Bürk
Category:
-
Target version:
-
Start date:
2024-09-29
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
11
PHP Version:
Tags:
Complexity:
Sprint Focus:

Description

PHP 8.4.0 RC1 emits E_DEPRECATED when the fifth paramater
$escape is not explicitly set and the default value used
for fgetcsv()

See: https://github.com/php/php-src/blob/ebee8df27edf7ecfbdc7ff87662f2f0d30859120/UPGRADING#L617-L622

Using the default value for $escape parameter of:
 - fputcsv()
 - fgetcsv()
 - str_getcsv()
 is now deprecated. It must be passed explicitly either positionally or via named arguments.
 RFC: https://wiki.php.net/rfc/deprecations_php_8_4#deprecate_proprietary_csv_escaping_mechanism

[1] https://github.com/php/php-src/pull/15569
[2] https://github.com/php/php-src/blob/ebee8df27ed/UPGRADING#L617-L622

Actions
Copy link
 #1

Updated by Gerrit Code Review about 2 months ago

  • Status changed from New to Under Review

Patch set 1 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/86373

Actions
Copy link
 #2

Updated by Stefan Bürk about 2 months ago

  • Status changed from Under Review to Closed

Integrated into #105155, closing this issue now.

Actions
Copy link

Also available in: Atom PDF