Project

General

Profile

Actions

Bug #88980

open

Translating records with FAL relation where allowLanguageSynchronization set to true

Added by Felix Hilmes over 5 years ago. Updated 3 months ago.

Status:
Under Review
Priority:
Should have
Assignee:
-
Category:
File Abstraction Layer (FAL)
Target version:
-
Start date:
2019-08-20
Due date:
% Done:

0%

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

Description

When translating a record with a immage (FAL relation) where allowLanguageSynchronization is set to true in the TCA configuration typo3 throws the exception #1486233164: Child record was not processed and the record gets some new immage relation entries

Easiest to Reproduce with the georgringer/news extension
Steps to reproduce:
1. Create a new (news) record, set language to [All]
2. Save
3. Add immage as fal relation,
4. Save
5. Set language to Default [0]
6. Save
7. Translate (In the News Administration section)
8. Admire the Exception/CMS/1486233164 page and the new copies of the FAL relation

Tested in Typo3 8.7, 9.5 and 10.0


Files

Screenshot 2019-08-20 at 09.34.24.png (31 KB) Screenshot 2019-08-20 at 09.34.24.png list of related files after exception Felix Hilmes, 2019-08-20 09:34
Screenshot 2019-08-20 at 09.33.27.png (58.5 KB) Screenshot 2019-08-20 at 09.33.27.png error message Felix Hilmes, 2019-08-20 09:34

Related issues 4 (3 open1 closed)

Related to TYPO3 Core - Bug #97505: Duplicate translated relations after moving default language content elementClosed2022-04-28

Actions
Related to TYPO3 Core - Bug #103970: Inline relations do not set the actual languageUnder Review2024-06-04

Actions
Related to TYPO3 Core - Bug #103969: Update language of inline elements is missing, leading to follow up issuesUnder Review2024-06-04

Actions
Related to TYPO3 Core - Bug #89197: FileCollector and allowLanguageSynchronization for FAL fieldsNew2019-09-18

Actions
Actions #1

Updated by Christian Eßl over 5 years ago

We got the same problem here with the pages -> media fal field. Trying to translate such a page produces pretty chaotic results by duplicating all image relations.

The same error occurs when we go to the Access module and try to change the access permissions of such a page. (Click for example "New pages" icon)
  • The server will answer with a 500 error and the message "Child record was not processed"
  • All inline image relations were duplicated.

When setting allowLanguageSynchronization to false, the problems disappear.

Actions #2

Updated by Andreas Wolf over 5 years ago

I have seen a similar behaviour in some instances and could fix it for my usecases.

Could you try if this patch https://review.typo3.org/c/Packages/TYPO3.CMS/+/62192 works for you as well? It should be applicable to 9.5 as well, not sure about 8, though.

Actions #3

Updated by Felix Hilmes over 5 years ago

Andreas Wolf wrote:

I have seen a similar behaviour in some instances and could fix it for my usecases.

Could you try if this patch https://review.typo3.org/c/Packages/TYPO3.CMS/+/62192 works for you as well? It should be applicable to 9.5 as well, not sure about 8, though.

Works with the news extension in 9.5

Actions #4

Updated by Gerrit Code Review almost 5 years ago

  • Status changed from New to Under Review

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

Actions #5

Updated by Gerrit Code Review almost 5 years ago

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

Actions #6

Updated by Joerg Kummer over 4 years ago

Can confirm for TYPO3 10.4.9, with 4 languages and pages.media, where allowLanguageSynchronization is true, and sys_file_reference is given for default language only. It occurse right after upgrade from TYPO3 8.7

Actions #7

Updated by Gerrit Code Review over 3 years ago

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

Actions #8

Updated by Gerrit Code Review over 3 years ago

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

Actions #9

Updated by Gerrit Code Review over 3 years ago

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

Actions #10

Updated by Gerrit Code Review over 3 years ago

Patch set 9 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/+/62192

Actions #11

Updated by Rudy Gnodde almost 3 years ago

  • Related to Bug #97505: Duplicate translated relations after moving default language content element added
Actions #12

Updated by Gerrit Code Review almost 2 years ago

Patch set 10 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/+/62192

Actions #13

Updated by Gerrit Code Review almost 2 years ago

Patch set 11 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/+/62192

Actions #14

Updated by Daniel Siepmann almost 2 years ago

We have the same issue with V12 and V11. I guess v11 won't receive the patch?

Actions #15

