Bug #86938
closed
Implicit type cast through ArrayUtility::stripTagsFromValuesRecursive
100%
Description
`\TYPO3\CMS\Core\Utility\ArrayUtility::stripTagsFromValuesRecursive` does not care if the value is a number, so any integers are returned as strings.
That's a problem e.g. in the form extension, which uses this method in `\TYPO3\CMS\Form\Controller\FormEditorController::transformFormDefinitionForFormEditor`.
Because of this implicit type cast the strict comparison for default value keys in Javascript fails (even though the yaml file has the correct keys, aka integers set).
The method should only change strings.
Updated by Gerrit Code Review over 5 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/58900
Updated by Gerrit Code Review over 5 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/58900
Updated by Gerrit Code Review over 5 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/58900
Updated by Gerrit Code Review over 5 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/58900
Updated by Gerrit Code Review over 5 years ago
Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/58900
Updated by Gerrit Code Review over 5 years ago
Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/58900
Updated by Gerrit Code Review over 5 years ago
Patch set 7 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/58900
Updated by Gerrit Code Review over 5 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/58914
Updated by Wolfgang Klinger over 5 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset a295c186584918b12fc4b29e12f8bcf6a126af65.
Updated by Benni Mack almost 5 years ago
- Status changed from Resolved to Closed