Bug #82051

Error in EXT:form documentation regarding deletion of YAML configuration

Added by Peter Kraume 4 months ago. Updated 9 days ago.

Status:
Under Review
Priority:
Should have
Category:
Form Framework
Target version:
-
Start date:
2017-08-05
Due date:
% Done:

0%

TYPO3 Version:
8
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:
Remote Sprint

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


Related issues

Related to TYPO3 Core - Feature #82089: EXT:form Support "imports" in form configuration Under Review 2017-08-11

History

#1 Updated by Gerrit Code Review 4 months 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

#2 Updated by Bjoern Jacob 3 months ago

  • Sprint Focus set to Remote Sprint

#3 Updated by Ralf Zimmermann 3 months ago

  • Assignee set to Ralf Zimmermann

#4 Updated by Peter Kraume 3 months ago

So there was the EXT:form code sprint last week. Any progress on this issue? I assumed it to be a nobrainer!

#5 Updated by Ralf Zimmermann 3 months 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.

#6 Updated by Peter Kraume about 1 month ago

Any news on this issue? Maybe it makes sense to merge the patch to 8.7 branch only?

#7 Updated by Bjoern Jacob 9 days ago

  • Related to Feature #82089: EXT:form Support "imports" in form configuration added

#8 Updated by Bjoern Jacob 9 days 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.

Also available in: Atom PDF