Bug #39383

Bug #46389: Faulty Workspace Placeholders

Cannot delete record moved in draft

Added by Bart Dubelaar over 7 years ago. Updated over 1 year ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Miscellaneous
Target version:
-
Start date:
2012-07-30
Due date:
% Done:

100%

TYPO3 Version:
4.5
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

In a draft workspace:

1. Create two pages A + B and move B into A.
2. Delete B.
3. You will now see B twice, once at the old location and once inside A.

The point is that the draft element is deleted, but the move-to placeholder remains and is shown as a real record.

Hold for 4.5 up till 6.0


Related issues

Related to TYPO3 Core - Bug #52636: Workspace: Moved records are deleted if source page is deleted. Closed 2013-10-09
Related to TYPO3 Core - Bug #53619: Moving Elements in draft - now my live is a mess Accepted 2013-11-13
Related to TYPO3 Core - Task #69969: Extend workspaces functional tests concerning move-placeholders Closed 2015-09-19
Related to TYPO3 Core - Bug #72273: In 6.2.16, publishing a deleted page containing a moved subpage will delete the whole web site! Closed 2015-12-16

Associated revisions

Revision f78adb23 (diff)
Added by Bart Dubelaar over 4 years ago

[BUGFIX] Delete move placeholders upon record deletion

If a record was moved in a draft workspace and subsequently
removed in the live workspace, then the move placeholder would
remain and show up as an actual record in the backend. To the
end user it appears as if the record was not deleted as one
would now see the move-placeholder.

If the mentioned moved record would be removed instead in the
very same draft workspace, the move-to-pointer was removed
correctly, but the move-placeholder stayed as well.

Resolves: #39383
Releases: master, 6.2
Change-Id: I3b0593332650795539c2d7eac075515dbd981b45
Reviewed-on: http://review.typo3.org/13496
Reviewed-by: Daniel Goerz <>
Tested-by: Daniel Goerz <>
Reviewed-by: Oliver Hader <>
Tested-by: Oliver Hader <>

Revision 27ec0fce (diff)
Added by Bart Dubelaar over 4 years ago

[BUGFIX] Delete move placeholders upon record deletion

If a record was moved in a draft workspace and subsequently
removed in the live workspace, then the move placeholder would
remain and show up as an actual record in the backend. To the
end user it appears as if the record was not deleted as one
would now see the move-placeholder.

If the mentioned moved record would be removed instead in the
very same draft workspace, the move-to-pointer was removed
correctly, but the move-placeholder stayed as well.

Resolves: #39383
Releases: master, 6.2
Change-Id: I3b0593332650795539c2d7eac075515dbd981b45
Reviewed-on: http://review.typo3.org/43479
Reviewed-by: Oliver Hader <>
Tested-by: Oliver Hader <>

History

#1 Updated by Bart Dubelaar over 7 years ago

Case II:
When the original record is removed in the live workspace, then the placeholder also remains and becomes visible.

#2 Updated by Gerrit Code Review over 7 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/13496

#3 Updated by Gerrit Code Review over 7 years ago

Patch set 2 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/13496

#4 Updated by Oliver Hader almost 7 years ago

  • Parent task set to #46389

#5 Updated by Gerrit Code Review over 6 years ago

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

#6 Updated by Gerrit Code Review over 6 years ago

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

#7 Updated by Gerrit Code Review over 6 years ago

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

#8 Updated by Gerrit Code Review over 6 years ago

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

#9 Updated by Gerrit Code Review over 6 years ago

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

#10 Updated by Sascha Egerer over 6 years ago

Testplan:

  • Create a new record "foo" in live workspace on
    a page where already a record exists
  • switch to a workspace
  • move the new record "foo" on the page (this will
    create a placeholder and a move placeholder in
    the database)
  • switch back to the live workspace
  • delete the record "foo"
  • switch back to the workspace where you have moved
    the record

Current result:

  • The move placeholder is shown in the page

Expected result:

  • The record should be removed completely

#11 Updated by Gerrit Code Review over 6 years ago

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

#12 Updated by Gerrit Code Review about 5 years ago

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

#13 Updated by Gerrit Code Review over 4 years ago

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

#14 Updated by Gerrit Code Review over 4 years ago

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

#15 Updated by Gerrit Code Review over 4 years ago

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

#16 Updated by Gerrit Code Review over 4 years ago

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

#17 Updated by Gerrit Code Review over 4 years 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/13496

#18 Updated by Bart Dubelaar over 4 years ago

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

#19 Updated by Gerrit Code Review over 4 years ago

  • Status changed from Resolved to Under Review

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

#20 Updated by Bart Dubelaar over 4 years ago

  • Status changed from Under Review to Resolved

#21 Updated by Benni Mack over 1 year ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF