[BUGFIX] SimpleFileBackend::setCacheDirectory returns valid path name 48/46148/6
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:05:07 +0000 (15:05 +0100)
commitb74bd367809083a6c582beeef64de1dd162bb99c
treea8f87f48d70ea254afc306a8774bb94777ba230c
parentce475a615135a71b95858845d720595a88768fc2
[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/46148
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
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