[!!!][TASK] Remove deprecated code from EXT:setup 48/59248/2
authorWouter Wolters <typo3@wouterwolters.nl>
Thu, 20 Dec 2018 20:36:50 +0000 (21:36 +0100)
committerGeorg Ringer <georg.ringer@gmail.com>
Fri, 21 Dec 2018 04:43:53 +0000 (05:43 +0100)
Resolves: #87250
Releases: master
Change-Id: I72b85074e4e9b31f3bb72391f035d4aae8932d41
Reviewed-on: https://review.typo3.org/59248
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
typo3/sysext/core/Documentation/Changelog/master/Breaking-87193-DeprecatedFunctionalityRemoved.rst
typo3/sysext/setup/Classes/Controller/SetupModuleController.php

index 4e2c5da..3f4426e 100644 (file)
@@ -339,6 +339,8 @@ The following PHP class methods that have been previously deprecated for v9 have
 * :php:`TYPO3\CMS\Impexp\Controller\ImportExportController->handleExternalFunctionValue()`
 * :php:`TYPO3\CMS\Impexp\Controller\ImportExportController->menuConfig()`
 * :php:`TYPO3\CMS\Impexp\Controller\ImportExportController->mergeExternalItems()`
+* :php:`TYPO3\CMS\Setup\Controller\SetupModuleController->getFormProtection()`
+* :php:`TYPO3\CMS\Setup\Controller\SetupModuleController->simulateUser()`
 
 
 The following PHP static class methods that have been previously deprecated for v9 have been removed:
@@ -584,6 +586,12 @@ The following class properties have changed visibility:
 * :php:`TYPO3\CMS\Impexp\Controller\ImportExportController->id` changed from public to protected
 * :php:`TYPO3\CMS\Impexp\Controller\ImportExportController->pageinfo` changed from public to protected
 * :php:`TYPO3\CMS\Impexp\Controller\ImportExportController->perms_clause` changed from public to protected
+* :php:`TYPO3\CMS\Setup\Controller\SetupModuleController->OLD_BE_USER` changed from public to protected
+* :php:`TYPO3\CMS\Setup\Controller\SetupModuleController->MOD_MENU` changed from public to protected
+* :php:`TYPO3\CMS\Setup\Controller\SetupModuleController->MOD_SETTINGS` changed from public to protected
+* :php:`TYPO3\CMS\Setup\Controller\SetupModuleController->content` changed from public to protected
+* :php:`TYPO3\CMS\Setup\Controller\SetupModuleController->overrideConf` changed from public to protected
+* :php:`TYPO3\CMS\Setup\Controller\SetupModuleController->languageUpdate` changed from public to protected
 * :php:`TYPO3\CMS\Tstemplate\Controller\TypoScriptTemplateModuleController->access` changed from public to protected
 * :php:`TYPO3\CMS\Tstemplate\Controller\TypoScriptTemplateModuleController->CMD` changed from public to protected
 * :php:`TYPO3\CMS\Tstemplate\Controller\TypoScriptTemplateModuleController->content` changed from public to protected
index 2272f4b..86de42a 100644 (file)
@@ -21,9 +21,6 @@ use TYPO3\CMS\Backend\Module\ModuleLoader;
 use TYPO3\CMS\Backend\Routing\UriBuilder;
 use TYPO3\CMS\Backend\Template\ModuleTemplate;
 use TYPO3\CMS\Backend\Utility\BackendUtility;
-use TYPO3\CMS\Core\Authentication\BackendUserAuthentication;
-use TYPO3\CMS\Core\Compatibility\PublicMethodDeprecationTrait;
-use TYPO3\CMS\Core\Compatibility\PublicPropertyDeprecationTrait;
 use TYPO3\CMS\Core\Core\Environment;
 use TYPO3\CMS\Core\Crypto\PasswordHashing\InvalidPasswordHashException;
 use TYPO3\CMS\Core\Crypto\PasswordHashing\PasswordHashFactory;
