Project

General

Profile

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 8 months ago. Updated 7 months ago.

Status:
Resolved
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:
12
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.

Actions
Copy link
 #1

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

Actions
Copy link
 #2

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

Actions
Copy link
 #3

Updated by Gerrit Code Review 7 months 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
 #4

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

Actions
Copy link
 #5

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

Actions
Copy link
 #6

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

Actions
Copy link
 #7

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

Actions
Copy link
 #8

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

Actions
Copy link
 #9

Updated by Anonymous 7 months ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
Actions
Copy link

Also available in: Atom PDF