[BUGFIX] Avoid CKEditor RequireJS information loaded in FE 87/51987/2
authorBenni Mack <benni@typo3.org>
Thu, 9 Mar 2017 09:16:16 +0000 (10:16 +0100)
committerAndreas Fernandez <typo3@scripting-base.de>
Fri, 10 Mar 2017 12:51:51 +0000 (13:51 +0100)
The CKEditor default configuration for RequireJS path
resolving is only relevant in BE, Frontend Page Rendering
needs to do that separately for now, to avoid
loading of too many unnecessary config files if only
requireJS should be used in the Frontend.

The patch changes the functionality to only load
RequireJS in Backend (and not in CLI mode).

Resolves: #80183
Releases: master
Change-Id: I035ddb6bc83ebfb2a8f45de71fadd8f0e0b2d0a3
Reviewed-on: https://review.typo3.org/51987
Reviewed-by: Frank N├Ągler <frank.naegler@typo3.org>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Tymoteusz Motylewski <t.motylewski@gmail.com>
Reviewed-by: Mathias Brodala <mbrodala@pagemachine.de>
Reviewed-by: Christer V <cvi@systime.dk>
Tested-by: Christer V <cvi@systime.dk>
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Tested-by: Andreas Fernandez <typo3@scripting-base.de>
typo3/sysext/rte_ckeditor/ext_localconf.php

index b7e1151..324b8b0 100644 (file)
@@ -8,7 +8,7 @@ $GLOBALS['TYPO3_CONF_VARS']['SYS']['formEngine']['nodeResolver'][1480314091] = [
     'class' => \TYPO3\CMS\RteCKEditor\Form\Resolver\RichTextNodeResolver::class,
 ];
 
-if ((TYPO3_REQUESTTYPE & TYPO3_REQUESTTYPE_CLI) === 0) {
+if (TYPO3_MODE === 'BE' && !(TYPO3_REQUESTTYPE & TYPO3_REQUESTTYPE_CLI)) {
     \TYPO3\CMS\Core\Utility\GeneralUtility::makeInstance(
         \TYPO3\CMS\Core\Page\PageRenderer::class
     )->addRequireJsConfiguration([