@@ -47,8 +44,6 @@ use TYPO3\CMS\Core\Utility\GeneralUtility;
  */
 class SetupModuleController
 {
-    use PublicPropertyDeprecationTrait;
-    use PublicMethodDeprecationTrait;
 
     /**
      * Flag if password has not been updated
@@ -72,39 +67,6 @@ class SetupModuleController
     const PASSWORD_OLD_WRONG = 3;
 
     /**
-     * Properties which have been moved to protected status from public
-     *
-     * @var array
-     */
-    private $deprecatedPublicProperties = [
-        'OLD_BE_USER' => 'Using $OLD_BE_USER of class SetupModuleController from the outside is discouraged, the variable will be removed.',
-        'MOD_MENU' => 'Using $MOD_MENU of class SetupModuleController from the outside is discouraged, the variable will be removed.',
-        'MOD_SETTINGS' => 'Using $MOD_SETTINGS of class SetupModuleController from the outside is discouraged, the variable will be removed.',
-        'content' => 'Using $content of class SetupModuleController from the outside is discouraged, as this variable is only used for internal storage.',
-        'overrideConf' => 'Using $overrideConf of class SetupModuleController from the outside is discouraged, as this variable is only used for internal storage.',
-        'languageUpdate' => 'Using $languageUpdate of class SetupModuleController from the outside is discouraged, as this variable is only used for internal storage.',
-    ];
-
-    /**
-     * @var array
-     */
-    private $deprecatedPublicMethods = [
-        'storeIncomingData' => 'Using SetupModuleController::storeIncomingData() is deprecated and will not be possible anymore in TYPO3 v10.0.',
-        'main' => 'Using SetupModuleController::main() is deprecated and will not be possible anymore in TYPO3 v10.0.',
-        'init' => 'Using SetupModuleController::init() is deprecated and will not be possible anymore in TYPO3 v10.0.',
-    ];
-
-    /**
-     * @var array
-     */
-    protected $MOD_MENU = [];
-
-    /**
-     * @var array
-     */
-    protected $MOD_SETTINGS = [];
-
-    /**
      * @var string
      */
     protected $content;
@@ -115,11 +77,6 @@ class SetupModuleController
     protected $overrideConf;
 
     /**
-     * @deprecated will be removed in TYPO3 v10.0
-     */
-    protected $OLD_BE_USER;
-
-    /**
      * @var bool
      */
     protected $languageUpdate;
@@ -212,18 +169,6 @@ class SetupModuleController
     }
 
     /**
-     * Getter for the form protection instance.
-     *
-     * @return \TYPO3\CMS\Core\FormProtection\BackendFormProtection
-     * @deprecated since TYPO3 v9, will be removed in TYPO3 v10.0.
-     */
-    public function getFormProtection()
-    {
-        trigger_error('SetupModuleController->getFormProtection() will be removed in TYPO3 v10.0.', E_USER_DEPRECATED);
-        return $this->formProtection;
-    }
-
-    /**
      * If settings are submitted to _POST[DATA], store them
      * NOTICE: This method is called before the \TYPO3\CMS\Backend\Template\ModuleTemplate
      * is included. See bottom of document.
@@ -745,65 +690,6 @@ class SetupModuleController
     }
 
     /**
-     * Will make the simulate-user selector if the logged in user is administrator.
-     * It will also set the GLOBAL(!) BE_USER to the simulated user selected if any (and set $this->OLD_BE_USER to logged in user)
-     *
-     * @deprecated since TYPO3 v9, will be removed in TYPO3 v10.0.
-     */
-    public function simulateUser()
-    {
-        trigger_error('SetupModuleController->simulateUser() will be removed in TYPO3 v10.0.', E_USER_DEPRECATED);
-        // If admin, allow simulation of another user
-        $this->simUser = 0;
-        $this->simulateSelector = '';
-        unset($this->OLD_BE_USER);
-        $currentBeUser = $this->getBackendUser();
-        if ($currentBeUser->isAdmin()) {
-            $this->simUser = (int)GeneralUtility::_GP('simUser');
-            $queryBuilder = GeneralUtility::makeInstance(ConnectionPool::class)->getQueryBuilderForTable('be_users');
-            $users = $queryBuilder
-                ->select('*')
-                ->from('be_users')
-                ->where(
-                    $queryBuilder->expr()->neq(
-                        'uid',
-                        $queryBuilder->createNamedParameter($currentBeUser->user['uid'], \PDO::PARAM_INT)
-                    ),
-                    $queryBuilder->expr()->notLike(
-                        'username',
-                        $queryBuilder->createNamedParameter(
-                            $queryBuilder->escapeLikeWildcards('_cli_') . '%',
-                            \PDO::PARAM_STR
-                        )
-                    )
-                )
-                ->orderBy('username')
-                ->execute()
-                ->fetchAll();
-            $opt = [];
-            foreach ($users as $rr) {
-                $label = $rr['username'] . ($rr['realName'] ? ' (' . $rr['realName'] . ')' : '');
-                $opt[] = '<option value="' . (int)$rr['uid'] . '"' . ($this->simUser === (int)$rr['uid'] ? ' selected="selected"' : '') . '>' . htmlspecialchars($label) . '</option>';
-            }
-            if (!empty($opt)) {
-                $uriBuilder = GeneralUtility::makeInstance(UriBuilder::class);
-                $this->simulateSelector = '<select id="field_simulate" class="form-control" name="simulateUser" onchange="window.location.href=' . GeneralUtility::quoteJSvalue((string)$uriBuilder->buildUriFromRoute('user_setup') . '&simUser=') . '+this.options[this.selectedIndex].value;"><option></option>' . implode('', $opt) . '</select>';
-            }
-        }
-        // This can only be set if the previous code was executed.
-        if ($this->simUser > 0) {
-            // Save old user...
-            $this->OLD_BE_USER = $currentBeUser;
-            // Unset current
-            // New backend user object
-            $currentBeUser = GeneralUtility::makeInstance(BackendUserAuthentication::class);
-            $currentBeUser->setBeUserByUid($this->simUser);
-            $currentBeUser->fetchGroupData();
-            $currentBeUser->backendSetUC();
-        }
-    }
-
-    /**
      * Returns access check (currently only "admin" is supported)
      *
      * @param array $config Configuration of the field, access mode is defined in key 'access'