Bug #84688
closed
TYPO3\CMS\Core\Resource\File->getPublicUrl returns boolean instead of string
100%
Description
The getPublicUrl method of TYPO3\CMS\Core\Resource\File returns a boolean "false" if the file is missing (https://github.com/TYPO3/TYPO3.CMS/blob/master/typo3/sysext/core/Classes/Resource/File.php#L364).
This return type is wrong and breaks in TYPO3\CMS\Frontend\Typolink\FileOrFolderLinkBuilder with rawurldecode() (https://github.com/TYPO3/TYPO3.CMS/blob/master/typo3/sysext/frontend/Classes/Typolink/FileOrFolderLinkBuilder.php#L45).
Updated by Gerrit Code Review over 6 years ago
- Status changed from New to Under Review
Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/56643
Updated by Gerrit Code Review over 6 years ago
Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/56643
Updated by Gerrit Code Review over 6 years ago
Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/56643
Updated by Gerrit Code Review over 6 years ago
Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/56643
Updated by Gerrit Code Review over 6 years ago
Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/56643
Updated by Gerrit Code Review over 6 years ago
Patch set 1 for branch TYPO3_8-7 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/56672
Updated by Kevin Ditscheid over 6 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset a850e4ac1f4b3636118296d75c92b614dfa25494.
Updated by