[BUGFIX] SimpleFileBackend::setCacheDirectory returns valid path name 57/46157/2
authorJan Helke <typo3@helke.de>
Fri, 22 Jan 2016 09:58:24 +0000 (10:58 +0100)
committerFrank Nägler <frank.naegler@typo3.org>
Fri, 22 Jan 2016 14:12:26 +0000 (15:12 +0100)
commit344e1484eb51bfab2da1199df3f0310e13595772
treea7fd52498f4e6f118d6df0a15bfa1053bd7ac5b8
parentb1b74ee257238e1b729a4ac4fed1f01837d94ef4
[BUGFIX] SimpleFileBackend::setCacheDirectory returns valid path name

The comment for the method
\TYPO3\CMS\Core\Cache\Backend\SimpleFileBackend::setCacheDirectory()
states clearly, that it is possible to provide an absolute path as
cache directory. However, in the lower part of the function, it is
stated, that if ($cacheDirectory0 == '/') the documentRoot should be
set to '/'. That results in a returned path like '//...'.
This causes problems within
\TYPO3\CMS\Core\Utility\GeneralUtility::validPathStr(), because pathes
with '//' in it are always invalid.

Resolves: #72635
Releases: master, 7.6, 6.2
Change-Id: I30e7743ba9835c99382b7c04153f91e688ead5fc
Reviewed-on: https://review.typo3.org/46157
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Reviewed-by: Jan Helke <typo3@helke.de>
Reviewed-by: Frank Nägler <frank.naegler@typo3.org>
Tested-by: Frank Nägler <frank.naegler@typo3.org>
typo3/sysext/core/Classes/Cache/Backend/SimpleFileBackend.php
typo3/sysext/core/Tests/Unit/Cache/Backend/FileBackendTest.php