Bug #101918: CKEditor5 migration for contentsCss with cache busting query string does not load the actual CSS file - TYPO3 Core - TYPO3 Forge

Actions

Copy link

Bug #101918

closed

Epic #99669: CKEditor5 Collection

Story #101905: CKEditor4 to CKEditor5 Migration Issues

CKEditor5 migration for contentsCss with cache busting query string does not load the actual CSS file

Added by Benjamin Franzke about 1 year ago. Updated 4 months ago.

Status:

Closed

Priority:

Should have

Assignee:

Category:

RTE (rtehtmlarea + ckeditor)

Target version:

Candidate for patchlevel

Start date:

2023-09-14

Due date:

% Done:

100%

Estimated time:

TYPO3 Version:

PHP Version:

Tags:

Complexity:

Is Regression:

Sprint Focus:

Description

With the following valid TYPO3 v11 ckeditor configuration in a v12 installation…

editor:
    config:
        contentsCss: "EXT:site/Resources/Public/Page/css/rte.css?v=4"

…the CSS file is not taken into account, but a PHP warning is emitted instead.

The happens because the URL is read from the filesystem (by the ScssProcessor) instead of being requested directly via web request, without stripping the query string:

PHP Warning
Core: Error handler (BE): PHP Warning: sha1_file(/var/www/html/typo3conf/ext/site/Resources/Public/Page/css/rte.css?v=4): Failed to open stream: No such file or directory in /var/www/html/typo3/sysext/rte_ckeditor/Classes/Form/Element/RichTextElement.php line 135

Note: A query string needed to be appended to RTE contentCss configuration in order to enforce browser cache busting on updates of the CSS file, when the default TYPO3 .htaccess configuration is used, which applies long-lived cache headers for resources by default.

Related issues 2 (1 open — 1 closed)

Actions

Copy link

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

Actions

Copy link

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

Actions

Copy link

Updated by Gerrit Code Review about 1 year ago

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/+/81295

Actions

Copy link

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

Actions

Copy link

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

Actions

Copy link

Updated by Gerrit Code Review about 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/+/81295

Actions

Copy link

Updated by Gerrit Code Review about 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/+/81295

Actions

Copy link

Updated by Gerrit Code Review about 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/+/81308

Actions

Copy link