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 about 6 years ago
- File linkval4.png linkval4.png added
- Description updated (diff)
Updated by Lina Wolf over 4 years ago
- Related to Feature #89177: Change TsConfig Defaults of Linkvalidator and Enable all core fields containing links added
Updated by Riccardo De Contardi about 4 years ago
- Related to Bug #90760: Link validator throws exception with Page TSConfig configuration on sys_file_reference added
Updated by Sybille Peters 25 days ago
- Subject changed from Linkvalidator should handle sys_file_reference to Linkvalidator should check in sys_file_reference.link
Updated by Sybille Peters 25 days 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 25 days ago
- Related to Bug #103478: Linkvalidator should check fields with type "file" added
Updated by Gerrit Code Review 23 days 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 23 days 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 23 days ago
- Related to Bug #103494: Linkvalidator uses tstamp field directly without checking TCA in recheckLinks added
Updated by Sybille Peters 23 days 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 11 days 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 11 days ago
- Blocked by Bug #103559: Opening broken link list as non-admin user throws exception if searchFields.sys_file_reference = link is set added