Bug #52917

Broken links for the page record show up on the parent page

Added by Dmitry Dulepov about 7 years ago. Updated 11 months ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Linkvalidator
Target version:
-
Start date:
2013-10-17
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
10
PHP Version:
Tags:
Complexity:
Is Regression:
No
Sprint Focus:

Description

If broken links are in the pages table, they will not be displayed in the report for this page but will appear on the parent page.


Files

52917.diff (877 Bytes) 52917.diff Dmitry Dulepov, 2013-10-17 12:58

Related issues

Related to TYPO3 Core - Bug #89267: Broken links found in pages table are associated with wrong pageClosedSybille Peters2019-09-25

Actions
#1

Updated by Dmitry Dulepov about 7 years ago

Added patch.

#2

Updated by Michael Stucki almost 7 years ago

  • Category set to Linkvalidator
#3

Updated by Michael Stucki almost 7 years ago

  • Project changed from Linkvalidator to TYPO3 Core
  • Category changed from Linkvalidator to Linkvalidator
#4

Updated by Mathias Schreiber almost 6 years ago

  • Target version set to 7.4 (Backend)
  • Is Regression set to No
#5

Updated by Susanne Moog over 5 years ago

  • Target version changed from 7.4 (Backend) to 7.5
#6

Updated by Benni Mack about 5 years ago

  • Target version deleted (7.5)
#7

Updated by Lina Ourima about 1 year ago

  • TYPO3 Version changed from 6.2 to 10

This is still the case on 10.1.0dev

#8

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

#9

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

#10

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

#11

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

#12

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

#13

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

#14

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

#15

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

#16

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

#17

Updated by Gerrit Code Review about 1 year ago

Patch set 10 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/+/61709

#18

Updated by Gerrit Code Review about 1 year ago

Patch set 11 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/+/61709

#19

Updated by Sybille Peters about 1 year ago

Can you add how to reproduce the problem, specifically how to produce a detecable broken link for pages? Thanks.

#20

Updated by Lina Ourima about 1 year ago

To reproduce this in TYPO3 10, configure the linkvalidator to check a field of pages that has softref = typolink in its TCA configuration i.e. canonical_link. Please note that the field has to be configured via PageTS Config:

mod.linkvalidator.searchFields {
pages = canonical_link
}

Create a rootpage and a subpage of the rootpage. On the subpage of the rootpage fill the field of the canonical link with an url pointing to a page which returns 404 i.e. https://typo3.org/xyz/ go to the subpage and then check for broken links in the linkvalidator. The broken canonical url is not visible. Now go one level up to the rootpage and hit refresh display. The link is visible.

After applying the patch the broken link is also visible when you hit refresh on the subpage itself.

#21

Updated by Gerrit Code Review about 1 year ago

Patch set 12 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/+/61709

#22

Updated by Sybille Peters about 1 year ago

  • Related to Bug #89267: Broken links found in pages table are associated with wrong page added
#23

Updated by Gerrit Code Review about 1 year ago

Patch set 13 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/+/61709

#24

Updated by Gerrit Code Review about 1 year ago

Patch set 14 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/+/61709

#25

Updated by Gerrit Code Review about 1 year ago

Patch set 15 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/+/61709

#26

Updated by Gerrit Code Review about 1 year ago

Patch set 16 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/+/61709

#27

Updated by Gerrit Code Review about 1 year ago

Patch set 17 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/+/61709

#28

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

#29

Updated by Anonymous about 1 year ago

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

Updated by Benni Mack 11 months ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF