Task #82669
closedResolve backend route path inconsistencies
100%
Description
There are inconsistencies in working with route path names depending on how these routes have been provided:
- path
/file/editcontent
(no trailing slash) is the result of a direct route configuration in typo3_src/typo3/sysext/backend/Configuration/Backend/Routes.php - path
/web/WorkspacesWorkspaces/
(with trailing slash) is the result of a backend module configuration which builds the path name internally- see \TYPO3\CMS\Extbase\Utility\ExtensionUtility::registerModule
- see \TYPO3\CMS\Core\Utility\ExtensionManagementUtility::addModule
- find
$path = '/' . trim($path, '/') . '/';
- thus always containing a slash in the end
To have a proper schema of using routes, either the slash should be stripped from the end or added in all cases.
Updated by Gerrit Code Review over 5 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/c/Packages/TYPO3.CMS/+/60433
Updated by Gerrit Code Review over 5 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/c/Packages/TYPO3.CMS/+/60433
Updated by Gerrit Code Review over 5 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/c/Packages/TYPO3.CMS/+/60433
Updated by Gerrit Code Review over 5 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/c/Packages/TYPO3.CMS/+/60433
Updated by Benni Mack over 5 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 351e83fd62164ba6b11e0ccac63d3e4b6bf1c255.