Bug #90916
closed
Page type external URL, removes fragments from relativ URLs
100%
Description
After updating to TYPO3 9.5.15
pages with external URL and a relativ URL with a Fragment do not resolve correctly
we have urls like this `/home/#fragment` used in Menus. to comunicate a the routing information to an Javascript app (angular).
before TYPO3 9.5.15 everything worked as expected.
as a fix we have changed the relativ URLs to Absolute URLs: `https://domain.tld/home/#fragment`
i suspect the following commit introduced the problem:
https://github.com/TYPO3/TYPO3.CMS/commit/0597c09389eacb9ce77ab32ae618171fe5cd350b
Updated by Christian Eßl over 4 years ago
- Related to Bug #75213: Missing Anchors for PDFs added
Updated by Riccardo De Contardi over 4 years ago
- Category set to Site Handling, Site Sets & Routing
Updated by Christian Eßl over 4 years ago
That's interesting. Page type "External Url" isn''t supposed to be used for relative urls, but apparently it just "works".
What happens is : because the url does not contain double slashes, that make it recognizable as "external url", it is then treated as "file or folder" in the code. The code then fails to find a file or folder for the url and then falls back to type "unknown" and just directly outputs the relative url without modifying it.
The url fragment changes were only made in the code that treats "file or folder" urls. And this now breaks the behaviour for this unusual use of the external doktype as well because of the reasons stated above.
Updated by Gerrit Code Review over 4 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/+/64411
Updated by Gerrit Code Review over 4 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/+/64411
Updated by Gerrit Code Review over 4 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/+/64411
Updated by Gerrit Code Review over 4 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/+/64411
Updated by Benni Mack over 4 years ago
- Related to Bug #91357: Typolink strips # added
Updated by Gerrit Code Review over 4 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/c/Packages/TYPO3.CMS/+/64411
Updated by Gerrit Code Review over 4 years ago
Patch set 6 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/+/64411
Updated by Gerrit Code Review over 4 years ago
Patch set 7 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/+/64411
Updated by Gerrit Code Review over 4 years ago
Patch set 8 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/+/64411
Updated by Gerrit Code Review over 4 years ago
Patch set 9 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/+/64411
Updated by Gerrit Code Review over 4 years ago
Patch set 1 for branch 9.5 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/64368
Updated by Benni Mack over 4 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 71440487f772fa8404fb5ac894914d487ff40c27.