Bug #90850

typolink generates links for non-reachable pages

Added by Markus Klein over 1 year ago. Updated over 1 year ago.

Status:
Closed
Priority:
Must have
Assignee:
-
Category:
Content Rendering
Target version:
-
Start date:
2020-03-27
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
8
PHP Version:
Tags:
Complexity:
medium
Is Regression:
Sprint Focus:

Description

Problem

typolink generates URLs to pages, which end up in a 404 error when using this link

Setup

FE|hidePagesIfNotTranslatedByDefault = true
fallbackType: fallback (in site config)

Reproduction

  • Create pages "A" and "B".
  • Add content (e.g. header CE) to both pages
  • Create a link from page "A" to "B" in the content
  • Translate page "A".
  • View page "A" in FE in the translated language
  • Click the link

Expected behaviour

If the link target cannot be reached by definition, the link must not be created at all.

Notes

If FE|hidePagesIfNotTranslatedByDefault = false then the created link is of course legitimate as the fallback content will be shown.

The same problem exists if fallbackType: strict (independent of FE|hidePagesIfNotTranslatedByDefault)


Related issues

Related to TYPO3 Core - Bug #87260: Untranslated records should not be linkedUnder ReviewMathias Brodala2018-12-21

Actions
Related to TYPO3 Core - Bug #85063: \TYPO3\CMS\Frontend\Typolink\PageLinkBuilder::build doesn't honor l18n_cfg settingsClosed2018-05-22

Actions
#1

Updated by Markus Klein over 1 year ago

  • Related to Bug #87260: Untranslated records should not be linked added
#2

Updated by Markus Klein over 1 year ago

  • Description updated (diff)
#3

Updated by Gerrit Code Review over 1 year 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/+/63946

#4

Updated by Markus Klein over 1 year ago

  • Related to Bug #85063: \TYPO3\CMS\Frontend\Typolink\PageLinkBuilder::build doesn't honor l18n_cfg settings added
#5

Updated by Gerrit Code Review over 1 year 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/c/Packages/TYPO3.CMS/+/63950

#6

Updated by Gerrit Code Review over 1 year 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/+/63946

#7

Updated by Gerrit Code Review over 1 year 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/+/64058

#8

Updated by Gerrit Code Review over 1 year ago

Patch set 2 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/+/64058

#9

Updated by Gerrit Code Review over 1 year 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/+/63946

#10

Updated by Gerrit Code Review over 1 year ago

Patch set 3 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/+/64058

#11

Updated by Markus Klein over 1 year ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
#12

Updated by Benni Mack over 1 year ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF