Task #71607
closed
Story #69712: Further FormEngine development
FormEngine: Flex section deletion
100%
Description
Deleting a section in flex currently does:
- add a t3js-flex-section-deleted to the section div
- the fields itself are kept and are probably (?) transmitted to dataHandler
This leads to funny fixes like https://review.typo3.org/#/c/44732/1 where validation did validate the "deleted" fields ...
This is also ugly: dataHandler still receives all field data, but probably just skips processing after it found out that the section itself is deleted.
Why can't delete just a) set the hidden "i'm a deleted section" marker and b) delete the single field dom altogether?
Updated by Riccardo De Contardi over 8 years ago
- Category set to FormEngine aka TCEforms
Updated by
Updated by Christian Kuhn almost 8 years ago
- Parent task changed from #69617 to #69712
Updated by Gerrit Code Review about 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/c/Packages/TYPO3.CMS/+/63830
Updated by Gerrit Code Review about 4 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/c/Packages/TYPO3.CMS/+/63830
Updated by Gerrit Code Review about 4 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/c/Packages/TYPO3.CMS/+/63830
Updated by Gerrit Code Review about 4 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/c/Packages/TYPO3.CMS/+/63830
Updated by Gerrit Code Review about 4 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/c/Packages/TYPO3.CMS/+/63830
Updated by Andreas Fernandez about 4 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 18c5d9e54cde5d5c35c274a74366118ebbf59143.
Updated by Benni Mack almost 4 years ago
- Status changed from Resolved to Closed