Bug #82051

Error in EXT:form documentation regarding deletion of YAML configuration

Added by Peter Kraume almost 4 years ago. Updated over 2 years ago.

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

100%

Estimated time:
TYPO3 Version:
8
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

#1

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

#2

Updated by Bjoern Jacob almost 4 years ago

  • Sprint Focus set to Remote Sprint
#3

Updated by Ralf Zimmermann almost 4 years ago

  • Assignee set to Ralf Zimmermann
#4

Updated by Peter Kraume almost 4 years 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 almost 4 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.

#6

Updated by Peter Kraume over 3 years ago

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

#7

Updated by Bjoern Jacob over 3 years ago

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

Updated by Bjoern Jacob over 3 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.

#9

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

#10

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

#11

Updated by Peter Kraume over 3 years ago

Thx for taking care!

#12

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

#13

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

#14

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

#15

Updated by Susanne Moog over 3 years ago

  • Target version set to 8.7.10
#16

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

#17

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

#18

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

#19

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

#20

Updated by Ralf Zimmermann over 3 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
#21

Updated by Oliver Hader over 3 years ago

  • Status changed from Resolved to Under Review
#22

Updated by Ralf Zimmermann over 3 years ago

  • Status changed from Under Review to Resolved
#23

Updated by Bjoern Jacob over 3 years ago

  • Sprint Focus deleted (Remote Sprint)
#24

Updated by Ralf Zimmermann over 3 years ago

  • Related to deleted (Feature #82089: EXT:form Support "imports" in form configuration)
#25

Updated by Benni Mack over 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF