Project

General

Profile

Actions

Bug #88858

open

Richtext configuration set in a flexform gets overwritten by RTE.default.preset

Added by Jan Kornblum over 4 years ago. Updated over 3 years ago.

Status:
Accepted
Priority:
Should have
Category:
RTE (rtehtmlarea + ckeditor)
Target version:
-
Start date:
2019-07-30
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
8
PHP Version:
7.0
Tags:
rte
Complexity:
Is Regression:
Sprint Focus:

Description

A richtext configuration is set in a flexform (attached to pi_flexform) field:

<settings.anyRichtextField>
    <exclude>1</exclude>
    <label>Custom RTE enabled field</label>
    <config>
        <type>text</type>
        <rows>5</rows>
        <cols>30</cols>
        <eval>trim,required</eval>
        <enableRichtext>1</enableRichtext>
        <richtextConfiguration>custom_config</richtextConfiguration>
    </config>
</settings.anyRichtextField>

As long as no global RTE preset is set anywhere else, the configuration is loaded fine. But as soon as...

RTE.default.preset = minimal

...is set anywhere else, the per-flexform-field rte configuration gets ignored. Also, it is not possible to explicitly set the preset by tsconfig like this additionally...

TCEFORM.[tableName].[fieldName].[dataStructureKey].[flexSheet].[flexFieldName].[propertyName] 

...due to limitations of the "config" section mentioned here: https://docs.typo3.org/m/typo3/reference-tsconfig/master/en-us/PageTsconfig/TceForm.html


Related issues 2 (1 open1 closed)

Related to TYPO3 Core - Bug #88655: richtextConfiguration, set via TCA for a text field is ignoredClosed2019-06-29

Actions
Related to TYPO3 Core - Feature #91690: Override Flexform via Page TsConfigNew2020-06-22

Actions
Actions #1

Updated by Jan Kornblum over 4 years ago

  • Description updated (diff)
  • Category changed from FormEngine aka TCEforms to RTE (rtehtmlarea + ckeditor)
Actions #2

Updated by Daniel Kirsch over 4 years ago

  • Tags set to rte

TYPO3 v9 also affected.

Easy example:

RTE {
    default.preset = full
    config {
        tx_news_domain_model_news.preset = minimal
        tt_content.type.textmedia.preset = default
    }
}
Actions #3

Updated by Benni Mack over 4 years ago

Daniel Kirsch, the syntax you provided is not correct.

This is the correct syntax for regular fields:


RTE {
    default.preset = full
    config {
       # override for one field
        tt_content.bodytext.preset = normal
      # override for one field of a specific type
        tt_content.bodytext.types.textmedia.preset = minimal
    }
}

Actions #5

Updated by Daniel Siepmann over 4 years ago

  • Assignee set to Daniel Siepmann
Actions #6

Updated by Gerrit Code Review over 4 years 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/c/Packages/TYPO3.CMS/+/62251

Actions #7

Updated by Gerrit Code Review over 4 years ago

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

Actions #8

Updated by Daniel Siepmann over 4 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
Actions #9

Updated by Benni Mack over 4 years ago

  • Related to Bug #88655: richtextConfiguration, set via TCA for a text field is ignored added
Actions #10

Updated by Benni Mack over 4 years ago

  • Status changed from Resolved to Accepted

I keep this open as this is currently not supported to override FlexForms via PageTS.

Actions #11

Updated by Jan Kornblum over 4 years ago

Thanks for fixing! So the first issue is fixed (custom flexform rte config won't get overwritten by global tsconfig any longer), but the second issue (overwrite rte config for flexform field by tsconfig) is still open?

Actions #12

Updated by Ioulia Kondratovitch over 4 years ago

I have found 2 mistakes and left my comments in the patch (can't set review = -1, since it's megred).
How to proceed? Are the comments enough or should I create a new ticket?

Daniel Siepmann wrote:

Applied in changeset e89e1a5a1c08e4878e9182814d98895d75545ebf.

Actions #13

Updated by Daniel Siepmann about 4 years ago

@Ioulia Kondratovitch could you just provide another patch with your proposed changes?

Actions #14

Updated by Benni Mack over 3 years ago

Sorry, but someone(tm) need to open up a new issue for the left-over changes, so we can provide a new patch

Actions #15

Updated by Jan Kornblum over 3 years ago

Opened a new issue for this: https://forge.typo3.org/issues/91690

Actions #16

Updated by Jan Kornblum over 3 years ago

  • Related to Feature #91690: Override Flexform via Page TsConfig added
Actions #17

Updated by Jan Kornblum over 3 years ago

  • Related to Feature #91690: Override Flexform via Page TsConfig added
Actions #18

Updated by Jan Kornblum over 3 years ago

  • Related to deleted (Feature #91690: Override Flexform via Page TsConfig)
Actions

Also available in: Atom PDF