[!!!][TASK] Remove deprecated Fluid Viewhelper arguments 20/52520/6
authorBenni Mack <benni@typo3.org>
Wed, 17 May 2017 05:30:18 +0000 (07:30 +0200)
committerWouter Wolters <typo3@wouterwolters.nl>
Wed, 17 May 2017 08:14:44 +0000 (10:14 +0200)
Remove arguments within f:be.container and f:be.pageRenderer which
have been marked as deprecated before:

* f:be.container->enableClickMenu
* f:be.container->loadExtJs
* f:be.container->loadExtJsTheme
* f:be.container->enableExtJsDebug
* f:be.container->loadJQuery
* f:be.container->jQueryNamespace
* f:be.pageRenderer->loadExtJs
* f:be.pageRenderer->loadExtJsTheme
* f:be.pageRenderer->enableExtJsDebug
* f:be.pageRenderer->loadJQuery
* f:be.pageRenderer->jQueryNamespace

Resolves: #80707
Releases: master
Change-Id: If1f36b3fb197e108cb12351145e312a5aaca6447
Reviewed-on: https://review.typo3.org/52520
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
typo3/sysext/core/Documentation/Changelog/master/Breaking-80700-DeprecatedFunctionalityRemoved.rst
typo3/sysext/fluid/Classes/ViewHelpers/Be/ContainerViewHelper.php
typo3/sysext/fluid/Classes/ViewHelpers/Be/PageRendererViewHelper.php

index b56d62f..3e34351 100644 (file)
@@ -340,6 +340,19 @@ The following icon identifiers have been removed:
 * actions-document-close
 * actions-edit-add
 
+The following Fluid ViewHelper arguments have been removed:
+* f:be.container->enableClickMenu
+* f:be.container->loadExtJs
+* f:be.container->loadExtJsTheme
+* f:be.container->enableExtJsDebug
+* f:be.container->loadJQuery
+* f:be.container->jQueryNamespace
+* f:be.pageRenderer->loadExtJs
+* f:be.pageRenderer->loadExtJsTheme
+* f:be.pageRenderer->enableExtJsDebug
+* f:be.pageRenderer->loadJQuery
+* f:be.pageRenderer->jQueryNamespace
+
 The following requireJS modules have been removed:
 * TYPO3/CMS/Core/QueryGenerator
 
index 63cdf25..c074f08 100644 (file)
@@ -14,7 +14,6 @@ namespace TYPO3\CMS\Fluid\ViewHelpers\Be;
  * The TYPO3 project - inspiring people to share!
  */
 
-use TYPO3\CMS\Core\Page\PageRenderer;
 use TYPO3\CMS\Core\Utility\GeneralUtility;
 use TYPO3\CMS\Extbase\Utility\LocalizationUtility;
 
@@ -32,13 +31,13 @@ use TYPO3\CMS\Extbase\Utility\LocalizationUtility;
  * </output>
  *
  * <code title="All options">
- * <f:be.container pageTitle="foo" enableClickMenu="false" loadExtJs="true" loadExtJsTheme="false" enableExtJsDebug="true" loadJQuery="true" includeCssFiles="{0: '{f:uri.resource(path:\'Css/Styles.css\')}'}" includeJsFiles="{0: '{f:uri.resource(path:\'JavaScript/Library1.js\')}', 1: '{f:uri.resource(path:\'JavaScript/Library2.js\')}'}" addJsInlineLabels="{0: 'label1', 1: 'label2'}">your module content</f:be.container>
+ * <f:be.container pageTitle="foo" includeCssFiles="{0: '{f:uri.resource(path:\'Css/Styles.css\')}'}" includeJsFiles="{0: '{f:uri.resource(path:\'JavaScript/Library1.js\')}', 1: '{f:uri.resource(path:\'JavaScript/Library2.js\')}'}" addJsInlineLabels="{0: 'label1', 1: 'label2'}">your module content</f:be.container>
  * </code>
  * <output>
  * "your module content" wrapped with proper head & body tags.
  * Custom CSS file EXT:your_extension/Resources/Public/Css/styles.css and
  * JavaScript files EXT:your_extension/Resources/Public/JavaScript/Library1.js and EXT:your_extension/Resources/Public/JavaScript/Library2.js
- * will be loaded, plus ExtJS and jQuery and some inline labels for usage in JS code.
+ * will be loaded, plus some inline labels for usage in JS code.
  * </output>
  */
 class ContainerViewHelper extends AbstractBackendViewHelper
@@ -57,16 +56,10 @@ class ContainerViewHelper extends AbstractBackendViewHelper
     {
         parent::initializeArguments();
         $this->registerArgument('pageTitle', 'string', 'Title tag of the module. Not required by default, as BE modules are shown in a frame', false, '');
-        $this->registerArgument('enableClickMenu', 'bool', 'If TRUE, loads clickmenu.js required by BE context menus. Defaults to TRUE. This option will be removed in TYPO3 v9', false, true);
-        $this->registerArgument('loadExtJs', 'bool', 'Specifies whether to load ExtJS library. Defaults to FALSE. This option will be removed in TYPO3 v9', false, false);
-        $this->registerArgument('loadExtJsTheme', 'bool', 'Whether to load ExtJS "grey" theme. Defaults to FALSE. This option will be removed in TYPO3 v9', false, true);
-        $this->registerArgument('enableExtJsDebug', 'bool', 'If TRUE, debug version of ExtJS is loaded. Use this for development only. This option will be removed in TYPO3 v9', false, false);
-        $this->registerArgument('loadJQuery', 'bool', 'Whether to load jQuery library. Defaults to FALSE. This option will be removed in TYPO3 v9', false, false);
         $this->registerArgument('includeCssFiles', 'array', 'List of custom CSS file to be loaded');
         $this->registerArgument('includeJsFiles', 'array', 'List of custom JavaScript file to be loaded');
         $this->registerArgument('addJsInlineLabels', 'array', 'Custom labels to add to JavaScript inline labels');
         $this->registerArgument('includeRequireJsModules', 'array', 'List of RequireJS modules to be loaded');
-        $this->registerArgument('jQueryNamespace', 'string', 'Store the jQuery object in a specific namespace. This option will be removed in TYPO3 v9');
     }
 
     /**
@@ -79,56 +72,15 @@ class ContainerViewHelper extends AbstractBackendViewHelper
     public function render()
     {
         $pageTitle = $this->arguments['pageTitle'];
-        $enableClickMenu = $this->arguments['enableClickMenu'];
-        $loadExtJs = $this->arguments['loadExtJs'];
-        $loadExtJsTheme = $this->arguments['loadExtJsTheme'];
-        $enableExtJsDebug = $this->arguments['enableExtJsDebug'];
-        $loadJQuery = $this->arguments['loadJQuery'];
         $includeCssFiles = $this->arguments['includeCssFiles'];
         $includeJsFiles = $this->arguments['includeJsFiles'];
         $addJsInlineLabels = $this->arguments['addJsInlineLabels'];
         $includeRequireJsModules = $this->arguments['includeRequireJsModules'];
-        $jQueryNamespace = $this->arguments['jQueryNamespace'];
 
         $pageRenderer = $this->getPageRenderer();
         $doc = $this->getDocInstance();
         $doc->JScode .= GeneralUtility::wrapJS($doc->redirectUrls());
 
-        // Load various standard libraries
-        if ($enableClickMenu) {
-            GeneralUtility::logDeprecatedViewHelperAttribute(
-                'enableClickMenu',
-                $this->renderingContext,
-                'Setting "enableClickMenu" in Container ViewHelper is deprecated, the option will be removed in TYPO3 v9'
-            );
-            $pageRenderer->loadJquery();
-            $pageRenderer->loadRequireJsModule('TYPO3/CMS/Backend/ContextMenu');
-        }
-        if ($loadExtJs) {
-            GeneralUtility::logDeprecatedViewHelperAttribute(
-                'loadExtJs',
-                $this->renderingContext,
-                'Setting "loadExtJs" and "loadExtJsTheme" in Container ViewHelper is deprecated, the option will be removed in TYPO3 v9'
-            );
-            $pageRenderer->loadExtJS(true, $loadExtJsTheme);
-            if ($enableExtJsDebug) {
-                GeneralUtility::logDeprecatedViewHelperAttribute(
-                    'enableExtJsDebug',
-                    $this->renderingContext,
-                    'Setting "enableExtJsDebug" in Container ViewHelper is deprecated, the option will be removed in TYPO3 v9'
-                );
-                $pageRenderer->enableExtJsDebug();
-            }
-        }
-        if ($loadJQuery) {
-            GeneralUtility::logDeprecatedViewHelperAttribute(
-                'loadjQuery',
-                $this->renderingContext,
-                'Setting "loadjQuery" and "jQueryNamespace" in Container ViewHelper are deprecated, the option will be removed in TYPO3 v9'
-            );
-            $jQueryNamespace = $jQueryNamespace ?: PageRenderer::JQUERY_NAMESPACE_DEFAULT;
-            $pageRenderer->loadJquery(null, null, $jQueryNamespace);
-        }
         // Include custom CSS and JS files
         if (is_array($includeCssFiles) && count($includeCssFiles) > 0) {
             foreach ($includeCssFiles as $addCssFile) {
index 2036f1b..eb60cc0 100644 (file)
@@ -15,7 +15,6 @@ namespace TYPO3\CMS\Fluid\ViewHelpers\Be;
  */
 
 use TYPO3\CMS\Core\Page\PageRenderer;
-use TYPO3\CMS\Core\Utility\GeneralUtility;
 use TYPO3\CMS\Extbase\Utility\LocalizationUtility;
 use TYPO3\CMS\Fluid\Core\ViewHelper\AbstractViewHelper;
 
@@ -25,13 +24,13 @@ use TYPO3\CMS\Fluid\Core\ViewHelper\AbstractViewHelper;
  * = Examples =
  *
  * <code title="All options">
- * <f:be.pageRenderer pageTitle="foo" loadExtJs="true" loadExtJsTheme="false" extJsAdapter="jQuery" enableExtJsDebug="true" loadJQuery="true" includeCssFiles="0: '{f:uri.resource(path:\'Css/Styles.css\')}'" includeJsFiles="0: '{f:uri.resource(path:\'JavaScript/Library1.js\')}', 1: '{f:uri.resource(path:\'JavaScript/Library2.js\')}'" addJsInlineLabels="{0: 'label1', 1: 'label2'}" />
+ * <f:be.pageRenderer pageTitle="foo" includeCssFiles="0: '{f:uri.resource(path:\'Css/Styles.css\')}'" includeJsFiles="0: '{f:uri.resource(path:\'JavaScript/Library1.js\')}', 1: '{f:uri.resource(path:\'JavaScript/Library2.js\')}'" addJsInlineLabels="{0: 'label1', 1: 'label2'}" />
  * </code>
  * <output>
  *
  * Custom CSS file EXT:your_extension/Resources/Public/Css/styles.css and
  * JavaScript files EXT:your_extension/Resources/Public/JavaScript/Library1.js and EXT:your_extension/Resources/Public/JavaScript/Library2.js
- * will be loaded, plus ExtJS and jQuery and some inline labels for usage in JS code.
+ * will be loaded, plus some inline labels for usage in JS code.
  * </output>
  */
 class PageRendererViewHelper extends AbstractViewHelper
