Project

General

Profile

Actions
Copy link

Bug #102245

closed

Epic #99669: CKEditor5 Collection

CKEditor5 removePlugins configuration is not properly serialized as JSON array when some array items are migrated

Added by Benjamin Franzke 6 months ago. Updated 6 months ago.

Status:
Resolved
Priority:
Should have
Assignee:
Benjamin Franzke
Category:
RTE (rtehtmlarea + ckeditor)
Target version:
Candidate for patchlevel
Start date:
2023-10-24
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
12
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

Reported by Johannes Schlier:

Example configuration:

editor:
  config:
    removePlugins:
      - image
      - specialchar

"image" is migrated which leaves "specialchar", but the array is encoded to JSON as

{"1":"specialchar"}

Instead of
["specialchar"]

Resulting in a CKEditor5 runtime error because the array method some() is tried to be executed on an object:

VM2902 ckeditor5-core.js:17 Uncaught (in promise) TypeError: e.some is not a function
    at v (VM2902 ckeditor5-core.js:17:2271)
    at VM2902 ckeditor5-core.js:17:1146
    at Array.filter (<anonymous>)
    at x.init (VM2902 ckeditor5-core.js:17:1134)
    at w.initPlugins (VM2902 ckeditor5-core.js:37:2082)
    at VM2846 ckeditor5-editor-classic.js:14:636
    at new Promise (<anonymous>)
    at w.create (VM2846 ckeditor5-editor-classic.js:14:593)
    at CKEditor5Element.firstUpdated (VM2808 ckeditor5.js:13:3194)

Related issues 2 (0 open — 2 closed)

Related to TYPO3 Core - Task #102099: Unbundle CKEditor5ClosedBenjamin Franzke2023-01-12

Actions
Related to TYPO3 Core - Bug #102230: Former CKEditor definitions no longer work after updating to TYPO3 v12.4.7ResolvedBenjamin Franzke2023-10-23

Actions
Actions
Copy link
 #1

Updated by Benjamin Franzke 6 months ago

Actions
Copy link
 #2

Updated by Benjamin Franzke 6 months ago

  • Description updated (diff)
Actions
Copy link
 #3

Updated by Gerrit Code Review 6 months ago

  • Status changed from New to Under Review

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

Actions
Copy link
 #4

Updated by Gerrit Code Review 6 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/+/81552

Actions
Copy link
 #5

Updated by Benjamin Franzke 6 months ago

  • Description updated (diff)
Actions
Copy link
 #6

Updated by Jasmina Ließmann 6 months ago

  • Related to Bug #102230: Former CKEditor definitions no longer work after updating to TYPO3 v12.4.7 added
Actions
Copy link
 #7

Updated by Gerrit Code Review 6 months ago

Patch set 2 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/+/81538

Actions
Copy link
 #8

Updated by Gerrit Code Review 6 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/+/81552

Actions
Copy link
 #9

Updated by Gerrit Code Review 6 months ago

Patch set 3 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/+/81538

Actions
Copy link
 #10

Updated by Anonymous 6 months ago

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

Also available in: Atom PDF