Bug #88980
openTranslating records with FAL relation where allowLanguageSynchronization set to true
Added by Felix Hilmes over 5 years ago. Updated about 2 months ago.
0%
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 |
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.
Updated by Andreas Wolf about 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.
Updated by Felix Hilmes about 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
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
Updated by Gerrit Code Review over 4 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
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
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
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
Updated by Gerrit Code Review about 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
Updated by Gerrit Code Review about 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
Updated by Rudy Gnodde almost 3 years ago
- Related to Bug #97505: Duplicate translated relations after moving default language content element added
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
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
Updated by Daniel Siepmann over 1 year ago
We have the same issue with V12 and V11. I guess v11 won't receive the patch?
Updated by Joerg Boesche over 1 year 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.
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
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
Updated by Gerrit Code Review 4 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
Updated by Gerrit Code Review about 2 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
Updated by Gerrit Code Review about 2 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
Updated by Gerrit Code Review about 2 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
Updated by Gerrit Code Review about 2 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
Updated by Gerrit Code Review about 2 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
Updated by Sybille Peters about 2 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.
Updated by Gerrit Code Review about 2 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
Updated by Gerrit Code Review about 2 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
Updated by Gerrit Code Review about 2 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
Updated by Sybille Peters about 2 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 | +-----+------------------+-------------+------------+-------------+--------+------------------------------+------------------+
Updated by Sybille Peters about 2 months ago
- Related to Bug #103970: Inline relations do not set the actual language added
Updated by Sybille Peters about 2 months ago
- Related to Bug #103969: Update language of inline elements is missing, leading to follow up issues added
Updated by Gerrit Code Review about 2 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
Updated by Gerrit Code Review about 2 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
Updated by Gerrit Code Review about 2 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
Updated by Gerrit Code Review about 2 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
Updated by Gerrit Code Review about 2 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
Updated by Gerrit Code Review about 2 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
Updated by Gerrit Code Review about 2 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
Updated by Gerrit Code Review about 2 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
Updated by Gerrit Code Review about 2 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
Updated by Gerrit Code Review about 2 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