Bug #96503
closedCommented out TypoScript imports still resolved.
100%
Description
I have a bunch of imports in my setup.typoscript that for some reason mess up my system and made the backend freeze due to a circular include.
It turns out that an import within a block-comment is still executed:
/* @import 'EXT:fluid_styled_content/Configuration/TypoScript/setup.typoscript' ... */
/* ### @import 'EXT:fluid_styled_content/Configuration/TypoScript/setup.typoscript' begin ### # Include setup ### <INCLUDE_TYPOSCRIPT: source="FILE:EXT:fluid_styled_content/Configuration/TypoScript/Helper/ContentElement.typoscript"> BEGIN: # Default configuration for content elements which are using FLUIDTEMPLATE directly lib.contentElement > lib.contentElement = FLUIDTEMPLATE ...
So a circular includes within a block-comment still makes the backend freeze.
imports within single line comments are however not executed:
# @import 'EXT:fluid_styled_content/Configuration/TypoScript/setup.typoscript'
If this is not a bug but a feature we would need to document it better...
Updated by Gerrit Code Review almost 3 years ago
- Status changed from New 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/+/73019
Updated by Christian Kuhn over 2 years ago
- Related to Feature #97816: New TypoScript parser added
Updated by Gerrit Code Review over 2 years ago
Patch set 7 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/+/74987
Updated by Gerrit Code Review over 2 years ago
Patch set 8 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/+/74987
Updated by Gerrit Code Review over 2 years ago
Patch set 9 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/+/74987
Updated by Gerrit Code Review over 2 years ago
Patch set 10 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/+/74987
Updated by Christian Kuhn over 2 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 09c61884bf2f15b0f209f8b177268620fc77cc8a.