[!!!][TASK] Remove option SYS/maxFileNameLength 10/45710/2
authorBenni Mack <benni@typo3.org>
Fri, 8 Jan 2016 14:01:39 +0000 (15:01 +0100)
committerChristian Kuhn <lolli@schwarzbu.ch>
Fri, 8 Jan 2016 15:48:25 +0000 (16:48 +0100)
The option $TYPO3_CONF_VARS[SYS][maxFileNameLength]
is not in use anymore and is removed.

Resolves: #72604
Releases: master
Change-Id: Ie9fa64e38bea8d5775aff5c897d36339a10c9ec8
Reviewed-on: https://review.typo3.org/45710
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Oliver Klee <typo3-coding@oliverklee.de>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
typo3/sysext/core/Classes/Utility/File/BasicFileUtility.php
typo3/sysext/core/Configuration/DefaultConfiguration.php
typo3/sysext/core/Documentation/Changelog/master/Breaking-72604-RemoveOptionMaxFileNameLength.rst [new file with mode: 0644]
typo3/sysext/install/Classes/Service/SilentConfigurationUpgradeService.php

index e1058d9..6f11491 100644 (file)
@@ -58,13 +58,6 @@ class BasicFileUtility
     public $uniquePrecision = 6;
 
     /**
-     * This is the maximum length of names treated by cleanFileName()
-     *
-     * @var int
-     */
-    public $maxInputNameLen = 60;
-
-    /**
      * Temp-foldername. A folder in the root of one of the mounts with this name is regarded a TEMP-folder (used for upload from clipboard)
      *
      * @var string
@@ -146,7 +139,6 @@ class BasicFileUtility
         $this->mounts = (!empty($mounts) ? $mounts : array());
         $this->webPath = GeneralUtility::getIndpEnv('TYPO3_DOCUMENT_ROOT');
         $this->isInit = 1;
-        $this->maxInputNameLen = $GLOBALS['TYPO3_CONF_VARS']['SYS']['maxFileNameLength'] ?: $this->maxInputNameLen;
     }
 
     /**
index fcee59d..d92121f 100644 (file)
@@ -100,7 +100,6 @@ return array(
         'systemLog' => '',                        // <p>String: semi-colon separated list. Defines one or more logging methods. Possible methods:</p><dl><dt>file,&lt;abs-path-to-file&gt;[,&lt;level&gt;]</dt><dd>logs to a file</dd><dt>mail,&lt;to&gt;[/&lt;from&gt;][,&lt;level&gt;]</dt><dd>sends the log entries via mail</dd><dt>syslog,&lt;facility&gt;,[,&lt;level&gt;]</dt><dd>uses the operating system's log. Facility may be one of LOCAL0..LOCAL7, USER (on Windows USER is the only valid type).</dd><dt>error_log[,,&lt;level&gt;]</dt><dd>uses the PHP error log</dd></dl><p>The &lt;level&gt; is the individual logging level (see <a href="#SYS-systemLogLevel">[SYS][systemLogLevel]</a>).</p>
         'systemLogLevel' => 0,                    // <p>Integer (0, 1, 2, 3, 4): Only messages with same or higher severity are logged.</p><ul><li>0: info</li><li>1: notice</li><li>2: warning</li><li>3: error</li><li>4: fatal error</li></ul>
         'enableDeprecationLog' => '',                // If set, this configuration enables the logging of deprecated methods and functions. The following options are allowed: <dl><dt>String: &quot;file&quot; (or integer &quot;1&quot;)</dt><dd>The log file will be written to typo3conf/deprecation_[hash-value].log</dd><dt>String: &quot;devlog&quot;</dt><dd>The log will be written to the development log</dd><dt>String: &quot;console&quot;<dt><dd>The log will be displayed in the Backend's Debug Console.</dd></dl>Logging options &quot;file&quot;, &quot;devlog&quot; and &quot;console&quot; can be combined by comma-separating them.
-        'maxFileNameLength' => 60,                // Integer: This is the maximum file name length. The value will be taken into account by basic file operations like renaming or creation of files and folders.
         'UTF8filesystem' => false,                // Boolean: If TRUE then TYPO3 uses utf-8 to store file names. This allows for accented Latin letters as well as any other non-latin characters like Cyrillic and Chinese.
         'systemLocale' => '',                    // String: locale used for certain system related functions, e.g. escaping shell commands. If problems with filenames containing special characters occur, the value of this option is probably wrong. See <a href="http://php.net/manual/en/function.setlocale.php" target="_blank">setlocale()</a>.
         'reverseProxyIP' => '',                    // String: list of IP addresses. If TYPO3 is behind one or more (intransparent) reverse proxies the IP addresses must be added here.
diff --git a/typo3/sysext/core/Documentation/Changelog/master/Breaking-72604-RemoveOptionMaxFileNameLength.rst b/typo3/sysext/core/Documentation/Changelog/master/Breaking-72604-RemoveOptionMaxFileNameLength.rst
new file mode 100644 (file)
index 0000000..e3cc2cf
--- /dev/null
@@ -0,0 +1,14 @@
+==================================================
+Breaking: #72604 - Remove option maxFileNameLength
+==================================================
+
+Description
+===========
+
+The option ``$TYPO3_CONF_VARS[SYS][maxFileNameLength]`` was removed.
+
+
+Impact
+======
+
+Using the option will have no effect anymore.
\ No newline at end of file
index 338b8f6..78eb595 100755 (executable)
@@ -79,6 +79,8 @@ class SilentConfigurationUpgradeService
         // #72337
         'SYS/t3lib_cs_utils',
         'SYS/t3lib_cs_convMethod',
+        // #72604
+        'SYS/maxFileNameLength',
     );
 
     /**