Project

General

Profile

Actions

Bug #68623

closed

Childrecords pid of inline relation broken for pages_language_overlay

Added by Anja Leichsenring over 8 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Should have
Category:
-
Target version:
-
Start date:
2015-07-29
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
7
PHP Version:
Tags:
Complexity:
Is Regression:
No
Sprint Focus:

Description

During the refactoring of FormEngine a replacement error happend, that led to the codeline $record['pid'] = $$inlineFirstPid; (mind the $$, resulting in NULL for this variable).

The part of the code belongs to translation activity of pages and led to sys_file_reference records with pid 0.

How to reproduce:
  • create a page and fill in at least one image into the media field.
  • have at least one sys_language additionally to default language.
  • translate the page, save the pages_language_overlay record.
  • the images shown in the media field of the translated page record are independent from those in the default language record, they are shown and all seems well.
  • check in the database -> the pid for the sys_file_reference records of the translated page record are all 0.

This seems to be no big deal, since the related child records are visible inside the parent record. But it might lead to missing records in list view and maybe access incidents for other TCA constructs, using the same approach (dedicated translation table, inline records). So better have it fixed.

Actions #1

Updated by Gerrit Code Review over 8 years ago

  • Status changed from New to Under Review

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

Actions #2

Updated by Anja Leichsenring over 8 years ago

  • Description updated (diff)
Actions #3

Updated by Anja Leichsenring over 8 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
Actions #4

Updated by Benni Mack over 5 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF