Bug #42075

Copying fails when page has new draft elements 2 or more levels deep

Added by Bart Dubelaar almost 3 years ago. Updated 7 months ago.

Status:Under Review Start date:2012-10-17
Priority:Must have Due date:
Assigned To:- % Done:

0%

Category:DataHandler aka TCEmain Spent time: -
Target version:-
TYPO3 Version:4.5 Is Regression:
PHP Version: Sprint Focus:
Complexity:

Description

When a page A has a draft subpage B which again has a draft subpage C, then a recursive copy of A will fail in the live workspace. This is because the initial placeholders for the drafts are included in the copy but the parent check fails for these. If the initial placeholders are excluded then the copy will succeed.

Steps to reproduce:
1. Enable recursive copying for user
2: Create a page A in the live workspace
3: Add a subpage B to A in a draft workspace
4: Add a subpage C to B in a draft workspace
5: Add a subpage D to A in the live workspace
6. Copy and paste A in the live workspace

You will get an error message and you will see that even though A is copied it does not include D, this is because the copying process has exited after the failure on the initial placeholders.

Holds for 4.5 up till 6.0.


Related issues

related to Core - Bug #36946: Copy paste in live makes draft placeholders visible. Under Review 2012-05-08

History

#1 Updated by Bart Dubelaar almost 3 years ago

Probably fixable by extending the patch for #36946 to also omit initial placeholders when copying pages, which is handled in a separate copy function and not the general one.

#2 Updated by Gerrit Code Review over 2 years ago

  • Status changed from New to Under Review

Patch set 8 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/11076

#3 Updated by Gerrit Code Review about 2 years ago

Patch set 9 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/11076

#4 Updated by Gerrit Code Review about 2 years ago

Patch set 10 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/11076

#5 Updated by Gerrit Code Review over 1 year ago

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

#6 Updated by Gerrit Code Review over 1 year ago

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

#7 Updated by Gerrit Code Review over 1 year ago

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

#8 Updated by Gerrit Code Review 7 months ago

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

#9 Updated by Gerrit Code Review 7 months ago

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

Also available in: Atom PDF