Bug #102064
closed
TypoScript module does not report errors
100%
Description
Using an invalid import in TS does not cause any integrator feedback.
I would expect that either "Active TypoScript" or "Included TypoScript" would warn me about invalid TS.
Example of invalid import (path traversal):<INCLUDE_TYPOSCRIPT: source="DIR:../global/Setup/">
Updated by Markus Klein 7 months ago
- Related to Bug #94695: Template Analyzer stoped Highlighting syntax errors added
Updated by Markus Klein 7 months ago
- Related to Bug #102065: Invalid TypoScript import causes the complete file to be ignored added
Updated by Markus Klein 7 months ago
https://github.com/TYPO3/typo3/blob/main/typo3/sysext/core/Classes/TypoScript/IncludeTree/TreeFromLineStreamBuilder.php#L437
should probably trigger the error
Updated by Christian Kuhn 7 months ago
- Related to Feature #97816: New TypoScript parser added
Updated by Christian Kuhn 7 months ago
- Related to Feature #100218: Basic TypoScript syntax scanner added
Updated by Christian Kuhn 7 months ago
IncludeTreeSyntaxScannerVisitor :) See #100218. I'll try to implement something.
Updated by Gerrit Code Review 7 months 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/+/81352
Updated by Gerrit Code Review 7 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/+/81352
Updated by Christian Kuhn 7 months ago
- Related to Task #102102: Refactor TS IncludeTree Import detail added
Updated by Christian Kuhn 7 months ago
- Related to Task #102103: Refactor TS IncludeTree condition detail added
Updated by Gerrit Code Review 7 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/+/81352
Updated by Gerrit Code Review 7 months ago
Patch set 4 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/+/81352
Updated by Gerrit Code Review 7 months ago
Patch set 5 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/+/81352
Updated by Gerrit Code Review 7 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/+/81358
Updated by Christian Kuhn 7 months ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 0e64ea58bd0faab0cc77725d711acf5ee7466d28.
Updated by Christian Kuhn 6 months ago
- Related to Bug #102191: Missing error messages for failed imports in TypoScript module added