Project

General

Profile

Actions

Bug #85576

closed

Linkvalidator not checking linked content elements with TypoLink

Added by Patrick Broens over 6 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Must have
Assignee:
-
Category:
Linkvalidator
Target version:
-
Start date:
2018-07-16
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
9
PHP Version:
7.2
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

If an anchor is added to a TypoLink, using the new TypoLink markup t3://page?uid=x#y, the content element is never checked on its existence.
The InternalLinktype is only checking the page, but not the content element, since it is only checking on recordRef = pages, while the SoftReferenceIndex is breaking this into pages and tt_content, calling the InternalLinkType twice.

How to reproduce:
Have a link in for instance the tt_content field "header_link" like t3://page?uid=x#y pointing to a valid page but not to a valid anchor
Run the checks in the module "Web > Info > Linkvalidator".
No error for the non-existing content element will show up.


Related issues 1 (0 open1 closed)

Related to TYPO3 Core - Bug #84016: impexp: page links are parsed / replaced incorrectely due to error in SoftReferenceIndexClosed2018-02-22

Actions
Actions #1

Updated by Patrick Broens over 6 years ago

  • Related to Bug #84016: impexp: page links are parsed / replaced incorrectely due to error in SoftReferenceIndex added
Actions #2

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/57612

Actions #3

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/57612

Actions #4

Updated by Patrick Broens over 6 years ago

  • Related to Epic #85006: Reduce falsely reported broken links added
Actions #5

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/57612

Actions #6

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/57612

Actions #7

Updated by Gerrit Code Review over 6 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/57612

Actions #8

Updated by Gerrit Code Review about 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/58735

Actions #9

Updated by Patrick Broens about 6 years ago

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

Updated by Gerrit Code Review about 6 years ago

  • Status changed from Resolved to Under Review

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

Actions #11

Updated by Patrick Broens about 6 years ago

  • Status changed from Under Review to Resolved
Actions #12

Updated by Benni Mack over 5 years ago

  • Status changed from Resolved to Closed
Actions #13

Updated by Sybille Peters about 5 years ago

  • Related to deleted (Epic #85006: Reduce falsely reported broken links)
Actions

Also available in: Atom PDF