TYPO3 Core

Hi Kai,

this same problem accours to me, with the latest maintenance release of the 7 version starting with 7.6.17+.
This is really bad, because such a thing sneaks in into production system easily (which happened to us).

Uwe

After some digging I found the part, which breaks the behavoir.
In TYPO3\CMS\Backend\Form\FormDataProvider\DatabaseEditRow::addData on line 34 the code now look like

        if ($result['command'] !== 'edit' || !empty($result['databaseRow'])) {

before that it was

        if ($result['command'] !== 'edit') {

Removing the second condition makes the localize all function work again.

The code change was introduced here:
https://review.typo3.org/#/c/52003/

I just found out, the transOrigPointerField field is not set, too. Even with the applied patch. Could that be a related problem?

Hi Guys,

I just also want to confirm that "localize all records" is broken with version 7.6.17 and above.
Rolling back to 7.6.16 makes it work again.

I also can confirm the mentioned behavior for example when you want to localize fal_media in news.

Thanks in advance to roll this functionality back.

Cheers,
merzilla

I could also reproduce this error in TYPO3 8.7.1 LTS

I can confirm that the problem is definitely in change https://review.typo3.org/#/c/52003/5 as described at https://forge.typo3.org/issues/80855#note-2

Ajax handler for "Localize" and "Synchronoize" buttons does not add the full record for the database row, it just sets the uid:

FormInlineAjaxController::synchronizeLocalizeAction:

$formDataCompilerInputForParent = [
                'vanillaUid' => (int)$parent['uid'],
                'command' => 'edit',
                'tableName' => $parent['table'],
                'databaseRow' => [
                    // TcaInlineExpandCollapseState needs this
                    'uid' => (int)$parent['uid'],
                ],
                'inlineFirstPid' => $inlineFirstPid,
                'columnsToProcess' => [
                    $parentFieldName
                ],
                // @todo: still needed? NO!
                'inlineStructure' => $inlineStackProcessor->getStructure(),
                // Do not compile existing children, we don't need them now
                'inlineCompileExistingChildren' => false,
            ];

So that change breaks localization in forms.

So the actual bug is

'databaseRow' => [
    // TcaInlineExpandCollapseState needs this
    'uid' => (int)$parent['uid'],
],

'databaseRow' => $parent

IMHO this should be fixed in all those places that do not provide the full record.

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/52543

Patch set 1 for branch TYPO3_8-7 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/53209