Bug #92908
closedUsing EXT: in allowedExtensionPaths causes exception with 10.4.10
100%
Description
FormEngineSetup.yaml:
TYPO3:
CMS:
Form:
persistenceManager:
allowedExtensionPaths:
10: 'EXT:my_form/Resources/Private/Forms/'
allowSaveToExtensionPaths: false
allowDeleteFromExtensionPaths: false
This worked up to and including 10.4.9.
However, upon installing 10.4.10, the frontend now instead displays the following exception upon loading a page with a predefined form:
(1/1) #1314516810 TYPO3\CMS\Core\Resource\Exception\FolderDoesNotExistException
Folder "/my_form/Resources/Private/Forms/" does not exist.
I traced the relevant change back to /typo3/sysext/core/Classes/Resource/ResourceFactory.php::getFolderObjectFromCombinedIdentifier()
At line 504, what used to be
$storageUid = $parts[0];
has been changed to $storageUid = (int)$parts[0];
in commit https://github.com/TYPO3/TYPO3.CMS/commit/a79c665c9abe112fbdf8f8a45ce734b6030a0606
This causes the EXT part of the config to be cast to 0, meaning that the is_numeric() in TYPO3\CMS\Core\Resource\ResourceFactory::getStorageObject() is no longer throwing an Exception - which itself was caught by TYPO3\CMS\Form\Mvc\Property\PropertyMappingConfiguration::checkSaveFileMountAccess() -, and instead the path is resolved and checked against storage uid 0, where it naturally can't be found.
Updated by Mathias Brodala about 4 years ago
- Related to Task #92264: Fix phpstan checkFunctionArgumentTypes errors in ext:core Resource added
Updated by Gerrit Code Review about 4 years ago
- Status changed from New to Under Review
Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/66852
Updated by Gerrit Code Review about 4 years ago
Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/66852
Updated by Gerrit Code Review about 4 years ago
Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/66852
Updated by Gerrit Code Review almost 4 years ago
Patch set 1 for branch 10.4 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/66918
Updated by Daniel Goerz almost 4 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 5f92958bed82f7250b4ce08a6233fd4ab9e90f6f.
Updated by Benni Mack almost 4 years ago
- Status changed from Resolved to Closed