Updated by Joerg Boesche almost 2 years ago

Daniel Siepmann wrote in #note-14:

We have the same issue with V12 and V11. I guess v11 won't receive the patch?

I also confirm the bug in T3 v12 and PHP 8.1. If you have a content element with a flexform and element of type "file", the exception "Child record was not processed" is thrown. Occurs on click for a new translation from the default language, with "allowLanguageSynchronization" enabled.

Actions #16

Updated by Gerrit Code Review over 1 year ago

Patch set 12 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/+/62192

Actions #17

Updated by Gerrit Code Review over 1 year ago

Patch set 13 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/+/62192

Actions #18

Updated by Gerrit Code Review 5 months ago

Patch set 14 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/+/62192

Actions #19

Updated by Gerrit Code Review 3 months ago

Patch set 15 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/+/62192

Actions #20

Updated by Gerrit Code Review 3 months ago

Patch set 16 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/+/62192

Actions #21

Updated by Gerrit Code Review 3 months ago

Patch set 17 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/+/62192

Actions #22

Updated by Gerrit Code Review 3 months ago

Patch set 18 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/+/62192

Actions #23

Updated by Gerrit Code Review 3 months ago

Patch set 19 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/+/62192

Actions #24

Updated by Sybille Peters 3 months ago

I created an extension for convenience in reprocing the bug: https://github.com/sypets/fields_with_allowlangsync

This way you can test the behaviour with core v14 fields, e.g. tt_content.assets.

Actions #25

Updated by Gerrit Code Review 3 months ago

Patch set 20 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/+/62192

Actions #26

Updated by Gerrit Code Review 3 months ago

Patch set 21 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/+/62192

Actions #27

Updated by Gerrit Code Review 3 months ago

Patch set 22 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/+/62192

Actions #28

Updated by Sybille Peters 3 months ago · Edited

The problem is already before the element is translated, so I think the current patch fixes it too late: https://review.typo3.org/c/Packages/TYPO3.CMS/+/62192

After performing steps 1-6 (see above): with tt_content.assets with allowLanguageSynchronization

1. Create a new textmedia, set language to [All]
2. Save
3. Add asset as fal relation,
4. Save
5. Set language to Default [0]
6. Save

and then we look in the DB: we expect the sys_file_reference.sys_language_uid to be have value 0, but it has value -1:

select r.uid,r.sys_language_uid,r.l10n_parent,r.l10n_state,r.uid_foreign,r.hidden,t.header,t.sys_language_uid from sys_file_reference r inner join tt_content t on r.uid_foreign=t.uid where r.pid=904 and not r.deleted order by t.uid;
+-----+------------------+-------------+------------+-------------+--------+------------------------------+------------------+
| uid | sys_language_uid | l10n_parent | l10n_state | uid_foreign | hidden | header                       | sys_language_uid |
+-----+------------------+-------------+------------+-------------+--------+------------------------------+------------------+
| 415 |               -1 |           0 | NULL       |         149 |      0 | another Element              |                0 |
+-----+------------------+-------------+------------+-------------+--------+------------------------------+------------------+

Actions #29

Updated by Sybille Peters 3 months ago

  • Related to Bug #103970: Inline relations do not set the actual language added
Actions #30

Updated by Sybille Peters 3 months ago

  • Related to Bug #103969: Update language of inline elements is missing, leading to follow up issues added
Actions #31

Updated by Gerrit Code Review 3 months ago

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/+/87595

Actions #32

Updated by Gerrit Code Review 3 months 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/+/87595

Actions #33

Updated by Gerrit Code Review 3 months 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/+/87595

Actions #34

Updated by Gerrit Code Review 3 months 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/+/87595

Actions #35

Updated by Gerrit Code Review 3 months 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/+/87595

Actions #36

Updated by Gerrit Code Review 3 months 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/+/87595

Actions #37

Updated by Gerrit Code Review 3 months ago

Patch set 7 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/+/87595

Actions #38

Updated by Gerrit Code Review 3 months ago

Patch set 8 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/+/87595

Actions #39

Updated by Gerrit Code Review 3 months ago

Patch set 9 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/+/87595

Actions #40

Updated by Gerrit Code Review 3 months ago

Patch set 10 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/+/87595

Actions #41

Updated by Benni Mack 2 days ago

  • Related to Bug #89197: FileCollector and allowLanguageSynchronization for FAL fields added
Actions

Also available in: Atom PDF