TYPO3 Core

I think I just found a bug with 'sys_language_mergeIfNotBlank' but I must say I'm a little confused currently of the way it is handled.

When fetching content with cType CONTENT, the query for selecting the content elements is built by a function 'tslib_content::getWhere' which (and only if $conf['languageField'] is expicit defined) switches the language-selection in the where-clause from the current sys_language to '0,-1' (default,all languages) which means that you get the content-element in the default language back and not the originaly asked one.

This element is later (in cType CONTENT) passed to another if-clause that merges the actual current sys_language over the given one, which HAS TO BE either 'default' or 'all languages' (therefore it had been switched before).

Isn't that double-reverse-thing a little unhandy?

Well, I think it is, as in this way it is impossible to get language-merged content-elements that are not passed through the CONTENT cType, which would be for example cType RECORDS. Because cType RECORDS can have direct references to translated content-elements which aren't fetched over the before mentioned function 'tslib_content::getWhere' which reverses the language.

So you will most likely not get language-merged content-elements over the function call 't3lib_page::getRecordOverlay' in cType RECORDS which leads to a big problem with all TemplaVoila sites, that fecht almost every content over cType RECORDS.

I stumbled over this as I coulnd't get 'sys_language_mergeIfNotBlank' work in TemplaVoila but it has worked with common rendering (page.30 < styles.content.get).

I'm no core developer, so I don't know what the actual idea was behind this double-language-conversion whatever thing, but I think it would be much wiser to remove this language conversion stuff and handle the overlay in one central function, which would be 't3lib_page::getRecordOverlay'.

The second thing is:
Currently the 'language conversion' in 'tslib_content::getWhere' is ONLY done when the given $conf-Array contains a field called 'languageField'. As I don't think that every developer is aware of this wouldn't it be better to simply check the TCA for the definition of that field (as it is done one line after: 6586, class.tslib_content.php) and optional check an override of the variable 'languageField'?

Here are my changes to my local system that made it work as it is supposed to be. I testet it with workspaces, versioning and language content-fallback. Everything seems to be ok, but it would be good if more people with different setups could test it before implementing in core.

----- class.tslib_content.php around line 6584 ------------

1. from:
   if ($conf['languageField']) {
   if ($GLOBALS['TSFE']->sys_language_contentOL && $TCA[$table] && $TCA[$table]['ctrl']['languageField'] && $TCA[$table]['ctrl']['transOrigPointerField']) {
   // Sys language content is set to zero/-1 - and it is expected that whatever routine processes the output will OVERLAY the records with localized versions!
   $sys_language_content = '0,-1';
   } else {
   $sys_language_content = intval($GLOBALS['TSFE']->sys_language_content);
   }
   $query.=' AND '.$conf['languageField'].' IN ('.$sys_language_content.')';
   }

1. to:
   if($TCA[$table] && ($TCA[$table]['ctrl']['languageField'] || $conf['languageField'])) {
   $lField = $conf['languageField']? $conf['languageField'] : $TCA[$table]['ctrl']['languageField'];
   $query.=' AND '.$lField.' IN ('.$GLOBALS['TSFE']->sys_language_content.')';
   }
   -----------------------------------------------------------
   maybe a additional check should be added if the defined 'languageField' in $conf really exists in $TCA[$table]['columns'].

----- class.t3lib_page.php line 324 -----------------------

1. from: if ($row[$TCA[$table]['ctrl']['languageField']]<=0) {
2. to: if ($row[$TCA[$table]['ctrl']['languageField']] > 0) {
   -----------------------------------------------------------

----- class.t3lib_page.php around line 330-----------------

1. from:
   ' AND '.$TCA[$table]['ctrl']['languageField'].'='.intval($sys_language_content).
   ' AND '.$TCA[$table]['ctrl']['transOrigPointerField'].'='.intval($row['uid']).

1. to:
   ' AND '.$TCA[$table]['ctrl']['languageField'].' IN (0,-1)'.
   ' AND uid='.intval($row['l18n_parent']).
   -----------------------------------------------------------

----- class.t3lib_page.php around line 345-----------------

1. from:
   if ($fN!='uid' && $fN!='pid' && isset($olrow[$fN])) {
   if ($GLOBALS['TSFE']->TCAcachedExtras[$table]['l10n_mode'][$fN]!='exclude'
   && ($GLOBALS['TSFE']->TCAcachedExtras[$table]['l10n_mode'][$fN]!='mergeIfNotBlank' || strcmp(trim($olrow[$fN]),''))) {
   $row[$fN] = $olrow[$fN];
   }
   }

1. to:
   if ($fN!='uid' && $fN!='pid' && isset($row[$fN])) {
   if (($GLOBALS['TSFE']->TCAcachedExtras[$table]['l10n_mode'][$fN]=='exclude' || $GLOBALS['TSFE']->TCAcachedExtras[$table]['l10n_mode'][$fN]=='mergeIfNotBlank') && !strcmp(trim($row[$fN]),'')) {
   $row[$fN] = $olrow[$fN];
   }
   }
   -----------------------------------------------------------

I'll provide diff-files as soon as I know how to create them correct.
(issue imported from #M2773)