@@ -58,15 +57,10 @@ class PageRendererViewHelper extends AbstractViewHelper
     {
         parent::initializeArguments();
         $this->registerArgument('pageTitle', 'string', 'title tag of the module. Not required by default, as BE modules are shown in a frame', false, '');
-        $this->registerArgument('loadExtJs', 'bool', 'specifies whether to load ExtJS library. Defaults to FALSE. This option will be removed in TYPO3 v9', false, false);
-        $this->registerArgument('loadExtJsTheme', 'bool', 'whether to load ExtJS "grey" theme. Defaults to FALSE. This option will be removed in TYPO3 v9', false, true);
-        $this->registerArgument('enableExtJsDebug', 'bool', 'if TRUE, debug version of ExtJS is loaded. Use this for development only. This option will be removed in TYPO3 v9', false, false);
-        $this->registerArgument('loadJQuery', 'bool', 'whether to load jQuery library. Defaults to FALSE. This option will be removed in TYPO3 v9', false, false);
         $this->registerArgument('includeCssFiles', 'array', 'List of custom CSS file to be loaded');
         $this->registerArgument('includeJsFiles', 'array', 'List of custom JavaScript file to be loaded');
         $this->registerArgument('addJsInlineLabels', 'array', 'Custom labels to add to JavaScript inline labels');
         $this->registerArgument('includeRequireJsModules', 'array', 'List of RequireJS modules to be loaded');
-        $this->registerArgument('jQueryNamespace', 'string', 'Store the jQuery object in a specific namespace. This option will be removed in TYPO3 v9');
         $this->registerArgument('addInlineSettings', 'array', 'Adds Javascript Inline Setting');
     }
 
@@ -76,45 +70,15 @@ class PageRendererViewHelper extends AbstractViewHelper
     public function render()
     {
         $pageTitle = $this->arguments['pageTitle'];
-        $loadExtJs = $this->arguments['loadExtJs'];
-        $loadExtJsTheme = $this->arguments['loadExtJsTheme'];
-        $enableExtJsDebug = $this->arguments['enableExtJsDebug'];
-        $loadJQuery = $this->arguments['loadJQuery'];
         $includeCssFiles = $this->arguments['includeCssFiles'];
         $includeJsFiles = $this->arguments['includeJsFiles'];
         $addJsInlineLabels = $this->arguments['addJsInlineLabels'];
         $includeRequireJsModules = $this->arguments['includeRequireJsModules'];
-        $jQueryNamespace = $this->arguments['jQueryNamespace'];
         $addInlineSettings = $this->arguments['addInlineSettings'];
 
         if ($pageTitle) {
             $this->pageRenderer->setTitle($pageTitle);
         }
-        if ($loadExtJs) {
-            GeneralUtility::logDeprecatedViewHelperAttribute(
-                'loadExtJs',
-                $this->renderingContext,
-                'Setting "loadExtJs" and "loadExtJsTheme" in PageRenderer ViewHelper is deprecated, the option will be removed in TYPO3 v9'
-            );
-            $this->pageRenderer->loadExtJS(true, $loadExtJsTheme);
-            if ($enableExtJsDebug) {
-                GeneralUtility::logDeprecatedViewHelperAttribute(
-                    'enableExtJsDebug',
-                    $this->renderingContext,
-                    'Setting "enableExtJsDebug" in PageRenderer ViewHelper is deprecated, the option will be removed in TYPO3 v9'
-                );
-                $this->pageRenderer->enableExtJsDebug();
-            }
-        }
-        if ($loadJQuery) {
-            GeneralUtility::logDeprecatedViewHelperAttribute(
-                'loadjQuery',
-                $this->renderingContext,
-                'Setting "loadjQuery" and "jQueryNamespace" in PageRenderer ViewHelper are deprecated, the option will be removed in TYPO3 v9'
-            );
-            $jQueryNamespace = $jQueryNamespace ?: PageRenderer::JQUERY_NAMESPACE_DEFAULT;
-            $this->pageRenderer->loadJquery(null, null, $jQueryNamespace);
-        }
         // Include custom CSS and JS files
         if (is_array($includeCssFiles) && count($includeCssFiles) > 0) {
             foreach ($includeCssFiles as $addCssFile) {