[TASK] Clean up Admin Panel Code 70/57870/5
authorBenni Mack <benni@typo3.org>
Sat, 11 Aug 2018 09:42:35 +0000 (11:42 +0200)
committerChristian Kuhn <lolli@schwarzbu.ch>
Sat, 11 Aug 2018 18:05:56 +0000 (20:05 +0200)
Due to the Admin Panel refactoring, some helper methods
were deprecated, but need a better description.

Also, the $BE_USER->uc property is set to "AdminPanel"
instead of cryptic string "TSFE_adminConfig" since
AdminPanel is not related to TSFE anymore.

Additionally, some helper methods and properties which
are not in use are removed.

Resolves: #85815
Releases: master
Change-Id: I7f875acd0f883690f69e895c814b6ad335d86601
Reviewed-on: https://review.typo3.org/57870
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
14 files changed:
typo3/sysext/adminpanel/Classes/Controller/AjaxController.php
typo3/sysext/adminpanel/Classes/Controller/MainController.php
typo3/sysext/adminpanel/Classes/Middleware/AdminPanelInitiator.php
typo3/sysext/adminpanel/Classes/Modules/AbstractModule.php
typo3/sysext/adminpanel/Classes/Modules/AbstractSubModule.php
typo3/sysext/adminpanel/Classes/Modules/CacheModule.php
typo3/sysext/adminpanel/Classes/Modules/EditModule.php
typo3/sysext/adminpanel/Classes/Modules/Info/GeneralInformation.php
typo3/sysext/adminpanel/Classes/Service/ConfigurationService.php
typo3/sysext/adminpanel/Classes/View/AdminPanelView.php
typo3/sysext/adminpanel/Classes/ViewHelpers/IsArrayViewHelper.php
typo3/sysext/adminpanel/Tests/Unit/Middleware/AdminPanelInitiatorTest.php
typo3/sysext/adminpanel/Tests/Unit/Service/ConfigurationServiceTest.php
typo3/sysext/fluid/Classes/Core/ViewHelper/ViewHelperResolver.php

index 0bb0362..ed9b563 100644 (file)
@@ -36,12 +36,12 @@ class AjaxController
     protected $adminPanelModuleConfiguration;
 
     /**
-     * @var \TYPO3\CMS\Adminpanel\Service\ModuleLoader
+     * @var ModuleLoader
      */
     protected $moduleLoader;
 
     /**
-     * @var \TYPO3\CMS\Adminpanel\Service\ConfigurationService
+     * @var ConfigurationService
      */
     private $configurationService;
 
