Bug #81263

CKEditor: Configuration of extraPlugins is overwritten

Added by Marcus Schwemer 2 months ago. Updated about 2 months ago.

Status:
Under Review
Priority:
Should have
Assignee:
-
Category:
RTE (rtehtmlarea + ckeditor)
Target version:
-
Start date:
2017-05-18
Due date:
% Done:

0%

TYPO3 Version:
8
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

If you add an extra plugin and configure it in the section

editor:
  config:
    <pluginname>: 
        <pluginoption>:

the configuration of the plugin in a preset is not taken into account.

Reason is that currently the configuration of the plugin overwrites the values of the output of "prepareConfigurationForEditor".

Solution is to check, whether there are values in both array. If so merge or overwrite the configuration for the editor, depending on the combination.

I will provide a patch and a extension on github to test it.

History

#1 Updated by Gerrit Code Review 2 months ago

  • Status changed from New to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/52857

#2 Updated by Gerrit Code Review 2 months ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/52857

#3 Updated by Marcus Schwemer 2 months ago

You can find the example plugin to experiment with here:

https://github.com/mschwemer/rte_ckeditor_wordcount

#4 Updated by Marcus Schwemer 2 months ago

  • Subject changed from Configuration of extraPlugins is overwritten to CKEditor: Configuration of extraPlugins is overwritten
  • Category set to RTE (rtehtmlarea + ckeditor)

Here is the extension for testing: https://github.com/mschwemer/rte_ckeditor_wordcount

#5 Updated by Gerrit Code Review about 2 months ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/52857

Also available in: Atom PDF