Bug #79101
closedStory #69617: FormEngine bugs
InvalidPointerFieldValueException needs to be caught
100%
Description
\TYPO3\CMS\Core\Configuration\FlexForm\FlexFormTools::getDataStructureIdentifierFromRecord
tries to resolve the data structure of a record from another record in the database. This is a common practice in TemplaVoilà for instance.
If the identifier is an empty string or 0 (zero), a InvalidPointerFieldValueException is thrown, which is the right thing to do in the mentioned method.
Problem is, that the Exception is not caught in \TYPO3\CMS\Core\Configuration\FlexForm\FlexFormTools::getDataStructureIdentifier
.
The exception says, the problem is most likely a wrong configured TCA, which is not right if there is just no relation (yet) to a datastructure record. Thus, the exception needs to be handled properly during further execution.
Usual use case:
During the localization of a content element, the overlay usually does not get the same identifier value as its parent, thus the whole localization fails.
Updated by Gerrit Code Review about 8 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/51061
Updated by Gerrit Code Review almost 8 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/51061
Updated by Gerrit Code Review about 7 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/51061
Updated by Gerrit Code Review about 7 years ago
Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/51061
Updated by Gerrit Code Review about 7 years ago
Patch set 1 for branch TYPO3_8-7 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/54875
Updated by Anonymous about 7 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 7ee93637146d119eabd7797e9acfbc85e910c3cf.
Updated by Alexander Opitz almost 7 years ago
Hint:
Copy&Paste elements with such a flexible DS field (as described above) still fails after this patch for elements which do not have a DS selected.
For example in TemplaVoilà all tt_content elements which aren't FCEs of TemplaVoilà.