TYPO3 Core

problem identifiet: tt_content, field image is empty in copy but should have the count of images inside

Georg Ringer wrote:

problem identifiet: tt_content, field image is empty in copy but should have the count of images inside

I think this is more an IRRE/TCEmain problem than really related to FAL, isn't it?

I think I found the problem. Quote form core mailing list:

Hi guys,

I'm not sure but I think I found a conceptional problem with inline records that are stored in the same page as their parents.

I found this by researching the cause of:
http://forge.typo3.org/issues/44795 and
http://forge.typo3.org/issues/44964

Lets assume we have a page with a tt_content element that uses a sys_file_reference. We now copy that page. This is what happens:

DataHandler->copySpecificPage() loops over all configured tables and copies the records in that page.

If sys_file_reference appears first in the tables array (which seems to be the case currently) the sys_file_reference record is copied (a new record with identical data but a different PID appears in the sys_file_reference table). The records still points to the content element in the old page though.

Now the tt_content element is copied which has a reference to an inline sys_file_reference record. This is done by copyRecord_procBasedOnFieldType() which finally calls copyRecord_raw(). In this method it is checked, if the record is already copied by calling isRecordCopied(). Since the record WAS already copied further processing is skipped.

Has anyone a good idea how to solve this? Maybe in copyRecord_raw() override the data of the related record isRecordCopied() returns TRUE?

I wish you all a merry Christmas

Cheers,
Alex

The patch pushed to Gerrit makes sure that even if the records have been copied before when looping over the tables the inline records still get updated.

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/26552

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

Is their a reason why this fix is not included in TYPO3 v6.1?

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

Good point, David. Patch for 6.1 was pushed to Gerrit.

Hi Alexander,
will this patch be available with the next update?

Thanks and greetings

David

EDIT:
Ah, it's allready their. =) Thanks

Andreas, did the patch solve your issue?