Project

General

Profile

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:
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

Actions
Copy link
 #1

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
 #2

Updated by Björn Jacob over 6 years ago

  • Sprint Focus set to Remote Sprint
Actions
Copy link
 #3

Updated by Ralf Zimmermann over 6 years ago

  • Assignee set to Ralf Zimmermann
Actions
Copy link
 #4

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
 #5

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
 #6

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
 #7

Updated by Björn Jacob over 6 years ago

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

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
 #9

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

Updated by Ralf Zimmermann about 6 years ago

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

Updated by Oliver Hader about 6 years ago

  • Status changed from Resolved to Under Review
Actions
Copy link
 #22

Updated by Ralf Zimmermann about 6 years ago

  • Status changed from Under Review to Resolved
Actions
Copy link
 #23

Updated by Björn Jacob about 6 years ago

  • Sprint Focus deleted (Remote Sprint)
Actions
Copy link
 #24

Updated by Ralf Zimmermann about 6 years ago

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

Updated by Benni Mack over 5 years ago

  • Status changed from Resolved to Closed
Actions
Copy link

Also available in: Atom PDF