Project

General

Profile

Actions

Bug #97499

open

Wrong notation for value of allowedAttribs in Processing.yaml

Added by Eric Harrer about 2 years ago. Updated about 1 year ago.

Status:
Under Review
Priority:
Should have
Assignee:
-
Category:
RTE (rtehtmlarea + ckeditor)
Target version:
-
Start date:
2022-04-28
Due date:
% Done:

0%

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

Description

Contrary to the default configuration in EXT:rte_ckeditor/Configuration/RTE/Processing.yaml, the class attribute is removed within hr-tags in RTE fields when saving the record.

This is due to incorrect spelling of the value of property allowedAttribs. According to HtmlParser::HTMLcleaner() the value needs to be a commalist of attributes (string value) instead of an array as currently defined in the Processing.yaml file.


Related issues 3 (2 open1 closed)

Related to TYPO3 Core - Bug #96485: CKEditor : allowAttribs in YAML not taken into account / attributes are removed at saveUnder Review2022-01-07

Actions
Related to TYPO3 Core - Bug #92943: RTE ckeditor does not respect YAML configurationNew2020-11-27

Actions
Has duplicate TYPO3 Core - Task #97478: [BUGFIX] Replace array notation with string for value of allowedAttribsClosed2022-04-26

Actions
Actions #1

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

Actions #2

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

Actions #3

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

Actions #4

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

Actions #5

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

Actions #6

Updated by Eric Harrer about 2 years ago

  • Has duplicate Task #97478: [BUGFIX] Replace array notation with string for value of allowedAttribs added
Actions #7

Updated by Eric Harrer about 2 years ago

  • Related to Bug #96485: CKEditor : allowAttribs in YAML not taken into account / attributes are removed at save added
Actions #8

Updated by Eric Harrer about 2 years ago

  • Related to Bug #88303: lang-attribute for CKEditor not saved added
Actions #9

Updated by Eric Harrer about 2 years ago

  • Related to Bug #92943: RTE ckeditor does not respect YAML configuration added
Actions #10

Updated by Eric Harrer about 2 years ago

  • Related to deleted (Bug #88303: lang-attribute for CKEditor not saved)
Actions #11

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

Actions #12

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

Actions #13

Updated by Gerrit Code Review about 2 years ago

Patch set 8 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/+/74473

Actions #14

Updated by Gerrit Code Review about 2 years ago

Patch set 9 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/+/74473

Actions #15

Updated by Gerrit Code Review about 2 years ago

Patch set 10 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/+/74473

Actions #16

Updated by Gerrit Code Review about 2 years ago

Patch set 11 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/+/74473

Actions #17

Updated by Riccardo De Contardi almost 2 years ago

Are these related? #87314 , #87400 ?

Actions #18

Updated by Gerrit Code Review about 1 year ago

Patch set 12 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/+/74473

Actions #19

Updated by Gerrit Code Review about 1 year ago

Patch set 13 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/+/74473

Actions

Also available in: Atom PDF