Bug #85302
closedResourceStorage throws wrong exception on folder move between storages
100%
Description
The method ResourceStorage::moveFolderBetweenStorages()
is currently not implemented and it throws a RuntimeException
.
However, the ExtendedFileUtility::func_move()
method implies that this should be a BadMethodCallException
instead:
catch (\BadMethodCallException $e) {
$this->writeLog(3, 1, 126, 'The function to move a file between storages is not yet implemented', []);
$this->addMessageToFlashMessageQueue('FileUtility.TheFunctionToMoveAFileBetweenStoragesIsNotYetImplemented');
}
Due to this users get a misleading error messages which hints at write permission problems (FileUtility.DirectoryWasNotMovedTo
) instead of the dedicated message which informs that this action is not yet implemented (FileUtility.TheFunctionToMoveAFileBetweenStoragesIsNotYetImplemented
).
Updated by Mathias Brodala over 6 years ago
- Related to Task #54230: Refactor Driver API added
Updated by Mathias Brodala over 6 years ago
- Related to Task #54231: The Driver Registry should check for the implementation of the Interface added
Updated by Mathias Brodala over 6 years ago
- Related to deleted (Task #54231: The Driver Registry should check for the implementation of the Interface)
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/57258
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/57258
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/57258
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/57258
Updated by Mathias Brodala over 6 years ago
- Sprint Focus set to On Location Sprint
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/57309
Updated by Mathias Brodala over 6 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset c2d666da5ff537e1f17763c81f12217b86b348a4.