Bug #91761
openLinks with fragment and without parameters broken
0%
Description
When linking from a page with parameters to the same page without parameters but with a fragment, the link only contains the fragment. This means the browser does not follow the link, but stays on the same page.
This is a regression introduced with #89068
The problem is, that the check for queryParameters does not check if the current page has parameters:
PageLinkBuilder.php#L399
This breaks the "Return to login form" in felogin on the "Forgot password" page, if the following config is used:
plugin.tx_felogin_pi1.linkConfig.section.field = uid
The Back Link on the Forgot-Password Page will not lead back to the login form, because the tx_felogin_pi1[forgot]=1
parameter is still present in the url.
Current workaround is:
plugin.tx_felogin_pi1.linkConfig.forceAbsoluteUrl = 1
Updated by Robert Vock over 4 years ago
- Related to Bug #89068: New routing breaks fragment-only links added
Updated by Alexander Opitz about 4 years ago
Also working with RouteEnhancers got problematic.
We have
employees as page which shows content of listAction
employees/detail which is same page but inside the detailAction
On the employees page is a content element with a rte field which contains following link t3://page?uid=17#231 which leads to the output <a href="#231">List</a> so the user never comes back from RouteEnhancers subpath.
Updated by Oliver Hader about 2 years ago
- Sprint Focus set to On Location Sprint
Updated by Gerrit Code Review about 2 years ago
- Status changed from New to Under Review
Patch set 1 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/76794
Updated by Gerrit Code Review about 2 years ago
Patch set 2 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/76794
Updated by Gerrit Code Review almost 2 years ago
Patch set 3 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/76794
Updated by Gerrit Code Review almost 2 years ago
Patch set 4 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/76794
Updated by Gerrit Code Review almost 2 years ago
Patch set 5 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/76794
Updated by Gerrit Code Review almost 2 years ago
Patch set 6 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/76794
Updated by Gerrit Code Review almost 2 years ago
Patch set 7 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/76794
Updated by Gerrit Code Review almost 2 years ago
Patch set 8 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/76794
Updated by Gerrit Code Review over 1 year ago
Patch set 9 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/76794
Updated by Gerrit Code Review over 1 year ago
Patch set 10 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/76794
Updated by Benni Mack over 1 year ago
- Status changed from Under Review to Accepted
Updated by Benni Mack over 1 year ago
- Sprint Focus deleted (
On Location Sprint)