Bug #70956
closedStory #69617: FormEngine bugs
Behavior of keepItems changed
100%
Description
I just noticed that the behavior of the keepItems setting in page TSconfig changed from TYPO3 6.2 to TYPO3 7.x (tested in 7.5).
Example 1:
TCEFORM.tt_content { header_layout { keepItems = addItems { 1 = Ebene 1 (nur einmal auf der Seite verwenden) 0 = Ebene 2 12 = Ebene 2 (Akzentfarbe) 3 = Ebene 3 13 = Ebene 3 (Akzentfarbe) 4 = Ebene 4 14 = Ebene 4 (Akzentfarbe) 100 = Verborgen } } }
- TYPO3 6.2: Removes all existing items from the select and adds all new ones
- TYPO3 7.5: Adds all new ones and keeps all existing items
Example 2:
TCEFORM.tt_content { header_layout { keepItems = 1 addItems { 1 = Ebene 1 (nur einmal auf der Seite verwenden) 0 = Ebene 2 12 = Ebene 2 (Akzentfarbe) 3 = Ebene 3 13 = Ebene 3 (Akzentfarbe) 4 = Ebene 4 14 = Ebene 4 (Akzentfarbe) 100 = Verborgen } } }
- TYPO3 6.2: Removes all existing items except for 1 from the select and adds all new ones
- TYPO3 7.5: Adds all new ones and then removes both existing and new ones except for both 1s
Example 3:
TCEFORM.tt_content { header_layout { removeItems = 0,1,2,3,4,5,6,100 addItems { 1 = Ebene 1 (nur einmal auf der Seite verwenden) 0 = Ebene 2 12 = Ebene 2 (Akzentfarbe) 3 = Ebene 3 13 = Ebene 3 (Akzentfarbe) 4 = Ebene 4 14 = Ebene 4 (Akzentfarbe) 100 = Verborgen } } }
- TYPO3 6.2: Adds all new ones and then removes both existing and new ones except for 12, 13 and 14
- TYPO3 7.5: the same
In my opinion the way TYPO3 6.2 interpreted the keepItems setting was more flexible because it allowed me to rename and reorder items without messing with the css_styled_content TypoScript.
Updated by Christian Kuhn about 9 years ago
- Category changed from TypoScript to FormEngine aka TCEforms
Updated by Morton Jonuschat about 9 years ago
- Status changed from New to Accepted
- Assignee set to Morton Jonuschat
Updated by Gerrit Code Review about 9 years ago
- Status changed from Accepted 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/44570
Updated by Gerrit Code Review about 9 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/44570
Updated by Gerrit Code Review about 9 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/44570
Updated by Morton Jonuschat about 9 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 2da8b474a37f987706b511250299ccc887c9f80c.
Updated by Riccardo De Contardi about 7 years ago
- Status changed from Resolved to Closed
Updated by Helmut Hummel about 7 years ago
- Related to Bug #82980: Handling of mixed string and 0 broken for keepItems and removeItems page ts added