Bug #102760
closed
ext_conf_template.txt is parsed inconsistently, leading to missing configuration fields
100%
Description
If I have an extension with the following ext_conf_template.txt:
# cat=foo//0; type=boolean; label=some config bar = 0 # cat=foo//1; type=boolean; label=other config baz = 0
only the second configuration option is shown in the Backend. However, if I replace the double slashes with single ones, like so:
# cat=foo/0; type=boolean; label=some config bar = 0 # cat=foo/1; type=boolean; label=other config baz = 0
both options are shown. As the documentation defines ext_conf_template.txt's syntax to be similar to the one used for the TS constants editor, I think this is a bug. The examples above are comparable to those in the documentation (https://docs.typo3.org/m/typo3/reference-typoscript/main/en-us/UsingSetting/TheConstantEditor.html#id3).
Updated by Christian Kuhn 11 months ago
- Related to Feature #97816: New TypoScript parser added
Updated by Georg Ringer 5 months ago
ยท Edited
still buggy in 12
with
# cat=Backend; type=string; label=LLL:EXT:backend/Resources/Private/Language/locallang.xlf:config.backendFavicon backendFavicon = # cat=foo//0; type=boolean; label=some config bar = 0 # cat=foo//1; type=boolean; label=other config baz = 0
all 3 settings are shown.
if 1st one is removed, only setting baz is shown
Updated by Georg Ringer 5 months ago
- Status changed from New to Accepted
- Assignee set to Christian Kuhn
Updated by Gerrit Code Review 5 months ago
- Status changed from Accepted to Under Review
Patch set 1 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/84912
Updated by Garvin Hicking 5 months ago
This also happened in v13/main.
Note that the problem is not really caused by using double slash notation or having another element come before, but the "//0" leading to a falsy evaluation.
A single slash would just hide this problem because then automatic indexing would take place. As well as placing any other element before it, which would also affect the ordering.
Please check out the patch to see if that helps.
Updated by Gerrit Code Review 5 months ago
Patch set 2 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/84912
Updated by Gerrit Code Review 5 months ago
Patch set 3 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/84912
Updated by Gerrit Code Review 5 months ago
Patch set 1 for branch 12.4 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/85027
Updated by Garvin Hicking 5 months ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 3efe471df41bd099b3ced2443c9a1a7aa44bf22e.
Updated by