Task #60654

Execute remap handling of nested children earlier

Added by Oliver Hader over 5 years ago. Updated over 1 year ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
FormEngine aka TCEforms
Target version:
-
Start date:
2014-07-29
Due date:
% Done:

100%

TYPO3 Version:
6.2
PHP Version:
5.3
Tags:
Complexity:
Sprint Focus:

Description

If dealing with nested parent-child-structures, it happens that the proper UID of a record (either parent or child) has not been persistend yet and therefore it's only a "NEW..." which will be replaces in a later step during several "remapping" actions.

If copying a page which many sub-pages and sub-records this remap action is processed very late. Results are invalid record relations (like content elements now having duplicated images since the remapping has not updated the real parent pointer yet).


Related issues

Related to TYPO3 Core - Bug #58307: Temporary id for inline records in flexforms seems to be persisting too long. Closed 2014-04-29

Associated revisions

Revision 8b17e1c4 (diff)
Added by Oliver Hader over 5 years ago

[!!!][TASK] Execute remap handling of nested children earlier

If dealing with nested parent-child-structures, it happens that
the proper UID of a record (either parent or child) has not been
persisted yet and therefore it's only a "NEW..." which will be
replaced in a later step during several "remapping" actions.

If copying a page with many sub-pages and sub-records this
remap action is processed very late. Results are invalid record
relations (like content elements now having duplicated images
since the remapping has not updated the real parent pointer yet).

This patch is marked with [!!!] since side-effects can appear.
In case this change gets merged and it turns out that DataHandling
got worse, it's target to be reverted again.

Resolves: #60654
Releases: master
Change-Id: I9dc77630e832e2b9ab65840daf44931640c6d816
Reviewed-on: http://review.typo3.org/31891
Reviewed-by: Wouter Wolters <>
Tested-by: Wouter Wolters <>
Reviewed-by: Markus Klein <>
Tested-by: Markus Klein <>

History

#1 Updated by Gerrit Code Review over 5 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/31891

#2 Updated by Gerrit Code Review over 5 years ago

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

#3 Updated by Gerrit Code Review over 5 years ago

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

#4 Updated by Oliver Hader over 5 years ago

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

#5 Updated by Benni Mack over 1 year ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF