Bug #71609

RTE Options dont work

Added by Rainer Becker over 3 years ago. Updated 3 months ago.

Status:
Needs Feedback
Priority:
Should have
Assignee:
-
Category:
FormEngine aka TCEforms
Start date:
2015-11-16
Due date:
% Done:

0%

TYPO3 Version:
7
PHP Version:
5.5
Tags:
Complexity:
Is Regression:
No
Sprint Focus:

Description

It seems that the page TS options

RTE.default.buttons.link {
targetSelector.disabled = 1
popupSelector.disabled = 1
options.removeItems = folder
}

don’t have any effect anymore.


Related issues

Related to TYPO3 Core - Bug #71248: TSconfig: RTE.classesAnchor seems not to be working anymore Closed 2015-11-02
Related to TYPO3 Core - Bug #71978: $RTEProperties is never initialized Closed 2015-11-30

History

#1 Updated by Martin R. Krause over 3 years ago

  • Target version set to 7 LTS

In fact the whole TYPO3\CMS\Rtehtmlarea\Controller\BrowseLinksController class is not working properly. I debugged the issues and it is a miracle that the LinkBrowser for the RTE works at all.

In TYPO3 7.5 configuring classes for links was still possible, now the link property for classes does not show up at all.

The class BrowseLinksController that inherits from AbstractLinkBrowserController does not load the configuration. The order is wrong (e.g. $this->RTEProperties is not set at all, should be $RTEsetup['properties'] in method initVariables()). The $this->currentLinkHandlerId is never set, therefor button classes configuration could not be found etc.

Proposal for solution

The RTE should not have link configurations on its own anyway. It always was very tedious to configure both link handler (for RTE and TCA Link Wizard). Now that we have the TCEMAIN.linkhandler configuration in PageTS, this should also be the place to configure link options (classes, target, etc.). Getting rid of the cryptic RTE configuration for links would simplify integration and the development of the LinkHandler classes. In the RTE no <a /> should be added, just TYPO3 <link /> tags that have a syntax that is consistent with typolink.

#2 Updated by Andreas Allacher over 3 years ago

Added #71248 as related.
I don't think it is a duplicate as it is related to different properties but maybe the same solution works (not tested - maybe someone wants to test, if so this ticket could be added in Resolves list)

Regarding currentLinkHandlerId there is another property for that (displayLinkHandlerId) which should be used instead, see also attached patch for related ticket.

#3 Updated by Eric Chavaillaz over 3 years ago

Hi,

The problem is described here : https://forge.typo3.org/issues/71978

Thanks

#4 Updated by Stephan Brun over 3 years ago

targetSelector.disabled works with 7.6.1

Could not test the other two options.

#5 Updated by Riccardo De Contardi over 3 years ago

  • Target version changed from 7 LTS to Candidate for patchlevel

#6 Updated by Riccardo De Contardi about 3 years ago

These are my findings:

6.2.22

RTE.default.buttons.link.targetSelector.disabled = 1 WORKS
RTE.default.buttons.link.popupSelector.disabled = 1 DOES NOT WORK (tested with RTE.default.buttons.link.targetSelector.disabled = 0)
RTE.default.buttons.link.options.removeItems = folder WORKS

7.6.6

RTE.default.buttons.link.targetSelector.disabled = 1 WORKS
RTE.default.buttons.link.popupSelector.disabled = 1 DOES NOT WORK (tested with RTE.default.buttons.link.targetSelector.disabled = 0)
RTE.default.buttons.link.options.removeItems = folder WORKS

8.2-dev

RTE.default.buttons.link.targetSelector.disabled = 1 WORKS
RTE.default.buttons.link.popupSelector.disabled = 1 DOES NOT WORK (tested with RTE.default.buttons.link.targetSelector.disabled = 0)
RTE.default.buttons.link.options.removeItems = folder WORKS

#7 Updated by Christian Kuhn over 1 year ago

  • Assignee deleted (Christian Kuhn)

#8 Updated by Benni Mack 3 months ago

  • Related to Bug #71978: $RTEProperties is never initialized added

#9 Updated by Benni Mack 3 months ago

  • Status changed from New to Needs Feedback

Does this also occur with CKeditor?

Also available in: Atom PDF