lookupTable and languagField / transOrigPointerField does not work correctly

[AndreasA]

The check in regards to the languageField is:

 if ($cfg['languageGetVar'] && $cfg['transOrigPointerField'] && $cfg['languageField'])

languageGetVar is wrong here because it is not part of the lookUpTable configuration but of the general pagePath setting. Simplest check is probably if $this->detectedLanguage >= 0

Might be used somewhere else in the method too.

[helhum]

Please add information on scenario, configuration, expectation and actual behavior

[AndreasA]

$GLOBALS['TYPO3_CONF_VARS']['EXTCONF']['realurl']['_DEFAULT'] = [
    'pagePath' => [
        'languageGetVar'    => 'L'
    ],
'postVarSets' => [
        '_DEFAULT' => [
            'news' => [
                [
                    'GETvar'      => 'tx_news_pi1[news]',
                    'lookUpTable' => [
                        'table'                 => 'tx_news_domain_model_news',
                        'id_field'              => 'uid',
                        'alias_field'           => 'title',
                        'addWhereClause'        => ' AND NOT deleted',
                        'useUniqueCache'        => 1,
                        'languageField'         => 'sys_language_uid',
                        'transOrigPointerField' => 'l10n_parent',
                        'useUniqueCache_conf'   => [
                            'strtolower'     => 1,
                            'spaceCharacter' => '-',
                        ],
                    ],
                ],
            ],
        ],
    ],

This configuration doesn't translate the news title to non-default language although languageField and transOrigPointerField are configured.

What is required is to modify the lookupTable configuration to add
'languageGetVar' => 'L',
like this:

 'news' => [
                [
                    'GETvar'      => 'tx_news_pi1[news]',
                    'lookUpTable' => [
                        'table'                 => 'tx_news_domain_model_news',
                        'id_field'              => 'uid',
                        'alias_field'           => 'title',
                        'addWhereClause'        => ' AND NOT deleted',
                        'useUniqueCache'        => 1,
                        'languageGetVar'        => 'L', 
                        'languageField'         => 'sys_language_uid',
                        'transOrigPointerField' => 'l10n_parent',
                        'useUniqueCache_conf'   => [
                            'strtolower'     => 1,
                            'spaceCharacter' => '-',
                        ],
                    ],
                ],

Which doesn't make sense as the languageGetVar is already specified in the pagePath section and it is also not mentioned in the realurl manual

I mean maybe that is the expected behaviour but I would expect the languageGetVar to be the one defined in pagePath to be used for lookupTable entries too.

Maybe with the possibility to override it but definitely not as requirement.