Bug #82051: Error in EXT:form documentation regarding deletion of YAML configuration - TYPO3 Core - TYPO3 Forge

Custom queries

12 LTS
Accessibility
Dashboard issues
Easy tasks
Extbase Issues
FAL Issues
FAL Sprint 2023
Hierarchical Issues
Issues reported for v11
JavaScript issues
JavaScript Issues (w/o Epics/Stories)
Most wanted Bugfixes
Most wanted Features
My open issues
New & Unassigned
No feedback within 90 days
Open Bugs
Open issues of 10
Page Tree Regressions after 2020-07-28
Recently modified
Regressions
Stabilization Issues

Actions

Copy link

Bug #82051

closed

Error in EXT:form documentation regarding deletion of YAML configuration

Added by Peter Kraume over 6 years ago. Updated over 5 years ago.

Status:

Closed

Priority:

Should have

Assignee:

Ralf Zimmermann

Category:

Form Framework

Target version:

8.7.10

Start date:

2017-08-05

Due date:

% Done:

100%

Estimated time:

TYPO3 Version:

PHP Version:

Tags:

Complexity:

Is Regression:

Sprint Focus:

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

History
Notes
Property changes
Associated revisions

Actions

Copy link

Updated by Gerrit Code Review over 6 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

Actions

Copy link

Updated by Björn Jacob over 6 years ago

Sprint Focus set to Remote Sprint

Actions

Copy link

Updated by Ralf Zimmermann over 6 years ago

Assignee set to Ralf Zimmermann

Actions

Copy link

Updated by Peter Kraume over 6 years ago

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

Actions

Copy link

Updated by Ralf Zimmermann over 6 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.

Actions

Copy link

Updated by Peter Kraume over 6 years ago

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

Actions

Copy link

Updated by Björn Jacob over 6 years ago

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

Actions

Copy link

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.

Actions

Copy link

Updated by Ralf Zimmermann about 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?

Actions

Copy link

#10

Updated by Ralf Zimmermann about 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.

Actions

Copy link

#11

Updated by Peter Kraume about 6 years ago

Thx for taking care!

Actions

Copy link

#12

Updated by Gerrit Code Review about 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

Actions

Copy link

#13

Updated by Gerrit Code Review about 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

Actions

Copy link

#14

Updated by Gerrit Code Review about 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

Actions

Copy link

#15

Updated by Susanne Moog about 6 years ago

Target version set to 8.7.10

Actions

Copy link

#16

Updated by Gerrit Code Review about 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

Actions

Copy link

#17

Updated by Gerrit Code Review about 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

Actions

Copy link

#18

Updated by Gerrit Code Review about 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

Actions

Copy link

#19

Updated by Gerrit Code Review about 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

Actions

Copy link

#20