[BUGFIX] Remove wrong references in Install Tool ViewHelper 97/42097/2
authorBenjamin Mack <benni@typo3.org>
Wed, 29 Jul 2015 15:16:00 +0000 (17:16 +0200)
committerWouter Wolters <typo3@wouterwolters.nl>
Wed, 29 Jul 2015 15:30:06 +0000 (17:30 +0200)
The ContainerViewHelper from the install tool has the option
to enable the click menu, which references to a file which is
now an AMD module (= needs to be loaded via RequireJS).

Additionally, the ExtJS logic is removed from the install tool
completely as well, as it is unused.

Releases: master
Resolves: #68628
Change-Id: I1aab35c49e98bd2fb42b16f5fe4a16c7b28acb92
Reviewed-on: http://review.typo3.org/42097
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
typo3/sysext/install/Classes/ViewHelpers/Be/ContainerViewHelper.php
typo3/sysext/install/Resources/Private/Layouts/BackendModule.html

index b92796d..3bed2bb 100644 (file)
@@ -30,7 +30,7 @@ namespace TYPO3\CMS\Install\ViewHelpers\Be;
  * </output>
  *
  * <code title="All options">
- * <f:be.container pageTitle="foo" enableClickMenu="false" loadExtJs="true" loadExtJsTheme="false" enableExtJsDebug="true">your module content</f:be.container>
+ * <f:be.container pageTitle="foo">your module content</f:be.container>
  * </code>
  * <output>
  * "your module content" wrapped with proper head & body tags.
@@ -45,32 +45,17 @@ class ContainerViewHelper extends \TYPO3\CMS\Fluid\ViewHelpers\Be\AbstractBacken
         * Render start page with \TYPO3\CMS\Backend\Template\DocumentTemplate and pageTitle
         *
         * @param string $pageTitle title tag of the module. Not required by default, as BE modules are shown in a frame
-        * @param bool $enableClickMenu If TRUE, loads clickmenu.js required by BE context menus. Defaults to TRUE
-        * @param bool $loadExtJs specifies whether to load ExtJS library. Defaults to FALSE
-        * @param bool $loadExtJsTheme whether to load ExtJS "grey" theme. Defaults to FALSE
-        * @param bool $enableExtJsDebug if TRUE, debug version of ExtJS is loaded. Use this for development only
         * @param array $addCssFiles Custom CSS files to be loaded
         * @param array $addJsFiles Custom JavaScript files to be loaded
-        * @param array $triggers Defined triggers to be forwarded to client (e.g. refreshing backend widgets)
         *
         * @return string
         * @see \TYPO3\CMS\Backend\Template\DocumentTemplate
         * @see \TYPO3\CMS\Core\Page\PageRenderer
         */
-       public function render($pageTitle = '', $enableClickMenu = TRUE, $loadExtJs = FALSE, $loadExtJsTheme = TRUE, $enableExtJsDebug = FALSE, $addCssFiles = array(), $addJsFiles = array(), $triggers = array()) {
+       public function render($pageTitle = '', $addCssFiles = array(), $addJsFiles = array()) {
                $doc = $this->getDocInstance();
                $pageRenderer = $this->getPageRenderer();
 
-               $doc->JScode .= $doc->wrapScriptTags($doc->redirectUrls());
-               if ($enableClickMenu) {
-                       $doc->loadJavascriptLib('sysext/backend/Resources/Public/JavaScript/clickmenu.js');
-               }
-               if ($loadExtJs) {
-                       $pageRenderer->loadExtJS(TRUE, $loadExtJsTheme);
-                       if ($enableExtJsDebug) {
-                               $pageRenderer->enableExtJsDebug();
-                       }
-               }
                if (is_array($addCssFiles) && !empty($addCssFiles)) {
                        foreach ($addCssFiles as $addCssFile) {
                                $pageRenderer->addCssFile($addCssFile);
@@ -81,13 +66,6 @@ class ContainerViewHelper extends \TYPO3\CMS\Fluid\ViewHelpers\Be\AbstractBacken
                                $pageRenderer->addJsFile($addJsFile);
                        }
                }
-               // Handle triggers
-               if (!empty($triggers[\TYPO3\CMS\Extensionmanager\Controller\AbstractController::TRIGGER_RefreshModuleMenu])) {
-                       $pageRenderer->addJsInlineCode(
-                               \TYPO3\CMS\Extensionmanager\Controller\AbstractController::TRIGGER_RefreshModuleMenu,
-                               'if (top.TYPO3ModuleMenu.refreshMenu) { top.TYPO3ModuleMenu.refreshMenu(); }'
-                       );
-               }
                $output = $this->renderChildren();
                $output = $doc->startPage($pageTitle) . $output;
                $output .= $doc->endPage();
index a5f2039..9a9199f 100644 (file)
@@ -7,9 +7,6 @@
 
 <i:be.container
        pageTitle="Install tool"
-       enableClickMenu="false"
-       loadExtJs="false"
-       loadExtJsTheme="false"
        addCssFiles="{
                0:'{f:uri.resource(path:\'Css/BackendModule/ShowEnableInstallToolButton.css\')}'
        }"