Bug #83835
openLinkvalidator should check more fields by default, such as sys_file_reference.link
0%
Description
Tested (25.03.2024): should just work if added to the searchFields (with TYPO3 v12 and v13):
mod.linkvalidator.serachFields.sys_file_reference = link
Since TYPO3 v12, linkvalidator also checks fields if type = "link".
Reproduce¶
1. Change default page TSconfig for linkvalidator to include sys_file_reference:
mod.linkvalidator { searchFields { pages = media,url tt_content = bodytext,header_link,records sys_file_reference = link } ... }
2. Create a "text & images" or "text & media", add an image, add a dead (not existing link) in the "Link" field for the image metadata.
3. Restart "Check link", the dead link should now appear in the Reports
Results¶
Clicking the pen will jump to the edit wizard for sys_file_reference, but you can't change the link there, you need the edit wizard for tt_content.
So, Linkvalidator does not properly handle sys_file_reference records (and probably wasn't intended to because it was developed before FAL).
Files
Updated by Sybille Peters over 6 years ago
- File linkval4.png linkval4.png added
- Description updated (diff)
Updated by Lina Wolf almost 5 years ago
- Related to Feature #89177: Change TsConfig Defaults of Linkvalidator and Enable all core fields containing links added
Updated by Riccardo De Contardi over 4 years ago
- Related to Bug #90760: Link validator throws exception with Page TSConfig configuration on sys_file_reference added
Updated by Sybille Peters 5 months ago
- Subject changed from Linkvalidator should handle sys_file_reference to Linkvalidator should check in sys_file_reference.link
Updated by Sybille Peters 5 months ago
- Related to Task #103477: Documentation for Link Validator searchFields says it is only possible to check fields with a defined "softref" added
Updated by Sybille Peters 5 months ago
- Related to Bug #103478: Linkvalidator should check fields with type "file" added
Updated by Gerrit Code Review 5 months 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/+/83612
Updated by Gerrit Code Review 5 months 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/+/83612
Updated by Sybille Peters 5 months ago
- Related to Bug #103494: Linkvalidator uses tstamp field directly without checking TCA in recheckLinks added
Updated by Sybille Peters 5 months ago
- Subject changed from Linkvalidator should check in sys_file_reference.link to Linkvalidator should check more fields by default, such as sys_file_reference.link
Updated by Sybille Peters 4 months ago
There is a problem when checking sys_file_reference.link, issue https://forge.typo3.org/issues/103559 must be fixed first
Updated by Sybille Peters 4 months ago
- Blocked by Bug #103559: Opening broken link list as non-admin user throws exception if searchFields.sys_file_reference = link is set added
Updated by Gerrit Code Review about 1 month 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/+/83612