Bug #106661
opentypolink softref parser handles links of type "record" but typolink_tag does not
0%
Description
The softref parser TypolinkSoftReferenceParser specifically handles links of type LinkService::TYPE_RECORD. The softref parser TypolinkTagSoftReferenceParser (used for example in RTE) does not.
Reproduce¶
- use EXT:news or some other extension which allows links of type record.
- for news, configure it in TSConfig (see below)
- add a news entry
- now create a "textmedia" content element with link to news in the bodytext (via RTE) => uid=84
- create another textmedia content element. This time add the link to the header_link field => uid=85
- check reference index
SELECT * FROM sys_refindex WHERE tablename = 'tt_content' AND ref_table='tx_news_domain_model_news';
Expected results: there are 2 new entries in refindex for both links
Actual results: There is only one new entry for the link in header_link.
TSconfig linkhandler config:
TCEMAIN{ linkHandler{ tx_news { handler = TYPO3\CMS\Backend\LinkHandler\RecordLinkHandler label = News configuration { table = tx_news_domain_model_news # set storagePid to page of news entries storagePid = 250 # Hide the page tree by setting it to 1 hidePageTree = 0 } scanAfter = page } } }
Impact¶
- if record-links are detected by typolink soft reference parser, they are written to the reference index, if detected by typolink_tag parser, they are not. This is inconsistent
- may also have impact in linkvalidator that record links in RTE fields are not detected (though the record link checking currently does not work anyway, see other issues, e.g. https://forge.typo3.org/issues/103403)
Versions¶
reproduced with:
- version 12.4.25-dev (latest 12.4 branch)
- version 13.4.10-dev (latest 13.4 branch)
Did not test with latest v14 / main, as there is no working "news" available yet.
Updated by Sybille Peters 13 days ago
- Related to Feature #103403: Make it possible to check custom record links with linkvalidator added
Updated by Sybille Peters 13 days ago
- Related to Bug #102468: Linkvalidator doesn't check record/custom links within RTEs at all added
Updated by Gerrit Code Review 13 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/+/89312
Updated by Gerrit Code Review 12 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/+/89312
Updated by Gerrit Code Review 12 days 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/+/89312
Updated by Gerrit Code Review 12 days ago
Patch set 4 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/+/89312
Updated by Gerrit Code Review 12 days ago
Patch set 5 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/+/89312
Updated by Gerrit Code Review 12 days ago
Patch set 6 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/+/89312