Bug #99868: INCLUDE_TYPOSCRIPT - TYPO3 Core - TYPO3 Forge

Actions

Copy link

Bug #99868

closed

INCLUDE_TYPOSCRIPT

Added by Christian Racan almost 2 years ago. Updated 5 months ago.

Status:

Closed

Priority:

Should have

Assignee:

Category:

TypoScript

Target version:

Start date:

2023-02-07

Due date:

% Done:

100%

Estimated time:

TYPO3 Version:

PHP Version:

8.1

Tags:

Complexity:

Is Regression:

Sprint Focus:

Description

Hello TYPO3 core team,
when I want to import a .tsconfig with the old method into my sitepackage in the ext_localconf.php file, this is not possible - only the new method works:

\TYPO3\CMS\Core\Utility\ExtensionManagementUtility::addUserTSConfig('@import "EXT:bpsitepackage/Configuration/TsConfig/Page/All.tsconfig"');

This one doesn't work:
\TYPO3\CMS\Core\Utility\ExtensionManagementUtility::addPageTSConfig('<INCLUDE_TYPOSCRIPT: source="FILE:EXT:bpsitepackage/Configuration/TsConfig/Page/All.tsconfig">');

tested with TYPO3 v12.2

Actions

Copy link

Updated by Stefan Bürk almost 2 years ago

Works pretty fine for me on current main (not far away from 12.2)

I used the "doesn't work copy&paste from you, created the corresponding file in my extension and just replaced the extension key ..

Ensure you have cleared all caches after changing it, and you may have a look into the PageTsConfig module (extracted) -> Active PageTsConfig

Instead of using the addPageTsConfig() call, you may be interessted in #96614 Feature: #96614 - Automatic inclusion of page TSconfig of extensions

Actions

Copy link

Updated by Gerrit Code Review almost 2 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/+/77775

Actions

Copy link

Updated by Gerrit Code Review almost 2 years 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/+/77775

Actions

Copy link

Updated by Stefan Bürk almost 2 years ago

added patch with functional tests for this

Actions

Copy link

Updated by Christian Racan almost 2 years ago

Today I tested it again with ddev restart and now both spellings work.
Please excuse the inconvenience caused.

Actions

Copy link

Updated by Gerrit Code Review almost 2 years 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/+/77775

Actions

Copy link

Updated by Gerrit Code Review over 1 year 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/+/77775

Actions

Copy link

Updated by Gerrit Code Review over 1 year 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/+/77775

Actions

Copy link

Updated by Gerrit Code Review over 1 year ago

Patch set 6 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/+/77775

Actions

Copy link

#10

Updated by Gerrit Code Review over 1 year 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/+/77775

Actions

Copy link

#11

Updated by Gerrit Code Review over 1 year 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/+/79814

Actions

Copy link

#12