[BUGFIX] Minor code cleanups 69/59269/2
authorChristian Kuhn <lolli@schwarzbu.ch>
Sat, 22 Dec 2018 12:20:04 +0000 (13:20 +0100)
committerAnja Leichsenring <aleichsenring@ab-softlab.de>
Sat, 22 Dec 2018 14:24:21 +0000 (15:24 +0100)
A small series of minor cleanups:
* Setting a no longer existing property
* Slight variable type improvements
* Unused variable assignment

Resolves: #87267
Releases: master
Change-Id: I07c43b31b8fb2d8bacd80dc023595860fa77abf4
Reviewed-on: https://review.typo3.org/59269
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Alexander Schnitzler <typo3@alexanderschnitzler.de>
Tested-by: Alexander Schnitzler <typo3@alexanderschnitzler.de>
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
typo3/sysext/adminpanel/Classes/Utility/StateUtility.php
typo3/sysext/backend/Classes/Controller/EditDocumentController.php
typo3/sysext/core/Classes/Authentication/AbstractUserAuthentication.php
typo3/sysext/core/Classes/Authentication/BackendUserAuthentication.php

index b04a26e..4149030 100644 (file)
@@ -35,8 +35,6 @@ class StateUtility
         $beUser = $GLOBALS['BE_USER'] ?? null;
         if ($beUser instanceof FrontendBackendUserAuthentication) {
             $adminPanelConfiguration = $beUser->getTSConfig()['admPanel.'] ?? [];
-            // set legacy config
-            $beUser->extAdminConfig = $adminPanelConfiguration;
             if (isset($adminPanelConfiguration['enable.'])) {
                 // only enabled if at least one module is enabled.
                 return (bool)array_filter($adminPanelConfiguration['enable.']);
index 4368192..fbf37a5 100644 (file)
@@ -726,7 +726,7 @@ class EditDocumentController
             || isset($parsedBody['_saveandclosedok'])
         ) {
             // Redirect if element should be closed after save
-            return $this->closeDocument(abs($this->closeDoc), $request);
+            return $this->closeDocument((int)abs($this->closeDoc), $request);
         }
         return null;
     }
index ef481be..6d42bff 100644 (file)
@@ -1409,8 +1409,8 @@ abstract class AbstractUserAuthentication implements LoggerAwareInterface
      * @param string $details Default text that follows the message
      * @param array $data Data that follows the log. Might be used to carry special information. If an array the first 5 entries (0-4) will be sprintf'ed the details-text...
      * @param string $tablename Special field used by tce_main.php. These ($tablename, $recuid, $recpid) holds the reference to the record which the log-entry is about. (Was used in attic status.php to update the interface.)
-     * @param int $recuid Special field used by tce_main.php. These ($tablename, $recuid, $recpid) holds the reference to the record which the log-entry is about. (Was used in attic status.php to update the interface.)
-     * @param int $recpid Special field used by tce_main.php. These ($tablename, $recuid, $recpid) holds the reference to the record which the log-entry is about. (Was used in attic status.php to update the interface.)
+     * @param int|string $recuid Special field used by tce_main.php. These ($tablename, $recuid, $recpid) holds the reference to the record which the log-entry is about. (Was used in attic status.php to update the interface.)
+     * @param int|string $recpid Special field used by tce_main.php. These ($tablename, $recuid, $recpid) holds the reference to the record which the log-entry is about. (Was used in attic status.php to update the interface.)
      */
     public function writelog($type, $action, $error, $details_nr, $details, $data, $tablename, $recuid, $recpid)
     {
index 17db690..8bc97ae 100644 (file)
@@ -1299,7 +1299,6 @@ class BackendUserAuthentication extends AbstractUserAuthentication
             // Parsing the user TSconfig (or getting from cache)
             $hash = md5('userTS:' . $this->userTS_text);
             $cache = GeneralUtility::makeInstance(CacheManager::class)->getCache('cache_hash');
-            $cachedContent = $cache->get($hash);
             $parseObj = GeneralUtility::makeInstance(\TYPO3\CMS\Core\TypoScript\Parser\TypoScriptParser::class);
             $parseObj->parse($this->userTS_text);
             $this->userTS = $parseObj->setup;