[!!!][TASK] Remove deprecations in DataHandler class 03/52503/6
authorThomas Hohn <thomas@hohn.dk>
Wed, 19 Apr 2017 12:48:08 +0000 (14:48 +0200)
committerBenni Mack <benni@typo3.org>
Thu, 20 Apr 2017 10:44:20 +0000 (12:44 +0200)
Removed deprecated methods `rmComma, destPathFromUploadFolder,
noRecordsFromUnallowedTables` and property `$checkWorkspaceCache`.

In addition changes visibility for internal properties:
`$recUpdateAccessCache, $recInsertAccessCache, $isRecordInWebMount_Cache,
$isInWebMount_Cache, $cachedTSconfig, $pageCache`.

Resolves: #80709
Releases: master
Change-Id: I9432c9586ff3478128b5e349d43ec18d9ed5a3c6
Reviewed-on: https://review.typo3.org/52503
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Thomas Hohn <thomas@hohn.dk>
Tested-by: Thomas Hohn <thomas@hohn.dk>
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
typo3/sysext/core/Classes/DataHandling/DataHandler.php
typo3/sysext/core/Documentation/Changelog/master/Breaking-80700-DeprecatedFunctionalityRemoved.rst

index 801907d..47dff90 100644 (file)
@@ -469,64 +469,43 @@ class DataHandler
      * Used by function checkRecordUpdateAccess() to store whether a record is updatable or not.
      *
      * @var array
-     *
-     * @deprecated since TYPO3 v8, visibility will change to protected or to a run-time cache in TYPO3 v9
      */
-    public $recUpdateAccessCache = [];
+    protected $recUpdateAccessCache = [];
 
     /**
      * User by function checkRecordInsertAccess() to store whether a record can be inserted on a page id
      *
      * @var array
-     *
-     * @deprecated since TYPO3 v8, visibility will change to protected or to a run-time cache in TYPO3 v9
      */
-    public $recInsertAccessCache = [];
+    protected $recInsertAccessCache = [];
 
     /**
      * Caching array for check of whether records are in a webmount
      *
      * @var array
-     *
-     * @deprecated since TYPO3 v8, visibility will change to protected or to a run-time cache in TYPO3 v9
      */
-    public $isRecordInWebMount_Cache = [];
+    protected $isRecordInWebMount_Cache = [];
 
     /**
      * Caching array for page ids in webmounts
      *
      * @var array
-     *
-     * @deprecated since TYPO3 v8, visibility will change to protected or to a run-time cache in TYPO3 v9
      */
-    public $isInWebMount_Cache = [];
+    protected $isInWebMount_Cache = [];
 
     /**
      * Caching for collecting TSconfig for page ids
      *
      * @var array
-     *
-     * @deprecated since TYPO3 v8, visibility will change to protected or to a run-time cache in TYPO3 v9
      */
-    public $cachedTSconfig = [];
+    protected $cachedTSconfig = [];
 
     /**
      * Used for caching page records in pageInfo()
      *
      * @var array
-     *
-     * @deprecated since TYPO3 v8, visibility will change to protected or to a run-time cache in TYPO3 v9
-     */
-    public $pageCache = [];
-
-    /**
-     * Array caching workspace access for BE_USER
-     *
-     * @var array
-     *
-     * @deprecated since TYPO3 v8, no references could be found in class will be removed in TYPO3 v9
      */
