Project

General

Profile

Actions

Bug #106661

open

typolink softref parser handles links of type "record" but typolink_tag does not

Added by Sybille Peters 13 days ago. Updated 12 days ago.

Status:
Under Review
Priority:
Should have
Assignee:
-
Category:
Link Handling & Redirect Handling
Target version:
-
Start date:
2025-05-07
Due date:
% Done:

0%

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

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

  1. use EXT:news or some other extension which allows links of type record.
  2. for news, configure it in TSConfig (see below)
  3. add a news entry
  4. now create a "textmedia" content element with link to news in the bodytext (via RTE) => uid=84
  5. create another textmedia content element. This time add the link to the header_link field => uid=85
  6. 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.


Related issues 2 (2 open0 closed)

Related to TYPO3 Core - Feature #103403: Make it possible to check custom record links with linkvalidatorUnder ReviewSybille Peters2024-03-15

Actions
Related to TYPO3 Core - Bug #102468: Linkvalidator doesn't check record/custom links within RTEs at allNew2023-11-22

Actions
Actions #1

Updated by Sybille Peters 13 days ago

  • Description updated (diff)
Actions #2

Updated by Sybille Peters 13 days ago

  • Related to Feature #103403: Make it possible to check custom record links with linkvalidator added
Actions #3

Updated by Sybille Peters 13 days ago

  • Related to Bug #102468: Linkvalidator doesn't check record/custom links within RTEs at all added
Actions #4

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

Actions #5

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

Actions #6

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

Actions #7

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

Actions #8

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

Actions #9

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

Actions

Also available in: Atom PDF