[TASK] Mark some places internal instead of deprecated 85/59485/2
authorChristian Kuhn <lolli@schwarzbu.ch>
Fri, 18 Jan 2019 13:27:28 +0000 (14:27 +0100)
committerWouter Wolters <typo3@wouterwolters.nl>
Fri, 18 Jan 2019 14:22:35 +0000 (15:22 +0100)
Some methods in BasicFileUtility have been marked as
deprecated for various versions already but never removed
since they are still used within core.
Switch these to @internal to give core freedom to remove
these at will.

Resolves: #87487
Releases: master
Change-Id: I837dfbbfdc7069dd89f86133c75ad9b0fd45e046
Reviewed-on: https://review.typo3.org/59485
Tested-by: TYPO3com <noreply@typo3.com>
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
typo3/sysext/core/Classes/Utility/File/BasicFileUtility.php

index 1f50b0a..f7ce30b 100644 (file)
@@ -116,8 +116,7 @@ class BasicFileUtility
      * @param string $_ not in use anymore
      * @param string $filename Filename to check against TYPO3_CONF_VARS[BE][fileDenyPattern]
      * @return bool TRUE if extension/filename is allowed
-     * @todo Deprecate, but still in use by DataHandler
-     * @deprecated but still in use in the Core. Don't use in your extensions!
+     * @internal May be removed without further notice. Method has been marked as deprecated for various versions but is still used in core.
      */
     public function checkIfAllowed($ext, $_, $filename = '')
     {
@@ -152,8 +151,7 @@ class BasicFileUtility
      * @param bool $dontCheckForUnique If set the filename is returned with the path prepended without checking whether it already existed!
      * @return string The destination absolute filepath (not just the name!) of a unique filename/foldername in that path.
      * @see \TYPO3\CMS\Core\DataHandling\DataHandler::checkValue()
-     * @todo Deprecate, but still in use by the Core (DataHandler...)
-     * @deprecated but still in use in the Core. Don't use in your extensions!
+     * @internal May be removed without further notice. Method has been marked as deprecated for various versions but is still used in core.
      */
     public function getUniqueName($theFile, $theDest, $dontCheckForUnique = false)
     {
@@ -206,8 +204,7 @@ class BasicFileUtility
      *
      * @param string $fileName Input string, typically the body of a filename
      * @return string Output string with any characters not matching [.a-zA-Z0-9_-] is substituted by '_' and trailing dots removed
-     * @todo Deprecate, but still in use by the core
-     * @deprecated but still in use in the Core. Don't use in your extensions!
+     * @internal May be removed without further notice. Method has been marked as deprecated for various versions but is still used in core.
      */
     public function cleanFileName($fileName)
     {