-    public $checkWorkspaceCache = [];
+    protected $pageCache = [];
 
     // Other arrays:
     /**
@@ -7654,19 +7633,6 @@ class DataHandler
     }
 
     /**
-     * Returns the $input string without a comma in the end
-     *
-     * @param string $input Input string
-     * @return string Output string with any comma in the end removed, if any.
-     * @deprecated since TYPO3 v8, will be removed in TYPO3 v9
-     */
-    public function rmComma($input)
-    {
-        GeneralUtility::logDeprecatedFunction();
-        return rtrim($input, ',');
-    }
-
-    /**
      * Converts a HTML entity (like &#123;) to the character '123'
      *
      * @param string $input Input string
@@ -7689,19 +7655,6 @@ class DataHandler
     }
 
     /**
-     * Returns absolute destination path for the upload folder, $folder
-     *
-     * @param string $folder Upload folder name, relative to PATH_site
-     * @return string Input string prefixed with PATH_site
-     * @deprecated since TYPO3 v8, will be removed in TYPO3 v9, can be simplified by just prepending the PATH_site constant
-     */
-    public function destPathFromUploadFolder($folder)
-    {
-        GeneralUtility::logDeprecatedFunction();
-        return PATH_site . $folder;
-    }
-
-    /**
      * Disables the delete clause for fetching records.
      * In general only undeleted records will be used. If the delete
      * clause is disabled, also deleted records are taken into account.
@@ -8187,19 +8140,6 @@ class DataHandler
     }
 
     /**
-     * Used by the deleteFunctions to check if there are records from disallowed tables under the pages to be deleted.
-     *
-     * @param string $inList List of page integers
-     * @return bool Return TRUE, if permission granted
-     * @deprecated since TYPO3 v8, will be removed in TYPO3 v9
-     */
-    public function noRecordsFromUnallowedTables($inList)
-    {
-        GeneralUtility::logDeprecatedFunction();
-        return $this->checkForRecordsFromDisallowedTables(GeneralUtility::intExplode(',', $inList, true));
-    }
-
-    /**
      * Check if there are records from tables on the pages to be deleted which the current user is not allowed to
      *
      * @param int[] $pageIds IDs of pages which should be checked
index 1a82de8..44267a1 100644 (file)
@@ -53,6 +53,9 @@ The following PHP class methods that have been previously deprecated for v8 have
 * TYPO3\CMS\Core\Charset\CharsetConverter->utf8_strrpos()
 * TYPO3\CMS\Core\Charset\CharsetConverter->utf8_strtrunc()
 * TYPO3\CMS\Core\Charset\CharsetConverter->utf8_substr()
+* TYPO3\CMS\Core\DataHandling\DataHandler::rmComma()
+* TYPO3\CMS\Core\DataHandling\DataHandler::destPathFromUploadFolder()
+* TYPO3\CMS\Core\DataHandling\DataHandler::noRecordsFromUnallowedTables()
 * TYPO3\CMS\Core\TypoScript\TemplateService->splitConfArray()
 * TYPO3\CMS\Core\TypoScript\TemplateService->fileContent()
 * TYPO3\CMS\Core\TypoScript\TemplateService->removeQueryString()
@@ -154,6 +157,7 @@ The following methods changed signature according to previous deprecations in v8
 The following public class properties have been dropped:
 * TYPO3\CMS\Core\Charset\CharsetConverter->charSetArray
 * TYPO3\CMS\Core\Charset\CharsetConverter->fourByteSets
+* TYPO3\CMS\Core\DataHandling\DataHandler->checkWorkspaceCache
 * TYPO3\CMS\Frontend\Controller\TypoScriptFrontendController->compensateFieldWidth
 * TYPO3\CMS\Frontend\Controller\TypoScriptFrontendController->excludeCHashVars
 * TYPO3\CMS\Frontend\Controller\TypoScriptFrontendController->scriptParseTime
@@ -165,6 +169,14 @@ The following public class properties have been dropped:
 * TYPO3\CMS\Lang\LanguageService->moduleLabels
 * TYPO3\CMS\Lang\LanguageService->parserFactory
 
+The following class properties have changed visibility:
+* TYPO3\CMS\Core\DataHandling\DataHandler->recUpdateAccessCache changed from public to protected
+* TYPO3\CMS\Core\DataHandling\DataHandler->recInsertAccessCache changed from public to protected
+* TYPO3\CMS\Core\DataHandling\DataHandler->isRecordInWebMount_Cache changed from public to protected
+* TYPO3\CMS\Core\DataHandling\DataHandler->isInWebMount_Cache changed from public to protected
+* TYPO3\CMS\Core\DataHandling\DataHandler->cachedTSconfig changed from public to protected
+* TYPO3\CMS\Core\DataHandling\DataHandler->pageCache changed from public to protected
+
 The following configuration options are not evaluated anymore:
 * $TYPO3_CONF_VARS[SC_OPTIONS][GLOBAL][cliKeys]
 * $TYPO3_CONF_VARS[FE][noPHPscriptInclude]