Task #98321
closedAddress "undefined array key" issues for PHP 8
100%
Description
I'm pretty sure there are still a bunch of "undefined array key" warnings hidden in the TYPO3 core when used in combination with PHP 8.
To identify and address those issues, I created a PsalmPHP plugin tweaking some static code analysis topics (e.g. static/self references, or ignoring $GLOBALS
) for creating a list with potential code locations that could be enhanced. This list is quite long and some aspects might feel okayish... Anyway, I'm just posting the results as TXT and SARIF file.
Processing¶
# process with max. 1 thread bin/psalm --memory-limit=8G --threads=1 --no-cache \ --report=results.sarif --output-format=github # Visual Studio Code SARIF needs special JSON and absolute paths (@sed@ on macOS neeeds `-i ''`, on Linux just `-i`) sed -i '' \ -e 's,\\/,/,g' \ -e's,{"uri":"typo3/,{"uri":"'"$(pwd)"'/typo3/,g' results.sarif
Tools¶
Subtasks¶
The subtasks of this issue are basically all system extensions, that have been approached systematically. Arbitrary one-liners can be referenced, but not added as subtasks.
Files
Updated by Gerrit Code Review about 2 years 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/+/75682
Updated by Gerrit Code Review about 2 years ago
Patch set 2 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/+/75682
Updated by Gerrit Code Review about 2 years ago
Patch set 3 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/+/75682
Updated by Oliver Hader about 2 years ago
- Related to Bug #98361: PHP Warning: Undefined array key "_CURRENT_VERSION" in WorkspaceVersionRecordsCommand added
Updated by Daniel Siepmann about 2 years ago
- Related to Bug #98337: Undefined array key "_CURRENT_VERSION" in WorkspaceVersionRecordsCommand added
Updated by Daniel Siepmann about 2 years ago
- Related to Bug #98358: Undefined array key "exec" in ExtensionManagementUtility added
Updated by Daniel Siepmann about 2 years ago
- Related to Bug #98359: PHP Warning: Undefined array key 0 in SchedulerCommand.php added
Updated by Daniel Siepmann about 2 years ago
- Related to Bug #97619: Undefined array key "_CURRENT_VERSION" in CleanFlexFormsCommand added
Updated by Gerrit Code Review about 2 years ago
Patch set 4 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/+/75682
Updated by Gerrit Code Review about 2 years ago
Patch set 1 for branch 11.5 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/75777
Updated by Daniel Siepmann about 2 years ago
- Related to Bug #98387: PHP Warning: Undefined array key "XYZ" in File line 184 added
Updated by Daniel Siepmann about 2 years ago
- Related to Bug #98404: PHP Warning: Undefined array key "sys_language_uid" in PageRepository added
Updated by Daniel Siepmann about 2 years ago
- Related to Bug #98406: PHP Warning: Undefined array key X in AbstractItemProvider added
Updated by Oliver Hader about 2 years ago
- Related to Bug #98274: Undefined array key "skipIfValueIsEmpty" in SaveToDatabaseFinisher (PHP 8) added
Updated by Daniel Siepmann about 2 years ago
- Related to Bug #98412: PHP Warning: Undefined array key "xyz" in ProcessedFile added
Updated by Christian Kuhn about 2 years ago
- Related to deleted (Bug #98404: PHP Warning: Undefined array key "sys_language_uid" in PageRepository)
Updated by Oliver Hader over 1 year ago
- Status changed from Under Review to Rejected