Bug #82051
closedError in EXT:form documentation regarding deletion of YAML configuration
Added by Peter Kraume about 7 years ago. Updated almost 6 years ago.
100%
Description
There is an error in the EXT:form documentation:
https://docs.typo3.org/typo3cms/drafts/code.tritum.de/TYPO3.CMS/Form_Documentation/Concepts/Configuration/Index.html?highlight=null#inheritances
Finally, all configuration entries with a value of `null` are deleted.
It’s not `null` but `__UNSET`.
This is done in `TYPO3\CMS\Core\Utility\ArrayUtility::mergeRecursiveWithOverrule()` line 613:
https://github.com/TYPO3/TYPO3.CMS/blob/TYPO3_8-7/typo3/sysext/core/Classes/Utility/ArrayUtility.php#L613
Updated by Gerrit Code Review about 7 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/53647
Updated by Peter Kraume almost 7 years ago
So there was the EXT:form code sprint last week. Any progress on this issue? I assumed it to be a nobrainer!
Updated by Ralf Zimmermann almost 7 years ago
Its more than change the documentation. We have to work on the code because there are some cases on which only "_UNSET" works but this is not the intended behavior. I am working on this and i will provide a patchset within the next days.
Updated by Peter Kraume almost 7 years ago
Any news on this issue? Maybe it makes sense to merge the patch to 8.7 branch only?
Updated by Björn Jacob over 6 years ago
- Related to Feature #82089: EXT:form Support "imports" in form configuration added
Updated by Björn Jacob over 6 years ago
For v9 we are improving the situation with #82089.
It is very difficult to solve for v8. For this version, we would need a proper documentation. But the problem is hard to tackle/ describe. It depends on the stuff you are doing -> different situation -> different solution.
Updated by Ralf Zimmermann over 6 years ago
- Status changed from Under Review to Needs Feedback
- Assignee changed from Ralf Zimmermann to Peter Kraume
Hi Peter,
can you please tell me an example configuration where the error can be reproduced?
Updated by Ralf Zimmermann over 6 years ago
- Status changed from Needs Feedback to In Progress
- Assignee changed from Peter Kraume to Ralf Zimmermann
I found it myself again ;) Thank you anyway.
Updated by Gerrit Code Review over 6 years ago
- Status changed from In Progress 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/55487
Updated by Gerrit Code Review over 6 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/55487
Updated by Gerrit Code Review over 6 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/55487
Updated by Gerrit Code Review over 6 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/55487
Updated by Gerrit Code Review over 6 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/55487
Updated by Gerrit Code Review over 6 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/55487
Updated by Gerrit Code Review over 6 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/55565
Updated by Ralf Zimmermann over 6 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 77899870b671e2f625f122b67e265a73655faffc.
Updated by Oliver Hader over 6 years ago
- Status changed from Resolved to Under Review
Updated by Ralf Zimmermann over 6 years ago
- Status changed from Under Review to Resolved
Applied in changeset a509d71bd5da02cda3c3e9b812affed70792bf01.
Updated by Ralf Zimmermann over 6 years ago
- Related to deleted (Feature #82089: EXT:form Support "imports" in form configuration)
Updated by Benni Mack almost 6 years ago
- Status changed from Resolved to Closed