TYPO3 Core

There won't be any change for 10 in that regard because it is not that critical.

I would considere this critical, as all links to translated records/pages that have fallbacks are gone! But those a very commen (e.g. pt_BR or en_US overlays, that hardly have translation records).

 if ($languageAspect->doOverlays()) {
                $overlay = $tsfe->sys_page->getRecordOverlay(
                    $databaseTable,
                    $record,
                    $languageAspect->getContentId(),
                    $languageAspect->getLegacyOverlayType()
                );

                if (empty($overlay['_LOCALIZED_UID'])) {
                    $record = 0;
                }
            }

This whole code DatabaseRecordLinkBuilder.php:58ff is a mystery. We load the overlay (which is expensive!) just to check that it exists, but only if we want to do overlay - which means that we work based on the default language anyways - then why check for the overlay record at all? Just link to record with its default uid - always.

Hi,

this bug can also be found in Typo3 version 11.

I've managed to resolve it by replacing

typo3/sysext/frontend/Classes/Typolink/DatabaseRecordLinkBuilder.php

Line 56-68

            $languageAspect = $tsfe->getContext()->getAspect('language');

            if ($languageAspect->doOverlays()) {
                $overlay = $tsfe->sys_page->getRecordOverlay(
                    $databaseTable,
                    $record,
                    $languageAspect
                );

                if (empty($overlay['_LOCALIZED_UID'])) {
                    $record = 0;
                }
            }

with

            $tableControl = $GLOBALS['TCA'][$databaseTable]['ctrl'] ?? [];
            if (!empty($tableControl['languageField'])) {
              if ($record[$tableControl['languageField']] != 0 && !empty($record[$tableControl['transOrigPointerField']])) {
                $record = $tsfe->sys_page->checkRecord($databaseTable, $record[$tableControl['transOrigPointerField']]);
              }
            }

Please have a look

Thanks

To catch records with language all -1 the line

if ($record[$tableControl['languageField']] != 0 && !empty($record[$tableControl['transOrigPointerField']])) {

from my patch needs to be changed to

if ($record[$tableControl['languageField']] > 0 && !empty($record[$tableControl['transOrigPointerField']])) {

Just link to record with its default uid - always

This was the case in old TYPO3 versions. You linked to the default language UID and the L-parameter defined the language.
That's where this comes from. (I'm very sure).