@@ -76,15 +76,15 @@ class AjaxController
     /**
      * Toggle admin panel active state via UC
      *
-     * @return \TYPO3\CMS\Core\Http\JsonResponse
+     * @return JsonResponse
      */
     public function toggleActiveState(): JsonResponse
     {
         $backendUser = $this->getBackendUser();
-        if ($backendUser->uc['TSFE_adminConfig']['display_top'] ?? false) {
-            $backendUser->uc['TSFE_adminConfig']['display_top'] = false;
+        if ($backendUser->uc['AdminPanel']['display_top'] ?? false) {
+            $backendUser->uc['AdminPanel']['display_top'] = false;
         } else {
-            $backendUser->uc['TSFE_adminConfig']['display_top'] = true;
+            $backendUser->uc['AdminPanel']['display_top'] = true;
         }
         $backendUser->writeUC();
         return new JsonResponse(['success' => true]);
@@ -93,7 +93,7 @@ class AjaxController
     /**
      * Returns the current BE user.
      *
-     * @return \TYPO3\CMS\Core\Authentication\BackendUserAuthentication
+     * @return BackendUserAuthentication
      */
     protected function getBackendUser(): BackendUserAuthentication
     {
index 95accbe..a3a5ebe 100644 (file)
@@ -23,12 +23,10 @@ use TYPO3\CMS\Adminpanel\Service\ModuleLoader;
 use TYPO3\CMS\Adminpanel\View\AdminPanelView;
 use TYPO3\CMS\Backend\FrontendBackendUserAuthentication;
 use TYPO3\CMS\Backend\Routing\UriBuilder;
-use TYPO3\CMS\Core\Localization\LanguageService;
 use TYPO3\CMS\Core\SingletonInterface;
 use TYPO3\CMS\Core\Utility\GeneralUtility;
 use TYPO3\CMS\Core\Utility\PathUtility;
 use TYPO3\CMS\Fluid\View\StandaloneView;
-use TYPO3\CMS\Frontend\Controller\TypoScriptFrontendController;
 
 /**
  * Main controller for the admin panel
@@ -113,7 +111,7 @@ class MainController implements SingletonInterface
      */
     public function render(): string
     {
-        // legacy handling
+        // legacy handling, deprecated, will be removed in TYPO3 v10.0.
         $adminPanelView = GeneralUtility::makeInstance(AdminPanelView::class);
         $hookObjectContent = $adminPanelView->callDeprecatedHookObject();
         // end legacy handling
@@ -245,16 +243,6 @@ class MainController implements SingletonInterface
     }
 
     /**
-     * Returns LanguageService
-     *
-     * @return LanguageService
-     */
-    protected function getLanguageService(): LanguageService
-    {
-        return $GLOBALS['LANG'];
-    }
-
-    /**
      * Return a string with tags for main admin panel resources
      *
      * @return string
@@ -270,14 +258,6 @@ class MainController implements SingletonInterface
     }
 
     /**
-     * @return TypoScriptFrontendController
-     */
-    protected function getTypoScriptFrontendController(): TypoScriptFrontendController
-    {
-        return $GLOBALS['TSFE'];
-    }
-
-    /**
      * Returns true if admin panel was activated
      * (switched "on" via GUI)
      *
@@ -285,6 +265,6 @@ class MainController implements SingletonInterface
      */
     protected function isAdminPanelActivated(): bool
     {
-        return (bool)($this->getBackendUser()->uc['TSFE_adminConfig']['display_top'] ?? false);
+        return (bool)($this->getBackendUser()->uc['AdminPanel']['display_top'] ?? false);
     }
 }
index 1db10f3..d2b2cfc 100644 (file)
@@ -49,7 +49,7 @@ class AdminPanelInitiator implements MiddlewareInterface
             $beUser->extAdminConfig = $beUser->getTSConfig()['admPanel.'] ?? [];
             $adminPanelConfiguration = $beUser->extAdminConfig;
             if (isset($adminPanelConfiguration['enable.']) &&
-                ($beUser->uc['TSFE_adminConfig']['display_top'] ?? false)
+                ($beUser->uc['AdminPanel']['display_top'] ?? false)
             ) {
                 // only initialize if at least one module is enabled.
                 foreach ($adminPanelConfiguration['enable.'] as $value) {
index b8d4ba4..a140075 100644 (file)
@@ -32,7 +32,7 @@ abstract class AbstractModule implements AdminPanelModuleInterface
 {
 
     /**
-     * @var \TYPO3\CMS\Adminpanel\Modules\AdminPanelSubModuleInterface[]
+     * @var AdminPanelSubModuleInterface[]
      */
     protected $subModules = [];
 
@@ -44,7 +44,7 @@ abstract class AbstractModule implements AdminPanelModuleInterface
     protected $mainConfiguration;
 
     /**
-     * @var \TYPO3\CMS\Adminpanel\Service\ConfigurationService
+     * @var ConfigurationService
      */
     protected $configurationService;
 
@@ -104,22 +104,6 @@ abstract class AbstractModule implements AdminPanelModuleInterface
     }
 
     /**
-     * Translate given key
-     *
-     * @param string $key Key for a label in the $LOCAL_LANG array of "sysext/lang/Resources/Private/Language/locallang_tsfe.xlf
-     * @param bool $convertWithHtmlspecialchars If TRUE the language-label will be sent through htmlspecialchars
-     * @return string The value for the $key
-     */
-    protected function extGetLL($key, $convertWithHtmlspecialchars = true): string
-    {
-        $labelStr = $this->getLanguageService()->getLL($key);
-        if ($convertWithHtmlspecialchars) {
-            $labelStr = htmlspecialchars($labelStr, ENT_QUOTES | ENT_HTML5);
-        }
-        return $labelStr;
-    }
-
-    /**
      * Returns the current BE user.
      *
      * @return BackendUserAuthentication|FrontendBackendUserAuthentication
@@ -132,7 +116,7 @@ abstract class AbstractModule implements AdminPanelModuleInterface
     /**
      * Returns LanguageService
      *
-     * @return \TYPO3\CMS\Core\Localization\LanguageService
+     * @return LanguageService
      */
     protected function getLanguageService(): LanguageService
     {
index ca409b3..be28541 100644 (file)
@@ -51,7 +51,7 @@ abstract class AbstractSubModule implements AdminPanelSubModuleInterface
     /**
      * Returns LanguageService
      *
-     * @return \TYPO3\CMS\Core\Localization\LanguageService
+     * @return LanguageService
      */
     protected function getLanguageService(): LanguageService
     {
index b08a453..dd93cef 100644 (file)
@@ -47,8 +47,8 @@ class CacheModule extends AbstractModule
 
         $view->assignMultiple(
             [
-                'isEnabled' => $this->getBackendUser()->uc['TSFE_adminConfig']['display_cache'],
-                'noCache' => $this->getBackendUser()->uc['TSFE_adminConfig']['cache_noCache'],
+                'isEnabled' => $this->getBackendUser()->uc['AdminPanel']['display_cache'],
+                'noCache' => $this->getBackendUser()->uc['AdminPanel']['cache_noCache'],
                 'currentId' => $this->getTypoScriptFrontendController()->id,
                 'clearPageCacheUrl' => $feCacheClear ? (string)$uriBuilder->buildUriFromRoute('tce_db', ['cacheCmd' => 'pages']) : '',
                 'clearCurrentPageCacheUrl' => (string)$uriBuilder->buildUriFromRoute(
index 8dc6378..fee1d00 100644 (file)
@@ -118,7 +118,7 @@ class EditModule extends AbstractModule
         $typoScriptFrontend->displayEditIcons = $this->configurationService->getConfigurationOption('edit', 'displayIcons');
         $typoScriptFrontend->displayFieldEditIcons = $this->configurationService->getConfigurationOption('edit', 'displayFieldIcons');
 
-        if (GeneralUtility::_GP('ADMCMD_editIcons')) {
+        if ($request->getQueryParams()['ADMCMD_editIcons'] ?? $request->getParsedBody()['ADMCMD_editIcons'] ?? false) {
             $typoScriptFrontend->displayFieldEditIcons = '1';
         }
         if ($extFeEditLoaded && $typoScriptFrontend->displayEditIcons) {
index bd8339b..99c76a0 100644 (file)
@@ -57,11 +57,11 @@ class GeneralInformation extends AbstractSubModule
                     'pageType' => $tsfe->type,
                     'groupList' => implode(',', $frontendUserAspect->getGroupIds()),
                     'noCache' => $this->isNoCacheEnabled(),
-                    'countUserInt' => \count($tsfe->config['INTincScript'] ?? []),
+                    'countUserInt' => count($tsfe->config['INTincScript'] ?? []),
                     'totalParsetime' => $this->getTimeTracker()->getParseTime(),
                     'feUser' => [
-                        'uid' => $tsfe->fe_user->user['uid'] ?? 0,
-                        'username' => $tsfe->fe_user->user['username'] ?? ''
+                        'uid' => $frontendUserAspect->get('id') ?: 0,
+                        'username' => $frontendUserAspect->get('username') ?: ''
                     ],
                     'imagesOnPage' => $this->collectImagesOnPage(),
                     'documentSize' => $this->collectDocumentSize()
index df4f443..4434ab1 100644 (file)
@@ -66,7 +66,7 @@ class ConfigurationService implements SingletonInterface
         if (isset($this->mainConfiguration['override.'][$identifier . '.'][$option])) {
             $returnValue = $this->mainConfiguration['override.'][$identifier . '.'][$option];
         } else {
-            $returnValue = $this->getBackendUser()->uc['TSFE_adminConfig'][$identifier . '_' . $option] ?? '';
+            $returnValue = $this->getBackendUser()->uc['AdminPanel'][$identifier . '_' . $option] ?? '';
         }
 
         return (string)$returnValue;
@@ -96,11 +96,11 @@ class ConfigurationService implements SingletonInterface
             }
         }
         // Settings
-        $beUser->uc['TSFE_adminConfig'] = array_merge(
-            !is_array($beUser->uc['TSFE_adminConfig']) ? [] : $beUser->uc['TSFE_adminConfig'],
+        $beUser->uc['AdminPanel'] = array_merge(
+            !is_array($beUser->uc['AdminPanel']) ? [] : $beUser->uc['AdminPanel'],
             $configurationToSave
         );
-        unset($beUser->uc['TSFE_adminConfig']['action']);
+        unset($beUser->uc['AdminPanel']['action']);
         // Saving
         $beUser->writeUC();
     }
index 9323071..1b933f9 100644 (file)
@@ -17,6 +17,7 @@ namespace TYPO3\CMS\Adminpanel\View;
 
 use TYPO3\CMS\Adminpanel\Modules\AdminPanelModuleInterface;
 use TYPO3\CMS\Adminpanel\Service\EditToolbarService;
+use TYPO3\CMS\Backend\FrontendBackendUserAuthentication;
 use TYPO3\CMS\Core\Cache\CacheManager;
 use TYPO3\CMS\Core\Utility\GeneralUtility;
 use TYPO3\CMS\Core\Utility\PathUtility;
@@ -29,13 +30,6 @@ use TYPO3\CMS\Core\Utility\PathUtility;
 class AdminPanelView
 {
     /**
-     * Determines whether the update button should be shown.
-     *
-     * @var bool
-     */
-    protected $extNeedUpdate = false;
-
-    /**
      * Force preview
      *
      * @var bool
@@ -43,11 +37,6 @@ class AdminPanelView
     protected $ext_forcePreview = false;
 
     /**
-     * @var string
-     */
-    protected $extJSCODE = '';
-
-    /**
      * Array of adminPanel modules
      *
      * @var AdminPanelModuleInterface[]
@@ -79,7 +68,7 @@ class AdminPanelView
      */
     protected function isAdminPanelActivated(): bool
     {
-        return $this->getBackendUser()->uc['TSFE_adminConfig']['display_top'] ?? false;
+        return $this->getBackendUser()->uc['AdminPanel']['display_top'] ?? false;
     }
 
     /**
@@ -95,21 +84,13 @@ class AdminPanelView
     /**
      * Returns the current BE user.
      *
-     * @return \TYPO3\CMS\Backend\FrontendBackendUserAuthentication
+     * @return FrontendBackendUserAuthentication
      */
     protected function getBackendUser()
     {
         return $GLOBALS['BE_USER'];
     }
 
-    /**
-     * @return \TYPO3\CMS\Frontend\Controller\TypoScriptFrontendController
-     */
-    protected function getTypoScriptFrontendController()
-    {
-        return $GLOBALS['TSFE'];
-    }
-
     /*****************************************************
      * Admin Panel: Deprecated API
      ****************************************************/
@@ -208,7 +189,7 @@ class AdminPanelView
         $onclick = 'document.TSFE_ADMIN_PANEL_FORM[' .
                    GeneralUtility::quoteJSvalue('TSFE_ADMIN_PANEL[display_' . $identifier . ']') .
                    '].value=' .
-                   ($this->getBackendUser()->uc['TSFE_adminConfig']['display_' . $identifier] ? '0' : '1') .
+                   ($this->getBackendUser()->uc['AdminPanel']['display_' . $identifier] ? '0' : '1') .
                    ';document.TSFE_ADMIN_PANEL_FORM.submit();return false;';
 
         $output = [];
@@ -245,9 +226,9 @@ class AdminPanelView
     /**
      * Translate given key
      *
-     * @param string $key Key for a label in the $LOCAL_LANG array of "sysext/lang/Resources/Private/Language/locallang_tsfe.xlf
+     * @param string $key Key for a label in the $LOCAL_LANG array of "EXT:core/Resources/Private/Language/locallang_tsfe.xlf
      * @param bool $convertWithHtmlspecialchars If TRUE the language-label will be sent through htmlspecialchars
-     * @deprecated Since TYPO3 v9 - only used in deprecated methods
+     * @deprecated Since TYPO3 v9, will be removed in TYPO3 v10.0 - only used in deprecated methods
      * @return string The value for the $key
      */
     protected function extGetLL($key, $convertWithHtmlspecialchars = true)
@@ -263,14 +244,11 @@ class AdminPanelView
      * Add an additional stylesheet
      *
      * @return string
-     * @deprecated since TYPO3 v9 - implement AdminPanelModules via the new API (see AdminPanelModuleInterface)
+     * @deprecated Since TYPO3 v9, will be removed in TYPO3 v10.0 - implement AdminPanelModules via the new API (see AdminPanelModuleInterface)
      */
     public function getAdminPanelHeaderData()
     {
-        trigger_error(
-            'Deprecated since TYPO3 v9 - implement AdminPanelModules via the new API (see AdminPanelModuleInterface)',
-            E_USER_DEPRECATED
-        );
+        trigger_error('AdminPanelView->' . __METHOD__ . ' will be removed in TYPO3 v10.0. Implement AdminPanelModules via the new API (see AdminPanelModuleInterface).', E_USER_DEPRECATED);
         $result = '';
         if (!empty($GLOBALS['TBE_STYLES']['stylesheets']['admPanel'])) {
             $stylesheet = GeneralUtility::locationHeaderUrl($GLOBALS['TBE_STYLES']['stylesheets']['admPanel']);
@@ -284,15 +262,12 @@ class AdminPanelView
      * Checks if an Admin Panel section ("module") is available for the user. If so, TRUE is returned.
      *
      * @param string $key The module key, eg. "edit", "preview", "info" etc.
-     * @deprecated
      * @return bool
+     * @deprecated Since TYPO3 v9, will be removed in TYPO3 v10.0 - implement AdminPanelModules via the new API (see AdminPanelModuleInterface)
      */
     public function isAdminModuleEnabled($key)
     {
-        trigger_error(
-            'Deprecated since TYPO3 v9 - implement AdminPanelModules via the new API (see AdminPanelModuleInterface)',
-            E_USER_DEPRECATED
-        );
+        trigger_error('AdminPanelView->' . __METHOD__ . ' will be removed in TYPO3 v10.0. Implement AdminPanelModules via the new API (see AdminPanelModuleInterface).', E_USER_DEPRECATED);
         $result = false;
         // Returns TRUE if the module checked is "preview" and the forcePreview flag is set.
         if ($key === 'preview' && $this->ext_forcePreview) {
@@ -308,23 +283,20 @@ class AdminPanelView
     /**
      * Saves any change in settings made in the Admin Panel.
      *
-     * @deprecated since TYPO3 v9 - implement AdminPanelModules via the new API (see AdminPanelModuleInterface)
+     * @deprecated Since TYPO3 v9, will be removed in TYPO3 v10.0 - implement AdminPanelModules via the new API (see AdminPanelModuleInterface)
      */
     public function saveConfigOptions()
     {
-        trigger_error(
-            'Deprecated since TYPO3 v9 - implement AdminPanelModules via the new API (see AdminPanelModuleInterface)',
-            E_USER_DEPRECATED
-        );
+        trigger_error('AdminPanelView->' . __METHOD__ . ' will be removed in TYPO3 v10.0. Implement AdminPanelModules via the new API (see AdminPanelModuleInterface).', E_USER_DEPRECATED);
         $input = GeneralUtility::_GP('TSFE_ADMIN_PANEL');
         $beUser = $this->getBackendUser();
         if (is_array($input)) {
             // Setting
-            $beUser->uc['TSFE_adminConfig'] = array_merge(
-                !is_array($beUser->uc['TSFE_adminConfig']) ? [] : $beUser->uc['TSFE_adminConfig'],
+            $beUser->uc['AdminPanel'] = array_merge(
+                !is_array($beUser->uc['AdminPanel']) ? [] : $beUser->uc['AdminPanel'],
                 $input
             );
-            unset($beUser->uc['TSFE_adminConfig']['action']);
+            unset($beUser->uc['AdminPanel']['action']);
 
             foreach ($this->modules as $module) {
                 if ($module->isEnabled()) {
@@ -347,12 +319,13 @@ class AdminPanelView
      * @param string $sectionName Module key
      * @param string $val Setting key
      * @return mixed The setting value
-     * @deprecated Since TYPO3 v9 - implement AdminPanelModules via the new API (see AdminPanelModuleInterface)
+     * @deprecated Since TYPO3 v9, will be removed in TYPO3 v10.0 - implement AdminPanelModules via the new API (see AdminPanelModuleInterface)
      */
     public function extGetFeAdminValue($sectionName, $val = '')
     {
+        trigger_error('AdminPanelView->' . __METHOD__ . ' will be removed in TYPO3 v10.0. Implement AdminPanelModules via the new API (see AdminPanelModuleInterface).', E_USER_DEPRECATED);
         trigger_error(
-            'Deprecated since TYPO3 v9 - implement AdminPanelModules via the new API (see AdminPanelModuleInterface)',
+            'Deprecated since TYPO3 v9 - ',
             E_USER_DEPRECATED
         );
         if (!$this->isAdminModuleEnabled($sectionName)) {
@@ -380,7 +353,7 @@ class AdminPanelView
             return $this->configuration['override.'][$sectionName];
         }
 
-        $returnValue = $val ? $beUser->uc['TSFE_adminConfig'][$sectionName . '_' . $val] : 1;
+        $returnValue = $val ? $beUser->uc['AdminPanel'][$sectionName . '_' . $val] : 1;
 
         // Exception for preview
         if ($sectionName === 'preview' && $this->ext_forcePreview) {
@@ -394,11 +367,11 @@ class AdminPanelView
     /**
      * Enables the force preview option.
      *
-     * @deprecated since TYPO3 v9 - see AdminPanelModule: Preview
+     * @deprecated since TYPO3 v9, will be removed in TYPO3 v10.0 - see AdminPanelModule: Preview
      */
     public function forcePreview()
     {
-        trigger_error('Deprecated since TYPO3 v9, see AdminPanelModule: Preview', E_USER_DEPRECATED);
+        trigger_error('AdminPanelView->' . __METHOD__ . ' will be removed in TYPO3 v10.0. Use new AdminPanel Preview Module instead.', E_USER_DEPRECATED);
         $this->ext_forcePreview = true;
     }
 
@@ -407,13 +380,13 @@ class AdminPanelView
      *
      * @param string $key Module key
      * @return bool TRUE, if the admin panel is open for the specified admin panel module key.
-     * @deprecated Since TYPO3 v9 - implement AdminPanelModules via the new API
+     * @deprecated since TYPO3 v9, will be removed in TYPO3 v10.0 - use new AdminPanel API instead
      */
     public function isAdminModuleOpen($key)
     {
-        trigger_error('since TYPO3 v9 - use new AdminPanel API instead', E_USER_DEPRECATED);
-        return $this->getBackendUser()->uc['TSFE_adminConfig']['display_top'] &&
-               $this->getBackendUser()->uc['TSFE_adminConfig']['display_' . $key];
+        trigger_error('AdminPanelView->' . __METHOD__ . ' will be removed in TYPO3 v10.0. Use new AdminPanel API instead.', E_USER_DEPRECATED);
+        return $this->getBackendUser()->uc['AdminPanel']['display_top'] &&
+               $this->getBackendUser()->uc['AdminPanel']['display_' . $key];
     }
 
     /**
@@ -427,11 +400,11 @@ class AdminPanelView
      * @param string $outerDivClass The Class attribute for the tr element.
      * @return string HTML table row.
      * @see extGetHead()
-     * @deprecated since TYPO3 v9 - use new AdminPanel API instead
+     * @deprecated since TYPO3 v9, will be removed in TYPO3 v10.0 - use new AdminPanel API instead
      */
     public function extGetItem($title, $content = '', $checkbox = '', $outerDivClass = null, $innerDivClass = null)
     {
-        trigger_error('since TYPO3 v9 - use new AdminPanel API instead', E_USER_DEPRECATED);
+        trigger_error('AdminPanelView->' . __METHOD__ . ' will be removed in TYPO3 v10.0. Use new AdminPanel API instead.', E_USER_DEPRECATED);
         $title = $title ? '<label for="' . htmlspecialchars($title) . '">' . $this->extGetLL($title) . '</label>' : '';
         $out = '';
         $out .= (string)$outerDivClass ? '<div class="' . htmlspecialchars($outerDivClass) . '">' : '<div>';
@@ -448,11 +421,11 @@ class AdminPanelView
      * @param string $sectionSuffix The suffix to the display_ label. Also selects the label from the LOCAL_LANG array.
      * @return string HTML table row.
      * @see extGetItem()
-     * @deprecated since TYPO3 v9 - use new AdminPanel API instead
+     * @deprecated since TYPO3 v9, will be removed in TYPO3 v10.0 - use new AdminPanel API instead
      */
     public function extGetHead($sectionSuffix)
     {
-        trigger_error('since TYPO3 v9 - use new AdminPanel API instead', E_USER_DEPRECATED);
+        trigger_error('AdminPanelView->' . __METHOD__ . ' will be removed in TYPO3 v10.0. Use new AdminPanel API instead.', E_USER_DEPRECATED);
         return $this->linkSectionHeader($sectionSuffix, $this->extGetLL($sectionSuffix));
     }
 
@@ -464,15 +437,15 @@ class AdminPanelView
      * @param string $className The classname for the <a> tag
      * @return string $className Linked input string
      * @see extGetHead()
-     * @deprecated  since TYPO3 v9 - use new AdminPanel API instead
+     * @deprecated since TYPO3 v9, will be removed in TYPO3 v10.0 - use new AdminPanel API instead
      */
     public function linkSectionHeader($sectionSuffix, $sectionTitle, $className = '')
     {
-        trigger_error('since TYPO3 v9 - use new AdminPanel API instead', E_USER_DEPRECATED);
+        trigger_error('AdminPanelView->' . __METHOD__ . ' will be removed in TYPO3 v10.0. Use new AdminPanel API instead.', E_USER_DEPRECATED);
         $onclick = 'document.TSFE_ADMIN_PANEL_FORM[' .
                    GeneralUtility::quoteJSvalue('TSFE_ADMIN_PANEL[display_' . $sectionSuffix . ']') .
                    '].value=' .
-                   ($this->getBackendUser()->uc['TSFE_adminConfig']['display_' . $sectionSuffix] ? '0' : '1') .
+                   ($this->getBackendUser()->uc['AdminPanel']['display_' . $sectionSuffix] ? '0' : '1') .
                    ';document.TSFE_ADMIN_PANEL_FORM.submit();return false;';
 
         $output = [];
index 88b2989..c6ce8e4 100644 (file)
@@ -21,6 +21,8 @@ use TYPO3Fluid\Fluid\Core\ViewHelper\AbstractViewHelper;
 
 /**
  * Checks whether the given value is an array
+ *
+ * @internal
  */
 class IsArrayViewHelper extends AbstractViewHelper
 {
index 95ffae4..9c69401 100644 (file)
@@ -51,7 +51,7 @@ class AdminPanelInitiatorTest extends UnitTestCase
             ],
         ];
         $uc = [
-            'TSFE_adminConfig' => [
+            'AdminPanel' => [
                 'display_top' => true
             ]
         ];
@@ -91,7 +91,7 @@ class AdminPanelInitiatorTest extends UnitTestCase
             ],
         ];
         $uc = [
-            'TSFE_adminConfig' => [
+            'AdminPanel' => [
                 'display_top' => false
             ]
         ];
@@ -107,7 +107,7 @@ class AdminPanelInitiatorTest extends UnitTestCase
             'admPanel.' => [],
         ];
         $uc = [
-            'TSFE_adminConfig' => [
+            'AdminPanel' => [
                 'display_top' => true
             ]
         ];
index c344b75..739b661 100644 (file)
@@ -135,7 +135,7 @@ class ConfigurationServiceTest extends UnitTestCase
     {
         $this->setUpUserTsConfigForAdmPanel([]);
         $this->beUserProphecy->uc = [
-            'TSFE_adminConfig' => [
+            'AdminPanel' => [
                 'preview_showHiddenPages' => '1',
             ],
         ];
@@ -178,7 +178,7 @@ class ConfigurationServiceTest extends UnitTestCase
     {
         // existing configuration from UC
         $this->beUserProphecy->uc = [
-            'TSFE_adminConfig' => [
+            'AdminPanel' => [
                 'foo' => 'bar',
             ],
         ];
@@ -197,7 +197,7 @@ class ConfigurationServiceTest extends UnitTestCase
         $configurationService->saveConfiguration([], $requestProphecy->reveal());
 
         $expected = [
-            'TSFE_adminConfig' => [
+            'AdminPanel' => [
                 'foo' => 'bar',
                 'baz' => 'bam',
             ],
index 8e1a421..6afbfbc 100644 (file)
@@ -59,7 +59,7 @@ class ViewHelperResolver extends \TYPO3Fluid\Fluid\Core\ViewHelper\ViewHelperRes
     {
         $this->namespaces = $GLOBALS['TYPO3_CONF_VARS']['SYS']['fluid']['namespaces'];
         if (TYPO3_REQUESTTYPE & TYPO3_REQUESTTYPE_FE && $this->getBackendUser() instanceof BackendUserAuthentication) {
-            $configuration = $this->getBackendUser()->uc['TSFE_adminConfig'];
+            $configuration = $this->getBackendUser()->uc['AdminPanel'];
             if (isset($configuration['preview_showFluidDebug']) && $configuration['preview_showFluidDebug']) {
                 $this->namespaces['f'][] = 'TYPO3\\CMS\\Fluid\\ViewHelpers\\Debug';
             }